mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: David Woodhouse <dwmw2@infradead.org>
To: "Gao, Yunpeng" <yunpeng.gao@intel.com>
Cc: Greg KH <gregkh@suse.de>, Alan Cox <alan@lxorguk.ukuu.org.uk>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH 1/2]Intel Moorestown NAND driver patch for mainline
Date: Tue, 23 Jun 2009 12:05:11 +0100	[thread overview]
Message-ID: <1245755111.25547.2036.camel@macbook.infradead.org> (raw)
In-Reply-To: <38D9F46DFF92C54980D2F2C1E8EE313001A47964EB@pdsmsx503.ccr.corp.intel.com>

On Tue, 2009-06-23 at 18:59 +0800, Gao, Yunpeng wrote:
> 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?

None whatsoever.

There is absolutely no reason why integrating properly with the MTD
subsystem would prevent you from retaining on-medium compatibility with
the firmware. We manage it for DiskOnChip devices, for example.

-- 
David Woodhouse                            Open Source Technology Centre
David.Woodhouse@intel.com                              Intel Corporation


      reply	other threads:[~2009-06-23 11:05 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-06 16:44 Yunpeng Gao
2009-06-06 11:11 ` Jiri Slaby
2009-06-06 14:57   ` Greg KH
2009-06-08  3:34     ` Gao, Yunpeng
2009-06-08  3:08   ` Gao, Yunpeng
2009-06-06 15:04 ` Alan Cox
2009-06-08  4:23   ` Gao, Yunpeng
2009-06-08 10:47     ` David Woodhouse
2009-06-09  0:38       ` Greg KH
2009-06-09 10:48         ` Yunpeng Gao
2009-06-23 10:59         ` Gao, Yunpeng
2009-06-23 11:05           ` David Woodhouse [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1245755111.25547.2036.camel@macbook.infradead.org \
    --to=dwmw2@infradead.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=gregkh@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=yunpeng.gao@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®