From: "Rafael J. Wysocki" <rjw@sisk.pl> To: Linus Walleij <linus.walleij@linaro.org> Cc: Mark Brown <broonie@opensource.wolfsonmicro.com>, Magnus Damm <magnus.damm@gmail.com>, Felipe Balbi <balbi@ti.com>, Dmitry Torokhov <dmitry.torokhov@gmail.com>, Benoit Cousson <b-cousson@ti.com>, Sourav Poddar <sourav.poddar@ti.com>, tony@atomide.com, linux-omap@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree-discuss@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org, linux-input@vger.kernel.org, Arnd Bergmann <arnd@arndb.de>, Grant Likely <grant.likely@secretlab.ca> Subject: Re: [PATCHv2] Input: omap4-keypad: Add pinctrl support Date: Tue, 30 Oct 2012 23:57:41 +0100 [thread overview] Message-ID: <2288906.HJczx1g3gj@vostro.rjw.lan> (raw) In-Reply-To: <CACRpkdac=ztY0vXQ8Mu6H8fiAO4aFSSF-A6O1Q==zKOKEsJegw@mail.gmail.com> On Tuesday, October 30, 2012 10:51:11 PM Linus Walleij wrote: > On Tue, Oct 30, 2012 at 7:37 PM, Mark Brown > <broonie@opensource.wolfsonmicro.com> wrote: > > > More seriously the amount of time we seem to have been spending recently > > on changes which end up requiring us to go through essentially every > > driver and add code to them (often several times) doesn't seem like > > we're doing a good job here. > > If this is your main concern you should be made aware that there > are people out there planning to supplant the existing DT probe paths > that are now being added to each and every ARM-related driver > with an ACPI probe path as ARM servers come into the picture. That's correct. > > pinctrl is really noticable because it's > > new but it's not the only thing. As a subsystem maintainer this code > > just makes me want to add new subsystem features to pull the code out of > > drivers but obviously that's not something that should be being done at > > the subsystem level. > > We did manage to drag the power/voltage domain per se out > of the AMBA bus, and recommend that people (like us) do that > business using the power domains. > > I think most people (including OMAP) have bought > into the concept of using the runtime PM framework and power > domains to control the power domain switches. > > It's this wider concept of using the loose concept "PM resource > domains" to control also clocks and pins that is at stake, and so > far the runtime PM core people (Rafael and Magnus) has not said > much so I think we need some kind of indication from them as to > what is to happen, long-term, with drivers handling their own clocks > and pins. Should it be centralized or not? If it's to be centralized it > needs to become a large piece of infrastructure refactoring and > needs the attention of Linaro and the like to happen. Well, I personally think it should be centralized somehow. I'm not quite sure how to achieve that, though. Thanks, Rafael -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center.
WARNING: multiple messages have this Message-ID (diff)
From: rjw@sisk.pl (Rafael J. Wysocki) To: linux-arm-kernel@lists.infradead.org Subject: [PATCHv2] Input: omap4-keypad: Add pinctrl support Date: Tue, 30 Oct 2012 23:57:41 +0100 [thread overview] Message-ID: <2288906.HJczx1g3gj@vostro.rjw.lan> (raw) In-Reply-To: <CACRpkdac=ztY0vXQ8Mu6H8fiAO4aFSSF-A6O1Q==zKOKEsJegw@mail.gmail.com> On Tuesday, October 30, 2012 10:51:11 PM Linus Walleij wrote: > On Tue, Oct 30, 2012 at 7:37 PM, Mark Brown > <broonie@opensource.wolfsonmicro.com> wrote: > > > More seriously the amount of time we seem to have been spending recently > > on changes which end up requiring us to go through essentially every > > driver and add code to them (often several times) doesn't seem like > > we're doing a good job here. > > If this is your main concern you should be made aware that there > are people out there planning to supplant the existing DT probe paths > that are now being added to each and every ARM-related driver > with an ACPI probe path as ARM servers come into the picture. That's correct. > > pinctrl is really noticable because it's > > new but it's not the only thing. As a subsystem maintainer this code > > just makes me want to add new subsystem features to pull the code out of > > drivers but obviously that's not something that should be being done at > > the subsystem level. > > We did manage to drag the power/voltage domain per se out > of the AMBA bus, and recommend that people (like us) do that > business using the power domains. > > I think most people (including OMAP) have bought > into the concept of using the runtime PM framework and power > domains to control the power domain switches. > > It's this wider concept of using the loose concept "PM resource > domains" to control also clocks and pins that is at stake, and so > far the runtime PM core people (Rafael and Magnus) has not said > much so I think we need some kind of indication from them as to > what is to happen, long-term, with drivers handling their own clocks > and pins. Should it be centralized or not? If it's to be centralized it > needs to become a large piece of infrastructure refactoring and > needs the attention of Linaro and the like to happen. Well, I personally think it should be centralized somehow. I'm not quite sure how to achieve that, though. Thanks, Rafael -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center.
next prev parent reply other threads:[~2012-10-30 22:53 UTC|newest] Thread overview: 162+ messages / expand[flat|nested] mbox.gz Atom feed top 2012-10-22 13:13 [PATCHv2] Input: omap4-keypad: Add pinctrl support Sourav Poddar 2012-10-22 13:13 ` Sourav Poddar 2012-10-22 13:13 ` Sourav Poddar 2012-10-22 15:50 ` Dmitry Torokhov 2012-10-22 15:50 ` Dmitry Torokhov 2012-10-23 9:13 ` Linus Walleij 2012-10-23 9:13 ` Linus Walleij 2012-10-23 9:35 ` Benoit Cousson 2012-10-23 9:35 ` Benoit Cousson 2012-10-23 9:35 ` Benoit Cousson 2012-10-23 10:04 ` Linus Walleij 2012-10-23 10:04 ` Linus Walleij 2012-10-23 10:03 ` Felipe Balbi 2012-10-23 10:03 ` Felipe Balbi 2012-10-23 10:03 ` Felipe Balbi 2012-10-23 10:23 ` Thomas Petazzoni 2012-10-23 10:23 ` Thomas Petazzoni 2012-10-23 10:29 ` Linus Walleij 2012-10-23 10:29 ` Linus Walleij 2012-10-23 10:29 ` Felipe Balbi 2012-10-23 10:29 ` Felipe Balbi 2012-10-23 10:29 ` Felipe Balbi 2012-10-23 10:45 ` Linus Walleij 2012-10-23 10:45 ` Linus Walleij 2012-10-23 10:42 ` Felipe Balbi 2012-10-23 10:42 ` Felipe Balbi 2012-10-23 10:42 ` Felipe Balbi 2012-10-23 11:11 ` Thomas Petazzoni 2012-10-23 11:11 ` Thomas Petazzoni 2012-10-23 17:02 ` Mitch Bradley 2012-10-23 17:02 ` Mitch Bradley 2012-10-23 17:20 ` Felipe Balbi 2012-10-23 17:20 ` Felipe Balbi 2012-10-23 17:20 ` Felipe Balbi 2012-10-23 17:51 ` Mitch Bradley 2012-10-23 17:51 ` Mitch Bradley 2012-10-23 17:51 ` Felipe Balbi 2012-10-23 17:51 ` Felipe Balbi 2012-10-23 17:51 ` Felipe Balbi 2012-10-23 9:18 ` Benoit Cousson 2012-10-23 9:18 ` Benoit Cousson 2012-10-23 9:18 ` Benoit Cousson 2012-10-23 20:02 ` Dmitry Torokhov 2012-10-23 20:02 ` Dmitry Torokhov 2012-10-24 8:37 ` Felipe Balbi 2012-10-24 8:37 ` Felipe Balbi 2012-10-24 8:37 ` Felipe Balbi 2012-10-24 16:14 ` Dmitry Torokhov 2012-10-24 16:14 ` Dmitry Torokhov 2012-10-24 16:51 ` Linus Walleij 2012-10-24 16:51 ` Linus Walleij 2012-10-24 17:28 ` Dmitry Torokhov 2012-10-24 17:28 ` Dmitry Torokhov 2012-10-24 18:58 ` Felipe Balbi 2012-10-24 18:58 ` Felipe Balbi 2012-10-24 18:58 ` Felipe Balbi 2012-10-25 20:59 ` Mark Brown 2012-10-25 20:59 ` Mark Brown 2012-10-25 20:59 ` Mark Brown 2012-10-26 6:20 ` Felipe Balbi 2012-10-26 6:20 ` Felipe Balbi 2012-10-26 6:20 ` Felipe Balbi 2012-10-26 16:03 ` Mark Brown 2012-10-26 16:03 ` Mark Brown 2012-10-29 19:49 ` Felipe Balbi 2012-10-29 19:49 ` Felipe Balbi 2012-10-29 19:49 ` Felipe Balbi 2012-10-30 11:24 ` Mark Brown 2012-10-30 11:24 ` Mark Brown 2012-10-30 11:49 ` Felipe Balbi 2012-10-30 11:49 ` Felipe Balbi 2012-10-30 11:49 ` Felipe Balbi 2012-10-30 14:07 ` Mark Brown 2012-10-30 14:07 ` Mark Brown 2012-10-30 14:16 ` Linus Walleij 2012-10-30 14:16 ` Linus Walleij 2012-10-30 14:54 ` Mark Brown 2012-10-30 14:54 ` Mark Brown 2012-10-30 15:16 ` Felipe Balbi 2012-10-30 15:16 ` Felipe Balbi 2012-10-30 15:16 ` Felipe Balbi 2012-10-30 15:58 ` Mark Brown 2012-10-30 15:58 ` Mark Brown 2012-10-30 17:25 ` Felipe Balbi 2012-10-30 17:25 ` Felipe Balbi 2012-10-30 17:25 ` Felipe Balbi 2012-10-30 18:20 ` Dmitry Torokhov 2012-10-30 18:20 ` Dmitry Torokhov 2012-10-30 18:48 ` Felipe Balbi 2012-10-30 18:48 ` Felipe Balbi 2012-10-30 18:48 ` Felipe Balbi 2012-10-30 18:37 ` Mark Brown 2012-10-30 18:37 ` Mark Brown 2012-10-30 21:51 ` Linus Walleij 2012-10-30 21:51 ` Linus Walleij 2012-10-30 22:57 ` Rafael J. Wysocki [this message] 2012-10-30 22:57 ` Rafael J. Wysocki 2012-11-02 18:26 ` Mark Brown 2012-11-02 18:26 ` Mark Brown 2012-10-30 14:11 ` Linus Walleij 2012-10-30 14:11 ` Linus Walleij 2012-10-28 20:12 ` Linus Walleij 2012-10-28 20:12 ` Linus Walleij 2012-10-30 11:34 ` Mark Brown 2012-10-30 11:34 ` Mark Brown 2012-10-30 11:34 ` Mark Brown 2012-10-30 14:02 ` Linus Walleij 2012-10-30 14:02 ` Linus Walleij 2012-10-30 14:37 ` Mark Brown 2012-10-30 14:37 ` Mark Brown 2012-10-31 20:10 ` Kevin Hilman 2012-10-31 20:10 ` Kevin Hilman [not found] ` <87obji8kta.fsf-1D3HCaltpLuhEniVeURVKkEOCMrvLtNR@public.gmane.org> 2012-11-01 8:54 ` Linus Walleij 2012-11-01 8:54 ` Linus Walleij 2012-11-01 8:56 ` Fwd: " Linus Walleij 2012-11-01 8:56 ` Linus Walleij 2012-11-01 11:42 ` Kevin Hilman 2012-11-01 11:42 ` Kevin Hilman 2012-11-01 13:22 ` Linus Walleij 2012-11-01 13:22 ` Linus Walleij 2012-11-01 12:07 ` Mark Brown 2012-11-01 12:07 ` Mark Brown 2012-11-01 14:01 ` Linus Walleij 2012-11-01 14:01 ` Linus Walleij 2012-11-01 14:19 ` Mark Brown 2012-11-01 14:19 ` Mark Brown 2012-11-11 12:32 ` Linus Walleij 2012-11-11 12:32 ` Linus Walleij 2012-10-31 13:19 ` Jean-Christophe PLAGNIOL-VILLARD 2012-10-31 13:19 ` Jean-Christophe PLAGNIOL-VILLARD 2012-10-24 16:52 ` Felipe Balbi 2012-10-24 16:52 ` Felipe Balbi 2012-10-24 16:52 ` Felipe Balbi 2012-10-24 17:13 ` Linus Walleij 2012-10-24 17:13 ` Linus Walleij 2012-10-24 17:34 ` Dmitry Torokhov 2012-10-24 17:34 ` Dmitry Torokhov 2012-10-24 17:46 ` Benoit Cousson 2012-10-24 17:46 ` Benoit Cousson 2012-10-24 17:46 ` Benoit Cousson 2012-10-24 12:54 ` Linus Walleij 2012-10-24 12:54 ` Linus Walleij 2012-10-24 16:18 ` Dmitry Torokhov 2012-10-24 16:18 ` Dmitry Torokhov 2012-10-24 16:57 ` Felipe Balbi 2012-10-24 16:57 ` Felipe Balbi 2012-10-24 16:57 ` Felipe Balbi 2012-10-24 17:18 ` Linus Walleij 2012-10-24 17:18 ` Linus Walleij 2012-10-24 17:58 ` Dmitry Torokhov 2012-10-24 17:58 ` Dmitry Torokhov 2012-10-24 19:10 ` Felipe Balbi 2012-10-24 19:10 ` Felipe Balbi 2012-10-24 19:10 ` Felipe Balbi 2012-10-24 19:38 ` Dmitry Torokhov 2012-10-24 19:38 ` Dmitry Torokhov 2012-10-24 19:38 ` Dmitry Torokhov 2012-10-24 19:51 ` Felipe Balbi 2012-10-24 19:51 ` Felipe Balbi 2012-10-24 19:51 ` Felipe Balbi 2012-10-24 17:01 ` Linus Walleij 2012-10-24 17:01 ` Linus Walleij
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=2288906.HJczx1g3gj@vostro.rjw.lan \ --to=rjw@sisk.pl \ --cc=arnd@arndb.de \ --cc=b-cousson@ti.com \ --cc=balbi@ti.com \ --cc=broonie@opensource.wolfsonmicro.com \ --cc=devicetree-discuss@lists.ozlabs.org \ --cc=dmitry.torokhov@gmail.com \ --cc=grant.likely@secretlab.ca \ --cc=linus.walleij@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-input@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-omap@vger.kernel.org \ --cc=magnus.damm@gmail.com \ --cc=sourav.poddar@ti.com \ --cc=tony@atomide.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: linkBe 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.