linux-kernel - Re: [PATCH 1/1] dt-bindings: mmc: Convert fsl-esdhc.txt to yaml

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Sat, 1 Jun 2024 17:25:46 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Frank Li <Frank.Li@....com>, Ulf Hansson <ulf.hansson@...aro.org>,
 Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>,
 "open list:MULTIMEDIA CARD (MMC), SECURE DIGITAL (SD) AND..."
 <linux-mmc@...r.kernel.org>,
 "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
 <devicetree@...r.kernel.org>, open list <linux-kernel@...r.kernel.org>
Cc: imx@...ts.linux.dev
Subject: Re: [PATCH 1/1] dt-bindings: mmc: Convert fsl-esdhc.txt to yaml

On 31/05/2024 21:37, Frank Li wrote:
> Convert layerscape fsl-esdhc binding doc from txt to yaml format.
> 
> Addtional change during convert:
> - deprecate "sdhci,wp-inverted", "sdhci,1-bit-only".
> - Add "reg" and "interrupts" property.
> - change example "sdhci@...00" to "mmc@...00".
> - compatible string require fsl,<chip>-esdhc followed by fsl,esdhc to match
> most existed dts file.
> 


> -};
> diff --git a/Documentation/devicetree/bindings/mmc/fsl-ls-esdhc.yaml b/Documentation/devicetree/bindings/mmc/fsl-ls-esdhc.yaml
> new file mode 100644
> index 0000000000000..cafc09c4f1234
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/mmc/fsl-ls-esdhc.yaml

Filename: fsl,esdhc.yaml

> @@ -0,0 +1,98 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2

...

> +  clock-frequency:
> +    $ref: /schemas/types.yaml#/definitions/uint32
> +    description: specifies eSDHC base clock frequency.
> +
> +  sdhci,wp-inverted:
> +    $ref: /schemas/types.yaml#/definitions/flag
> +    deprecated: true
> +    description:
> +      specifies that eSDHC controller reports
> +      inverted write-protect state; New devices should use the generic
> +      "wp-inverted" property.
> +
> +  sdhci,1-bit-only:
> +    $ref: /schemas/types.yaml#/definitions/flag
> +    deprecated: true
> +    description:
> +      specifies that a controller can only handle
> +      1-bit data transfers. New devices should use the generic
> +      "bus-width = <1>" property.
> +
> +  sdhci,auto-cmd12:
> +    $ref: /schemas/types.yaml#/definitions/flag
> +    description:
> +      specifies that a controller can only handle auto CMD12.
> +
> +  voltage-ranges:
> +    $ref: /schemas/types.yaml#/definitions/uint32-matrix
> +    items:
> +      items:
> +        - description: specifies minimum slot voltage (mV).
> +        - description: specifies maximum slot voltage (mV).
> +
> +  little-endian:
> +    $ref: /schemas/types.yaml#/definitions/flag
> +    description:
> +      If the host controller is little-endian mode, specify
> +      this property. The default endian mode is big-endian.
> +
> +required:
> +  - compatible
> +  - reg
> +  - interrupts

clock-frequency was required. Isn't it required now?

> +
> +allOf:
> +  - $ref: sdhci-common.yaml#
> +
> +unevaluatedProperties: false
> +
> +examples:
> +  - |
> +    mmc@...00 {
> +        compatible = "fsl,mpc8378-esdhc", "fsl,esdhc";
> +        reg = <0x2e000 0x1000>;
> +        interrupts = <42 0x8>;
> +        interrupt-parent = <&ipic>;
> +        /* Filled in by U-Boot */
> +        clock-frequency = <0>;

Please provide complete (final) example.

> +        voltage-ranges = <3300 3300>;
> +    };

Best regards,
Krzysztof