Re: [PATCH] [MTD] NAND: add ts7xxx driver

[Alexander Clouter <alex@digriz.org.uk>]

Hi,

* Alexander Clouter <alex@digriz.org.uk> [2009-02-08 10:52:26+0000]:
> 
> * Alexander Clouter <alex@digriz.org.uk> [2009-02-08 10:39:04+0000]:
> > 
> > * Lennert Buytenhek <buytenh@wantstofly.org> [2009-02-08 11:21:47+0100]:
> > > 
> > > On Sun, Feb 08, 2009 at 08:59:19AM +0000, Alexander Clouter wrote:
> > > >
> > > > This patch adds support for the NAND found in Technologic Systems ARM
> > > > boards[1].  The platform specific parts (IO address and parititoning
> > > > schemes) have been moved into platform specific files whilst the driver
> > > > it's self can be used as a complete replacement for the ts7250 NAND
> > > > driver.
> > > 
> > > Why don't you morph the ts7250 into the form you want instead of adding
> > > a parallel driver for the same thing?
> > >
> > 
> > [snipped rant]
> > 
> > The driver I threw together makes a rather radical change to put all the 
> > partitioning and iomem resources ranges back into the platform specific 
> > code where they belong.  It was just easier to start clean, plus I 
> > learnt a good deal about the MTD subsystem in the process, although 
> > that's not a good reason for submission I know.
> > 
> Attached is the kind of change that affects the platform code.  
> Meanwhile if I was to morph the ts7250 driver my first step would be to 
> trim a huge chunk of the platform specific code that's in the NAND 
> driver and then at the same time get the ARM platform driver amended 
> otherwise you end up with a NAND less board.
> 
Unsure if it even attached so incase it did not:

http://stuff.digriz.org.uk/ts7xxx-nand/0002--ARM-ep93xx-add-hooks-to-ts7xxx-NAND-driver.patch

Cheers