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

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

Hi,

* 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?
>
...because when I sent this to you last year for some offlist comment 
action you did not grumble that it was a separate driver.  Then after a 
year of reposting twice to this list with no comment or feedback what so 
ever I guess I made the mistake of assuming it was not 'displeasing' to 
the community.

Sorry to sound grumpy, but after trying to post cpufreq, timeriomem, 
nand and arm patches over the past two years with a 90% resulting in the 
silent treatment it feels like a brick has hit me in the face with "hey 
instead just morph the current driver".

Bah.

</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.

As the feedback from ts72[056]0 users has been pretty...well...dire and 
I have no access to the platform I cannot be certain this patch safely 
works for those users.  I had the view that once I can get a solid 
confirmation that the ts72[056]0 users are happy I was hoping that the 
7250.c driver would just become obsolete?

So, is it back to square one for me?

Cheers

-- 
Alexander Clouter
.sigmonster says: You will never know hunger.