From: Geert Uytterhoeven <geert@linux-m68k.org> To: Ulf Hansson <ulf.hansson@linaro.org> Cc: "Rafael J. Wysocki" <rafael@kernel.org>, Linus Torvalds <torvalds@linux-foundation.org>, Arnd Bergmann <arnd@arndb.de>, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, Olof Johansson <olof@lixom.net>, soc@kernel.org, linux-arm-kernel@lists.infradead.org, Sebastian Reichel <sebastian.reichel@collabora.com> Subject: Re: [GIT PULL] ARM: SoC/genpd driver updates for v6.6 Date: Mon, 11 Sep 2023 13:48:29 +0200 [thread overview] Message-ID: <CAMuHMdUZcrtHZz7daKjL7Oz2MEmCrjJ_ovO3ttTnZ7xuRWgiRA@mail.gmail.com> (raw) In-Reply-To: <CAPDyKFpUUJeGD77GTYCk59cvW=SdGTkOEXH4dap1JQWekAa0Eg@mail.gmail.com> Hi Ulf, On Mon, Sep 11, 2023 at 1:28 PM Ulf Hansson <ulf.hansson@linaro.org> wrote: > On Mon, 11 Sept 2023 at 09:52, Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > On Thu, Aug 31, 2023 at 1:39 PM Ulf Hansson <ulf.hansson@linaro.org> wrote: > > > On Thu, 31 Aug 2023 at 11:33, Rafael J. Wysocki <rafael@kernel.org> wrote: > > > > If I may suggest something, I would call this "pmdomain" instead of > > > > "genpd". I don't think that /drivers/power/ is a particularly > > > > suitable location for it, because it doesn't really have much to do > > > > with power supplies and more to do with device PM. > > > > > > "pmdomain" is probably giving a reasonable good hint of what goes on > > > in this subsystem. This works fine for me, thanks! > > > > > > Also, I would move drivers/base/power/domain.c to drivers/pmdomain/ > > > > (and rename it to something like core.c), because it would be a better > > > > location for that fiile IMO. > > > > > > We could certainly do that, let's discuss it a bit more. > > > > > > Although, at this point I want to focus on the genpd providers, as to > > > release some of the burden from arm-soc maintainers. > > > > > > > I can also handle future pull requests for this if that's fine with everyone. > > > > > > Thanks a lot for your offer! However, if a re-route is preferred (I > > > think not?), this is probably better suited via arm-soc, as most > > > changes are going to be arm platform specific. > > > > Which brings me to the final question: what is the upstream path > > for changes to drivers/genpd/*/ (or whatever it's gonna be called)? > > Before, we sent PRs to (arm-)soc. Do you expect us to send them to > > you? There's usually quite some interaction between drivers/soc/reneas/ > > and drivers/genpd/renesas (and there are DT binding definitions), > > but not more than with e.g. drivers/clk/renesas/. > > I would be happy to pick this up and funnel this via my new genpd > tree. As long as it's coupled with changes affecting "genpd > providers", of course. > > I can certainly also collect patches directly from the > mailing-list/patch-tracker too. Whatever works for you the best. Of > course, in that case I need your acks before I pick up the relevant > patches. > > If we need "immutable" branches, let's discuss that on a case by case basis. At least for Renesas SoCs, every new SoC comes with a DT binding definitions file under include/dt-bindings/power/, to be shared by genpd driver and DTS (the same is true for clocks). So PRs will work best. > > And I just realized you moved the code and Makefiles to drivers/genpd/, > > but not the Kconfig symbols and logic, which still lives under > > drivers/soc/. So resolving that (and the name) is something that > > should be resolved sooner rather than later... > > In regards to the name, I am relying on input from Linus to make a > final decision before I send a patch. In regards to this, I have also > started working on a documentation patch for genpd. It needs some more > polishing before I can send it though. > > When it comes to the Kconfig move, I will send out a series for it, this week. OK. Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
WARNING: multiple messages have this Message-ID (diff)
From: Geert Uytterhoeven <geert@linux-m68k.org> To: Ulf Hansson <ulf.hansson@linaro.org> Cc: "Rafael J. Wysocki" <rafael@kernel.org>, Linus Torvalds <torvalds@linux-foundation.org>, Arnd Bergmann <arnd@arndb.de>, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, Olof Johansson <olof@lixom.net>, soc@kernel.org, linux-arm-kernel@lists.infradead.org, Sebastian Reichel <sebastian.reichel@collabora.com> Subject: Re: [GIT PULL] ARM: SoC/genpd driver updates for v6.6 Date: Mon, 11 Sep 2023 13:48:29 +0200 [thread overview] Message-ID: <CAMuHMdUZcrtHZz7daKjL7Oz2MEmCrjJ_ovO3ttTnZ7xuRWgiRA@mail.gmail.com> (raw) In-Reply-To: <CAPDyKFpUUJeGD77GTYCk59cvW=SdGTkOEXH4dap1JQWekAa0Eg@mail.gmail.com> Hi Ulf, On Mon, Sep 11, 2023 at 1:28 PM Ulf Hansson <ulf.hansson@linaro.org> wrote: > On Mon, 11 Sept 2023 at 09:52, Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > On Thu, Aug 31, 2023 at 1:39 PM Ulf Hansson <ulf.hansson@linaro.org> wrote: > > > On Thu, 31 Aug 2023 at 11:33, Rafael J. Wysocki <rafael@kernel.org> wrote: > > > > If I may suggest something, I would call this "pmdomain" instead of > > > > "genpd". I don't think that /drivers/power/ is a particularly > > > > suitable location for it, because it doesn't really have much to do > > > > with power supplies and more to do with device PM. > > > > > > "pmdomain" is probably giving a reasonable good hint of what goes on > > > in this subsystem. This works fine for me, thanks! > > > > > > Also, I would move drivers/base/power/domain.c to drivers/pmdomain/ > > > > (and rename it to something like core.c), because it would be a better > > > > location for that fiile IMO. > > > > > > We could certainly do that, let's discuss it a bit more. > > > > > > Although, at this point I want to focus on the genpd providers, as to > > > release some of the burden from arm-soc maintainers. > > > > > > > I can also handle future pull requests for this if that's fine with everyone. > > > > > > Thanks a lot for your offer! However, if a re-route is preferred (I > > > think not?), this is probably better suited via arm-soc, as most > > > changes are going to be arm platform specific. > > > > Which brings me to the final question: what is the upstream path > > for changes to drivers/genpd/*/ (or whatever it's gonna be called)? > > Before, we sent PRs to (arm-)soc. Do you expect us to send them to > > you? There's usually quite some interaction between drivers/soc/reneas/ > > and drivers/genpd/renesas (and there are DT binding definitions), > > but not more than with e.g. drivers/clk/renesas/. > > I would be happy to pick this up and funnel this via my new genpd > tree. As long as it's coupled with changes affecting "genpd > providers", of course. > > I can certainly also collect patches directly from the > mailing-list/patch-tracker too. Whatever works for you the best. Of > course, in that case I need your acks before I pick up the relevant > patches. > > If we need "immutable" branches, let's discuss that on a case by case basis. At least for Renesas SoCs, every new SoC comes with a DT binding definitions file under include/dt-bindings/power/, to be shared by genpd driver and DTS (the same is true for clocks). So PRs will work best. > > And I just realized you moved the code and Makefiles to drivers/genpd/, > > but not the Kconfig symbols and logic, which still lives under > > drivers/soc/. So resolving that (and the name) is something that > > should be resolved sooner rather than later... > > In regards to the name, I am relying on input from Linus to make a > final decision before I send a patch. In regards to this, I have also > started working on a documentation patch for genpd. It needs some more > polishing before I can send it though. > > When it comes to the Kconfig move, I will send out a series for it, this week. OK. Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2023-09-11 20:53 UTC|newest] Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-08-29 21:34 [GIT PULL] ARM: SoC/genpd driver updates for v6.6 Ulf Hansson 2023-08-29 21:34 ` Ulf Hansson 2023-08-30 0:18 ` Linus Torvalds 2023-08-30 0:18 ` Linus Torvalds 2023-08-30 0:31 ` Linus Torvalds 2023-08-30 0:31 ` Linus Torvalds 2023-08-30 0:47 ` Arnd Bergmann 2023-08-30 0:47 ` Arnd Bergmann 2023-08-30 1:19 ` Linus Torvalds 2023-08-30 1:19 ` Linus Torvalds 2023-08-30 8:33 ` Ulf Hansson 2023-08-30 8:33 ` Ulf Hansson 2023-08-30 15:07 ` Linus Torvalds 2023-08-30 15:07 ` Linus Torvalds 2023-08-31 0:08 ` Linus Torvalds 2023-08-31 0:08 ` Linus Torvalds 2023-08-31 11:29 ` Ulf Hansson 2023-08-31 11:29 ` Ulf Hansson 2023-08-31 9:32 ` Rafael J. Wysocki 2023-08-31 9:32 ` Rafael J. Wysocki 2023-08-31 11:37 ` Ulf Hansson 2023-08-31 11:37 ` Ulf Hansson 2023-08-31 12:58 ` Rafael J. Wysocki 2023-08-31 12:58 ` Rafael J. Wysocki 2023-09-11 7:52 ` Geert Uytterhoeven 2023-09-11 7:52 ` Geert Uytterhoeven 2023-09-11 11:28 ` Ulf Hansson 2023-09-11 11:28 ` Ulf Hansson 2023-09-11 11:48 ` Geert Uytterhoeven [this message] 2023-09-11 11:48 ` Geert Uytterhoeven 2023-09-11 12:06 ` Ulf Hansson 2023-09-11 12:06 ` Ulf Hansson 2023-09-11 13:06 ` Geert Uytterhoeven 2023-09-11 13:06 ` Geert Uytterhoeven 2023-09-11 13:57 ` Ulf Hansson 2023-09-11 13:57 ` Ulf Hansson 2023-09-12 13:57 ` Arnd Bergmann 2023-09-12 13:57 ` Arnd Bergmann 2023-09-12 22:19 ` Ulf Hansson 2023-09-12 22:19 ` Ulf Hansson 2023-08-31 0:10 ` pr-tracker-bot 2023-08-31 0:10 ` pr-tracker-bot 2023-09-26 21:03 ` patchwork-bot+linux-soc 2023-09-26 21:21 ` patchwork-bot+linux-soc
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=CAMuHMdUZcrtHZz7daKjL7Oz2MEmCrjJ_ovO3ttTnZ7xuRWgiRA@mail.gmail.com \ --to=geert@linux-m68k.org \ --cc=arnd@arndb.de \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-pm@vger.kernel.org \ --cc=olof@lixom.net \ --cc=rafael@kernel.org \ --cc=sebastian.reichel@collabora.com \ --cc=soc@kernel.org \ --cc=torvalds@linux-foundation.org \ --cc=ulf.hansson@linaro.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: 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.