[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240517203621.72b8b9c7@xps-13>
Date: Fri, 17 May 2024 20:36:21 +0200
From: Miquel Raynal <miquel.raynal@...tlin.com>
To: Frank Li <Frank.Li@....com>
Cc: Richard Weinberger <richard@....at>, Vignesh Raghavendra
<vigneshr@...com>, Rob Herring <robh@...nel.org>, Krzysztof Kozlowski
<krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, Han Xu
<han.xu@....com>, Vinod Koul <vkoul@...nel.org>, Shawn Guo
<shawnguo@...nel.org>, Sascha Hauer <s.hauer@...gutronix.de>, Pengutronix
Kernel Team <kernel@...gutronix.de>, Fabio Estevam <festevam@...il.com>,
Marek Vasut <marex@...x.de>, linux-mtd@...ts.infradead.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
dmaengine@...r.kernel.org, imx@...ts.linux.dev,
linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH 1/5] dt-bindings: mtd: gpmi-nand: Add
'fsl,imx8qxp-gpmi-nand' compatible string
Hi Frank,
Frank.Li@....com wrote on Fri, 17 May 2024 14:09:48 -0400:
> Add 'fsl,imx8qxp-gpmi-nand' compatible string and clock-names restriction.
>
> Signed-off-by: Frank Li <Frank.Li@....com>
> ---
> .../devicetree/bindings/mtd/gpmi-nand.yaml | 22 ++++++++++++++++++++++
> 1 file changed, 22 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/mtd/gpmi-nand.yaml b/Documentation/devicetree/bindings/mtd/gpmi-nand.yaml
> index 021c0da0b072f..f9eb1868ca1f4 100644
> --- a/Documentation/devicetree/bindings/mtd/gpmi-nand.yaml
> +++ b/Documentation/devicetree/bindings/mtd/gpmi-nand.yaml
> @@ -24,6 +24,7 @@ properties:
> - fsl,imx6q-gpmi-nand
> - fsl,imx6sx-gpmi-nand
> - fsl,imx7d-gpmi-nand
> + - fsl,imx8qxp-gpmi-nand
> - items:
> - enum:
> - fsl,imx8mm-gpmi-nand
> @@ -151,6 +152,27 @@ allOf:
> - const: gpmi_io
> - const: gpmi_bch_apb
>
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - fsl,imx8qxp-gpmi-nand
> + then:
> + properties:
> + clocks:
> + items:
> + - description: SoC gpmi io clock
> + - description: SoC gpmi apb clock
I believe these two clocks are mandatory?
> + - description: SoC gpmi bch clock
> + - description: SoC gpmi bch apb clock
> + clock-names:
> + items:
> + - const: gpmi_io
> + - const: gpmi_apb
> + - const: gpmi_bch
> + - const: gpmi_bch_apb
> +
> examples:
> - |
> nand-controller@...0c000 {
>
Thanks,
Miquèl
Powered by blists - more mailing lists