On Wed, 20 Aug 2008 20:37:19 -0400 "Josh Boyer" <[EMAIL PROTECTED]> wrote:
> On Thu, 21 Aug 2008 00:40:58 +0200 > Arnd Bergmann <[EMAIL PROTECTED]> wrote: > > > On Wednesday 20 August 2008, Sebastian Siewior wrote: > > > I didn't convert the NDFC driver to support OF because there are > > > non-OF-aware platforms with the ndfc chip. > > > All settings are mandatory except the oob layout. > > > > Are you aware of Sean's patch from > > http://patchwork.ozlabs.org/linuxppc/patch?id=20183 ? > > I need to look at that hard for .28 still. Please do, I know it works, but I am not sure it is correct. I haven't had a chance to really look at the new driver from Sebastian, but it is more complete than mine. It has to be decided if that is a good thing. I modeled my driver after the existing nand of drivers. They also limit, even more than mine, what can be configured. My gut feel is that it is better to let the upper level nand drivers default correctly based on the chip. For example, on the warp we need to let the ecc layout default since then it can handle the differences between 64M and 256M nand chips without a fixup at boot time. (I believe Sebastian's driver *does* allow this) I also believe that some fields are now obsolete when going to an of platform driver. For example, chip select is part of the reg entry. So my goal was to configure as little as possible and then add as needed. Cheers, Sean _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev
