From: Krzysztof Kozlowski <krzk@kernel.org> To: Roger Quadros <rogerq@kernel.org>, miquel.raynal@bootlin.com, robh+dt@kernel.org Cc: richard@nod.at, vigneshr@ti.com, kishon@ti.com, nm@ti.com, linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/3] dt-bindings: mtd: ti,elm: Convert to yaml Date: Thu, 24 Mar 2022 19:24:53 +0100 [thread overview] Message-ID: <622e5de3-12b3-0f5f-1c1b-6e062c39a626@kernel.org> (raw) In-Reply-To: <20220323111851.4699-2-rogerq@kernel.org> On 23/03/2022 12:18, Roger Quadros wrote: > Convert elm.txt to ti,elm.yaml. > > Signed-off-by: Roger Quadros <rogerq@kernel.org> > --- > Documentation/devicetree/bindings/mtd/elm.txt | 16 ------- > .../devicetree/bindings/mtd/ti,elm.yaml | 48 +++++++++++++++++++ > 2 files changed, 48 insertions(+), 16 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/mtd/elm.txt > create mode 100644 Documentation/devicetree/bindings/mtd/ti,elm.yaml > > diff --git a/Documentation/devicetree/bindings/mtd/elm.txt b/Documentation/devicetree/bindings/mtd/elm.txt > deleted file mode 100644 > index 59ddc61c1076..000000000000 > --- a/Documentation/devicetree/bindings/mtd/elm.txt > +++ /dev/null > @@ -1,16 +0,0 @@ > -Error location module > - > -Required properties: > -- compatible: Must be "ti,am3352-elm" > -- reg: physical base address and size of the registers map. > -- interrupts: Interrupt number for the elm. > - > -Optional properties: > -- ti,hwmods: Name of the hwmod associated to the elm > - > -Example: > -elm: elm@0 { > - compatible = "ti,am3352-elm"; > - reg = <0x48080000 0x2000>; > - interrupts = <4>; > -}; > diff --git a/Documentation/devicetree/bindings/mtd/ti,elm.yaml b/Documentation/devicetree/bindings/mtd/ti,elm.yaml > new file mode 100644 > index 000000000000..24ed0fb84204 > --- /dev/null > +++ b/Documentation/devicetree/bindings/mtd/ti,elm.yaml > @@ -0,0 +1,48 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/mtd/ti,elm.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Texas Instruments Error Location Module (ELM). > + > +maintainers: > + - Roger Quadros <rogerq@kernel.org> > + > +description: > + ELM module is used together with GPMC and NAND Flash to detect > + errors and the location of the error based on BCH algorithms > + so they can be corrected if possible. > + > +properties: > + compatible: > + items: No items, just const. > + - const: ti,am3352-elm > + > + reg: > + maxItems: 1 > + > + interrupts: > + maxItems: 1 > + > + ti,hwmods: > + description: > + Name of the HWMOD associated with ELM. This is for legacy > + platforms only. > + $ref: /schemas/types.yaml#/definitions/string > + deprecated: true It was not deprecated before. Could you explain it in commit msg because it is not a conversion only. > + > +required: > + - compatible > + - reg > + - interrupts > + > +additionalProperties: false > + > +examples: > + - | > + elm: elm@0 { This is error correction, right? So generic node name "ecc"? > + compatible = "ti,am3352-elm"; > + reg = <0x0 0x2000>; > + interrupts = <4>; > + }; Best regards, Krzysztof
WARNING: multiple messages have this Message-ID (diff)
From: Krzysztof Kozlowski <krzk@kernel.org> To: Roger Quadros <rogerq@kernel.org>, miquel.raynal@bootlin.com, robh+dt@kernel.org Cc: richard@nod.at, vigneshr@ti.com, kishon@ti.com, nm@ti.com, linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/3] dt-bindings: mtd: ti,elm: Convert to yaml Date: Thu, 24 Mar 2022 19:24:53 +0100 [thread overview] Message-ID: <622e5de3-12b3-0f5f-1c1b-6e062c39a626@kernel.org> (raw) In-Reply-To: <20220323111851.4699-2-rogerq@kernel.org> On 23/03/2022 12:18, Roger Quadros wrote: > Convert elm.txt to ti,elm.yaml. > > Signed-off-by: Roger Quadros <rogerq@kernel.org> > --- > Documentation/devicetree/bindings/mtd/elm.txt | 16 ------- > .../devicetree/bindings/mtd/ti,elm.yaml | 48 +++++++++++++++++++ > 2 files changed, 48 insertions(+), 16 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/mtd/elm.txt > create mode 100644 Documentation/devicetree/bindings/mtd/ti,elm.yaml > > diff --git a/Documentation/devicetree/bindings/mtd/elm.txt b/Documentation/devicetree/bindings/mtd/elm.txt > deleted file mode 100644 > index 59ddc61c1076..000000000000 > --- a/Documentation/devicetree/bindings/mtd/elm.txt > +++ /dev/null > @@ -1,16 +0,0 @@ > -Error location module > - > -Required properties: > -- compatible: Must be "ti,am3352-elm" > -- reg: physical base address and size of the registers map. > -- interrupts: Interrupt number for the elm. > - > -Optional properties: > -- ti,hwmods: Name of the hwmod associated to the elm > - > -Example: > -elm: elm@0 { > - compatible = "ti,am3352-elm"; > - reg = <0x48080000 0x2000>; > - interrupts = <4>; > -}; > diff --git a/Documentation/devicetree/bindings/mtd/ti,elm.yaml b/Documentation/devicetree/bindings/mtd/ti,elm.yaml > new file mode 100644 > index 000000000000..24ed0fb84204 > --- /dev/null > +++ b/Documentation/devicetree/bindings/mtd/ti,elm.yaml > @@ -0,0 +1,48 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/mtd/ti,elm.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Texas Instruments Error Location Module (ELM). > + > +maintainers: > + - Roger Quadros <rogerq@kernel.org> > + > +description: > + ELM module is used together with GPMC and NAND Flash to detect > + errors and the location of the error based on BCH algorithms > + so they can be corrected if possible. > + > +properties: > + compatible: > + items: No items, just const. > + - const: ti,am3352-elm > + > + reg: > + maxItems: 1 > + > + interrupts: > + maxItems: 1 > + > + ti,hwmods: > + description: > + Name of the HWMOD associated with ELM. This is for legacy > + platforms only. > + $ref: /schemas/types.yaml#/definitions/string > + deprecated: true It was not deprecated before. Could you explain it in commit msg because it is not a conversion only. > + > +required: > + - compatible > + - reg > + - interrupts > + > +additionalProperties: false > + > +examples: > + - | > + elm: elm@0 { This is error correction, right? So generic node name "ecc"? > + compatible = "ti,am3352-elm"; > + reg = <0x0 0x2000>; > + interrupts = <4>; > + }; Best regards, Krzysztof ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/
next prev parent reply other threads:[~2022-03-24 18:25 UTC|newest] Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-03-23 11:18 [PATCH 0/3] mtd: rawnand: omap_elm: convert DT binding to yaml Roger Quadros 2022-03-23 11:18 ` Roger Quadros 2022-03-23 11:18 ` [PATCH 1/3] dt-bindings: mtd: ti,elm: Convert " Roger Quadros 2022-03-23 11:18 ` Roger Quadros 2022-03-24 18:24 ` Krzysztof Kozlowski [this message] 2022-03-24 18:24 ` Krzysztof Kozlowski 2022-03-25 12:03 ` Roger Quadros 2022-03-25 12:03 ` Roger Quadros 2022-03-23 11:18 ` [PATCH 2/3] dt-bindings: mtd: ti,elm: Add support for AM64 ELM Roger Quadros 2022-03-23 11:18 ` Roger Quadros 2022-03-24 18:26 ` Krzysztof Kozlowski 2022-03-24 18:26 ` Krzysztof Kozlowski 2022-03-25 12:05 ` Roger Quadros 2022-03-25 12:05 ` Roger Quadros 2022-03-25 12:08 ` Krzysztof Kozlowski 2022-03-25 12:08 ` Krzysztof Kozlowski 2022-03-25 12:13 ` Roger Quadros 2022-03-25 12:13 ` Roger Quadros 2022-03-25 12:27 ` Krzysztof Kozlowski 2022-03-25 12:27 ` Krzysztof Kozlowski 2022-03-23 11:18 ` [PATCH 3/3] mtd: rawnand: omap_elm: Add compatible " Roger Quadros 2022-03-23 11:18 ` Roger Quadros
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=622e5de3-12b3-0f5f-1c1b-6e062c39a626@kernel.org \ --to=krzk@kernel.org \ --cc=devicetree@vger.kernel.org \ --cc=kishon@ti.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mtd@lists.infradead.org \ --cc=miquel.raynal@bootlin.com \ --cc=nm@ti.com \ --cc=richard@nod.at \ --cc=robh+dt@kernel.org \ --cc=rogerq@kernel.org \ --cc=vigneshr@ti.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.