[PATCH v3 25/28] ARM: mvebu: Add support for NAND controller in Armada 370/XP
Ezequiel Garcia
ezequiel.garcia at free-electrons.com
Tue Nov 5 08:51:46 EST 2013
On Tue, Nov 05, 2013 at 08:29:05AM -0500, Jason Cooper wrote:
> On Tue, Nov 05, 2013 at 09:55:32AM -0300, Ezequiel Garcia wrote:
> > The Armada 370 and Armada XP SoC have a NAND controller (aka NFCv2).
> > This commit adds support for it in Armada 370 and Armada XP SoC
> > common devicetree.
> >
> > Signed-off-by: Ezequiel Garcia <ezequiel.garcia at free-electrons.com>
> > ---
> > arch/arm/boot/dts/armada-370-xp.dtsi | 9 +++++++++
> > 1 file changed, 9 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/armada-370-xp.dtsi b/arch/arm/boot/dts/armada-370-xp.dtsi
> > index 01e69fc..b4e6898 100644
> > --- a/arch/arm/boot/dts/armada-370-xp.dtsi
> > +++ b/arch/arm/boot/dts/armada-370-xp.dtsi
> > @@ -258,6 +258,15 @@
> > status = "disabled";
> > };
> >
> > + nand at d0000 {
> > + compatible = "marvell,armada370-nand";
>
> Could you please provide a separate patch updating the devicetree
> binding documentation? You can also Cc the entire series to the
> devicetree ml as long as the documentation patch is easy to find in the
> series. eg 'dt: binding: ...'
>
Hm.. actually the controller already supports the new compatible string
so the binding documentation should be added now.
And I'd rather do that in a separate patch, to avoid cluttering the poor
devicetree people with an unrelated 28-piece patch :-)
BTW: who should take such a patch? I'm still a little lost regarding
who takes the binding or dts patches for a given subsystem.
--
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com
More information about the linux-arm-kernel
mailing list