From: Paul Mundt <lethal@linux-sh.org>
To: linux-sh@vger.kernel.org
Subject: Re: [PATCH 01/04] Driver Core: Add platform device arch data
Date: Fri, 29 May 2009 05:51:05 +0000 [thread overview]
Message-ID: <20090529055105.GD1122@linux-sh.org> (raw)
In-Reply-To: <20090527100633.29671.83531.sendpatchset@rx1.opensource.se>
On Fri, May 29, 2009 at 02:07:35PM +0900, Magnus Damm wrote:
> 2009/5/29 Rafael J. Wysocki <rjw@sisk.pl>:
> > On Wednesday 27 May 2009, Magnus Damm wrote:
> >> --- 0001/arch/Kconfig
> >> +++ work/arch/Kconfig 2009-05-15 14:51:35.000000000 +0900
> >> @@ -112,3 +112,6 @@ config HAVE_DMA_API_DEBUG
> >>
> >> ?config HAVE_DEFAULT_NO_SPIN_MUTEXES
> >> ? ? ? bool
> >> +
> >> +config HAVE_PLATFORM_DEVICE_ARCHDATA
> >
> > Hmm. ?Do we really need yet another CONFIG_ option for that?
>
> I don't think we need a total of 3 different config options for the
> platform device pm stuff, but I do think it makes sense to handle the
> platform archdata separately. Not sure which is the best way though,
> the #ifdefery is not very pretty. Maybe the best solution is just to
> drop the Kconfig stuff and add empty structures to all architecture
> versions of asm/device.h.
>
A stub in asm-generic/device.h will cover the majority of cases. That is
much cleaner than adding to the Kconfig mess.
next prev parent reply other threads:[~2009-05-29 5:51 UTC|newest]
Thread overview: 89+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-05-27 10:06 [PATCH 00/04][RFC] PM: Runtime platform device PM Magnus Damm
2009-05-27 10:06 ` [PATCH 01/04] Driver Core: Add platform device arch data Magnus Damm
2009-05-28 21:26 ` Rafael J. Wysocki
2009-05-28 21:26 ` Rafael J. Wysocki
2009-05-29 5:07 ` Magnus Damm
2009-05-29 5:07 ` Magnus Damm
2009-05-29 5:51 ` Paul Mundt
2009-05-29 5:51 ` Paul Mundt [this message]
2009-05-27 10:06 ` Magnus Damm
2009-05-27 10:06 ` [PATCH 02/04] Driver Core: Add idle and wakeup functions Magnus Damm
2009-05-27 10:06 ` Magnus Damm
2009-05-28 21:29 ` Rafael J. Wysocki
2009-05-28 21:29 ` Rafael J. Wysocki
2009-05-29 5:10 ` Magnus Damm
2009-05-29 5:10 ` Magnus Damm
2009-06-03 9:05 ` Rafael J. Wysocki
2009-06-03 9:05 ` Rafael J. Wysocki
2009-06-05 3:26 ` Magnus Damm
2009-06-05 3:26 ` Magnus Damm
2009-06-05 20:42 ` Rafael J. Wysocki
2009-06-05 20:42 ` Rafael J. Wysocki
2009-06-09 4:22 ` Magnus Damm
2009-06-09 4:22 ` Magnus Damm
2009-06-09 23:41 ` Rafael J. Wysocki
2009-06-09 23:41 ` Rafael J. Wysocki
2009-06-10 6:03 ` Magnus Damm
2009-06-10 6:03 ` Magnus Damm
2009-06-10 8:19 ` Rafael J. Wysocki
2009-06-10 8:19 ` Rafael J. Wysocki
2009-05-27 10:06 ` [PATCH 03/04] PM: Add platform bus runtime dev_pm_ops Magnus Damm
2009-05-27 10:06 ` Magnus Damm
2009-05-29 23:23 ` Rafael J. Wysocki
2009-06-02 13:37 ` Magnus Damm
2009-05-29 23:23 ` Rafael J. Wysocki
2009-06-02 13:37 ` Magnus Damm
2009-06-03 9:47 ` Rafael J. Wysocki
2009-06-05 10:40 ` Magnus Damm
2009-06-05 10:40 ` Magnus Damm
2009-06-05 21:24 ` Rafael J. Wysocki
2009-06-05 21:24 ` Rafael J. Wysocki
2009-05-27 10:06 ` [PATCH 04/04] sh: Runtime platform device PM mockup Magnus Damm
2009-05-27 12:10 ` [linux-pm] [PATCH 00/04][RFC] PM: Runtime platform device PM Mark Brown
2009-05-28 6:02 ` Magnus Damm
2009-05-27 12:10 ` Mark Brown
2009-05-27 14:30 ` Alan Stern
2009-05-28 6:14 ` Magnus Damm
2009-05-27 14:30 ` Alan Stern
2009-05-28 0:32 ` Kevin Hilman
2009-05-28 0:32 ` Kevin Hilman
2009-05-28 6:02 ` [linux-pm] " Magnus Damm
2009-05-28 6:14 ` Magnus Damm
2009-05-28 7:12 ` Rafael J. Wysocki
2009-05-28 15:33 ` Alan Stern
2009-05-28 7:12 ` Rafael J. Wysocki
2009-05-28 15:28 ` Alan Stern
2009-05-28 15:28 ` Alan Stern
2009-06-01 19:04 ` Rafael J. Wysocki
2009-05-28 15:33 ` Alan Stern
2009-05-29 7:41 ` Magnus Damm
2009-05-28 17:14 ` Kevin Hilman
2009-05-29 9:17 ` Magnus Damm
2009-05-28 17:14 ` Kevin Hilman
2009-05-29 7:41 ` Magnus Damm
2009-05-29 13:45 ` Alan Stern
2009-05-29 13:45 ` Alan Stern
2009-05-29 18:18 ` Rafael J. Wysocki
2009-05-29 9:17 ` Magnus Damm
2009-06-02 21:37 ` Pavel Machek
2009-05-29 18:18 ` Rafael J. Wysocki
2009-06-02 13:44 ` Magnus Damm
2009-06-01 19:04 ` Rafael J. Wysocki
2009-06-01 19:31 ` Alan Stern
2009-06-01 19:31 ` Alan Stern
2009-06-01 19:58 ` Rafael J. Wysocki
2009-06-01 19:58 ` Rafael J. Wysocki
2009-06-01 22:16 ` Alan Stern
2009-06-01 22:16 ` Alan Stern
2009-06-01 23:21 ` Rafael J. Wysocki
2009-06-01 23:21 ` Rafael J. Wysocki
2009-06-02 14:51 ` Alan Stern
2009-06-02 13:44 ` Magnus Damm
2009-06-02 14:51 ` Alan Stern
2009-06-04 16:30 ` Rafael J. Wysocki
2009-06-02 21:37 ` [linux-pm] " Pavel Machek
2009-06-04 10:03 ` Magnus Damm
2009-06-04 10:03 ` [linux-pm] " Magnus Damm
2009-06-04 16:30 ` Rafael J. Wysocki
2009-07-18 11:49 ` Pavel Machek
2009-07-18 11:49 ` [linux-pm] " Pavel Machek
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=20090529055105.GD1122@linux-sh.org \
--to=lethal@linux-sh.org \
--cc=linux-sh@vger.kernel.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.