linux-renesas-soc.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v3 0/4] pinctrl: sh-pfc: collect Renesas related CONFIGs in one place
@ 2020-08-24  2:29 Kuninori Morimoto
  2020-08-24  2:30 ` [PATCH v3 1/4] pinctrl: sh-pfc: tidyup Emma Mobile EV2 Kuninori Morimoto
                   ` (3 more replies)
  0 siblings, 4 replies; 7+ messages in thread
From: Kuninori Morimoto @ 2020-08-24  2:29 UTC (permalink / raw)
  To: Linus Walleij, Geert Uytterhoeven; +Cc: linux-gpio, linux-renesas-soc


Hi Linus, Geert

These are v3 patch for Renesas pinctrl Kconfig.

Renesas related pinctrl CONFIGs are located random places on menuconfig,
because of it, it is confusable.
This patch collects these into same place.

This patch moves pinctrl-rza{a1,a2,n1}.c to sh-pfc,
but not renamed "sh-pfc" to "renesas".

These are based on renesas-drivers-2020-08-18-v5.9-rc1

v2 -> v3
	- tidyup "AV2" to "EV2"
	- move pinctrl-rza{a1,a2,n1}.c to sh-pfc
	- sort all drivers

v1 -> v2
	- align driver description title
	- sort  driver description title

Kuninori Morimoto (4):
  pinctrl: sh-pfc: tidyup Emma Mobile EV2
  pinctrl: sh-pfc: collect Renesas related CONFIGs in one place
  pinctrl: sh-pfc: align driver description title
  pinctrl: sh-pfc: tidy up driver description title

 drivers/pinctrl/Kconfig                     |  32 ----
 drivers/pinctrl/Makefile                    |   3 -
 drivers/pinctrl/sh-pfc/Kconfig              | 181 ++++++++++++--------
 drivers/pinctrl/sh-pfc/Makefile             |   4 +
 drivers/pinctrl/{ => sh-pfc}/pinctrl-rza1.c |   0
 drivers/pinctrl/{ => sh-pfc}/pinctrl-rza2.c |   0
 drivers/pinctrl/{ => sh-pfc}/pinctrl-rzn1.c |   0
 7 files changed, 114 insertions(+), 106 deletions(-)
 rename drivers/pinctrl/{ => sh-pfc}/pinctrl-rza1.c (100%)
 rename drivers/pinctrl/{ => sh-pfc}/pinctrl-rza2.c (100%)
 rename drivers/pinctrl/{ => sh-pfc}/pinctrl-rzn1.c (100%)

-- 
2.25.1


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH v3 1/4] pinctrl: sh-pfc: tidyup Emma Mobile EV2
  2020-08-24  2:29 [PATCH v3 0/4] pinctrl: sh-pfc: collect Renesas related CONFIGs in one place Kuninori Morimoto
@ 2020-08-24  2:30 ` Kuninori Morimoto
  2020-08-24  2:30 ` [PATCH v3 2/4] pinctrl: sh-pfc: collect Renesas related CONFIGs in one place Kuninori Morimoto
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 7+ messages in thread
From: Kuninori Morimoto @ 2020-08-24  2:30 UTC (permalink / raw)
  To: Linus Walleij, Geert Uytterhoeven; +Cc: linux-gpio, linux-renesas-soc


From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

It is "Emma Mobile EV2", not "AV2".
This patch tidyup it.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 drivers/pinctrl/sh-pfc/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pinctrl/sh-pfc/Kconfig b/drivers/pinctrl/sh-pfc/Kconfig
index 7fdc7ed8bd2e..8b2b1e1a9047 100644
--- a/drivers/pinctrl/sh-pfc/Kconfig
+++ b/drivers/pinctrl/sh-pfc/Kconfig
@@ -66,7 +66,7 @@ config PINCTRL_SH_FUNC_GPIO
 	  This enables legacy function GPIOs for SH platforms
 
 config PINCTRL_PFC_EMEV2
-	bool "Emma Mobile AV2 pin control support" if COMPILE_TEST
+	bool "Emma Mobile EV2 pin control support" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A73A4
 	bool "R-Mobile APE6 pin control support" if COMPILE_TEST
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH v3 2/4] pinctrl: sh-pfc: collect Renesas related CONFIGs in one place
  2020-08-24  2:29 [PATCH v3 0/4] pinctrl: sh-pfc: collect Renesas related CONFIGs in one place Kuninori Morimoto
  2020-08-24  2:30 ` [PATCH v3 1/4] pinctrl: sh-pfc: tidyup Emma Mobile EV2 Kuninori Morimoto
@ 2020-08-24  2:30 ` Kuninori Morimoto
  2020-08-24  3:26   ` kernel test robot
  2020-08-24  2:31 ` [PATCH v3 3/4] pinctrl: sh-pfc: align driver description title Kuninori Morimoto
  2020-08-24  2:31 ` [PATCH v3 4/4] pinctrl: sh-pfc: tidy up " Kuninori Morimoto
  3 siblings, 1 reply; 7+ messages in thread
From: Kuninori Morimoto @ 2020-08-24  2:30 UTC (permalink / raw)
  To: Linus Walleij, Geert Uytterhoeven; +Cc: linux-gpio, linux-renesas-soc

From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Renesas related pinctrl CONFIGs are located many places,
and it is confusable.
This patch collects these into same place,
and group into "Renesas pinctrl drivers" menu.
This patch also moves pinctrl-rz{a1,a2,n1}.c into sh-pfc folder.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 drivers/pinctrl/Kconfig                     | 32 ------------------
 drivers/pinctrl/Makefile                    |  3 --
 drivers/pinctrl/sh-pfc/Kconfig              | 36 +++++++++++++++++++++
 drivers/pinctrl/sh-pfc/Makefile             |  4 +++
 drivers/pinctrl/{ => sh-pfc}/pinctrl-rza1.c |  0
 drivers/pinctrl/{ => sh-pfc}/pinctrl-rza2.c |  0
 drivers/pinctrl/{ => sh-pfc}/pinctrl-rzn1.c |  0
 7 files changed, 40 insertions(+), 35 deletions(-)
 rename drivers/pinctrl/{ => sh-pfc}/pinctrl-rza1.c (100%)
 rename drivers/pinctrl/{ => sh-pfc}/pinctrl-rza2.c (100%)
 rename drivers/pinctrl/{ => sh-pfc}/pinctrl-rzn1.c (100%)

diff --git a/drivers/pinctrl/Kconfig b/drivers/pinctrl/Kconfig
index 8828613c4e0e..f63c5a04a3f7 100644
--- a/drivers/pinctrl/Kconfig
+++ b/drivers/pinctrl/Kconfig
@@ -213,38 +213,6 @@ config PINCTRL_ROCKCHIP
 	select GENERIC_IRQ_CHIP
 	select MFD_SYSCON
 
-config PINCTRL_RZA1
-	bool "Renesas RZ/A1 gpio and pinctrl driver"
-	depends on OF
-	depends on ARCH_R7S72100 || COMPILE_TEST
-	select GPIOLIB
-	select GENERIC_PINCTRL_GROUPS
-	select GENERIC_PINMUX_FUNCTIONS
-	select GENERIC_PINCONF
-	help
-	  This selects pinctrl driver for Renesas RZ/A1 platforms.
-
-config PINCTRL_RZA2
-	bool "Renesas RZ/A2 gpio and pinctrl driver"
-	depends on OF
-	depends on ARCH_R7S9210 || COMPILE_TEST
-	select GPIOLIB
-	select GENERIC_PINCTRL_GROUPS
-	select GENERIC_PINMUX_FUNCTIONS
-	select GENERIC_PINCONF
-	help
-	  This selects GPIO and pinctrl driver for Renesas RZ/A2 platforms.
-
-config PINCTRL_RZN1
-	bool "Renesas RZ/N1 pinctrl driver"
-	depends on OF
-	depends on ARCH_RZN1 || COMPILE_TEST
-	select GENERIC_PINCTRL_GROUPS
-	select GENERIC_PINMUX_FUNCTIONS
-	select GENERIC_PINCONF
-	help
-	  This selects pinctrl driver for Renesas RZ/N1 devices.
-
 config PINCTRL_SINGLE
 	tristate "One-register-per-pin type device tree based pinctrl driver"
 	depends on OF
diff --git a/drivers/pinctrl/Makefile b/drivers/pinctrl/Makefile
index 1731b2154df9..1da9f28aecbd 100644
--- a/drivers/pinctrl/Makefile
+++ b/drivers/pinctrl/Makefile
@@ -30,9 +30,6 @@ obj-$(CONFIG_PINCTRL_PALMAS)	+= pinctrl-palmas.o
 obj-$(CONFIG_PINCTRL_PIC32)	+= pinctrl-pic32.o
 obj-$(CONFIG_PINCTRL_PISTACHIO)	+= pinctrl-pistachio.o
 obj-$(CONFIG_PINCTRL_ROCKCHIP)	+= pinctrl-rockchip.o
-obj-$(CONFIG_PINCTRL_RZA1)	+= pinctrl-rza1.o
-obj-$(CONFIG_PINCTRL_RZA2)	+= pinctrl-rza2.o
-obj-$(CONFIG_PINCTRL_RZN1)	+= pinctrl-rzn1.o
 obj-$(CONFIG_PINCTRL_SINGLE)	+= pinctrl-single.o
 obj-$(CONFIG_PINCTRL_SIRF)	+= sirf/
 obj-$(CONFIG_PINCTRL_SX150X)	+= pinctrl-sx150x.o
diff --git a/drivers/pinctrl/sh-pfc/Kconfig b/drivers/pinctrl/sh-pfc/Kconfig
index 8b2b1e1a9047..ff10bb2ed497 100644
--- a/drivers/pinctrl/sh-pfc/Kconfig
+++ b/drivers/pinctrl/sh-pfc/Kconfig
@@ -3,6 +3,8 @@
 # Renesas SH and SH Mobile PINCTRL drivers
 #
 
+menu "Renesas pinctrl drivers"
+
 config PINCTRL_SH_PFC
 	bool "Renesas SoC pin control support" if COMPILE_TEST && !(ARCH_RENESAS || SUPERH)
 	default y if ARCH_RENESAS || SUPERH
@@ -53,6 +55,38 @@ config PINCTRL_SH_PFC
 	help
 	  This enables pin control drivers for Renesas SuperH and ARM platforms
 
+config PINCTRL_RZA1
+	bool "RZ/A1 gpio and pinctrl driver"
+	depends on OF
+	depends on ARCH_R7S72100 || COMPILE_TEST
+	select GPIOLIB
+	select GENERIC_PINCTRL_GROUPS
+	select GENERIC_PINMUX_FUNCTIONS
+	select GENERIC_PINCONF
+	help
+	  This selects pinctrl driver for Renesas RZ/A1 platforms.
+
+config PINCTRL_RZA2
+	bool "RZ/A2 gpio and pinctrl driver"
+	depends on OF
+	depends on ARCH_R7S9210 || COMPILE_TEST
+	select GPIOLIB
+	select GENERIC_PINCTRL_GROUPS
+	select GENERIC_PINMUX_FUNCTIONS
+	select GENERIC_PINCONF
+	help
+	  This selects GPIO and pinctrl driver for Renesas RZ/A2 platforms.
+
+config PINCTRL_RZN1
+	bool "RZ/N1 pinctrl driver"
+	depends on OF
+	depends on ARCH_RZN1 || COMPILE_TEST
+	select GENERIC_PINCTRL_GROUPS
+	select GENERIC_PINMUX_FUNCTIONS
+	select GENERIC_PINCONF
+	help
+	  This selects pinctrl driver for Renesas RZ/N1 devices.
+
 config PINCTRL_SH_PFC_GPIO
 	select GPIOLIB
 	bool
@@ -203,3 +237,5 @@ config PINCTRL_PFC_SH7786
 config PINCTRL_PFC_SHX3
 	bool "SH-X3 pin control support" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
+
+endmenu
diff --git a/drivers/pinctrl/sh-pfc/Makefile b/drivers/pinctrl/sh-pfc/Makefile
index 7bb99187cd8e..0b5640cf457b 100644
--- a/drivers/pinctrl/sh-pfc/Makefile
+++ b/drivers/pinctrl/sh-pfc/Makefile
@@ -43,6 +43,10 @@ obj-$(CONFIG_PINCTRL_PFC_SH7785)	+= pfc-sh7785.o
 obj-$(CONFIG_PINCTRL_PFC_SH7786)	+= pfc-sh7786.o
 obj-$(CONFIG_PINCTRL_PFC_SHX3)		+= pfc-shx3.o
 
+obj-$(CONFIG_PINCTRL_RZA1)	+= pinctrl-rza1.o
+obj-$(CONFIG_PINCTRL_RZA2)	+= pinctrl-rza2.o
+obj-$(CONFIG_PINCTRL_RZN1)	+= pinctrl-rzn1.o
+
 ifeq ($(CONFIG_COMPILE_TEST),y)
 CFLAGS_pfc-sh7203.o	+= -I$(srctree)/arch/sh/include/cpu-sh2a
 CFLAGS_pfc-sh7264.o	+= -I$(srctree)/arch/sh/include/cpu-sh2a
diff --git a/drivers/pinctrl/pinctrl-rza1.c b/drivers/pinctrl/sh-pfc/pinctrl-rza1.c
similarity index 100%
rename from drivers/pinctrl/pinctrl-rza1.c
rename to drivers/pinctrl/sh-pfc/pinctrl-rza1.c
diff --git a/drivers/pinctrl/pinctrl-rza2.c b/drivers/pinctrl/sh-pfc/pinctrl-rza2.c
similarity index 100%
rename from drivers/pinctrl/pinctrl-rza2.c
rename to drivers/pinctrl/sh-pfc/pinctrl-rza2.c
diff --git a/drivers/pinctrl/pinctrl-rzn1.c b/drivers/pinctrl/sh-pfc/pinctrl-rzn1.c
similarity index 100%
rename from drivers/pinctrl/pinctrl-rzn1.c
rename to drivers/pinctrl/sh-pfc/pinctrl-rzn1.c
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH v3 3/4] pinctrl: sh-pfc: align driver description title
  2020-08-24  2:29 [PATCH v3 0/4] pinctrl: sh-pfc: collect Renesas related CONFIGs in one place Kuninori Morimoto
  2020-08-24  2:30 ` [PATCH v3 1/4] pinctrl: sh-pfc: tidyup Emma Mobile EV2 Kuninori Morimoto
  2020-08-24  2:30 ` [PATCH v3 2/4] pinctrl: sh-pfc: collect Renesas related CONFIGs in one place Kuninori Morimoto
@ 2020-08-24  2:31 ` Kuninori Morimoto
  2020-08-24  2:31 ` [PATCH v3 4/4] pinctrl: sh-pfc: tidy up " Kuninori Morimoto
  3 siblings, 0 replies; 7+ messages in thread
From: Kuninori Morimoto @ 2020-08-24  2:31 UTC (permalink / raw)
  To: Linus Walleij, Geert Uytterhoeven; +Cc: linux-gpio, linux-renesas-soc


From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Now, Renesas Pin Control drivers are under menu,
but current description are not aligned.
This patch align these.

	- Emma Mobile EV2 pin control support
	- R-Mobile APE6 pin control support
	- R-Mobile A1 pin control support
	- RZ/N1 pin control support
	- RZ/G1H pin control support
	- RZ/G1M pin control support

	+ pin control support for Emma Mobile EV2
	+ pin control support for R-Mobile APE6
	+ pin control support for R-Mobile A1
	+ pin control support for RZ/N1
	+ pin control support for RZ/G1H
	+ pin control support for RZ/G1M

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 drivers/pinctrl/sh-pfc/Kconfig | 88 +++++++++++++++++-----------------
 1 file changed, 44 insertions(+), 44 deletions(-)

diff --git a/drivers/pinctrl/sh-pfc/Kconfig b/drivers/pinctrl/sh-pfc/Kconfig
index ff10bb2ed497..c27861fa557d 100644
--- a/drivers/pinctrl/sh-pfc/Kconfig
+++ b/drivers/pinctrl/sh-pfc/Kconfig
@@ -56,7 +56,7 @@ config PINCTRL_SH_PFC
 	  This enables pin control drivers for Renesas SuperH and ARM platforms
 
 config PINCTRL_RZA1
-	bool "RZ/A1 gpio and pinctrl driver"
+	bool "gpio and pinctrl driver for RZ/A1"
 	depends on OF
 	depends on ARCH_R7S72100 || COMPILE_TEST
 	select GPIOLIB
@@ -67,7 +67,7 @@ config PINCTRL_RZA1
 	  This selects pinctrl driver for Renesas RZ/A1 platforms.
 
 config PINCTRL_RZA2
-	bool "RZ/A2 gpio and pinctrl driver"
+	bool "gpio and pinctrl driver for RZ/A2"
 	depends on OF
 	depends on ARCH_R7S9210 || COMPILE_TEST
 	select GPIOLIB
@@ -78,7 +78,7 @@ config PINCTRL_RZA2
 	  This selects GPIO and pinctrl driver for Renesas RZ/A2 platforms.
 
 config PINCTRL_RZN1
-	bool "RZ/N1 pinctrl driver"
+	bool "pin control support for RZ/N1"
 	depends on OF
 	depends on ARCH_RZN1 || COMPILE_TEST
 	select GENERIC_PINCTRL_GROUPS
@@ -100,142 +100,142 @@ config PINCTRL_SH_FUNC_GPIO
 	  This enables legacy function GPIOs for SH platforms
 
 config PINCTRL_PFC_EMEV2
-	bool "Emma Mobile EV2 pin control support" if COMPILE_TEST
+	bool "pin control support for Emma Mobile EV2" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A73A4
-	bool "R-Mobile APE6 pin control support" if COMPILE_TEST
+	bool "pin control support for R-Mobile APE6" if COMPILE_TEST
 	select PINCTRL_SH_PFC_GPIO
 
 config PINCTRL_PFC_R8A7740
-	bool "R-Mobile A1 pin control support" if COMPILE_TEST
+	bool "pin control support for R-Mobile A1" if COMPILE_TEST
 	select PINCTRL_SH_PFC_GPIO
 
 config PINCTRL_PFC_R8A7742
-	bool "RZ/G1H pin control support" if COMPILE_TEST
+	bool "pin control support for RZ/G1H" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A7743
-	bool "RZ/G1M pin control support" if COMPILE_TEST
+	bool "pin control support for RZ/G1M" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A7744
-	bool "RZ/G1N pin control support" if COMPILE_TEST
+	bool "pin control support for RZ/G1N" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A7745
-	bool "RZ/G1E pin control support" if COMPILE_TEST
+	bool "pin control support for RZ/G1E" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A77470
-	bool "RZ/G1C pin control support" if COMPILE_TEST
+	bool "pin control support for RZ/G1C" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A774A1
-	bool "RZ/G2M pin control support" if COMPILE_TEST
+	bool "pin control support for RZ/G2M" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A774B1
-	bool "RZ/G2N pin control support" if COMPILE_TEST
+	bool "pin control support for RZ/G2N" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A774C0
-	bool "RZ/G2E pin control support" if COMPILE_TEST
+	bool "pin control support for RZ/G2E" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A774E1
-	bool "RZ/G2H pin control support" if COMPILE_TEST
+	bool "pin control support for RZ/G2H" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A7778
-	bool "R-Car M1A pin control support" if COMPILE_TEST
+	bool "pin control support for R-Car M1A" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A7779
-	bool "R-Car H1 pin control support" if COMPILE_TEST
+	bool "pin control support for R-Car H1" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A7790
-	bool "R-Car H2 pin control support" if COMPILE_TEST
+	bool "pin control support for R-Car H2" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A7791
-	bool "R-Car M2-W pin control support" if COMPILE_TEST
+	bool "pin control support for R-Car M2-W" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A7792
-	bool "R-Car V2H pin control support" if COMPILE_TEST
+	bool "pin control support for R-Car V2H" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A7793
-	bool "R-Car M2-N pin control support" if COMPILE_TEST
+	bool "pin control support for R-Car M2-N" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A7794
-	bool "R-Car E2 pin control support" if COMPILE_TEST
+	bool "pin control support for R-Car E2" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A77950
-	bool "R-Car H3 ES1.x pin control support" if COMPILE_TEST
+	bool "pin control support for R-Car H3 ES1.x" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A77951
-	bool "R-Car H3 ES2.0+ pin control support" if COMPILE_TEST
+	bool "pin control support for R-Car H3 ES2.0+" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A77960
-	bool "R-Car M3-W pin control support" if COMPILE_TEST
+	bool "pin control support for R-Car M3-W" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A77961
-	bool "R-Car M3-W+ pin control support" if COMPILE_TEST
+	bool "pin control support for R-Car M3-W+" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A77965
-	bool "R-Car M3-N pin control support" if COMPILE_TEST
+	bool "pin control support for R-Car M3-N" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A77970
-	bool "R-Car V3M pin control support" if COMPILE_TEST
+	bool "pin control support for R-Car V3M" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A77980
-	bool "R-Car V3H pin control support" if COMPILE_TEST
+	bool "pin control support for R-Car V3H" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A77990
-	bool "R-Car E3 pin control support" if COMPILE_TEST
+	bool "pin control support for R-Car E3" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A77995
-	bool "R-Car D3 pin control support" if COMPILE_TEST
+	bool "pin control support for R-Car D3" if COMPILE_TEST
 
 config PINCTRL_PFC_SH7203
-	bool "SH7203 pin control support" if COMPILE_TEST
+	bool "pin control support for SH7203" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
 
 config PINCTRL_PFC_SH7264
-	bool "SH7264 pin control support" if COMPILE_TEST
+	bool "pin control support for SH7264" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
 
 config PINCTRL_PFC_SH7269
-	bool "SH7269 pin control support" if COMPILE_TEST
+	bool "pin control support for SH7269" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
 
 config PINCTRL_PFC_SH73A0
-	bool "SH-Mobile AG5 pin control support" if COMPILE_TEST
+	bool "pin control support for SH-Mobile AG5" if COMPILE_TEST
 	select PINCTRL_SH_PFC_GPIO
 	select REGULATOR
 
 config PINCTRL_PFC_SH7720
-	bool "SH7720 pin control support" if COMPILE_TEST
+	bool "pin control support for SH7720" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
 
 config PINCTRL_PFC_SH7722
-	bool "SH7722 pin control support" if COMPILE_TEST
+	bool "pin control support for SH7722" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
 
 config PINCTRL_PFC_SH7723
-	bool "SH-Mobile R2 pin control support" if COMPILE_TEST
+	bool "pin control support for SH-Mobile R2" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
 
 config PINCTRL_PFC_SH7724
-	bool "SH-Mobile R2R pin control support" if COMPILE_TEST
+	bool "pin control support for SH-Mobile R2R" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
 
 config PINCTRL_PFC_SH7734
-	bool "SH7734 pin control support" if COMPILE_TEST
+	bool "pin control support for SH7734" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
 
 config PINCTRL_PFC_SH7757
-	bool "SH7757 pin control support" if COMPILE_TEST
+	bool "pin control support for SH7757" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
 
 config PINCTRL_PFC_SH7785
-	bool "SH7785 pin control support" if COMPILE_TEST
+	bool "pin control support for SH7785" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
 
 config PINCTRL_PFC_SH7786
-	bool "SH7786 pin control support" if COMPILE_TEST
+	bool "pin control support for SH7786" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
 
 config PINCTRL_PFC_SHX3
-	bool "SH-X3 pin control support" if COMPILE_TEST
+	bool "pin control support for SH-X3" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
 
 endmenu
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH v3 4/4] pinctrl: sh-pfc: tidy up driver description title
  2020-08-24  2:29 [PATCH v3 0/4] pinctrl: sh-pfc: collect Renesas related CONFIGs in one place Kuninori Morimoto
                   ` (2 preceding siblings ...)
  2020-08-24  2:31 ` [PATCH v3 3/4] pinctrl: sh-pfc: align driver description title Kuninori Morimoto
@ 2020-08-24  2:31 ` Kuninori Morimoto
  3 siblings, 0 replies; 7+ messages in thread
From: Kuninori Morimoto @ 2020-08-24  2:31 UTC (permalink / raw)
  To: Linus Walleij, Geert Uytterhoeven; +Cc: linux-gpio, linux-renesas-soc


From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

This patch sorts each driver by description title alphabetical order.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 drivers/pinctrl/sh-pfc/Kconfig | 121 +++++++++++++++++----------------
 1 file changed, 62 insertions(+), 59 deletions(-)

diff --git a/drivers/pinctrl/sh-pfc/Kconfig b/drivers/pinctrl/sh-pfc/Kconfig
index c27861fa557d..5c2a8916e439 100644
--- a/drivers/pinctrl/sh-pfc/Kconfig
+++ b/drivers/pinctrl/sh-pfc/Kconfig
@@ -77,16 +77,6 @@ config PINCTRL_RZA2
 	help
 	  This selects GPIO and pinctrl driver for Renesas RZ/A2 platforms.
 
-config PINCTRL_RZN1
-	bool "pin control support for RZ/N1"
-	depends on OF
-	depends on ARCH_RZN1 || COMPILE_TEST
-	select GENERIC_PINCTRL_GROUPS
-	select GENERIC_PINMUX_FUNCTIONS
-	select GENERIC_PINCONF
-	help
-	  This selects pinctrl driver for Renesas RZ/N1 devices.
-
 config PINCTRL_SH_PFC_GPIO
 	select GPIOLIB
 	bool
@@ -102,13 +92,13 @@ config PINCTRL_SH_FUNC_GPIO
 config PINCTRL_PFC_EMEV2
 	bool "pin control support for Emma Mobile EV2" if COMPILE_TEST
 
-config PINCTRL_PFC_R8A73A4
-	bool "pin control support for R-Mobile APE6" if COMPILE_TEST
-	select PINCTRL_SH_PFC_GPIO
 
-config PINCTRL_PFC_R8A7740
-	bool "pin control support for R-Mobile A1" if COMPILE_TEST
-	select PINCTRL_SH_PFC_GPIO
+
+config PINCTRL_PFC_R8A77470
+	bool "pin control support for RZ/G1C" if COMPILE_TEST
+
+config PINCTRL_PFC_R8A7745
+	bool "pin control support for RZ/G1E" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A7742
 	bool "pin control support for RZ/G1H" if COMPILE_TEST
@@ -119,11 +109,11 @@ config PINCTRL_PFC_R8A7743
 config PINCTRL_PFC_R8A7744
 	bool "pin control support for RZ/G1N" if COMPILE_TEST
 
-config PINCTRL_PFC_R8A7745
-	bool "pin control support for RZ/G1E" if COMPILE_TEST
+config PINCTRL_PFC_R8A774C0
+	bool "pin control support for RZ/G2E" if COMPILE_TEST
 
-config PINCTRL_PFC_R8A77470
-	bool "pin control support for RZ/G1C" if COMPILE_TEST
+config PINCTRL_PFC_R8A774E1
+	bool "pin control support for RZ/G2H" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A774A1
 	bool "pin control support for RZ/G2M" if COMPILE_TEST
@@ -131,14 +121,25 @@ config PINCTRL_PFC_R8A774A1
 config PINCTRL_PFC_R8A774B1
 	bool "pin control support for RZ/G2N" if COMPILE_TEST
 
-config PINCTRL_PFC_R8A774C0
-	bool "pin control support for RZ/G2E" if COMPILE_TEST
+config PINCTRL_RZN1
+	bool "pin control support for RZ/N1"
+	depends on OF
+	depends on ARCH_RZN1 || COMPILE_TEST
+	select GENERIC_PINCTRL_GROUPS
+	select GENERIC_PINMUX_FUNCTIONS
+	select GENERIC_PINCONF
+	help
+	  This selects pinctrl driver for Renesas RZ/N1 devices.
 
-config PINCTRL_PFC_R8A774E1
-	bool "pin control support for RZ/G2H" if COMPILE_TEST
 
-config PINCTRL_PFC_R8A7778
-	bool "pin control support for R-Car M1A" if COMPILE_TEST
+config PINCTRL_PFC_R8A77995
+	bool "pin control support for R-Car D3" if COMPILE_TEST
+
+config PINCTRL_PFC_R8A7794
+	bool "pin control support for R-Car E2" if COMPILE_TEST
+
+config PINCTRL_PFC_R8A77990
+	bool "pin control support for R-Car E3" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A7779
 	bool "pin control support for R-Car H1" if COMPILE_TEST
@@ -146,23 +147,23 @@ config PINCTRL_PFC_R8A7779
 config PINCTRL_PFC_R8A7790
 	bool "pin control support for R-Car H2" if COMPILE_TEST
 
-config PINCTRL_PFC_R8A7791
-	bool "pin control support for R-Car M2-W" if COMPILE_TEST
+config PINCTRL_PFC_R8A77950
+	bool "pin control support for R-Car H3 ES1.x" if COMPILE_TEST
 
-config PINCTRL_PFC_R8A7792
-	bool "pin control support for R-Car V2H" if COMPILE_TEST
+config PINCTRL_PFC_R8A77951
+	bool "pin control support for R-Car H3 ES2.0+" if COMPILE_TEST
+
+config PINCTRL_PFC_R8A7778
+	bool "pin control support for R-Car M1A" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A7793
 	bool "pin control support for R-Car M2-N" if COMPILE_TEST
 
-config PINCTRL_PFC_R8A7794
-	bool "pin control support for R-Car E2" if COMPILE_TEST
-
-config PINCTRL_PFC_R8A77950
-	bool "pin control support for R-Car H3 ES1.x" if COMPILE_TEST
+config PINCTRL_PFC_R8A7791
+	bool "pin control support for R-Car M2-W" if COMPILE_TEST
 
-config PINCTRL_PFC_R8A77951
-	bool "pin control support for R-Car H3 ES2.0+" if COMPILE_TEST
+config PINCTRL_PFC_R8A77965
+	bool "pin control support for R-Car M3-N" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A77960
 	bool "pin control support for R-Car M3-W" if COMPILE_TEST
@@ -170,20 +171,22 @@ config PINCTRL_PFC_R8A77960
 config PINCTRL_PFC_R8A77961
 	bool "pin control support for R-Car M3-W+" if COMPILE_TEST
 
-config PINCTRL_PFC_R8A77965
-	bool "pin control support for R-Car M3-N" if COMPILE_TEST
-
-config PINCTRL_PFC_R8A77970
-	bool "pin control support for R-Car V3M" if COMPILE_TEST
+config PINCTRL_PFC_R8A7792
+	bool "pin control support for R-Car V2H" if COMPILE_TEST
 
 config PINCTRL_PFC_R8A77980
 	bool "pin control support for R-Car V3H" if COMPILE_TEST
 
-config PINCTRL_PFC_R8A77990
-	bool "pin control support for R-Car E3" if COMPILE_TEST
+config PINCTRL_PFC_R8A77970
+	bool "pin control support for R-Car V3M" if COMPILE_TEST
 
-config PINCTRL_PFC_R8A77995
-	bool "pin control support for R-Car D3" if COMPILE_TEST
+config PINCTRL_PFC_R8A7740
+	bool "pin control support for R-Mobile A1" if COMPILE_TEST
+	select PINCTRL_SH_PFC_GPIO
+
+config PINCTRL_PFC_R8A73A4
+	bool "pin control support for R-Mobile APE6" if COMPILE_TEST
+	select PINCTRL_SH_PFC_GPIO
 
 config PINCTRL_PFC_SH7203
 	bool "pin control support for SH7203" if COMPILE_TEST
@@ -197,11 +200,6 @@ config PINCTRL_PFC_SH7269
 	bool "pin control support for SH7269" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
 
-config PINCTRL_PFC_SH73A0
-	bool "pin control support for SH-Mobile AG5" if COMPILE_TEST
-	select PINCTRL_SH_PFC_GPIO
-	select REGULATOR
-
 config PINCTRL_PFC_SH7720
 	bool "pin control support for SH7720" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
@@ -210,14 +208,6 @@ config PINCTRL_PFC_SH7722
 	bool "pin control support for SH7722" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
 
-config PINCTRL_PFC_SH7723
-	bool "pin control support for SH-Mobile R2" if COMPILE_TEST
-	select PINCTRL_SH_FUNC_GPIO
-
-config PINCTRL_PFC_SH7724
-	bool "pin control support for SH-Mobile R2R" if COMPILE_TEST
-	select PINCTRL_SH_FUNC_GPIO
-
 config PINCTRL_PFC_SH7734
 	bool "pin control support for SH7734" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
@@ -234,6 +224,19 @@ config PINCTRL_PFC_SH7786
 	bool "pin control support for SH7786" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
 
+config PINCTRL_PFC_SH73A0
+	bool "pin control support for SH-Mobile AG5" if COMPILE_TEST
+	select PINCTRL_SH_PFC_GPIO
+	select REGULATOR
+
+config PINCTRL_PFC_SH7723
+	bool "pin control support for SH-Mobile R2" if COMPILE_TEST
+	select PINCTRL_SH_FUNC_GPIO
+
+config PINCTRL_PFC_SH7724
+	bool "pin control support for SH-Mobile R2R" if COMPILE_TEST
+	select PINCTRL_SH_FUNC_GPIO
+
 config PINCTRL_PFC_SHX3
 	bool "pin control support for SH-X3" if COMPILE_TEST
 	select PINCTRL_SH_FUNC_GPIO
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH v3 2/4] pinctrl: sh-pfc: collect Renesas related CONFIGs in one place
  2020-08-24  2:30 ` [PATCH v3 2/4] pinctrl: sh-pfc: collect Renesas related CONFIGs in one place Kuninori Morimoto
@ 2020-08-24  3:26   ` kernel test robot
  2020-08-24  5:28     ` Kuninori Morimoto
  0 siblings, 1 reply; 7+ messages in thread
From: kernel test robot @ 2020-08-24  3:26 UTC (permalink / raw)
  To: Kuninori Morimoto, Linus Walleij, Geert Uytterhoeven
  Cc: kbuild-all, linux-gpio, linux-renesas-soc

[-- Attachment #1: Type: text/plain, Size: 2741 bytes --]

Hi Kuninori,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on renesas-drivers/sh-pfc]
[also build test ERROR on pinctrl/devel v5.9-rc2 next-20200821]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Kuninori-Morimoto/pinctrl-sh-pfc-collect-Renesas-related-CONFIGs-in-one-place/20200824-103625
base:   https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git sh-pfc
config: arc-allyesconfig (attached as .config)
compiler: arc-elf-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arc 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

>> drivers/pinctrl/sh-pfc/pinctrl-rza1.c:30:10: fatal error: devicetree.h: No such file or directory
      30 | #include "devicetree.h"
         |          ^~~~~~~~~~~~~~
   compilation terminated.
--
>> drivers/pinctrl/sh-pfc/pinctrl-rza2.c:21:10: fatal error: pinmux.h: No such file or directory
      21 | #include "pinmux.h"
         |          ^~~~~~~~~~
   compilation terminated.
--
>> drivers/pinctrl/sh-pfc/pinctrl-rzn1.c:21:10: fatal error: pinconf.h: No such file or directory
      21 | #include "pinconf.h"
         |          ^~~~~~~~~~~
   compilation terminated.

# https://github.com/0day-ci/linux/commit/0eebf3dd4b399ae6e2278063620b198fb0a28b24
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Kuninori-Morimoto/pinctrl-sh-pfc-collect-Renesas-related-CONFIGs-in-one-place/20200824-103625
git checkout 0eebf3dd4b399ae6e2278063620b198fb0a28b24
vim +30 drivers/pinctrl/sh-pfc/pinctrl-rza1.c

5a49b644b3075f drivers/pinctrl/pinctrl-rza1.c Jacopo Mondi 2017-06-22  28  
5a49b644b3075f drivers/pinctrl/pinctrl-rza1.c Jacopo Mondi 2017-06-22  29  #include "core.h"
5a49b644b3075f drivers/pinctrl/pinctrl-rza1.c Jacopo Mondi 2017-06-22 @30  #include "devicetree.h"
5a49b644b3075f drivers/pinctrl/pinctrl-rza1.c Jacopo Mondi 2017-06-22  31  #include "pinconf.h"
5a49b644b3075f drivers/pinctrl/pinctrl-rza1.c Jacopo Mondi 2017-06-22  32  #include "pinmux.h"
5a49b644b3075f drivers/pinctrl/pinctrl-rza1.c Jacopo Mondi 2017-06-22  33  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 65326 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v3 2/4] pinctrl: sh-pfc: collect Renesas related CONFIGs in one place
  2020-08-24  3:26   ` kernel test robot
@ 2020-08-24  5:28     ` Kuninori Morimoto
  0 siblings, 0 replies; 7+ messages in thread
From: Kuninori Morimoto @ 2020-08-24  5:28 UTC (permalink / raw)
  To: kernel test robot
  Cc: Linus Walleij, Geert Uytterhoeven, kbuild-all, linux-gpio,
	linux-renesas-soc


Hi

> All errors (new ones prefixed by >>):
> 
> >> drivers/pinctrl/sh-pfc/pinctrl-rza1.c:30:10: fatal error: devicetree.h: No such file or directory
>       30 | #include "devicetree.h"
>          |          ^~~~~~~~~~~~~~
>    compilation terminated.
> --
> >> drivers/pinctrl/sh-pfc/pinctrl-rza2.c:21:10: fatal error: pinmux.h: No such file or directory
>       21 | #include "pinmux.h"
>          |          ^~~~~~~~~~
>    compilation terminated.
> --
> >> drivers/pinctrl/sh-pfc/pinctrl-rzn1.c:21:10: fatal error: pinconf.h: No such file or directory
>       21 | #include "pinconf.h"
>          |          ^~~~~~~~~~~
>    compilation terminated.

Grr my bad.
Will fix in v4

Thank you for your help !!

Best regards
---
Kuninori Morimoto

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2020-08-24  5:28 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-24  2:29 [PATCH v3 0/4] pinctrl: sh-pfc: collect Renesas related CONFIGs in one place Kuninori Morimoto
2020-08-24  2:30 ` [PATCH v3 1/4] pinctrl: sh-pfc: tidyup Emma Mobile EV2 Kuninori Morimoto
2020-08-24  2:30 ` [PATCH v3 2/4] pinctrl: sh-pfc: collect Renesas related CONFIGs in one place Kuninori Morimoto
2020-08-24  3:26   ` kernel test robot
2020-08-24  5:28     ` Kuninori Morimoto
2020-08-24  2:31 ` [PATCH v3 3/4] pinctrl: sh-pfc: align driver description title Kuninori Morimoto
2020-08-24  2:31 ` [PATCH v3 4/4] pinctrl: sh-pfc: tidy up " 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).