[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ebca0e5d-cc24-af1c-5308-8544682984ec@kernel.org>
Date: Fri, 25 Mar 2022 17:22:07 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Roger Quadros <rogerq@...nel.org>, miquel.raynal@...tlin.com,
robh+dt@...nel.org
Cc: richard@....at, vigneshr@...com, kishon@...com, nm@...com,
linux-mtd@...ts.infradead.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 2/3] dt-bindings: mtd: ti,elm: Add support for AM64 ELM
On 25/03/2022 13:37, Roger Quadros wrote:
> TI's AM64 SoC has the Error Locator Module. Add compatible and
> related properties to support ELM on AM64 SoC.
>
> Signed-off-by: Roger Quadros <rogerq@...nel.org>
> ---
> .../devicetree/bindings/mtd/ti,elm.yaml | 28 ++++++++++++++++++-
> 1 file changed, 27 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/mtd/ti,elm.yaml b/Documentation/devicetree/bindings/mtd/ti,elm.yaml
> index e36452b742c0..967c87f32c71 100644
> --- a/Documentation/devicetree/bindings/mtd/ti,elm.yaml
> +++ b/Documentation/devicetree/bindings/mtd/ti,elm.yaml
> @@ -16,7 +16,10 @@ description:
>
> properties:
> compatible:
> - const: ti,am3352-elm
> + items:
Why do the items appeared again?
Make it an enum in first patch, without items, with one entry. Now in
patch #2 you add one more compatible.
Best regards,
Krzysztof
Powered by blists - more mailing lists