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 09:52:30 +0200 [thread overview] Message-ID: <CAMuHMdVxLkxN0bmSRXW74R_dGNDSRDB4Z=zE0DoiXDOuweSM0A@mail.gmail.com> (raw) In-Reply-To: <CAPDyKFpXLj_2HAgyV_VJf+GPQVmxb_iiDe77Q2MY17MDNqy9fA@mail.gmail.com> Hi Ulf, 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! Sounds nice! All of this lives in <linux/pm_domain.h> (with underscore?) anyway, and "PM Domains" is the usual naming, as it covers both Power and Clock Domains. However, although I am quite familiar with genpd, I am still wondering what is the meaning of the "generic" part in the name? And what is a non-generic PM Domain? > > 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/. 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... 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 09:52:30 +0200 [thread overview] Message-ID: <CAMuHMdVxLkxN0bmSRXW74R_dGNDSRDB4Z=zE0DoiXDOuweSM0A@mail.gmail.com> (raw) In-Reply-To: <CAPDyKFpXLj_2HAgyV_VJf+GPQVmxb_iiDe77Q2MY17MDNqy9fA@mail.gmail.com> Hi Ulf, 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! Sounds nice! All of this lives in <linux/pm_domain.h> (with underscore?) anyway, and "PM Domains" is the usual naming, as it covers both Power and Clock Domains. However, although I am quite familiar with genpd, I am still wondering what is the meaning of the "generic" part in the name? And what is a non-generic PM Domain? > > 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/. 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... 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 7: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 [this message] 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 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='CAMuHMdVxLkxN0bmSRXW74R_dGNDSRDB4Z=zE0DoiXDOuweSM0A@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.