On 26.8.2016 01:01, Zach Brown wrote: > The sdhci controller on xilinx zynq devices will not function unless > the CD bit is provided. http://www.xilinx.com/support/answers/61064.html > In cases where it is impossible to provide the CD bit in hardware, > setting the controller to test mode and then setting inserted to true > will get the controller to function without the CD bit. > > Additionally in some situations it will make sense to use the > "broken-cd" and "non-removable" property together. > i.e the sd device is non-removable so polling doesn't make sense, but > the CD is broken and the controller will need to be in test mode to > function. > > This patch modifies the documentation to reflect those differences > from the standard behaviours. > > Signed-off-by: Zach Brown <zach.brown@xxxxxx> > --- > v2: > * improved commit messages > * removed fake-cd device property > * removed fake-cd quirk > * use broken-cd device property > * documented new usage of broken-cd > > Documentation/devicetree/bindings/mmc/arasan,sdhci.txt | 10 +++++++++- > 1 file changed, 9 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt b/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt > index 3404afa..b41c448 100644 > --- a/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt > +++ b/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt > @@ -37,6 +37,14 @@ Optional Properties: > in place we will export a clock representing the Card Clock. This clock > is expected to be consumed by our PHY. You must also specify > > +Card detection: > +The arasan controller does not work > +when the CD bit is not supplied unless in test mode. > +- broken-cd: The CD bit is not supplied and the controller will > + need to be set to test mode to function. > +- non-removable: like normal, except it can now be partnered with > + broken-cd, which will turn off polling, but keep test mode on. > + > Example: > sdhci@e0100000 { > compatible = "arasan,sdhci-8.9a"; > @@ -72,4 +80,4 @@ Example: > phy-names = "phy_arasan"; > #clock-cells = <0>; > status = "disabled"; > - }; > + } ; Probably you don't need this change. M -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html