[09/18] soc: renesas: Move power-domain drivers to the genpd dir
Commit Message
Cc: Geert Uytterhoeven <geert+renesas@glider.be>
Cc: Magnus Damm <magnus.damm@gmail.com>
Cc: <linux-renesas-soc@vger.kernel.org>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
---
MAINTAINERS | 1 +
drivers/genpd/Makefile | 1 +
drivers/genpd/renesas/Makefile | 30 +++++++++++++++++++
drivers/{soc => genpd}/renesas/r8a7742-sysc.c | 0
drivers/{soc => genpd}/renesas/r8a7743-sysc.c | 0
drivers/{soc => genpd}/renesas/r8a7745-sysc.c | 0
.../{soc => genpd}/renesas/r8a77470-sysc.c | 0
.../{soc => genpd}/renesas/r8a774a1-sysc.c | 0
.../{soc => genpd}/renesas/r8a774b1-sysc.c | 0
.../{soc => genpd}/renesas/r8a774c0-sysc.c | 0
.../{soc => genpd}/renesas/r8a774e1-sysc.c | 0
drivers/{soc => genpd}/renesas/r8a7779-sysc.c | 0
drivers/{soc => genpd}/renesas/r8a7790-sysc.c | 0
drivers/{soc => genpd}/renesas/r8a7791-sysc.c | 0
drivers/{soc => genpd}/renesas/r8a7792-sysc.c | 0
drivers/{soc => genpd}/renesas/r8a7794-sysc.c | 0
drivers/{soc => genpd}/renesas/r8a7795-sysc.c | 0
drivers/{soc => genpd}/renesas/r8a7796-sysc.c | 0
.../{soc => genpd}/renesas/r8a77965-sysc.c | 0
.../{soc => genpd}/renesas/r8a77970-sysc.c | 0
.../{soc => genpd}/renesas/r8a77980-sysc.c | 0
.../{soc => genpd}/renesas/r8a77990-sysc.c | 0
.../{soc => genpd}/renesas/r8a77995-sysc.c | 0
.../{soc => genpd}/renesas/r8a779a0-sysc.c | 0
.../{soc => genpd}/renesas/r8a779f0-sysc.c | 0
.../{soc => genpd}/renesas/r8a779g0-sysc.c | 0
.../{soc => genpd}/renesas/rcar-gen4-sysc.c | 0
.../{soc => genpd}/renesas/rcar-gen4-sysc.h | 0
drivers/{soc => genpd}/renesas/rcar-sysc.c | 0
drivers/{soc => genpd}/renesas/rcar-sysc.h | 0
drivers/{soc => genpd}/renesas/rmobile-sysc.c | 0
drivers/soc/renesas/Makefile | 27 -----------------
32 files changed, 32 insertions(+), 27 deletions(-)
create mode 100644 drivers/genpd/renesas/Makefile
rename drivers/{soc => genpd}/renesas/r8a7742-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a7743-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a7745-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a77470-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a774a1-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a774b1-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a774c0-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a774e1-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a7779-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a7790-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a7791-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a7792-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a7794-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a7795-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a7796-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a77965-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a77970-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a77980-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a77990-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a77995-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a779a0-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a779f0-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/r8a779g0-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/rcar-gen4-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/rcar-gen4-sysc.h (100%)
rename drivers/{soc => genpd}/renesas/rcar-sysc.c (100%)
rename drivers/{soc => genpd}/renesas/rcar-sysc.h (100%)
rename drivers/{soc => genpd}/renesas/rmobile-sysc.c (100%)
Comments
Hi Ulf,
On Fri, Jul 7, 2023 at 4:04 PM Ulf Hansson <ulf.hansson@linaro.org> wrote:
> Cc: Geert Uytterhoeven <geert+renesas@glider.be>
> Cc: Magnus Damm <magnus.damm@gmail.com>
> Cc: <linux-renesas-soc@vger.kernel.org>
> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
Thanks for your patch!
> ---
> MAINTAINERS | 1 +
> drivers/genpd/Makefile | 1 +
> drivers/genpd/renesas/Makefile | 30 +++++++++++++++++++
> drivers/{soc => genpd}/renesas/r8a7742-sysc.c | 0
> drivers/{soc => genpd}/renesas/r8a7743-sysc.c | 0
> drivers/{soc => genpd}/renesas/r8a7745-sysc.c | 0
> .../{soc => genpd}/renesas/r8a77470-sysc.c | 0
> .../{soc => genpd}/renesas/r8a774a1-sysc.c | 0
> .../{soc => genpd}/renesas/r8a774b1-sysc.c | 0
> .../{soc => genpd}/renesas/r8a774c0-sysc.c | 0
> .../{soc => genpd}/renesas/r8a774e1-sysc.c | 0
> drivers/{soc => genpd}/renesas/r8a7779-sysc.c | 0
> drivers/{soc => genpd}/renesas/r8a7790-sysc.c | 0
> drivers/{soc => genpd}/renesas/r8a7791-sysc.c | 0
> drivers/{soc => genpd}/renesas/r8a7792-sysc.c | 0
> drivers/{soc => genpd}/renesas/r8a7794-sysc.c | 0
> drivers/{soc => genpd}/renesas/r8a7795-sysc.c | 0
> drivers/{soc => genpd}/renesas/r8a7796-sysc.c | 0
> .../{soc => genpd}/renesas/r8a77965-sysc.c | 0
> .../{soc => genpd}/renesas/r8a77970-sysc.c | 0
> .../{soc => genpd}/renesas/r8a77980-sysc.c | 0
> .../{soc => genpd}/renesas/r8a77990-sysc.c | 0
> .../{soc => genpd}/renesas/r8a77995-sysc.c | 0
> .../{soc => genpd}/renesas/r8a779a0-sysc.c | 0
> .../{soc => genpd}/renesas/r8a779f0-sysc.c | 0
> .../{soc => genpd}/renesas/r8a779g0-sysc.c | 0
> .../{soc => genpd}/renesas/rcar-gen4-sysc.c | 0
> .../{soc => genpd}/renesas/rcar-gen4-sysc.h | 0
> drivers/{soc => genpd}/renesas/rcar-sysc.c | 0
> drivers/{soc => genpd}/renesas/rcar-sysc.h | 0
> drivers/{soc => genpd}/renesas/rmobile-sysc.c | 0
> drivers/soc/renesas/Makefile | 27 -----------------
> 32 files changed, 32 insertions(+), 27 deletions(-)
LGTM.
Is there any specific reason why you're not moving the SYSC_* symbols
from drivers/soc/renesas/Kconfig to drivers/genpd/renesas/Kconfig?
Thanks!
Gr{oetje,eeting}s,
Geert
On Mon, 10 Jul 2023 at 14:53, Geert Uytterhoeven <geert@linux-m68k.org> wrote:
>
> Hi Ulf,
>
> On Fri, Jul 7, 2023 at 4:04 PM Ulf Hansson <ulf.hansson@linaro.org> wrote:
> > Cc: Geert Uytterhoeven <geert+renesas@glider.be>
> > Cc: Magnus Damm <magnus.damm@gmail.com>
> > Cc: <linux-renesas-soc@vger.kernel.org>
> > Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
>
> Thanks for your patch!
>
> > ---
> > MAINTAINERS | 1 +
> > drivers/genpd/Makefile | 1 +
> > drivers/genpd/renesas/Makefile | 30 +++++++++++++++++++
> > drivers/{soc => genpd}/renesas/r8a7742-sysc.c | 0
> > drivers/{soc => genpd}/renesas/r8a7743-sysc.c | 0
> > drivers/{soc => genpd}/renesas/r8a7745-sysc.c | 0
> > .../{soc => genpd}/renesas/r8a77470-sysc.c | 0
> > .../{soc => genpd}/renesas/r8a774a1-sysc.c | 0
> > .../{soc => genpd}/renesas/r8a774b1-sysc.c | 0
> > .../{soc => genpd}/renesas/r8a774c0-sysc.c | 0
> > .../{soc => genpd}/renesas/r8a774e1-sysc.c | 0
> > drivers/{soc => genpd}/renesas/r8a7779-sysc.c | 0
> > drivers/{soc => genpd}/renesas/r8a7790-sysc.c | 0
> > drivers/{soc => genpd}/renesas/r8a7791-sysc.c | 0
> > drivers/{soc => genpd}/renesas/r8a7792-sysc.c | 0
> > drivers/{soc => genpd}/renesas/r8a7794-sysc.c | 0
> > drivers/{soc => genpd}/renesas/r8a7795-sysc.c | 0
> > drivers/{soc => genpd}/renesas/r8a7796-sysc.c | 0
> > .../{soc => genpd}/renesas/r8a77965-sysc.c | 0
> > .../{soc => genpd}/renesas/r8a77970-sysc.c | 0
> > .../{soc => genpd}/renesas/r8a77980-sysc.c | 0
> > .../{soc => genpd}/renesas/r8a77990-sysc.c | 0
> > .../{soc => genpd}/renesas/r8a77995-sysc.c | 0
> > .../{soc => genpd}/renesas/r8a779a0-sysc.c | 0
> > .../{soc => genpd}/renesas/r8a779f0-sysc.c | 0
> > .../{soc => genpd}/renesas/r8a779g0-sysc.c | 0
> > .../{soc => genpd}/renesas/rcar-gen4-sysc.c | 0
> > .../{soc => genpd}/renesas/rcar-gen4-sysc.h | 0
> > drivers/{soc => genpd}/renesas/rcar-sysc.c | 0
> > drivers/{soc => genpd}/renesas/rcar-sysc.h | 0
> > drivers/{soc => genpd}/renesas/rmobile-sysc.c | 0
> > drivers/soc/renesas/Makefile | 27 -----------------
> > 32 files changed, 32 insertions(+), 27 deletions(-)
>
> LGTM.
Thanks! I defer to your formal ack, before adding your tag.
>
> Is there any specific reason why you're not moving the SYSC_* symbols
> from drivers/soc/renesas/Kconfig to drivers/genpd/renesas/Kconfig?
I was looking into moving the corresponding Kconfig options for all
SoCs/platforms, but I found it being a bit controversial at this
point. One could argue that it looks easier to understand the Kconfig
files, by keeping the SoC specific options together as there are
simply always some kind of dependency that needs to be described too.
Moreover, we currently don't have a menu option for the
CONFIG_GENERIC_PM_DOMAINS. I guess that could be discussed too,
especially if we should want to move the Kconfig files to the genpd
directory.
That said, for now, I suggest we keep the Kconfig files in the soc
directory. Or at least address this separately on top of the $subject
series.
Kind regards
Uffe
@@ -2591,6 +2591,7 @@ F: arch/arm/include/debug/renesas-scif.S
F: arch/arm/mach-shmobile/
F: arch/arm64/boot/dts/renesas/
F: arch/riscv/boot/dts/renesas/
+F: drivers/genpd/renesas/
F: drivers/soc/renesas/
F: include/linux/soc/renesas/
K: \brenesas,
@@ -5,3 +5,4 @@ obj-y += apple/
obj-y += bcm/
obj-y += mediatek/
obj-y += qcom/
+obj-y += renesas/
new file mode 100644
@@ -0,0 +1,30 @@
+# SPDX-License-Identifier: GPL-2.0
+# SoC
+obj-$(CONFIG_SYSC_R8A7742) += r8a7742-sysc.o
+obj-$(CONFIG_SYSC_R8A7743) += r8a7743-sysc.o
+obj-$(CONFIG_SYSC_R8A7745) += r8a7745-sysc.o
+obj-$(CONFIG_SYSC_R8A77470) += r8a77470-sysc.o
+obj-$(CONFIG_SYSC_R8A774A1) += r8a774a1-sysc.o
+obj-$(CONFIG_SYSC_R8A774B1) += r8a774b1-sysc.o
+obj-$(CONFIG_SYSC_R8A774C0) += r8a774c0-sysc.o
+obj-$(CONFIG_SYSC_R8A774E1) += r8a774e1-sysc.o
+obj-$(CONFIG_SYSC_R8A7779) += r8a7779-sysc.o
+obj-$(CONFIG_SYSC_R8A7790) += r8a7790-sysc.o
+obj-$(CONFIG_SYSC_R8A7791) += r8a7791-sysc.o
+obj-$(CONFIG_SYSC_R8A7792) += r8a7792-sysc.o
+obj-$(CONFIG_SYSC_R8A7794) += r8a7794-sysc.o
+obj-$(CONFIG_SYSC_R8A7795) += r8a7795-sysc.o
+obj-$(CONFIG_SYSC_R8A77960) += r8a7796-sysc.o
+obj-$(CONFIG_SYSC_R8A77961) += r8a7796-sysc.o
+obj-$(CONFIG_SYSC_R8A77965) += r8a77965-sysc.o
+obj-$(CONFIG_SYSC_R8A77970) += r8a77970-sysc.o
+obj-$(CONFIG_SYSC_R8A77980) += r8a77980-sysc.o
+obj-$(CONFIG_SYSC_R8A77990) += r8a77990-sysc.o
+obj-$(CONFIG_SYSC_R8A77995) += r8a77995-sysc.o
+obj-$(CONFIG_SYSC_R8A779A0) += r8a779a0-sysc.o
+obj-$(CONFIG_SYSC_R8A779F0) += r8a779f0-sysc.o
+obj-$(CONFIG_SYSC_R8A779G0) += r8a779g0-sysc.o
+# Family
+obj-$(CONFIG_SYSC_RCAR) += rcar-sysc.o
+obj-$(CONFIG_SYSC_RCAR_GEN4) += rcar-gen4-sysc.o
+obj-$(CONFIG_SYSC_RMOBILE) += rmobile-sysc.o
similarity index 100%
rename from drivers/soc/renesas/r8a7742-sysc.c
rename to drivers/genpd/renesas/r8a7742-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a7743-sysc.c
rename to drivers/genpd/renesas/r8a7743-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a7745-sysc.c
rename to drivers/genpd/renesas/r8a7745-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a77470-sysc.c
rename to drivers/genpd/renesas/r8a77470-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a774a1-sysc.c
rename to drivers/genpd/renesas/r8a774a1-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a774b1-sysc.c
rename to drivers/genpd/renesas/r8a774b1-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a774c0-sysc.c
rename to drivers/genpd/renesas/r8a774c0-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a774e1-sysc.c
rename to drivers/genpd/renesas/r8a774e1-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a7779-sysc.c
rename to drivers/genpd/renesas/r8a7779-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a7790-sysc.c
rename to drivers/genpd/renesas/r8a7790-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a7791-sysc.c
rename to drivers/genpd/renesas/r8a7791-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a7792-sysc.c
rename to drivers/genpd/renesas/r8a7792-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a7794-sysc.c
rename to drivers/genpd/renesas/r8a7794-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a7795-sysc.c
rename to drivers/genpd/renesas/r8a7795-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a7796-sysc.c
rename to drivers/genpd/renesas/r8a7796-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a77965-sysc.c
rename to drivers/genpd/renesas/r8a77965-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a77970-sysc.c
rename to drivers/genpd/renesas/r8a77970-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a77980-sysc.c
rename to drivers/genpd/renesas/r8a77980-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a77990-sysc.c
rename to drivers/genpd/renesas/r8a77990-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a77995-sysc.c
rename to drivers/genpd/renesas/r8a77995-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a779a0-sysc.c
rename to drivers/genpd/renesas/r8a779a0-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a779f0-sysc.c
rename to drivers/genpd/renesas/r8a779f0-sysc.c
similarity index 100%
rename from drivers/soc/renesas/r8a779g0-sysc.c
rename to drivers/genpd/renesas/r8a779g0-sysc.c
similarity index 100%
rename from drivers/soc/renesas/rcar-gen4-sysc.c
rename to drivers/genpd/renesas/rcar-gen4-sysc.c
similarity index 100%
rename from drivers/soc/renesas/rcar-gen4-sysc.h
rename to drivers/genpd/renesas/rcar-gen4-sysc.h
similarity index 100%
rename from drivers/soc/renesas/rcar-sysc.c
rename to drivers/genpd/renesas/rcar-sysc.c
similarity index 100%
rename from drivers/soc/renesas/rcar-sysc.h
rename to drivers/genpd/renesas/rcar-sysc.h
similarity index 100%
rename from drivers/soc/renesas/rmobile-sysc.c
rename to drivers/genpd/renesas/rmobile-sysc.c
@@ -3,30 +3,6 @@
obj-$(CONFIG_SOC_RENESAS) += renesas-soc.o
# SoC
-obj-$(CONFIG_SYSC_R8A7742) += r8a7742-sysc.o
-obj-$(CONFIG_SYSC_R8A7743) += r8a7743-sysc.o
-obj-$(CONFIG_SYSC_R8A7745) += r8a7745-sysc.o
-obj-$(CONFIG_SYSC_R8A77470) += r8a77470-sysc.o
-obj-$(CONFIG_SYSC_R8A774A1) += r8a774a1-sysc.o
-obj-$(CONFIG_SYSC_R8A774B1) += r8a774b1-sysc.o
-obj-$(CONFIG_SYSC_R8A774C0) += r8a774c0-sysc.o
-obj-$(CONFIG_SYSC_R8A774E1) += r8a774e1-sysc.o
-obj-$(CONFIG_SYSC_R8A7779) += r8a7779-sysc.o
-obj-$(CONFIG_SYSC_R8A7790) += r8a7790-sysc.o
-obj-$(CONFIG_SYSC_R8A7791) += r8a7791-sysc.o
-obj-$(CONFIG_SYSC_R8A7792) += r8a7792-sysc.o
-obj-$(CONFIG_SYSC_R8A7794) += r8a7794-sysc.o
-obj-$(CONFIG_SYSC_R8A7795) += r8a7795-sysc.o
-obj-$(CONFIG_SYSC_R8A77960) += r8a7796-sysc.o
-obj-$(CONFIG_SYSC_R8A77961) += r8a7796-sysc.o
-obj-$(CONFIG_SYSC_R8A77965) += r8a77965-sysc.o
-obj-$(CONFIG_SYSC_R8A77970) += r8a77970-sysc.o
-obj-$(CONFIG_SYSC_R8A77980) += r8a77980-sysc.o
-obj-$(CONFIG_SYSC_R8A77990) += r8a77990-sysc.o
-obj-$(CONFIG_SYSC_R8A77995) += r8a77995-sysc.o
-obj-$(CONFIG_SYSC_R8A779A0) += r8a779a0-sysc.o
-obj-$(CONFIG_SYSC_R8A779F0) += r8a779f0-sysc.o
-obj-$(CONFIG_SYSC_R8A779G0) += r8a779g0-sysc.o
ifdef CONFIG_SMP
obj-$(CONFIG_ARCH_R9A06G032) += r9a06g032-smp.o
endif
@@ -34,6 +10,3 @@ endif
# Family
obj-$(CONFIG_PWC_RZV2M) += pwc-rzv2m.o
obj-$(CONFIG_RST_RCAR) += rcar-rst.o
-obj-$(CONFIG_SYSC_RCAR) += rcar-sysc.o
-obj-$(CONFIG_SYSC_RCAR_GEN4) += rcar-gen4-sysc.o
-obj-$(CONFIG_SYSC_RMOBILE) += rmobile-sysc.o