diff mbox series

[v2,2/2] dt-bindings: mmc: sdhci-am654: Add ti,suppress-v1p8-ena

Message ID 20250417182652.3521104-3-jm@ti.com
State New
Headers show
Series Fix V1P8_SIGNAL_ENA | expand

Commit Message

Judith Mendez April 17, 2025, 6:26 p.m. UTC
This patch documents ti,suppress-v1p8-ena which is a flag
used to suppress V1P8_SIGNAL_ENA in sdhci_am654 driver. This
quirk is necessary to fix fail init issues across various
types of SD cards tested on Sitara K3 SoCs.

Signed-off-by: Judith Mendez <jm@ti.com>
---
 Documentation/devicetree/bindings/mmc/sdhci-am654.yaml | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Francesco Dolcini April 17, 2025, 7:19 p.m. UTC | #1
Hello Judith,
thanks for the patch.

On Thu, Apr 17, 2025 at 01:26:52PM -0500, Judith Mendez wrote:
> This patch documents ti,suppress-v1p8-ena which is a flag
> used to suppress V1P8_SIGNAL_ENA in sdhci_am654 driver. This
> quirk is necessary to fix fail init issues across various
> types of SD cards tested on Sitara K3 SoCs.

bindings are supposed to describe the hardware, not the driver.

You should rephrase the commit message and the description of the property with
this in mind.

In addition, I think that the dt-bindings is supposed to be before the driver
patch in the series.
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml b/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml
index 676a746953893..0f92bbf8e13b3 100644
--- a/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml
+++ b/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml
@@ -201,6 +201,11 @@  properties:
       and the controller is required to be forced into Test mode
       to set the TESTCD bit.
 
+  ti,suppress-v1p8-ena:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description:
+      When present, V1P8_SIGNAL_ENA shall be suppressed.
+
 required:
   - compatible
   - reg