[PATCH 1/2] ARM: mvebu: enable the SDHCI interface on Armada 385
Andrew Lunn
andrew at lunn.ch
Fri Apr 4 10:23:08 PDT 2014
On Fri, Apr 04, 2014 at 06:15:08PM +0200, Thomas Petazzoni wrote:
> Dear Andrew Lunn,
>
> On Fri, 4 Apr 2014 18:00:13 +0200, Andrew Lunn wrote:
>
> > > diff --git a/arch/arm/boot/dts/armada-38x.dtsi b/arch/arm/boot/dts/armada-38x.dtsi
> > > index a064f59..2c1ff30 100644
> > > --- a/arch/arm/boot/dts/armada-38x.dtsi
> > > +++ b/arch/arm/boot/dts/armada-38x.dtsi
> > > @@ -355,6 +355,15 @@
> > > clocks = <&coredivclk 0>;
> > > status = "disabled";
> > > };
> > > +
> > > + sdhci at d8000 {
> > > + compatible = "marvell,armada-380-sdhci";
> > > + reg = <0xd8000 0x1000>, <0xdc000 0x100>;
> > > + interrupts = <0 25 0x4>;
> > > + clocks = <&gateclk 17>;
> > > + mrvl,clk-delay-cycles = <0x1F>;
> >
> > Hi Thomas
> >
> > Maybe this has been discussed as part of the previous patchset. Why
> > the mrvl prefix, not marvell? marvell is in vendor-prefixes.txt, but
> > mrvl does not appear to be listed.
>
> Because I'm not the one who introduced this part of the driver DT
> binding. It has been added two years ago by:
>
> commit b650352dd3df36164e3427bff3f33bc06ac47642
> Author: Chris Ball <cjb at laptop.org>
> Date: Tue Apr 10 22:34:33 2012 -0400
>
> mmc: sdhci-pxa: Add device tree support
>
> Tested on an OLPC XO-1.75. (MMP2, sdhci-pxav3, CONFIG_MACH_MMP2_DT=y)
>
> Signed-off-by: Chris Ball <cjb at laptop.org>
> Acked-by: Haojian Zhuang <haojian.zhuang at gmail.com>
>
> So I'm just merely using the existing DT binding for the driver. Of
> course, since I'm introducing a new compatible string, I could also
> change the driver to support the same property prefixed by "marvell,",
> but it would also have to support the old DT property forever, so is it
> really useful?
Hi Thomas
Thanks for the explanation.
Changing it makes no sense. We should just document this other vendor
prefix in vendor-prefixes.txt
Andrew
More information about the linux-arm-kernel
mailing list