[PATCH] mtd: nand: pxa3xx: Disable "armada370-nand" compatible support

Ezequiel Garcia ezequiel.garcia at free-electrons.com
Sat Dec 7 20:35:26 EST 2013


On Sat, Dec 07, 2013 at 06:23:15PM -0500, Jason Cooper wrote:
> > +/*
> > + * Currently, the armada370 support is incomplete and can cause the
> > + * system to crash. Disable it until all the infrastructure is in place.
> > + */
> > +#if 0
> >  	{
> >  		.compatible = "marvell,armada370-nand",
> >  		.data       = (void *)PXA3XX_NAND_VARIANT_ARMADA370,
> >  	},
> > +#endif
> 
> I prefer to just remove the lines.
> 

I'm actually fine either way. Did both (disable and remove) but since all
the code is already sitting and waiting for v3.14 thought it might be cleaner
to just "disable" it.

Brian (given you're going to take the patch): what do you think?

If we agree to remove the lines, shall we also revert the binding
documentation, removing the compatible from there as well?
-- 
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com



More information about the linux-arm-kernel mailing list