Message ID | 20221215150214.1109074-15-hugo@hugovil.com |
---|---|
State | Changes Requested |
Headers | show |
Series | rtc: pcf2127: add PCF2131 driver | expand |
On 15/12/2022 16:02, Hugo Villeneuve wrote: > From: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > Add support for new NXP RTC PCF2131. > > Signed-off-by: Hugo Villeneuve <hvilleneuve@dimonoff.com> > --- > Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
Den tor. 15. dec. 2022 kl. 16.19 skrev Hugo Villeneuve <hugo@hugovil.com>: > > From: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > Add support for new NXP RTC PCF2131. > > Signed-off-by: Hugo Villeneuve <hvilleneuve@dimonoff.com> > --- > Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml > index cde7b1675ead..a8f8c23da4d8 100644 > --- a/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml > +++ b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml > @@ -14,7 +14,9 @@ maintainers: > > properties: > compatible: > - const: nxp,pcf2127 > + enum: > + - nxp,pcf2127 > + - nxp,pcf2131 The enum is incomplete as pcf2127_of_match struct also contains: nxp,pcf2129 ncp,pca2129 /Bruno > > reg: > maxItems: 1 > -- > 2.30.2 >
On Mon, 19 Dec 2022 10:14:10 +0100 Bruno Thomsen <bruno.thomsen@gmail.com> wrote: > Den tor. 15. dec. 2022 kl. 16.19 skrev Hugo Villeneuve <hugo@hugovil.com>: > > > > From: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > > > Add support for new NXP RTC PCF2131. > > > > Signed-off-by: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > --- > > Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > diff --git a/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml > > index cde7b1675ead..a8f8c23da4d8 100644 > > --- a/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml > > +++ b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml > > @@ -14,7 +14,9 @@ maintainers: > > > > properties: > > compatible: > > - const: nxp,pcf2127 > > + enum: > > + - nxp,pcf2127 > > + - nxp,pcf2131 > > The enum is incomplete as pcf2127_of_match struct also contains: > nxp,pcf2129 > ncp,pca2129 > > /Bruno Hi, if I understand correctly, this means that the pca2129 and pcf2129 entries are already missing and should be added in a fix or a patch outside the scope of my new driver... Hugo. > > reg: > > maxItems: 1 > > -- > > 2.30.2 > > >
Den man. 19. dec. 2022 kl. 17.25 skrev Hugo Villeneuve <hugo@hugovil.com>: > > On Mon, 19 Dec 2022 10:14:10 +0100 > Bruno Thomsen <bruno.thomsen@gmail.com> wrote: > > > Den tor. 15. dec. 2022 kl. 16.19 skrev Hugo Villeneuve <hugo@hugovil.com>: > > > > > > From: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > > > > > Add support for new NXP RTC PCF2131. > > > > > > Signed-off-by: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > > --- > > > Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml | 4 +++- > > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > > > diff --git a/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml > > > index cde7b1675ead..a8f8c23da4d8 100644 > > > --- a/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml > > > +++ b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml > > > @@ -14,7 +14,9 @@ maintainers: > > > > > > properties: > > > compatible: > > > - const: nxp,pcf2127 > > > + enum: > > > + - nxp,pcf2127 > > > + - nxp,pcf2131 > > > > The enum is incomplete as pcf2127_of_match struct also contains: > > nxp,pcf2129 > > ncp,pca2129 > > > > /Bruno > > Hi, > if I understand correctly, this means that the pca2129 and pcf2129 entries are already missing and should be added in a fix or a patch outside the scope of my new driver... > Correct, I just noticed while doing review. > Hugo. > > > > > reg: > > > maxItems: 1 > > > -- > > > 2.30.2 > > > > > > > > -- > Hugo Villeneuve <hugo@hugovil.com>
On Mon, 19 Dec 2022 18:18:58 +0100 Bruno Thomsen <bruno.thomsen@gmail.com> wrote: > Den man. 19. dec. 2022 kl. 17.25 skrev Hugo Villeneuve <hugo@hugovil.com>: > > > > On Mon, 19 Dec 2022 10:14:10 +0100 > > Bruno Thomsen <bruno.thomsen@gmail.com> wrote: > > > > > Den tor. 15. dec. 2022 kl. 16.19 skrev Hugo Villeneuve <hugo@hugovil.com>: > > > > > > > > From: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > > > > > > > Add support for new NXP RTC PCF2131. > > > > > > > > Signed-off-by: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > > > --- > > > > Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml | 4 +++- > > > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > > > > > diff --git a/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml > > > > index cde7b1675ead..a8f8c23da4d8 100644 > > > > --- a/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml > > > > +++ b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml > > > > @@ -14,7 +14,9 @@ maintainers: > > > > > > > > properties: > > > > compatible: > > > > - const: nxp,pcf2127 > > > > + enum: > > > > + - nxp,pcf2127 > > > > + - nxp,pcf2131 > > > > > > The enum is incomplete as pcf2127_of_match struct also contains: > > > nxp,pcf2129 > > > ncp,pca2129 > > > > > > /Bruno > > > > Hi, > > if I understand correctly, this means that the pca2129 and pcf2129 entries are already missing and should be added in a fix or a patch outside the scope of my new driver... > > > > Correct, I just noticed while doing review. Hi, I will send a separate patch very soon. Thank you, Hugo. V. > > > > reg: > > > > maxItems: 1 > > > > -- > > > > 2.30.2 > > > > > > > > > > > > > -- > > Hugo Villeneuve <hugo@hugovil.com> >
diff --git a/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml index cde7b1675ead..a8f8c23da4d8 100644 --- a/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml +++ b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml @@ -14,7 +14,9 @@ maintainers: properties: compatible: - const: nxp,pcf2127 + enum: + - nxp,pcf2127 + - nxp,pcf2131 reg: maxItems: 1