| Message ID | 20170807160142.12134-6-kishon@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
* Kishon Vijay Abraham I <kishon@ti.com> [170807 09:03]: > Document the new compatible string "ti,dra7-sdhci" to be used for > MMC controllers in DRA7 and DRA72 SoCs. I wonder if this should really be documented for sdhci instead of ti-omap-hsmmc.txt? I agree it's better to use sdhci compatible and keep the hsmmc compatible around as the sdhci is still missing features for runtime PM and context save and restore etc. Regards, Tony > Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com> > --- > Documentation/devicetree/bindings/mmc/ti-omap-hsmmc.txt | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/mmc/ti-omap-hsmmc.txt b/Documentation/devicetree/bindings/mmc/ti-omap-hsmmc.txt > index 0e026c151c1c..db80fdfd05d7 100644 > --- a/Documentation/devicetree/bindings/mmc/ti-omap-hsmmc.txt > +++ b/Documentation/devicetree/bindings/mmc/ti-omap-hsmmc.txt > @@ -13,6 +13,7 @@ Required properties: > Should be "ti,omap3-pre-es3-hsmmc" for OMAP3 controllers pre ES3.0 > Should be "ti,omap4-hsmmc", for OMAP4 controllers > Should be "ti,am33xx-hsmmc", for AM335x controllers > + Should be "ti,dra7-sdhci" for DRA7 and DRA72 controllers > - ti,hwmods: Must be "mmc<n>", n is controller instance starting 1 > > Optional properties: > -- > 2.11.0 > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Tony, On Thursday 10 August 2017 03:42 AM, Tony Lindgren wrote: > * Kishon Vijay Abraham I <kishon@ti.com> [170807 09:03]: >> Document the new compatible string "ti,dra7-sdhci" to be used for >> MMC controllers in DRA7 and DRA72 SoCs. > > I wonder if this should really be documented for sdhci > instead of ti-omap-hsmmc.txt? hmm.. yeah, having a separate binding document for sdhci would make the binding really clean and we also don't have to carry legacy binding code. But we'll be never able to remove omap-hsmmc driver then and end up maintaining 2 drivers for the same controller. > > I agree it's better to use sdhci compatible and keep the > hsmmc compatible around as the sdhci is still missing features > for runtime PM and context save and restore etc. yeah, once we have all the features in omap_hsmmc added in sdhci-omap, we can add the omap-hsmmc compatible to sdhci-omap and remove omap-hsmmc driver. Thanks Kishon -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
* Kishon Vijay Abraham I <kishon@ti.com> [170816 22:44]: > Hi Tony, > > On Thursday 10 August 2017 03:42 AM, Tony Lindgren wrote: > > * Kishon Vijay Abraham I <kishon@ti.com> [170807 09:03]: > >> Document the new compatible string "ti,dra7-sdhci" to be used for > >> MMC controllers in DRA7 and DRA72 SoCs. > > > > I wonder if this should really be documented for sdhci > > instead of ti-omap-hsmmc.txt? > > hmm.. yeah, having a separate binding document for sdhci would make the binding > really clean and we also don't have to carry legacy binding code. But we'll be > never able to remove omap-hsmmc driver then and end up maintaining 2 drivers > for the same controller. Well I think it's best to be able to enable sdhci one board at a time after testing. Then eventually when fully supported, we can just have sdhci also parse the legacy omap binding and just drop the omap-hsmmc driver. Regards, Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/devicetree/bindings/mmc/ti-omap-hsmmc.txt b/Documentation/devicetree/bindings/mmc/ti-omap-hsmmc.txt index 0e026c151c1c..db80fdfd05d7 100644 --- a/Documentation/devicetree/bindings/mmc/ti-omap-hsmmc.txt +++ b/Documentation/devicetree/bindings/mmc/ti-omap-hsmmc.txt @@ -13,6 +13,7 @@ Required properties: Should be "ti,omap3-pre-es3-hsmmc" for OMAP3 controllers pre ES3.0 Should be "ti,omap4-hsmmc", for OMAP4 controllers Should be "ti,am33xx-hsmmc", for AM335x controllers + Should be "ti,dra7-sdhci" for DRA7 and DRA72 controllers - ti,hwmods: Must be "mmc<n>", n is controller instance starting 1 Optional properties:
Document the new compatible string "ti,dra7-sdhci" to be used for MMC controllers in DRA7 and DRA72 SoCs. Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com> --- Documentation/devicetree/bindings/mmc/ti-omap-hsmmc.txt | 1 + 1 file changed, 1 insertion(+)