Re: [PATCH v2] dt-bindings: mmc: fsl-imx-mmc: Document the required clocks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 22 Feb 2024 at 15:39, Fabio Estevam <festevam@xxxxxxxxx> wrote:
>
> From: Fabio Estevam <festevam@xxxxxxx>
>
> The fsl-imx-mmc hardware needs two clocks to operate: ipg and per.
>
> Document these required clocks.
>
> This fixes the following schema warning:
>
> imx27-apf27dev.dtb: mmc@10014000: Unevaluated properties are not allowed ('clock-names', 'clocks' were unexpected)
>         from schema $id: http://devicetree.org/schemas/mmc/fsl-imx-mmc.yaml#
>
> Signed-off-by: Fabio Estevam <festevam@xxxxxxx>

Applied for next, thanks!

Kind regards
Uffe


> ---
> Changes since v2:
> - None. Just fixed a typo in the dt list.
>
>  .../devicetree/bindings/mmc/fsl-imx-mmc.yaml         | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/mmc/fsl-imx-mmc.yaml b/Documentation/devicetree/bindings/mmc/fsl-imx-mmc.yaml
> index 221f5bc047bd..7911316fbd6a 100644
> --- a/Documentation/devicetree/bindings/mmc/fsl-imx-mmc.yaml
> +++ b/Documentation/devicetree/bindings/mmc/fsl-imx-mmc.yaml
> @@ -24,6 +24,14 @@ properties:
>    reg:
>      maxItems: 1
>
> +  clocks:
> +    maxItems: 2
> +
> +  clock-names:
> +    items:
> +      - const: ipg
> +      - const: per
> +
>    interrupts:
>      maxItems: 1
>
> @@ -34,6 +42,8 @@ properties:
>      const: rx-tx
>
>  required:
> +  - clocks
> +  - clock-names
>    - compatible
>    - reg
>    - interrupts
> @@ -46,6 +56,8 @@ examples:
>          compatible = "fsl,imx27-mmc", "fsl,imx21-mmc";
>          reg = <0x10014000 0x1000>;
>          interrupts = <11>;
> +        clocks = <&clks 29>, <&clks 60>;
> +        clock-names = "ipg", "per";
>          dmas = <&dma 7>;
>          dma-names = "rx-tx";
>          bus-width = <4>;
> --
> 2.34.1
>




[Index of Archives]     [Linux Memonry Technology]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux