From: u.kleine-koenig@pengutronix.de (Uwe Kleine-König)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] Add platform driver support to the CS890x driver
Date: Mon, 12 Sep 2011 14:30:23 +0200 [thread overview]
Message-ID: <20110912123023.GJ28624@pengutronix.de> (raw)
In-Reply-To: <CAGzjT4ce0gZ2e6cM1PfUb9JkZ=52PUCjPz1C9qjjRYxgSJckbQ@mail.gmail.com>
Hello Jaccon,
On Mon, Sep 12, 2011 at 12:52:52PM +0200, Jaccon Bastiaansen wrote:
> 2011/9/11 Uwe Kleine-K?nig <u.kleine-koenig@pengutronix.de>:
> > On some machines (e.g arm/imx/mx31ads) some magic constants
> > are defined somewhere in arch/ to fill in netcard_portlist and
> > cs8900_irq_map. On these machines the cs8900 driver does drive some
> > hardware, but not via the platform bus. This is what i want to have
> > changed.
> >
>
> Ok, so you would like to see the #ifdefs for the IXDP2351, IXDP2X01,
> QQ2440 and MX31ADS in the cs890x driver replaced by platform_device
> definitions in the platform specific code of those platforms (in the
> same way as I have done for the i.MX21ADS platform)? Is this correct?
More or less, yes. I would have started to keep the change local to
the cs8900 driver, but moving the devices to arch is the final goal.
The upside of first reworking the driver and only then move the devices
out is that it might be better/easier to bisect and review.
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-K?nig |
Industrial Linux Solutions | http://www.pengutronix.de/ |
next prev parent reply other threads:[~2011-09-12 12:30 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-07 10:22 [PATCH] Add platform driver support to the CS890x driver Jaccon Bastiaansen
2011-09-07 12:50 ` Uwe Kleine-König
2011-09-10 11:37 ` Jaccon Bastiaansen
2011-09-10 14:12 ` Uwe Kleine-König
2011-09-11 17:34 ` Jaccon Bastiaansen
2011-09-11 18:53 ` Uwe Kleine-König
2011-09-12 10:52 ` Jaccon Bastiaansen
2011-09-12 12:30 ` Uwe Kleine-König [this message]
[not found] ` <20110928083048.GA15311@glitch.intra.local>
2011-09-30 9:01 ` Jaccon Bastiaansen
2011-10-01 17:13 ` Russell King - ARM Linux
2011-10-12 14:28 ` Domenico Andreoli
2011-09-13 7:44 ` Sascha Hauer
2011-09-13 7:46 ` Uwe Kleine-König
2011-09-27 18:27 ` [PATCH V2] " Jaccon Bastiaansen
2011-10-09 20:51 ` Jaccon Bastiaansen
2011-11-03 8:06 ` Sascha Hauer
2011-11-07 12:14 ` Jaccon Bastiaansen
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=20110912123023.GJ28624@pengutronix.de \
--to=u.kleine-koenig@pengutronix.de \
--cc=linux-arm-kernel@lists.infradead.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.