From: Andrew Morton <akpm@linux-foundation.org>
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: linux-kernel@vger.kernel.org, david-b@pacbell.net,
s.hauer@pengutronix.de,
Andrea Paterniani <a.paterniani@swapp-eng.it>
Subject: Re: [PATCH 1/2] remove i.MX SPI driver
Date: Tue, 23 Jun 2009 15:58:40 -0700 [thread overview]
Message-ID: <20090623155840.c750b70b.akpm@linux-foundation.org> (raw)
In-Reply-To: <1245308072-14392-2-git-send-email-s.hauer@pengutronix.de>
On Thu, 18 Jun 2009 08:54:31 +0200
Sascha Hauer <s.hauer@pengutronix.de> wrote:
> This driver is in a non working state at the moment and will
> be replaced by a bitbang driver which can also handle the
> newer i.MX variants
hum. How did it get into a non-working state?
>From the logs, it looks like it was working OK for Andrea Paterniani
when he patched it in April last year.
Ordinarily I'd be asking whether this replacement of one driver with
another is a 100% seamless change. But I guess that the audience for
SPI drivers are sufficiently technical to be able to handle the odd
Kconfig changes, module parameter changes, module name changes, etc.
But I do think that if there are any such user-visible changes, they
should be described in the changelog. And I think there are such
changes - the module name at least?
Is it possible and desirable to retain both drivers for a while? Would
that ease the transition? It also gives people a fallback driver to
use, if your new driver doesn't work for them. Just like eepro100.c,
which lived for five years ;)
But it's really hard to make any decisions about this because the
changelog failed to provide any details about the "non working state".
next prev parent reply other threads:[~2009-06-23 22:59 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-18 6:54 Sascha Hauer
2009-06-18 6:54 ` [PATCH 1/2] remove i.MX SPI driver Sascha Hauer
2009-06-18 6:54 ` [PATCH 2/2] SPI: Add SPI driver for most known i.MX SoCs Sascha Hauer
2009-06-23 23:11 ` Andrew Morton
2009-06-24 9:09 ` Sascha Hauer
2009-06-23 22:58 ` Andrew Morton [this message]
2009-06-24 9:24 ` [PATCH 1/2] remove i.MX SPI driver Sascha Hauer
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=20090623155840.c750b70b.akpm@linux-foundation.org \
--to=akpm@linux-foundation.org \
--cc=a.paterniani@swapp-eng.it \
--cc=david-b@pacbell.net \
--cc=linux-kernel@vger.kernel.org \
--cc=s.hauer@pengutronix.de \
/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