linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt
@ 2020-08-27  9:14 Qiang Zhao
  2020-08-27  9:14 ` [PATCH 2/3] rtc: pcf2127: add "no-watchdog" property Qiang Zhao
                   ` (3 more replies)
  0 siblings, 4 replies; 12+ messages in thread
From: Qiang Zhao @ 2020-08-27  9:14 UTC (permalink / raw)
  To: a.zummo, alexandre.belloni, robh+dt
  Cc: linux-rtc, devicetree, linux-kernel, Zhao Qiang

From: Zhao Qiang <qiang.zhao@nxp.com>

Add bindings for nxp,rtc-2127

Signed-off-by: Zhao Qiang <qiang.zhao@nxp.com>
---
 Documentation/devicetree/bindings/rtc/nxp,rtc-2127.txt | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/rtc/nxp,rtc-2127.txt

diff --git a/Documentation/devicetree/bindings/rtc/nxp,rtc-2127.txt b/Documentation/devicetree/bindings/rtc/nxp,rtc-2127.txt
new file mode 100644
index 0000000..c822e10
--- /dev/null
+++ b/Documentation/devicetree/bindings/rtc/nxp,rtc-2127.txt
@@ -0,0 +1,18 @@
+NXP PCF2127 SPI Real Time Clock
+
+Required properties:
+- compatible: should be: "nxp,pcf2127"
+                      or "nxp,pcf2129"
+- reg: should be the SPI slave chipselect address
+
+Optional properties:
+- no-watchdog: PCF2127 is not used as watchdog
+
+Example:
+
+pcf2127: rtc@51 {
+	compatible = "nxp,pcf2127";
+	reg = <0x51>;
+	interrupts = <0 150 0x4>;
+	no-watchdog;
+};
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 2/3] rtc: pcf2127: add "no-watchdog" property
  2020-08-27  9:14 [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt Qiang Zhao
@ 2020-08-27  9:14 ` Qiang Zhao
  2020-08-27  9:14 ` [PATCH 3/3] arm64: dts: lx2160a: Add "no-watchdog" property to pcf2127 Qiang Zhao
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 12+ messages in thread
From: Qiang Zhao @ 2020-08-27  9:14 UTC (permalink / raw)
  To: a.zummo, alexandre.belloni, robh+dt
  Cc: linux-rtc, devicetree, linux-kernel, Zhao Qiang

From: Zhao Qiang <qiang.zhao@nxp.com>

Commit 0e735eaae165 ("rtc: pcf2127: add watchdog feature support")
adds watchdog feature for pcf2127, then it will be registered as
a watchdog device even though it is not used as watchdog.
Pcf2127 always be registered as watchdog0(default watchdog device),
even though it is not used as watchdog, meanwhile there is another
real watchdog device registered as watchdog1.
So add "no-watchdog" property, if the node has this property,
driver will not register it as a watchdog device

Signed-off-by: Zhao Qiang <qiang.zhao@nxp.com>
---
 drivers/rtc/rtc-pcf2127.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/drivers/rtc/rtc-pcf2127.c b/drivers/rtc/rtc-pcf2127.c
index 02b069c..38d30e2 100644
--- a/drivers/rtc/rtc-pcf2127.c
+++ b/drivers/rtc/rtc-pcf2127.c
@@ -476,9 +476,11 @@ static int pcf2127_probe(struct device *dev, struct regmap *regmap,
 	}
 
 #ifdef CONFIG_WATCHDOG
-	ret = devm_watchdog_register_device(dev, &pcf2127->wdd);
-	if (ret)
-		return ret;
+	if (!of_property_read_bool(dev->of_node, "no-watchdog")) {
+		ret = devm_watchdog_register_device(dev, &pcf2127->wdd);
+		if (ret)
+			return ret;
+	}
 #endif /* CONFIG_WATCHDOG */
 
 	/*
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 3/3] arm64: dts: lx2160a: Add "no-watchdog" property to pcf2127
  2020-08-27  9:14 [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt Qiang Zhao
  2020-08-27  9:14 ` [PATCH 2/3] rtc: pcf2127: add "no-watchdog" property Qiang Zhao
@ 2020-08-27  9:14 ` Qiang Zhao
  2020-09-09 20:24 ` [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt Rob Herring
  2020-09-11 21:34 ` Alexandre Belloni
  3 siblings, 0 replies; 12+ messages in thread
From: Qiang Zhao @ 2020-08-27  9:14 UTC (permalink / raw)
  To: a.zummo, alexandre.belloni, robh+dt
  Cc: linux-rtc, devicetree, linux-kernel, Zhao Qiang

From: Zhao Qiang <qiang.zhao@nxp.com>

pcf2127 on lx2160a are not used as watchdog.

Signed-off-by: Zhao Qiang <qiang.zhao@nxp.com>
---
 arch/arm64/boot/dts/freescale/fsl-lx2160a-rdb.dts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/freescale/fsl-lx2160a-rdb.dts b/arch/arm64/boot/dts/freescale/fsl-lx2160a-rdb.dts
index 2ec43c6..4d14c91 100644
--- a/arch/arm64/boot/dts/freescale/fsl-lx2160a-rdb.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-lx2160a-rdb.dts
@@ -149,6 +149,7 @@
 		reg = <0x51>;
 		// IRQ10_B
 		interrupts = <0 150 0x4>;
+		no-watchdog;
 	};
 };
 
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt
  2020-08-27  9:14 [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt Qiang Zhao
  2020-08-27  9:14 ` [PATCH 2/3] rtc: pcf2127: add "no-watchdog" property Qiang Zhao
  2020-08-27  9:14 ` [PATCH 3/3] arm64: dts: lx2160a: Add "no-watchdog" property to pcf2127 Qiang Zhao
@ 2020-09-09 20:24 ` Rob Herring
  2020-09-10  3:16   ` Qiang Zhao
  2020-09-11 21:34 ` Alexandre Belloni
  3 siblings, 1 reply; 12+ messages in thread
From: Rob Herring @ 2020-09-09 20:24 UTC (permalink / raw)
  To: Qiang Zhao
  Cc: a.zummo, alexandre.belloni, linux-rtc, devicetree, linux-kernel

On Thu, Aug 27, 2020 at 05:14:39PM +0800, Qiang Zhao wrote:
> From: Zhao Qiang <qiang.zhao@nxp.com>
> 
> Add bindings for nxp,rtc-2127
> 
> Signed-off-by: Zhao Qiang <qiang.zhao@nxp.com>
> ---
>  Documentation/devicetree/bindings/rtc/nxp,rtc-2127.txt | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/rtc/nxp,rtc-2127.txt

Bindings should be in DT schema format now.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* RE: [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt
  2020-09-09 20:24 ` [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt Rob Herring
@ 2020-09-10  3:16   ` Qiang Zhao
  2020-09-11 14:03     ` Rob Herring
  0 siblings, 1 reply; 12+ messages in thread
From: Qiang Zhao @ 2020-09-10  3:16 UTC (permalink / raw)
  To: Rob Herring
  Cc: a.zummo, alexandre.belloni, linux-rtc, devicetree, linux-kernel

On Thu, Sep 10, 2020 at 04:25AM, Rob Herring <robh@kernel.org> wrote:
> -----Original Message-----
> From: Rob Herring <robh@kernel.org>
> Sent: 2020年9月10日 4:25
> To: Qiang Zhao <qiang.zhao@nxp.com>
> Cc: a.zummo@towertech.it; alexandre.belloni@bootlin.com;
> linux-rtc@vger.kernel.org; devicetree@vger.kernel.org;
> linux-kernel@vger.kernel.org
> Subject: Re: [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for
> nxp,rtc-2127.txt
> 
> On Thu, Aug 27, 2020 at 05:14:39PM +0800, Qiang Zhao wrote:
> > From: Zhao Qiang <qiang.zhao@nxp.com>
> >
> > Add bindings for nxp,rtc-2127
> >
> > Signed-off-by: Zhao Qiang <qiang.zhao@nxp.com>
> > ---
> >  Documentation/devicetree/bindings/rtc/nxp,rtc-2127.txt | 18
> ++++++++++++++++++
> >  1 file changed, 18 insertions(+)
> >  create mode 100644
> Documentation/devicetree/bindings/rtc/nxp,rtc-2127.txt
> 
> Bindings should be in DT schema format now.

Is there any doc description for DT schema format or example 
So that I can take as a reference. Thank you!

Best Regards
Qiang Zhao

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt
  2020-09-10  3:16   ` Qiang Zhao
@ 2020-09-11 14:03     ` Rob Herring
  2020-09-14  7:08       ` Qiang Zhao
  0 siblings, 1 reply; 12+ messages in thread
From: Rob Herring @ 2020-09-11 14:03 UTC (permalink / raw)
  To: Qiang Zhao
  Cc: a.zummo, alexandre.belloni, linux-rtc, devicetree, linux-kernel

On Wed, Sep 9, 2020 at 9:16 PM Qiang Zhao <qiang.zhao@nxp.com> wrote:
>
> On Thu, Sep 10, 2020 at 04:25AM, Rob Herring <robh@kernel.org> wrote:
> > -----Original Message-----
> > From: Rob Herring <robh@kernel.org>
> > Sent: 2020年9月10日 4:25
> > To: Qiang Zhao <qiang.zhao@nxp.com>
> > Cc: a.zummo@towertech.it; alexandre.belloni@bootlin.com;
> > linux-rtc@vger.kernel.org; devicetree@vger.kernel.org;
> > linux-kernel@vger.kernel.org
> > Subject: Re: [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for
> > nxp,rtc-2127.txt
> >
> > On Thu, Aug 27, 2020 at 05:14:39PM +0800, Qiang Zhao wrote:
> > > From: Zhao Qiang <qiang.zhao@nxp.com>
> > >
> > > Add bindings for nxp,rtc-2127
> > >
> > > Signed-off-by: Zhao Qiang <qiang.zhao@nxp.com>
> > > ---
> > >  Documentation/devicetree/bindings/rtc/nxp,rtc-2127.txt | 18
> > ++++++++++++++++++
> > >  1 file changed, 18 insertions(+)
> > >  create mode 100644
> > Documentation/devicetree/bindings/rtc/nxp,rtc-2127.txt
> >
> > Bindings should be in DT schema format now.
>
> Is there any doc description for DT schema format or example
> So that I can take as a reference. Thank you!

Documentation/devicetree/writing-schema.rst and about 1000 examples in
the kernel tree.

Rob

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt
  2020-08-27  9:14 [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt Qiang Zhao
                   ` (2 preceding siblings ...)
  2020-09-09 20:24 ` [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt Rob Herring
@ 2020-09-11 21:34 ` Alexandre Belloni
  3 siblings, 0 replies; 12+ messages in thread
From: Alexandre Belloni @ 2020-09-11 21:34 UTC (permalink / raw)
  To: Qiang Zhao; +Cc: a.zummo, robh+dt, linux-rtc, devicetree, linux-kernel

Hi,

The correct subject prefix is dt-bindings: rtc: pcf2127

On 27/08/2020 17:14:39+0800, Qiang Zhao wrote:
> From: Zhao Qiang <qiang.zhao@nxp.com>
> 
> Add bindings for nxp,rtc-2127
> 
> Signed-off-by: Zhao Qiang <qiang.zhao@nxp.com>
> ---
>  Documentation/devicetree/bindings/rtc/nxp,rtc-2127.txt | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/rtc/nxp,rtc-2127.txt
> 
> diff --git a/Documentation/devicetree/bindings/rtc/nxp,rtc-2127.txt b/Documentation/devicetree/bindings/rtc/nxp,rtc-2127.txt
> new file mode 100644
> index 0000000..c822e10
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/rtc/nxp,rtc-2127.txt

The proper filename is nxp,pcf2127.

> @@ -0,0 +1,18 @@
> +NXP PCF2127 SPI Real Time Clock
> +
> +Required properties:
> +- compatible: should be: "nxp,pcf2127"
> +                      or "nxp,pcf2129"
> +- reg: should be the SPI slave chipselect address
> +
> +Optional properties:
> +- no-watchdog: PCF2127 is not used as watchdog
> +

Please Cc the watchdog maintainers as this should probably end up being
a common watchdog binding.

> +Example:
> +
> +pcf2127: rtc@51 {
> +	compatible = "nxp,pcf2127";
> +	reg = <0x51>;
> +	interrupts = <0 150 0x4>;
> +	no-watchdog;
> +};
> -- 
> 2.7.4
> 

-- 
Alexandre Belloni, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

^ permalink raw reply	[flat|nested] 12+ messages in thread

* RE: [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt
  2020-09-11 14:03     ` Rob Herring
@ 2020-09-14  7:08       ` Qiang Zhao
  2020-09-14 14:36         ` Rob Herring
  2020-09-17 19:29         ` Bruno Thomsen
  0 siblings, 2 replies; 12+ messages in thread
From: Qiang Zhao @ 2020-09-14  7:08 UTC (permalink / raw)
  To: Rob Herring
  Cc: a.zummo, alexandre.belloni, linux-rtc, devicetree, linux-kernel

On Fri, Sep 11, 2020 at 22:03, Rob Herring <robh@kernel.org> wrote:

> -----Original Message-----
> From: Rob Herring <robh@kernel.org>
> Sent: 2020年9月11日 22:03
> To: Qiang Zhao <qiang.zhao@nxp.com>
> Cc: a.zummo@towertech.it; alexandre.belloni@bootlin.com;
> linux-rtc@vger.kernel.org; devicetree@vger.kernel.org;
> linux-kernel@vger.kernel.org
> Subject: Re: [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for
> nxp,rtc-2127.txt
> 
> On Wed, Sep 9, 2020 at 9:16 PM Qiang Zhao <qiang.zhao@nxp.com> wrote:
> >
> > On Thu, Sep 10, 2020 at 04:25AM, Rob Herring <robh@kernel.org> wrote:
> > > -----Original Message-----
> > > From: Rob Herring <robh@kernel.org>
> > > Sent: 2020年9月10日 4:25
> > > To: Qiang Zhao <qiang.zhao@nxp.com>
> > > Cc: a.zummo@towertech.it; alexandre.belloni@bootlin.com;
> > > linux-rtc@vger.kernel.org; devicetree@vger.kernel.org;
> > > linux-kernel@vger.kernel.org
> > > Subject: Re: [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for
> > > nxp,rtc-2127.txt
> > >
> > > On Thu, Aug 27, 2020 at 05:14:39PM +0800, Qiang Zhao wrote:
> > > > From: Zhao Qiang <qiang.zhao@nxp.com>
> > > >

Please help to review as below, if it is ok, I will send the new version patch. Thank you!

diff --git a/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml
new file mode 100644
index 0000000..809dd59
--- /dev/null
+++ b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml
@@ -0,0 +1,38 @@
+# SPDX-License-Identifier: GPL-2.0
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/rtc/nxp,pcf2127.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: PCF RTCs
+
+maintainers:
+  - Qiang Zhao <qiang.zhao@nxp.com>
+
+allOf:
+  - $ref: "rtc.yaml#"
+
+properties:
+  compatible:
+    enum:
+      - nxp,pcf2127
+      - nxp,pcf2129
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    maxItems: 1
+
+  no-watchdog:
+    maxItems: 1
+
+  start-year: true
+
+required:
+  - compatible
+  - reg
+
+additionalProperties: false
+
+...

> 
> Documentation/devicetree/writing-schema.rst and about 1000 examples in the
> kernel tree.
> 
> Rob

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt
  2020-09-14  7:08       ` Qiang Zhao
@ 2020-09-14 14:36         ` Rob Herring
  2020-09-15  3:26           ` Qiang Zhao
  2020-09-17 19:29         ` Bruno Thomsen
  1 sibling, 1 reply; 12+ messages in thread
From: Rob Herring @ 2020-09-14 14:36 UTC (permalink / raw)
  To: Qiang Zhao
  Cc: a.zummo, alexandre.belloni, linux-rtc, devicetree, linux-kernel

On Mon, Sep 14, 2020 at 1:08 AM Qiang Zhao <qiang.zhao@nxp.com> wrote:
>
> On Fri, Sep 11, 2020 at 22:03, Rob Herring <robh@kernel.org> wrote:
>
> > -----Original Message-----
> > From: Rob Herring <robh@kernel.org>
> > Sent: 2020年9月11日 22:03
> > To: Qiang Zhao <qiang.zhao@nxp.com>
> > Cc: a.zummo@towertech.it; alexandre.belloni@bootlin.com;
> > linux-rtc@vger.kernel.org; devicetree@vger.kernel.org;
> > linux-kernel@vger.kernel.org
> > Subject: Re: [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for
> > nxp,rtc-2127.txt
> >
> > On Wed, Sep 9, 2020 at 9:16 PM Qiang Zhao <qiang.zhao@nxp.com> wrote:
> > >
> > > On Thu, Sep 10, 2020 at 04:25AM, Rob Herring <robh@kernel.org> wrote:
> > > > -----Original Message-----
> > > > From: Rob Herring <robh@kernel.org>
> > > > Sent: 2020年9月10日 4:25
> > > > To: Qiang Zhao <qiang.zhao@nxp.com>
> > > > Cc: a.zummo@towertech.it; alexandre.belloni@bootlin.com;
> > > > linux-rtc@vger.kernel.org; devicetree@vger.kernel.org;
> > > > linux-kernel@vger.kernel.org
> > > > Subject: Re: [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for
> > > > nxp,rtc-2127.txt
> > > >
> > > > On Thu, Aug 27, 2020 at 05:14:39PM +0800, Qiang Zhao wrote:
> > > > > From: Zhao Qiang <qiang.zhao@nxp.com>
> > > > >
>
> Please help to review as below, if it is ok, I will send the new version patch. Thank you!
>
> diff --git a/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml
> new file mode 100644
> index 0000000..809dd59
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml
> @@ -0,0 +1,38 @@
> +# SPDX-License-Identifier: GPL-2.0

Dual license new bindings please:

(GPL-2.0-only OR BSD-2-Clause)

> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/rtc/nxp,pcf2127.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: PCF RTCs
> +
> +maintainers:
> +  - Qiang Zhao <qiang.zhao@nxp.com>
> +
> +allOf:
> +  - $ref: "rtc.yaml#"
> +
> +properties:
> +  compatible:
> +    enum:
> +      - nxp,pcf2127
> +      - nxp,pcf2129
> +
> +  reg:
> +    maxItems: 1
> +
> +  interrupts:
> +    maxItems: 1
> +
> +  no-watchdog:
> +    maxItems: 1

maxItems is for arrays. What's the type here? It should have a description too.

> +
> +  start-year: true
> +
> +required:
> +  - compatible
> +  - reg
> +
> +additionalProperties: false
> +
> +...
>
> >
> > Documentation/devicetree/writing-schema.rst and about 1000 examples in the
> > kernel tree.
> >
> > Rob

^ permalink raw reply	[flat|nested] 12+ messages in thread

* RE: [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt
  2020-09-14 14:36         ` Rob Herring
@ 2020-09-15  3:26           ` Qiang Zhao
  0 siblings, 0 replies; 12+ messages in thread
From: Qiang Zhao @ 2020-09-15  3:26 UTC (permalink / raw)
  To: Rob Herring
  Cc: a.zummo, alexandre.belloni, linux-rtc, devicetree, linux-kernel

On Thu, Sep 14, 2020 at 22:36, Rob Herring <robh@kernel.org> wrote:

> -----Original Message-----
> From: Rob Herring <robh@kernel.org>
> Sent: 2020年9月14日 22:36
> To: Qiang Zhao <qiang.zhao@nxp.com>
> Cc: a.zummo@towertech.it; alexandre.belloni@bootlin.com;
> linux-rtc@vger.kernel.org; devicetree@vger.kernel.org;
> linux-kernel@vger.kernel.org
> Subject: Re: [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for
> nxp,rtc-2127.txt
> 
> > > > > >
> >
> > Please help to review as below, if it is ok, I will send the new version patch.
> Thank you!
> >
> > diff --git a/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml
> > b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml
> > new file mode 100644
> > index 0000000..809dd59
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml
> > @@ -0,0 +1,38 @@
> > +# SPDX-License-Identifier: GPL-2.0
> 
> Dual license new bindings please:
> > +
> > +title: PCF RTCs
> > +
> > +maintainers:
> > +  - Qiang Zhao <qiang.zhao@nxp.com>
> > +
> > +
> > +  no-watchdog:
> > +    maxItems: 1
> 
> maxItems is for arrays. What's the type here? It should have a description too.

Is it ok as below?
  no-watchdog:
    $ref: /schemas/types.yaml#/definitions/flag
    description:
      With this property, the device will not registered as a watchdog device.

> > >
> > > Documentation/devicetree/writing-schema.rst and about 1000 examples
> > > in the kernel tree.
> > >
> > > Rob
Best Regards
Qiang Zhao

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt
  2020-09-14  7:08       ` Qiang Zhao
  2020-09-14 14:36         ` Rob Herring
@ 2020-09-17 19:29         ` Bruno Thomsen
  2020-09-18  1:45           ` Qiang Zhao
  1 sibling, 1 reply; 12+ messages in thread
From: Bruno Thomsen @ 2020-09-17 19:29 UTC (permalink / raw)
  To: Qiang Zhao
  Cc: Rob Herring, a.zummo, alexandre.belloni, linux-rtc, devicetree,
	linux-kernel

Den man. 14. sep. 2020 kl. 09.08 skrev Qiang Zhao <qiang.zhao@nxp.com>:
>
> On Fri, Sep 11, 2020 at 22:03, Rob Herring <robh@kernel.org> wrote:

> Please help to review as below, if it is ok, I will send the new version patch. Thank you!
>
> diff --git a/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml
> new file mode 100644
> index 0000000..809dd59
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml
> @@ -0,0 +1,38 @@
> +# SPDX-License-Identifier: GPL-2.0
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/rtc/nxp,pcf2127.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: PCF RTCs
> +
> +maintainers:
> +  - Qiang Zhao <qiang.zhao@nxp.com>
> +
> +allOf:
> +  - $ref: "rtc.yaml#"
> +
> +properties:
> +  compatible:
> +    enum:
> +      - nxp,pcf2127
> +      - nxp,pcf2129

The device driver have 3 compatible strings, "nxp,pca2129" is missing.

/Bruno

> +
> +  reg:
> +    maxItems: 1
> +
> +  interrupts:
> +    maxItems: 1
> +
> +  no-watchdog:
> +    maxItems: 1
> +
> +  start-year: true
> +
> +required:
> +  - compatible
> +  - reg
> +
> +additionalProperties: false
> +
> +...
>
> >
> > Documentation/devicetree/writing-schema.rst and about 1000 examples in the
> > kernel tree.
> >
> > Rob

^ permalink raw reply	[flat|nested] 12+ messages in thread

* RE: [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt
  2020-09-17 19:29         ` Bruno Thomsen
@ 2020-09-18  1:45           ` Qiang Zhao
  0 siblings, 0 replies; 12+ messages in thread
From: Qiang Zhao @ 2020-09-18  1:45 UTC (permalink / raw)
  To: Bruno Thomsen
  Cc: Rob Herring, a.zummo, alexandre.belloni, linux-rtc, devicetree,
	linux-kernel


On Fri, Sep 18, 2020 at 03:29, Bruno Thomsen <bruno.thomsen@gmail.com> wrote:

> -----Original Message-----
> From: Bruno Thomsen <bruno.thomsen@gmail.com>
> Sent: 2020年9月18日 3:29
> To: Qiang Zhao <qiang.zhao@nxp.com>
> Cc: Rob Herring <robh@kernel.org>; a.zummo@towertech.it;
> alexandre.belloni@bootlin.com; linux-rtc@vger.kernel.org;
> devicetree@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: Re: [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for
> nxp,rtc-2127.txt
> 
> Den man. 14. sep. 2020 kl. 09.08 skrev Qiang Zhao <qiang.zhao@nxp.com>:
> >
> > On Fri, Sep 11, 2020 at 22:03, Rob Herring <robh@kernel.org> wrote:
> 
> > Please help to review as below, if it is ok, I will send the new version patch.
> Thank you!
> >
> > diff --git a/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml
> > b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml
> > new file mode 100644
> > index 0000000..809dd59
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/rtc/nxp,pcf2127.yaml
> > @@ -0,0 +1,38 @@
> > +# SPDX-License-Identifier: GPL-2.0
> > +%YAML 1.2
> > +---
> > +$id:
> > +https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevi
> >
> +cetree.org%2Fschemas%2Frtc%2Fnxp%2Cpcf2127.yaml%23&amp;data=02%7
> C01%7
> >
> +Cqiang.zhao%40nxp.com%7C5a271aa3d1ed46ea578108d85b3ff9f6%7C686e
> a1d3bc
> >
> +2b4c6fa92cd99c5c301635%7C0%7C0%7C637359677624002617&amp;sdata
> =UGZ9B6E
> > +xteIMvxc3zvPfr%2FjZVQuPD6DJ6b2kZ%2BOO%2BIA%3D&amp;reserved=0
> > +$schema:
> > +https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevi
> >
> +cetree.org%2Fmeta-schemas%2Fcore.yaml%23&amp;data=02%7C01%7Cqiang
> .zha
> >
> +o%40nxp.com%7C5a271aa3d1ed46ea578108d85b3ff9f6%7C686ea1d3bc2b4
> c6fa92c
> >
> +d99c5c301635%7C0%7C0%7C637359677624002617&amp;sdata=7P2m9OvIs
> FS6Q2By9
> > +DNmK9Oq5Ff6B%2BNSEA4F7otTT8g%3D&amp;reserved=0
> > +
> > +title: PCF RTCs
> > +
> > +maintainers:
> > +  - Qiang Zhao <qiang.zhao@nxp.com>
> > +
> > +allOf:
> > +  - $ref: "rtc.yaml#"
> > +
> > +properties:
> > +  compatible:
> > +    enum:
> > +      - nxp,pcf2127
> > +      - nxp,pcf2129
> 
> The device driver have 3 compatible strings, "nxp,pca2129" is missing.

Thank you for your reminder, I will add in next version.

> 
> /Bruno
> 
> > +
> > +  reg:
> > +    maxItems: 1
> > +
> > +  interrupts:
> > +    maxItems: 1
> > +
> > +  no-watchdog:
> > +    maxItems: 1
> > +
> > +  start-year: true
> > +
> > +required:
> > +  - compatible
> > +  - reg
> > +
> > +additionalProperties: false
> > +
> > +...
> >
> > >
> > > Documentation/devicetree/writing-schema.rst and about 1000 examples
> > > in the kernel tree.
> > >
> > > Rob
Best Regards
Qiang Zhao

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2020-09-18  1:45 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-27  9:14 [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt Qiang Zhao
2020-08-27  9:14 ` [PATCH 2/3] rtc: pcf2127: add "no-watchdog" property Qiang Zhao
2020-08-27  9:14 ` [PATCH 3/3] arm64: dts: lx2160a: Add "no-watchdog" property to pcf2127 Qiang Zhao
2020-09-09 20:24 ` [PATCH 1/3] dt-bindings: rtc-2127: Add bindings for nxp,rtc-2127.txt Rob Herring
2020-09-10  3:16   ` Qiang Zhao
2020-09-11 14:03     ` Rob Herring
2020-09-14  7:08       ` Qiang Zhao
2020-09-14 14:36         ` Rob Herring
2020-09-15  3:26           ` Qiang Zhao
2020-09-17 19:29         ` Bruno Thomsen
2020-09-18  1:45           ` Qiang Zhao
2020-09-11 21:34 ` Alexandre Belloni

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).