| Message ID | 20220418102040.4993-2-a-govindraju@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | MMC: Add quirk to set the TESTCD bit | expand |
On Mon, 18 Apr 2022 at 12:21, Aswath Govindraju <a-govindraju@ti.com> wrote: > > The ARASAN MMC controller on Keystone 3 class of devices needs the SDCD > line to be connected for proper functioning. Similar to the issue pointed > out in sdhci-of-arasan.c driver, commit 3794c542641f ("mmc: > sdhci-of-arasan: Set controller to test mode when no CD bit"). > > In cases where SDCD line is not connected, driver support has been added to > force the controller into test mode and set the TESTCD bit. In order to > implement this quirk the driver uses "ti,fails-without-test-cd" flag from > the device tree node. Therefore, update the bindings to document the above. Would you mind rephrasing this a bit. DT bindings is about describing the HW, not about what the software should do. Otherwise, this looks good to me. Kind regards Uffe > > Signed-off-by: Aswath Govindraju <a-govindraju@ti.com> > --- > Documentation/devicetree/bindings/mmc/sdhci-am654.yaml | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml b/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml > index 0566493c4def..0ab07759b472 100644 > --- a/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml > +++ b/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml > @@ -186,6 +186,13 @@ properties: > description: Clock Delay Buffer Select > $ref: "/schemas/types.yaml#/definitions/uint32" > > + ti,fails-without-test-cd: > + $ref: /schemas/types.yaml#/definitions/flag > + description: > + When present, indicates that the CD line is not connected > + and the controller is required to be forced into Test mode > + to set the TESTCD bit. > + > required: > - compatible > - reg > -- > 2.17.1 >
Hi Uffe, On 21/04/22 17:40, Ulf Hansson wrote: > On Mon, 18 Apr 2022 at 12:21, Aswath Govindraju <a-govindraju@ti.com> wrote: >> >> The ARASAN MMC controller on Keystone 3 class of devices needs the SDCD >> line to be connected for proper functioning. Similar to the issue pointed >> out in sdhci-of-arasan.c driver, commit 3794c542641f ("mmc: >> sdhci-of-arasan: Set controller to test mode when no CD bit"). >> >> In cases where SDCD line is not connected, driver support has been added to >> force the controller into test mode and set the TESTCD bit. In order to >> implement this quirk the driver uses "ti,fails-without-test-cd" flag from >> the device tree node. Therefore, update the bindings to document the above. > > Would you mind rephrasing this a bit. DT bindings is about describing > the HW, not about what the software should do. > Sure, will rephrase it in the respin to remove the aspects that indicate the sw support added. > Otherwise, this looks good to me. > Thank you for the review. Regards, Aswath > Kind regards > Uffe > >> >> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com> >> --- >> Documentation/devicetree/bindings/mmc/sdhci-am654.yaml | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml b/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml >> index 0566493c4def..0ab07759b472 100644 >> --- a/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml >> +++ b/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml >> @@ -186,6 +186,13 @@ properties: >> description: Clock Delay Buffer Select >> $ref: "/schemas/types.yaml#/definitions/uint32" >> >> + ti,fails-without-test-cd: >> + $ref: /schemas/types.yaml#/definitions/flag >> + description: >> + When present, indicates that the CD line is not connected >> + and the controller is required to be forced into Test mode >> + to set the TESTCD bit. >> + >> required: >> - compatible >> - reg >> -- >> 2.17.1 >>
diff --git a/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml b/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml index 0566493c4def..0ab07759b472 100644 --- a/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml +++ b/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml @@ -186,6 +186,13 @@ properties: description: Clock Delay Buffer Select $ref: "/schemas/types.yaml#/definitions/uint32" + ti,fails-without-test-cd: + $ref: /schemas/types.yaml#/definitions/flag + description: + When present, indicates that the CD line is not connected + and the controller is required to be forced into Test mode + to set the TESTCD bit. + required: - compatible - reg
The ARASAN MMC controller on Keystone 3 class of devices needs the SDCD line to be connected for proper functioning. Similar to the issue pointed out in sdhci-of-arasan.c driver, commit 3794c542641f ("mmc: sdhci-of-arasan: Set controller to test mode when no CD bit"). In cases where SDCD line is not connected, driver support has been added to force the controller into test mode and set the TESTCD bit. In order to implement this quirk the driver uses "ti,fails-without-test-cd" flag from the device tree node. Therefore, update the bindings to document the above. Signed-off-by: Aswath Govindraju <a-govindraju@ti.com> --- Documentation/devicetree/bindings/mmc/sdhci-am654.yaml | 7 +++++++ 1 file changed, 7 insertions(+)