From: Ulf Hansson <ulf.hansson@linaro.org>
To: neil.armstrong@linaro.org
Cc: Arnd Bergmann <arnd@arndb.de>, Olof Johansson <olof@lixom.net>,
linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, Kevin Hilman <khilman@baylibre.com>,
Jerome Brunet <jbrunet@baylibre.com>,
Martin Blumenstingl <martin.blumenstingl@googlemail.com>,
linux-amlogic@lists.infradead.org
Subject: Re: [PATCH 03/18] soc: amlogic: Move power-domain drivers to the genpd dir
Date: Fri, 7 Jul 2023 17:27:39 +0200 [thread overview]
Message-ID: <CAPDyKFr7Mqy5bisLcxcA_iEGWqL8SFt2mDDng7zYEaTD1vNisA@mail.gmail.com> (raw)
In-Reply-To: <1496b9c1-289a-c354-f0ae-e14fd4c9bcfa@linaro.org>
On Fri, 7 Jul 2023 at 17:12, <neil.armstrong@linaro.org> wrote:
>
> On 07/07/2023 16:54, Ulf Hansson wrote:
> > On Fri, 7 Jul 2023 at 16:42, Neil Armstrong <neil.armstrong@linaro.org> wrote:
> >>
> >> Hi,
> >>
> >> On 07/07/2023 16:04, Ulf Hansson wrote:
> >>> Cc: Neil Armstrong <neil.armstrong@linaro.org>
> >>> Cc: Kevin Hilman <khilman@baylibre.com>
> >>> Cc: Jerome Brunet <jbrunet@baylibre.com>
> >>> Cc: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
> >>> Cc: <linux-amlogic@lists.infradead.org>
> >>> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
> >>> ---
> >>> MAINTAINERS | 1 +
> >>> drivers/genpd/Makefile | 1 +
> >>> drivers/genpd/amlogic/Makefile | 4 ++++
> >>> drivers/{soc => genpd}/amlogic/meson-ee-pwrc.c | 0
> >>> drivers/{soc => genpd}/amlogic/meson-gx-pwrc-vpu.c | 0
> >>> drivers/{soc => genpd}/amlogic/meson-secure-pwrc.c | 0
> >>> drivers/soc/Makefile | 1 -
> >>> drivers/soc/amlogic/Makefile | 3 ---
> >>> 8 files changed, 6 insertions(+), 4 deletions(-)
> >>> create mode 100644 drivers/genpd/amlogic/Makefile
> >>> rename drivers/{soc => genpd}/amlogic/meson-ee-pwrc.c (100%)
> >>> rename drivers/{soc => genpd}/amlogic/meson-gx-pwrc-vpu.c (100%)
> >>> rename drivers/{soc => genpd}/amlogic/meson-secure-pwrc.c (100%)
> >>>
> >>> diff --git a/MAINTAINERS b/MAINTAINERS
> >>> index 588769fab516..9b8c9ae2375d 100644
> >>> --- a/MAINTAINERS
> >>> +++ b/MAINTAINERS
> >>> @@ -1843,6 +1843,7 @@ F: Documentation/devicetree/bindings/phy/amlogic*
> >>> F: arch/arm/boot/dts/amlogic/
> >>> F: arch/arm/mach-meson/
> >>> F: arch/arm64/boot/dts/amlogic/
> >>> +F: drivers/genpd/amlogic/
> >>> F: drivers/mmc/host/meson*
> >>> F: drivers/phy/amlogic/
> >>> F: drivers/pinctrl/meson/
> >>> diff --git a/drivers/genpd/Makefile b/drivers/genpd/Makefile
> >>> index a2d5b2095915..cdba3b9f0c75 100644
> >>> --- a/drivers/genpd/Makefile
> >>> +++ b/drivers/genpd/Makefile
> >>> @@ -1,2 +1,3 @@
> >>> # SPDX-License-Identifier: GPL-2.0-only
> >>> obj-y += actions/
> >>> +obj-y += amlogic/
> >>> diff --git a/drivers/genpd/amlogic/Makefile b/drivers/genpd/amlogic/Makefile
> >>> new file mode 100644
> >>> index 000000000000..3d58abd574f9
> >>> --- /dev/null
> >>> +++ b/drivers/genpd/amlogic/Makefile
> >>> @@ -0,0 +1,4 @@
> >>> +# SPDX-License-Identifier: GPL-2.0-only
> >>> +obj-$(CONFIG_MESON_GX_PM_DOMAINS) += meson-gx-pwrc-vpu.o
> >>> +obj-$(CONFIG_MESON_EE_PM_DOMAINS) += meson-ee-pwrc.o
> >>> +obj-$(CONFIG_MESON_SECURE_PM_DOMAINS) += meson-secure-pwrc.o
> >>> diff --git a/drivers/soc/amlogic/meson-ee-pwrc.c b/drivers/genpd/amlogic/meson-ee-pwrc.c
> >>> similarity index 100%
> >>> rename from drivers/soc/amlogic/meson-ee-pwrc.c
> >>> rename to drivers/genpd/amlogic/meson-ee-pwrc.c
> >>> diff --git a/drivers/soc/amlogic/meson-gx-pwrc-vpu.c b/drivers/genpd/amlogic/meson-gx-pwrc-vpu.c
> >>> similarity index 100%
> >>> rename from drivers/soc/amlogic/meson-gx-pwrc-vpu.c
> >>> rename to drivers/genpd/amlogic/meson-gx-pwrc-vpu.c
> >>> diff --git a/drivers/soc/amlogic/meson-secure-pwrc.c b/drivers/genpd/amlogic/meson-secure-pwrc.c
> >>> similarity index 100%
> >>> rename from drivers/soc/amlogic/meson-secure-pwrc.c
> >>> rename to drivers/genpd/amlogic/meson-secure-pwrc.c
> >>> diff --git a/drivers/soc/Makefile b/drivers/soc/Makefile
> >>> index 3b0f9fb3b5c8..dc93e1762ca7 100644
> >>> --- a/drivers/soc/Makefile
> >>> +++ b/drivers/soc/Makefile
> >>> @@ -23,7 +23,6 @@ obj-y += mediatek/
> >>> obj-y += microchip/
> >>> obj-y += nuvoton/
> >>> obj-y += pxa/
> >>> -obj-y += amlogic/
> >>> obj-y += qcom/
> >>> obj-y += renesas/
> >>> obj-y += rockchip/
> >>> diff --git a/drivers/soc/amlogic/Makefile b/drivers/soc/amlogic/Makefile
> >>> index 7b8c5d323f5c..c25f835e6a26 100644
> >>> --- a/drivers/soc/amlogic/Makefile
> >>> +++ b/drivers/soc/amlogic/Makefile
> >>> @@ -2,7 +2,4 @@
> >>> obj-$(CONFIG_MESON_CANVAS) += meson-canvas.o
> >>> obj-$(CONFIG_MESON_CLK_MEASURE) += meson-clk-measure.o
> >>> obj-$(CONFIG_MESON_GX_SOCINFO) += meson-gx-socinfo.o
> >>> -obj-$(CONFIG_MESON_GX_PM_DOMAINS) += meson-gx-pwrc-vpu.o
> >>> obj-$(CONFIG_MESON_MX_SOCINFO) += meson-mx-socinfo.o
> >>> -obj-$(CONFIG_MESON_EE_PM_DOMAINS) += meson-ee-pwrc.o
> >>> -obj-$(CONFIG_MESON_SECURE_PM_DOMAINS) += meson-secure-pwrc.o
> >>
> >> I've a few changes for v6.6, how shall we handle that ?
> >
> > I think we should continue to use the soc tree until v6.6-rc1 (or if
> > Arnd have some other ideas), there are some more thoughts around this
> > in the cover letter.
> >
> > Beyond v6.6-rc1 I plan to help with maintenance in practice and run
> > my own separate git tree.
>
> If it helps I can send you a PR with only the pwrc changes you can merge
> before applying this serie.
Right, that could be a good idea, but let's defer to Arnd before we
agree on the way forward.
Kind regards
Uffe
next prev parent reply other threads:[~2023-07-07 15:28 UTC|newest]
Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-07-07 14:04 [PATCH 00/18] genpd: Create a genpd directory to host genpd providers Ulf Hansson
2023-07-07 14:04 ` [PATCH 01/18] genpd: Create a new subsystem " Ulf Hansson
2023-07-07 14:04 ` [PATCH 02/18] soc: actions: Move power-domain driver to the genpd dir Ulf Hansson
2023-07-07 16:47 ` Manivannan Sadhasivam
2023-07-07 14:04 ` [PATCH 03/18] soc: amlogic: Move power-domain drivers " Ulf Hansson
2023-07-07 14:42 ` Neil Armstrong
2023-07-07 14:54 ` Ulf Hansson
2023-07-07 15:12 ` neil.armstrong
2023-07-07 15:27 ` Ulf Hansson [this message]
2023-07-09 2:25 ` Bjorn Andersson
2023-07-10 12:37 ` Ulf Hansson
2023-07-10 12:45 ` Arnd Bergmann
2023-07-13 0:48 ` Bjorn Andersson
2023-07-13 9:50 ` Ulf Hansson
2023-07-14 3:10 ` Bjorn Andersson
2023-07-07 19:04 ` Arnd Bergmann
2023-07-07 18:25 ` Martin Blumenstingl
2023-07-11 6:38 ` Ulf Hansson
2023-07-07 14:04 ` [PATCH 04/18] soc: apple: Move power-domain driver " Ulf Hansson
2023-07-07 14:04 ` [PATCH 05/18] soc: bcm: Move power-domain drivers " Ulf Hansson
2023-07-07 14:04 ` [PATCH 06/18] soc: imx: " Ulf Hansson
2023-07-07 14:25 ` Fabio Estevam
2023-07-07 14:33 ` Ulf Hansson
2023-07-18 6:54 ` Shawn Guo
2023-07-07 14:04 ` [PATCH 07/18] soc: mediatek: " Ulf Hansson
2023-07-07 14:45 ` Matthias Brugger
2023-07-07 14:59 ` Ulf Hansson
2023-07-07 14:04 ` [PATCH 08/18] soc: qcom: " Ulf Hansson
2023-07-07 14:04 ` [PATCH 09/18] soc: renesas: " Ulf Hansson
2023-07-10 12:53 ` Geert Uytterhoeven
2023-07-11 13:53 ` Ulf Hansson
2023-07-07 14:04 ` [PATCH 10/18] soc: rockchip: Mover power-domain driver " Ulf Hansson
2023-07-07 14:13 ` Heiko Stübner
2023-07-07 14:04 ` [PATCH 11/18] soc: samsung: Move " Ulf Hansson
2023-07-11 7:38 ` Krzysztof Kozlowski
2023-07-07 14:04 ` [PATCH 12/18] soc: starfive: Move the " Ulf Hansson
2023-07-07 15:51 ` Conor Dooley
2023-07-11 13:57 ` Ulf Hansson
2023-07-11 16:07 ` Conor Dooley
2023-07-07 14:04 ` [PATCH 13/18] soc: sunxi: Move " Ulf Hansson
2023-07-07 14:04 ` [PATCH 14/18] soc: tegra: Move powergate-bpmp " Ulf Hansson
2023-07-10 13:08 ` Thierry Reding
2023-07-07 14:04 ` [PATCH 15/18] soc: ti: Mover power-domain drivers " Ulf Hansson
2023-07-07 17:50 ` Nishanth Menon
2023-07-11 9:16 ` Ulf Hansson
2023-07-11 12:49 ` Nishanth Menon
2023-07-14 7:51 ` Tony Lindgren
2023-07-07 14:04 ` [PATCH 16/18] soc: xilinx: Move power-domain driver " Ulf Hansson
2023-07-07 14:04 ` [PATCH 17/18] ARM: ux500: Convert power-domain code into a regular platform driver Ulf Hansson
2023-07-07 14:04 ` [PATCH 18/18] ARM: ux500: Move power-domain driver to the genpd dir Ulf Hansson
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=CAPDyKFr7Mqy5bisLcxcA_iEGWqL8SFt2mDDng7zYEaTD1vNisA@mail.gmail.com \
--to=ulf.hansson@linaro.org \
--cc=arnd@arndb.de \
--cc=jbrunet@baylibre.com \
--cc=khilman@baylibre.com \
--cc=linux-amlogic@lists.infradead.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=martin.blumenstingl@googlemail.com \
--cc=neil.armstrong@linaro.org \
--cc=olof@lixom.net \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).