[PATCH v5 1/3] mtd: nand: gpio: Add DT property to automatically determine bus width

Brian Norris computersforpeace at gmail.com
Sat Nov 30 04:15:56 EST 2013


On Fri, Nov 29, 2013 at 09:25:52AM -0300, Ezequiel Garcia wrote:
> On Tue, Nov 26, 2013 at 05:23:38PM -0800, Brian Norris wrote:
> [..]
> > > 
> > > If we do resort to a new binding for auto-buswidth, it should be a
> > > generic one that all NAND drivers can use.
> 
> Why do we need yet another binding to describe something that's
> completely discoverable?

The DT property is not specified only for the sake of the flash chip
itself, but for the sake of the controller which supports it. I suppose
we've kind of overloaded its usage, but it is not entirely
auto-detectable.

> I'm working on *removing* any need to set the bus width, either from the
> driver or from the DT, so I see this patch as step backwards.

Well, I disagree with the removal ;)

> Can anyone help me understand if there's *any* valid use case where we
> want to specify a-priori the bus width, considering it's completely
> discoverable at run-time?

I think the primary use case should be to reflect a limitation in the
hardware (besides just the flash chip). It can mean that the controller
itself only supports one bus width, or that the board is only wired up
for x8, for instance.

Brian



More information about the linux-mtd mailing list