[PATCH 3/3] mmc: esdhc i.MX: Support 8bit mode
s.hauer at pengutronix.de
Tue Sep 25 03:38:08 EDT 2012
On Tue, Sep 25, 2012 at 03:33:19PM +0800, Shawn Guo wrote:
> On Tue, Sep 25, 2012 at 09:27:15AM +0200, Sascha Hauer wrote:
> > > > Optional properties:
> > > > - fsl,cd-controller : Indicate to use controller internal card detection
> > > > - fsl,wp-controller : Indicate to use controller internal write protection
> > > > +- bus-width : Maximum supported bus width. Defaults to 4 if omitted
> > > >
> > > This is a common mmc property documented in bindings/mmc/mmc.txt.
> > > Instead of duplicating the documentation, we should try to make our
> > > implementation conform to the common definition of the property.
> > It is conform to the common definition, so all I have to do is drop the
> > line duplicating the docs. That's easy ;)
> Maybe not. The common binding defines it as a required property while
> the patch implements it as an optional one.
So you want to make the code conform to the common spec which has
bus-width as a required property. Is this really worth it to add an
incompatible change to our devicetrees?
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
More information about the linux-arm-kernel