diff mbox series

dt-bindings: mtd: fsl-quadspi: Pass the qspi clock names

Message ID 1512340584-26894-1-git-send-email-festevam@gmail.com
State Accepted
Delegated to: Cyrille Pitchen
Headers show
Series dt-bindings: mtd: fsl-quadspi: Pass the qspi clock names | expand

Commit Message

Fabio Estevam Dec. 3, 2017, 10:36 p.m. UTC
From: Fabio Estevam <fabio.estevam@nxp.com>

In order to improve the bindings documentation, explicitly pass the name
of the clocks: "qspi_en" and "qspi", which are mandatory.

Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
---
 Documentation/devicetree/bindings/mtd/fsl-quadspi.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Cyrille Pitchen Dec. 4, 2017, 12:59 p.m. UTC | #1
Le 03/12/2017 à 23:36, Fabio Estevam a écrit :
> From: Fabio Estevam <fabio.estevam@nxp.com>
> 
> In order to improve the bindings documentation, explicitly pass the name
> of the clocks: "qspi_en" and "qspi", which are mandatory.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>

Acked-by: Cyrille Pitchen <cyrille.pitchen@wedev4u.fr>

> ---
>  Documentation/devicetree/bindings/mtd/fsl-quadspi.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/mtd/fsl-quadspi.txt b/Documentation/devicetree/bindings/mtd/fsl-quadspi.txt
> index c34aa6f..cc3f579 100644
> --- a/Documentation/devicetree/bindings/mtd/fsl-quadspi.txt
> +++ b/Documentation/devicetree/bindings/mtd/fsl-quadspi.txt
> @@ -12,7 +12,7 @@ Required properties:
>    - reg-names: Should contain the reg names "QuadSPI" and "QuadSPI-memory"
>    - interrupts : Should contain the interrupt for the device
>    - clocks : The clocks needed by the QuadSPI controller
> -  - clock-names : the name of the clocks
> +  - clock-names : Should contain the name of the clocks: "qspi_en" and "qspi".
>  
>  Optional properties:
>    - fsl,qspi-has-second-chip: The controller has two buses, bus A and bus B.
>
Rob Herring (Arm) Dec. 4, 2017, 10:36 p.m. UTC | #2
On Sun, Dec 03, 2017 at 08:36:24PM -0200, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@nxp.com>
> 
> In order to improve the bindings documentation, explicitly pass the name
> of the clocks: "qspi_en" and "qspi", which are mandatory.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
> ---
>  Documentation/devicetree/bindings/mtd/fsl-quadspi.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Rob Herring <robh@kernel.org>
Cyrille Pitchen Dec. 5, 2017, 8:11 p.m. UTC | #3
Le 04/12/2017 à 23:36, Rob Herring a écrit :
> On Sun, Dec 03, 2017 at 08:36:24PM -0200, Fabio Estevam wrote:
>> From: Fabio Estevam <fabio.estevam@nxp.com>
>>
>> In order to improve the bindings documentation, explicitly pass the name
>> of the clocks: "qspi_en" and "qspi", which are mandatory.
>>
>> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
>> ---
>>  Documentation/devicetree/bindings/mtd/fsl-quadspi.txt | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Reviewed-by: Rob Herring <robh@kernel.org>
> 
Applied to the spi-nor/next branch of l2-mtd

Thanks!
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/mtd/fsl-quadspi.txt b/Documentation/devicetree/bindings/mtd/fsl-quadspi.txt
index c34aa6f..cc3f579 100644
--- a/Documentation/devicetree/bindings/mtd/fsl-quadspi.txt
+++ b/Documentation/devicetree/bindings/mtd/fsl-quadspi.txt
@@ -12,7 +12,7 @@  Required properties:
   - reg-names: Should contain the reg names "QuadSPI" and "QuadSPI-memory"
   - interrupts : Should contain the interrupt for the device
   - clocks : The clocks needed by the QuadSPI controller
-  - clock-names : the name of the clocks
+  - clock-names : Should contain the name of the clocks: "qspi_en" and "qspi".
 
 Optional properties:
   - fsl,qspi-has-second-chip: The controller has two buses, bus A and bus B.