* [PATCH 2/2] pmdomain: renesas: sort Renesas Kconfig configs
@ 2025-06-29 23:18 Kuninori Morimoto
2025-07-01 9:49 ` Geert Uytterhoeven
0 siblings, 1 reply; 6+ messages in thread
From: Kuninori Morimoto @ 2025-06-29 23:18 UTC (permalink / raw)
To: Geert Uytterhoeven, Ulf Hansson, linux-pm, linux-renesas-soc
Renesas Kconfig is using "SoC serial number" for CONFIG symbol, but is
using "SoC chip name" for menu description. Because of it, it looks
random order when we run "make menuconfig".
commit 6d5aded8d57f ("soc: renesas: Sort driver description title")
sorted Renesas Kconfig is by menu description title order, but it makes
confusable to add new config.
Let's unify "System Controller support for ${CHIP_NUMBER} (${CHIP_NAME}).
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
drivers/pmdomain/renesas/Kconfig | 122 ++++++++++++++++---------------
1 file changed, 62 insertions(+), 60 deletions(-)
diff --git a/drivers/pmdomain/renesas/Kconfig b/drivers/pmdomain/renesas/Kconfig
index 70bd6605a97c..6d51dd2ab4e9 100644
--- a/drivers/pmdomain/renesas/Kconfig
+++ b/drivers/pmdomain/renesas/Kconfig
@@ -2,114 +2,116 @@
if SOC_RENESAS
menu "Renesas PM Domains"
+# SoC Type
config SYSC_RCAR
bool "System Controller support for R-Car" if COMPILE_TEST
config SYSC_RCAR_GEN4
bool "System Controller support for R-Car Gen4" if COMPILE_TEST
-config SYSC_R8A77995
- bool "System Controller support for R-Car D3" if COMPILE_TEST
+config SYSC_RMOBILE
+ bool "System Controller support for R-Mobile" if COMPILE_TEST
+
+# SoC
+config SYSC_R8A7742
+ bool "System Controller support for R8A7742 (RZ/G1H)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A7794
- bool "System Controller support for R-Car E2" if COMPILE_TEST
+config SYSC_R8A7743
+ bool "System Controller support for R8A7743 (RZ/G1M)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A77990
- bool "System Controller support for R-Car E3" if COMPILE_TEST
+config SYSC_R8A7745
+ bool "System Controller support for R8A7745 (RZ/G1E)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A7779
- bool "System Controller support for R-Car H1" if COMPILE_TEST
+config SYSC_R8A77470
+ bool "System Controller support for R8A77470 (RZ/G1C)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A7790
- bool "System Controller support for R-Car H2" if COMPILE_TEST
+config SYSC_R8A774A1
+ bool "System Controller support for R8A774A1 (RZ/G2M)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A7795
- bool "System Controller support for R-Car H3" if COMPILE_TEST
+config SYSC_R8A774B1
+ bool "System Controller support for R8A774B1 (RZ/G2N)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A7791
- bool "System Controller support for R-Car M2-W/N" if COMPILE_TEST
+config SYSC_R8A774C0
+ bool "System Controller support for R8A774C0 (RZ/G2E)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A77965
- bool "System Controller support for R-Car M3-N" if COMPILE_TEST
+config SYSC_R8A774E1
+ bool "System Controller support for R8A774E1 (RZ/G2H)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A77960
- bool "System Controller support for R-Car M3-W" if COMPILE_TEST
+config SYSC_R8A7779
+ bool "System Controller support for R8A7779 (R-Car H1)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A77961
- bool "System Controller support for R-Car M3-W+" if COMPILE_TEST
+config SYSC_R8A7790
+ bool "System Controller support for R8A7790 (R-Car H2)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A779F0
- bool "System Controller support for R-Car S4-8" if COMPILE_TEST
- select SYSC_RCAR_GEN4
+config SYSC_R8A7791
+ bool "System Controller support for R8A7791 (R-Car M2-W/N)" if COMPILE_TEST
+ select SYSC_RCAR
config SYSC_R8A7792
- bool "System Controller support for R-Car V2H" if COMPILE_TEST
+ bool "System Controller support for R8A7792 (R-Car V2H)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A77980
- bool "System Controller support for R-Car V3H" if COMPILE_TEST
+config SYSC_R8A7794
+ bool "System Controller support for R8A7794 (R-Car E2)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A77970
- bool "System Controller support for R-Car V3M" if COMPILE_TEST
+config SYSC_R8A7795
+ bool "System Controller support for R8A7795 (R-Car H3)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A779A0
- bool "System Controller support for R-Car V3U" if COMPILE_TEST
- select SYSC_RCAR_GEN4
-
-config SYSC_R8A779G0
- bool "System Controller support for R-Car V4H" if COMPILE_TEST
- select SYSC_RCAR_GEN4
-
-config SYSC_R8A779H0
- bool "System Controller support for R-Car V4M" if COMPILE_TEST
- select SYSC_RCAR_GEN4
-
-config SYSC_RMOBILE
- bool "System Controller support for R-Mobile" if COMPILE_TEST
-
-config SYSC_R8A77470
- bool "System Controller support for RZ/G1C" if COMPILE_TEST
+config SYSC_R8A77960
+ bool "System Controller support for R8A77960 (R-Car M3-W)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A7745
- bool "System Controller support for RZ/G1E" if COMPILE_TEST
+config SYSC_R8A77961
+ bool "System Controller support for R8A77961 (R-Car M3-W+)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A7742
- bool "System Controller support for RZ/G1H" if COMPILE_TEST
+config SYSC_R8A77965
+ bool "System Controller support for R8A77965 (R-Car M3-N)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A7743
- bool "System Controller support for RZ/G1M" if COMPILE_TEST
+config SYSC_R8A77970
+ bool "System Controller support for R8A77970 (R-Car V3M)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A774C0
- bool "System Controller support for RZ/G2E" if COMPILE_TEST
+config SYSC_R8A77980
+ bool "System Controller support for R8A77980 (R-Car V3H)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A774E1
- bool "System Controller support for RZ/G2H" if COMPILE_TEST
+config SYSC_R8A77990
+ bool "System Controller support for R8A77990 (R-Car E3)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A774A1
- bool "System Controller support for RZ/G2M" if COMPILE_TEST
+config SYSC_R8A77995
+ bool "System Controller support for R8A77995 (R-Car D3)" if COMPILE_TEST
select SYSC_RCAR
-config SYSC_R8A774B1
- bool "System Controller support for RZ/G2N" if COMPILE_TEST
- select SYSC_RCAR
+config SYSC_R8A779A0
+ bool "System Controller support for R8A779A0 (R-Car V3U)" if COMPILE_TEST
+ select SYSC_RCAR_GEN4
+
+config SYSC_R8A779F0
+ bool "System Controller support for R8A779F0 (R-Car S4-8)" if COMPILE_TEST
+ select SYSC_RCAR_GEN4
+
+config SYSC_R8A779G0
+ bool "System Controller support for R8A779G0 (R-Car V4H)" if COMPILE_TEST
+ select SYSC_RCAR_GEN4
+
+config SYSC_R8A779H0
+ bool "System Controller support for R8A779H0 (R-Car V4M)" if COMPILE_TEST
+ select SYSC_RCAR_GEN4
endmenu
endif
--
2.43.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] pmdomain: renesas: sort Renesas Kconfig configs
2025-06-29 23:18 [PATCH 2/2] pmdomain: renesas: sort Renesas Kconfig configs Kuninori Morimoto
@ 2025-07-01 9:49 ` Geert Uytterhoeven
2025-07-01 23:40 ` Kuninori Morimoto
2025-07-03 0:42 ` Kuninori Morimoto
0 siblings, 2 replies; 6+ messages in thread
From: Geert Uytterhoeven @ 2025-07-01 9:49 UTC (permalink / raw)
To: Kuninori Morimoto; +Cc: Ulf Hansson, linux-pm, linux-renesas-soc
Hi Morimoto-san,
On Mon, 30 Jun 2025 at 01:18, Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
> Renesas Kconfig is using "SoC serial number" for CONFIG symbol, but is
> using "SoC chip name" for menu description. Because of it, it looks
> random order when we run "make menuconfig".
>
> commit 6d5aded8d57f ("soc: renesas: Sort driver description title")
> sorted Renesas Kconfig is by menu description title order, but it makes
> confusable to add new config.
>
> Let's unify "System Controller support for ${CHIP_NUMBER} (${CHIP_NAME}).
>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Thanks for your patch!
> --- a/drivers/pmdomain/renesas/Kconfig
> +++ b/drivers/pmdomain/renesas/Kconfig
> @@ -2,114 +2,116 @@
> if SOC_RENESAS
> menu "Renesas PM Domains"
>
> +# SoC Type
"# Family"?
> config SYSC_RCAR
> bool "System Controller support for R-Car" if COMPILE_TEST
>
> config SYSC_RCAR_GEN4
> bool "System Controller support for R-Car Gen4" if COMPILE_TEST
>
> -config SYSC_R8A77995
> - bool "System Controller support for R-Car D3" if COMPILE_TEST
> +config SYSC_RMOBILE
> + bool "System Controller support for R-Mobile" if COMPILE_TEST
> +
> +# SoC
> +config SYSC_R8A7742
> + bool "System Controller support for R8A7742 (RZ/G1H)" if COMPILE_TEST
> select SYSC_RCAR
[...]
> -config SYSC_R8A779F0
> - bool "System Controller support for R-Car S4-8" if COMPILE_TEST
> - select SYSC_RCAR_GEN4
> +config SYSC_R8A7791
> + bool "System Controller support for R8A7791 (R-Car M2-W/N)" if COMPILE_TEST
"R8A7791/R8A7793"?
> + select SYSC_RCAR
>
The rest LGTM, so
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
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
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] pmdomain: renesas: sort Renesas Kconfig configs
2025-07-01 9:49 ` Geert Uytterhoeven
@ 2025-07-01 23:40 ` Kuninori Morimoto
2025-07-02 6:58 ` Geert Uytterhoeven
2025-07-03 0:42 ` Kuninori Morimoto
1 sibling, 1 reply; 6+ messages in thread
From: Kuninori Morimoto @ 2025-07-01 23:40 UTC (permalink / raw)
To: Geert Uytterhoeven; +Cc: Ulf Hansson, linux-pm, linux-renesas-soc
Hi Geert
Thank you for your review
> > +# SoC Type
>
> "# Family"?
(snip)
> > -config SYSC_R8A779F0
> > - bool "System Controller support for R-Car S4-8" if COMPILE_TEST
> > - select SYSC_RCAR_GEN4
> > +config SYSC_R8A7791
> > + bool "System Controller support for R8A7791 (R-Car M2-W/N)" if COMPILE_TEST
>
> "R8A7791/R8A7793"?
(snip)
> The rest LGTM, so
Should I post v2 patch ? Or can you adjust/fixup when you applying ?
Thank you for your help !!
Best regards
---
Kuninori Morimoto
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] pmdomain: renesas: sort Renesas Kconfig configs
2025-07-01 23:40 ` Kuninori Morimoto
@ 2025-07-02 6:58 ` Geert Uytterhoeven
2025-07-03 0:36 ` Kuninori Morimoto
0 siblings, 1 reply; 6+ messages in thread
From: Geert Uytterhoeven @ 2025-07-02 6:58 UTC (permalink / raw)
To: Kuninori Morimoto; +Cc: Ulf Hansson, linux-pm, linux-renesas-soc
Hi Morimoto-san,
On Wed, 2 Jul 2025 at 01:40, Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
> > > +# SoC Type
> >
> > "# Family"?
> (snip)
> > > -config SYSC_R8A779F0
> > > - bool "System Controller support for R-Car S4-8" if COMPILE_TEST
> > > - select SYSC_RCAR_GEN4
> > > +config SYSC_R8A7791
> > > + bool "System Controller support for R8A7791 (R-Car M2-W/N)" if COMPILE_TEST
> >
> > "R8A7791/R8A7793"?
> (snip)
> > The rest LGTM, so
>
> Should I post v2 patch ? Or can you adjust/fixup when you applying ?
That is up to the PM Domain maintainer (Ulf).
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
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] pmdomain: renesas: sort Renesas Kconfig configs
2025-07-02 6:58 ` Geert Uytterhoeven
@ 2025-07-03 0:36 ` Kuninori Morimoto
0 siblings, 0 replies; 6+ messages in thread
From: Kuninori Morimoto @ 2025-07-03 0:36 UTC (permalink / raw)
To: Geert Uytterhoeven; +Cc: Ulf Hansson, linux-pm, linux-renesas-soc
Hi Geert
> > > > +# SoC Type
> > >
> > > "# Family"?
> > (snip)
> > > > -config SYSC_R8A779F0
> > > > - bool "System Controller support for R-Car S4-8" if COMPILE_TEST
> > > > - select SYSC_RCAR_GEN4
> > > > +config SYSC_R8A7791
> > > > + bool "System Controller support for R8A7791 (R-Car M2-W/N)" if COMPILE_TEST
> > >
> > > "R8A7791/R8A7793"?
> > (snip)
> > > The rest LGTM, so
> >
> > Should I post v2 patch ? Or can you adjust/fixup when you applying ?
>
> That is up to the PM Domain maintainer (Ulf).
Ah, OK.
So, I will post v2 patch.
Thank you for your help !!
Best regards
---
Kuninori Morimoto
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] pmdomain: renesas: sort Renesas Kconfig configs
2025-07-01 9:49 ` Geert Uytterhoeven
2025-07-01 23:40 ` Kuninori Morimoto
@ 2025-07-03 0:42 ` Kuninori Morimoto
1 sibling, 0 replies; 6+ messages in thread
From: Kuninori Morimoto @ 2025-07-03 0:42 UTC (permalink / raw)
To: Geert Uytterhoeven; +Cc: Ulf Hansson, linux-pm, linux-renesas-soc
Hi Geert
> > -config SYSC_R8A779F0
> > - bool "System Controller support for R-Car S4-8" if COMPILE_TEST
> > - select SYSC_RCAR_GEN4
> > +config SYSC_R8A7791
> > + bool "System Controller support for R8A7791 (R-Car M2-W/N)" if COMPILE_TEST
>
> "R8A7791/R8A7793"?
I want to have both 91/93 for each, not squashed.
v2 will do it.
Thank you for your help !!
Best regards
---
Kuninori Morimoto
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2025-07-03 0:42 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-06-29 23:18 [PATCH 2/2] pmdomain: renesas: sort Renesas Kconfig configs Kuninori Morimoto
2025-07-01 9:49 ` Geert Uytterhoeven
2025-07-01 23:40 ` Kuninori Morimoto
2025-07-02 6:58 ` Geert Uytterhoeven
2025-07-03 0:36 ` Kuninori Morimoto
2025-07-03 0:42 ` Kuninori Morimoto
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).