From: "Gao, Yunpeng" <yunpeng.gao@intel.com>
To: Greg KH <gregkh@suse.de>, David Woodhouse <dwmw2@infradead.org>
CC: Alan Cox <alan@lxorguk.ukuu.org.uk>,
       "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Date: Tue, 23 Jun 2009 18:59:24 +0800
Subject: RE: [PATCH 1/2]Intel Moorestown NAND driver patch for mainline
Thread-Topic: [PATCH 1/2]Intel Moorestown NAND driver patch for mainline
Thread-Index: Acnom9fsbC6nmM8RQLmVFIrKq5NfOwLTVFEA
Message-ID: <38D9F46DFF92C54980D2F2C1E8EE313001A47964EB@pdsmsx503.ccr.corp.intel.com>
References: <20090606164455.GB27257@intel.com>
 <20090606160413.08c254cf@lxorguk.ukuu.org.uk>
 <38D9F46DFF92C54980D2F2C1E8EE313001A45F4F3C@pdsmsx503.ccr.corp.intel.com>
 <1244458030.30939.216.camel@macbook.infradead.org>
 <20090609003839.GB22766@suse.de>
In-Reply-To: <20090609003839.GB22766@suse.de>
Accept-Language: en-US
Content-Language: en-US
acceptlanguage: en-US
Content-Type: text/plain; charset="gb2312"
MIME-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
Content-Transfer-Encoding: 8bit
Content-Length: 2159
Lines: 48


>-----Original Message-----
>From: Greg KH [mailto:gregkh@suse.de]
>Sent: 2009??6??9?? 8:39
>To: David Woodhouse
>Cc: Gao, Yunpeng; Alan Cox; linux-kernel@vger.kernel.org
>Subject: Re: [PATCH 1/2]Intel Moorestown NAND driver patch for mainline
>
>On Mon, Jun 08, 2009 at 11:47:10AM +0100, David Woodhouse wrote:
>> On Mon, 2009-06-08 at 12:23 +0800, Gao, Yunpeng wrote:
>> > The Moorestown NAND flash driver is licensed from the Moorestown NAND
>> > controller vendor. It's a standalone NAND driver. According to the
>> > vendor, this driver has some advanced features specific to the
>> > controller hardware and is difficult to port to MTD subsystem.
>>
>> I don't believe it. If the MTD subsystem doesn't support certain
>> features, the MTD subsystem needs to be updated to cope.
>
>Based on this response, I am guessing that you do not want me to take
>this in the staging tree? :)
>
>If so, please let me know and I'll drop it.
>
>thanks,
>
>greg k-h

Hi Greg, David and Alan,

I have discussed with the architect of Moorestown NAND driver today, and we agreed that the Moorestown NAND driver can not be ported to MTD subsystem.
The reason is: 
The NAND driver has to keep compatibility with Moorestown IA Firmware (which also operates on NAND device and is close source), and if the NAND driver is ported to MTD, the driver has to be changed much and thus will fail to keep the compatibility with Firmware.

As this is a standalone NAND driver, I think I can re-submit it to the block subsystem (maintained by Jens Axboe). Thus the NAND driver need not to change its architecture and can keep the compatibility with Firmware.

Also, it'll be better if the NAND driver can be in Greg's staging tree before the driver becomes 'good enough' and accepted by the block subsystem. Because some customers and partners want to get the latest NAND driver code from the kernel tree before the driver is accepted finally.

Does this suggestion make sense?

Thanks for your time.

Rgds,
Yunpeng Gao


????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m????????????I?