linux-gpio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] pinctrl: mediatek: add missing options to PINCTRL_MT7981
@ 2023-02-18  6:51 arinc9.unal
  2023-02-18  6:51 ` [PATCH 2/2] pinctrl: mediatek: fix naming inconsistency arinc9.unal
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: arinc9.unal @ 2023-02-18  6:51 UTC (permalink / raw)
  To: Sean Wang, Linus Walleij, Matthias Brugger
  Cc: Arınç ÜNAL, linux-mediatek, linux-gpio,
	linux-kernel, linux-arm-kernel, Daniel Golle, erkin.bozoglu

From: Arınç ÜNAL <arinc.unal@arinc9.com>

There are options missing from PINCTRL_MT7981 whilst being on every other
pin controller. Add them.

Signed-off-by: Arınç ÜNAL <arinc.unal@arinc9.com>
---
 drivers/pinctrl/mediatek/Kconfig | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/pinctrl/mediatek/Kconfig b/drivers/pinctrl/mediatek/Kconfig
index f20c28334bcb..67818ba14d4e 100644
--- a/drivers/pinctrl/mediatek/Kconfig
+++ b/drivers/pinctrl/mediatek/Kconfig
@@ -130,6 +130,8 @@ config PINCTRL_MT7622
 config PINCTRL_MT7981
 	bool "Mediatek MT7981 pin control"
 	depends on OF
+	depends on ARM64 || COMPILE_TEST
+	default ARM64 && ARCH_MEDIATEK
 	select PINCTRL_MTK_MOORE
 
 config PINCTRL_MT7986
-- 
2.37.2


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

* [PATCH 2/2] pinctrl: mediatek: fix naming inconsistency
  2023-02-18  6:51 [PATCH 1/2] pinctrl: mediatek: add missing options to PINCTRL_MT7981 arinc9.unal
@ 2023-02-18  6:51 ` arinc9.unal
  2023-02-18 17:32   ` Daniel Golle
  2023-03-06 13:23   ` Linus Walleij
  2023-02-18 17:31 ` [PATCH 1/2] pinctrl: mediatek: add missing options to PINCTRL_MT7981 Daniel Golle
  2023-03-06 13:23 ` Linus Walleij
  2 siblings, 2 replies; 6+ messages in thread
From: arinc9.unal @ 2023-02-18  6:51 UTC (permalink / raw)
  To: Sean Wang, Linus Walleij, Matthias Brugger
  Cc: Arınç ÜNAL, linux-mediatek, linux-gpio,
	linux-kernel, linux-arm-kernel, Daniel Golle, erkin.bozoglu

From: Arınç ÜNAL <arinc.unal@arinc9.com>

Some options include "MediaTek", some "Mediatek". Rename all to "MediaTek"
to address the naming inconsistency.

Signed-off-by: Arınç ÜNAL <arinc.unal@arinc9.com>
---
 drivers/pinctrl/mediatek/Kconfig | 42 ++++++++++++++++----------------
 1 file changed, 21 insertions(+), 21 deletions(-)

diff --git a/drivers/pinctrl/mediatek/Kconfig b/drivers/pinctrl/mediatek/Kconfig
index 67818ba14d4e..a71874fed3d6 100644
--- a/drivers/pinctrl/mediatek/Kconfig
+++ b/drivers/pinctrl/mediatek/Kconfig
@@ -45,35 +45,35 @@ config PINCTRL_MTK_PARIS
 
 # For ARMv7 SoCs
 config PINCTRL_MT2701
-	bool "Mediatek MT2701 pin control"
+	bool "MediaTek MT2701 pin control"
 	depends on MACH_MT7623 || MACH_MT2701 || COMPILE_TEST
 	depends on OF
 	default MACH_MT2701
 	select PINCTRL_MTK
 
 config PINCTRL_MT7623
-	bool "Mediatek MT7623 pin control with generic binding"
+	bool "MediaTek MT7623 pin control with generic binding"
 	depends on MACH_MT7623 || COMPILE_TEST
 	depends on OF
 	default MACH_MT7623
 	select PINCTRL_MTK_MOORE
 
 config PINCTRL_MT7629
-	bool "Mediatek MT7629 pin control"
+	bool "MediaTek MT7629 pin control"
 	depends on MACH_MT7629 || COMPILE_TEST
 	depends on OF
 	default MACH_MT7629
 	select PINCTRL_MTK_MOORE
 
 config PINCTRL_MT8135
-	bool "Mediatek MT8135 pin control"
+	bool "MediaTek MT8135 pin control"
 	depends on MACH_MT8135 || COMPILE_TEST
 	depends on OF
 	default MACH_MT8135
 	select PINCTRL_MTK
 
 config PINCTRL_MT8127
-	bool "Mediatek MT8127 pin control"
+	bool "MediaTek MT8127 pin control"
 	depends on MACH_MT8127 || COMPILE_TEST
 	depends on OF
 	default MACH_MT8127
@@ -88,33 +88,33 @@ config PINCTRL_MT2712
 	select PINCTRL_MTK
 
 config PINCTRL_MT6765
-	tristate "Mediatek MT6765 pin control"
+	tristate "MediaTek MT6765 pin control"
 	depends on OF
 	depends on ARM64 || COMPILE_TEST
 	default ARM64 && ARCH_MEDIATEK
 	select PINCTRL_MTK_PARIS
 
 config PINCTRL_MT6779
-	tristate "Mediatek MT6779 pin control"
+	tristate "MediaTek MT6779 pin control"
 	depends on OF
 	depends on ARM64 || COMPILE_TEST
 	default ARM64 && ARCH_MEDIATEK
 	select PINCTRL_MTK_PARIS
 	help
 	  Say yes here to support pin controller and gpio driver
-	  on Mediatek MT6779 SoC.
+	  on MediaTek MT6779 SoC.
 	  In MTK platform, we support virtual gpio and use it to
 	  map specific eint which doesn't have real gpio pin.
 
 config PINCTRL_MT6795
-	bool "Mediatek MT6795 pin control"
+	bool "MediaTek MT6795 pin control"
 	depends on OF
 	depends on ARM64 || COMPILE_TEST
 	default ARM64 && ARCH_MEDIATEK
 	select PINCTRL_MTK_PARIS
 
 config PINCTRL_MT6797
-	bool "Mediatek MT6797 pin control"
+	bool "MediaTek MT6797 pin control"
 	depends on OF
 	depends on ARM64 || COMPILE_TEST
 	default ARM64 && ARCH_MEDIATEK
@@ -128,42 +128,42 @@ config PINCTRL_MT7622
 	select PINCTRL_MTK_MOORE
 
 config PINCTRL_MT7981
-	bool "Mediatek MT7981 pin control"
+	bool "MediaTek MT7981 pin control"
 	depends on OF
 	depends on ARM64 || COMPILE_TEST
 	default ARM64 && ARCH_MEDIATEK
 	select PINCTRL_MTK_MOORE
 
 config PINCTRL_MT7986
-	bool "Mediatek MT7986 pin control"
+	bool "MediaTek MT7986 pin control"
 	depends on OF
 	depends on ARM64 || COMPILE_TEST
 	default ARM64 && ARCH_MEDIATEK
 	select PINCTRL_MTK_MOORE
 
 config PINCTRL_MT8167
-	bool "Mediatek MT8167 pin control"
+	bool "MediaTek MT8167 pin control"
 	depends on OF
 	depends on ARM64 || COMPILE_TEST
 	default ARM64 && ARCH_MEDIATEK
 	select PINCTRL_MTK
 
 config PINCTRL_MT8173
-	bool "Mediatek MT8173 pin control"
+	bool "MediaTek MT8173 pin control"
 	depends on OF
 	depends on ARM64 || COMPILE_TEST
 	default ARM64 && ARCH_MEDIATEK
 	select PINCTRL_MTK
 
 config PINCTRL_MT8183
-	bool "Mediatek MT8183 pin control"
+	bool "MediaTek MT8183 pin control"
 	depends on OF
 	depends on ARM64 || COMPILE_TEST
 	default ARM64 && ARCH_MEDIATEK
 	select PINCTRL_MTK_PARIS
 
 config PINCTRL_MT8186
-	bool "Mediatek MT8186 pin control"
+	bool "MediaTek MT8186 pin control"
 	depends on OF
 	depends on ARM64 || COMPILE_TEST
 	default ARM64 && ARCH_MEDIATEK
@@ -182,28 +182,28 @@ config PINCTRL_MT8188
 	  map specific eint which doesn't have real gpio pin.
 
 config PINCTRL_MT8192
-	bool "Mediatek MT8192 pin control"
+	bool "MediaTek MT8192 pin control"
 	depends on OF
 	depends on ARM64 || COMPILE_TEST
 	default ARM64 && ARCH_MEDIATEK
 	select PINCTRL_MTK_PARIS
 
 config PINCTRL_MT8195
-	bool "Mediatek MT8195 pin control"
+	bool "MediaTek MT8195 pin control"
 	depends on OF
 	depends on ARM64 || COMPILE_TEST
 	default ARM64 && ARCH_MEDIATEK
 	select PINCTRL_MTK_PARIS
 
 config PINCTRL_MT8365
-	bool "Mediatek MT8365 pin control"
+	bool "MediaTek MT8365 pin control"
 	depends on OF
 	depends on ARM64 || COMPILE_TEST
 	default ARM64 && ARCH_MEDIATEK
 	select PINCTRL_MTK
 
 config PINCTRL_MT8516
-	bool "Mediatek MT8516 pin control"
+	bool "MediaTek MT8516 pin control"
 	depends on OF
 	depends on ARM64 || COMPILE_TEST
 	default ARM64 && ARCH_MEDIATEK
@@ -211,7 +211,7 @@ config PINCTRL_MT8516
 
 # For PMIC
 config PINCTRL_MT6397
-	bool "Mediatek MT6397 pin control"
+	bool "MediaTek MT6397 pin control"
 	depends on MFD_MT6397 || COMPILE_TEST
 	depends on OF
 	default MFD_MT6397
-- 
2.37.2


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

* Re: [PATCH 1/2] pinctrl: mediatek: add missing options to PINCTRL_MT7981
  2023-02-18  6:51 [PATCH 1/2] pinctrl: mediatek: add missing options to PINCTRL_MT7981 arinc9.unal
  2023-02-18  6:51 ` [PATCH 2/2] pinctrl: mediatek: fix naming inconsistency arinc9.unal
@ 2023-02-18 17:31 ` Daniel Golle
  2023-03-06 13:23 ` Linus Walleij
  2 siblings, 0 replies; 6+ messages in thread
From: Daniel Golle @ 2023-02-18 17:31 UTC (permalink / raw)
  To: arinc9.unal
  Cc: Sean Wang, Linus Walleij, Matthias Brugger,
	Arınç ÜNAL, linux-mediatek, linux-gpio,
	linux-kernel, linux-arm-kernel, erkin.bozoglu

On Sat, Feb 18, 2023 at 09:51:06AM +0300, arinc9.unal@gmail.com wrote:
> From: Arınç ÜNAL <arinc.unal@arinc9.com>
> 
> There are options missing from PINCTRL_MT7981 whilst being on every other
> pin controller. Add them.
> 

Acked-by: Daniel Golle <daniel@makrotopia.org>

> Signed-off-by: Arınç ÜNAL <arinc.unal@arinc9.com>
> ---
>  drivers/pinctrl/mediatek/Kconfig | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/pinctrl/mediatek/Kconfig b/drivers/pinctrl/mediatek/Kconfig
> index f20c28334bcb..67818ba14d4e 100644
> --- a/drivers/pinctrl/mediatek/Kconfig
> +++ b/drivers/pinctrl/mediatek/Kconfig
> @@ -130,6 +130,8 @@ config PINCTRL_MT7622
>  config PINCTRL_MT7981
>  	bool "Mediatek MT7981 pin control"
>  	depends on OF
> +	depends on ARM64 || COMPILE_TEST
> +	default ARM64 && ARCH_MEDIATEK
>  	select PINCTRL_MTK_MOORE
>  
>  config PINCTRL_MT7986
> -- 
> 2.37.2
> 

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

* Re: [PATCH 2/2] pinctrl: mediatek: fix naming inconsistency
  2023-02-18  6:51 ` [PATCH 2/2] pinctrl: mediatek: fix naming inconsistency arinc9.unal
@ 2023-02-18 17:32   ` Daniel Golle
  2023-03-06 13:23   ` Linus Walleij
  1 sibling, 0 replies; 6+ messages in thread
From: Daniel Golle @ 2023-02-18 17:32 UTC (permalink / raw)
  To: arinc9.unal
  Cc: Sean Wang, Linus Walleij, Matthias Brugger,
	Arınç ÜNAL, linux-mediatek, linux-gpio,
	linux-kernel, linux-arm-kernel, erkin.bozoglu

On Sat, Feb 18, 2023 at 09:51:07AM +0300, arinc9.unal@gmail.com wrote:
> From: Arınç ÜNAL <arinc.unal@arinc9.com>
> 
> Some options include "MediaTek", some "Mediatek". Rename all to "MediaTek"
> to address the naming inconsistency.
> 

Reviewed-by: Daniel Golle <daniel@makrotopia.org>

> Signed-off-by: Arınç ÜNAL <arinc.unal@arinc9.com>
> ---
>  drivers/pinctrl/mediatek/Kconfig | 42 ++++++++++++++++----------------
>  1 file changed, 21 insertions(+), 21 deletions(-)
> 
> diff --git a/drivers/pinctrl/mediatek/Kconfig b/drivers/pinctrl/mediatek/Kconfig
> index 67818ba14d4e..a71874fed3d6 100644
> --- a/drivers/pinctrl/mediatek/Kconfig
> +++ b/drivers/pinctrl/mediatek/Kconfig
> @@ -45,35 +45,35 @@ config PINCTRL_MTK_PARIS
>  
>  # For ARMv7 SoCs
>  config PINCTRL_MT2701
> -	bool "Mediatek MT2701 pin control"
> +	bool "MediaTek MT2701 pin control"
>  	depends on MACH_MT7623 || MACH_MT2701 || COMPILE_TEST
>  	depends on OF
>  	default MACH_MT2701
>  	select PINCTRL_MTK
>  
>  config PINCTRL_MT7623
> -	bool "Mediatek MT7623 pin control with generic binding"
> +	bool "MediaTek MT7623 pin control with generic binding"
>  	depends on MACH_MT7623 || COMPILE_TEST
>  	depends on OF
>  	default MACH_MT7623
>  	select PINCTRL_MTK_MOORE
>  
>  config PINCTRL_MT7629
> -	bool "Mediatek MT7629 pin control"
> +	bool "MediaTek MT7629 pin control"
>  	depends on MACH_MT7629 || COMPILE_TEST
>  	depends on OF
>  	default MACH_MT7629
>  	select PINCTRL_MTK_MOORE
>  
>  config PINCTRL_MT8135
> -	bool "Mediatek MT8135 pin control"
> +	bool "MediaTek MT8135 pin control"
>  	depends on MACH_MT8135 || COMPILE_TEST
>  	depends on OF
>  	default MACH_MT8135
>  	select PINCTRL_MTK
>  
>  config PINCTRL_MT8127
> -	bool "Mediatek MT8127 pin control"
> +	bool "MediaTek MT8127 pin control"
>  	depends on MACH_MT8127 || COMPILE_TEST
>  	depends on OF
>  	default MACH_MT8127
> @@ -88,33 +88,33 @@ config PINCTRL_MT2712
>  	select PINCTRL_MTK
>  
>  config PINCTRL_MT6765
> -	tristate "Mediatek MT6765 pin control"
> +	tristate "MediaTek MT6765 pin control"
>  	depends on OF
>  	depends on ARM64 || COMPILE_TEST
>  	default ARM64 && ARCH_MEDIATEK
>  	select PINCTRL_MTK_PARIS
>  
>  config PINCTRL_MT6779
> -	tristate "Mediatek MT6779 pin control"
> +	tristate "MediaTek MT6779 pin control"
>  	depends on OF
>  	depends on ARM64 || COMPILE_TEST
>  	default ARM64 && ARCH_MEDIATEK
>  	select PINCTRL_MTK_PARIS
>  	help
>  	  Say yes here to support pin controller and gpio driver
> -	  on Mediatek MT6779 SoC.
> +	  on MediaTek MT6779 SoC.
>  	  In MTK platform, we support virtual gpio and use it to
>  	  map specific eint which doesn't have real gpio pin.
>  
>  config PINCTRL_MT6795
> -	bool "Mediatek MT6795 pin control"
> +	bool "MediaTek MT6795 pin control"
>  	depends on OF
>  	depends on ARM64 || COMPILE_TEST
>  	default ARM64 && ARCH_MEDIATEK
>  	select PINCTRL_MTK_PARIS
>  
>  config PINCTRL_MT6797
> -	bool "Mediatek MT6797 pin control"
> +	bool "MediaTek MT6797 pin control"
>  	depends on OF
>  	depends on ARM64 || COMPILE_TEST
>  	default ARM64 && ARCH_MEDIATEK
> @@ -128,42 +128,42 @@ config PINCTRL_MT7622
>  	select PINCTRL_MTK_MOORE
>  
>  config PINCTRL_MT7981
> -	bool "Mediatek MT7981 pin control"
> +	bool "MediaTek MT7981 pin control"
>  	depends on OF
>  	depends on ARM64 || COMPILE_TEST
>  	default ARM64 && ARCH_MEDIATEK
>  	select PINCTRL_MTK_MOORE
>  
>  config PINCTRL_MT7986
> -	bool "Mediatek MT7986 pin control"
> +	bool "MediaTek MT7986 pin control"
>  	depends on OF
>  	depends on ARM64 || COMPILE_TEST
>  	default ARM64 && ARCH_MEDIATEK
>  	select PINCTRL_MTK_MOORE
>  
>  config PINCTRL_MT8167
> -	bool "Mediatek MT8167 pin control"
> +	bool "MediaTek MT8167 pin control"
>  	depends on OF
>  	depends on ARM64 || COMPILE_TEST
>  	default ARM64 && ARCH_MEDIATEK
>  	select PINCTRL_MTK
>  
>  config PINCTRL_MT8173
> -	bool "Mediatek MT8173 pin control"
> +	bool "MediaTek MT8173 pin control"
>  	depends on OF
>  	depends on ARM64 || COMPILE_TEST
>  	default ARM64 && ARCH_MEDIATEK
>  	select PINCTRL_MTK
>  
>  config PINCTRL_MT8183
> -	bool "Mediatek MT8183 pin control"
> +	bool "MediaTek MT8183 pin control"
>  	depends on OF
>  	depends on ARM64 || COMPILE_TEST
>  	default ARM64 && ARCH_MEDIATEK
>  	select PINCTRL_MTK_PARIS
>  
>  config PINCTRL_MT8186
> -	bool "Mediatek MT8186 pin control"
> +	bool "MediaTek MT8186 pin control"
>  	depends on OF
>  	depends on ARM64 || COMPILE_TEST
>  	default ARM64 && ARCH_MEDIATEK
> @@ -182,28 +182,28 @@ config PINCTRL_MT8188
>  	  map specific eint which doesn't have real gpio pin.
>  
>  config PINCTRL_MT8192
> -	bool "Mediatek MT8192 pin control"
> +	bool "MediaTek MT8192 pin control"
>  	depends on OF
>  	depends on ARM64 || COMPILE_TEST
>  	default ARM64 && ARCH_MEDIATEK
>  	select PINCTRL_MTK_PARIS
>  
>  config PINCTRL_MT8195
> -	bool "Mediatek MT8195 pin control"
> +	bool "MediaTek MT8195 pin control"
>  	depends on OF
>  	depends on ARM64 || COMPILE_TEST
>  	default ARM64 && ARCH_MEDIATEK
>  	select PINCTRL_MTK_PARIS
>  
>  config PINCTRL_MT8365
> -	bool "Mediatek MT8365 pin control"
> +	bool "MediaTek MT8365 pin control"
>  	depends on OF
>  	depends on ARM64 || COMPILE_TEST
>  	default ARM64 && ARCH_MEDIATEK
>  	select PINCTRL_MTK
>  
>  config PINCTRL_MT8516
> -	bool "Mediatek MT8516 pin control"
> +	bool "MediaTek MT8516 pin control"
>  	depends on OF
>  	depends on ARM64 || COMPILE_TEST
>  	default ARM64 && ARCH_MEDIATEK
> @@ -211,7 +211,7 @@ config PINCTRL_MT8516
>  
>  # For PMIC
>  config PINCTRL_MT6397
> -	bool "Mediatek MT6397 pin control"
> +	bool "MediaTek MT6397 pin control"
>  	depends on MFD_MT6397 || COMPILE_TEST
>  	depends on OF
>  	default MFD_MT6397
> -- 
> 2.37.2
> 

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

* Re: [PATCH 1/2] pinctrl: mediatek: add missing options to PINCTRL_MT7981
  2023-02-18  6:51 [PATCH 1/2] pinctrl: mediatek: add missing options to PINCTRL_MT7981 arinc9.unal
  2023-02-18  6:51 ` [PATCH 2/2] pinctrl: mediatek: fix naming inconsistency arinc9.unal
  2023-02-18 17:31 ` [PATCH 1/2] pinctrl: mediatek: add missing options to PINCTRL_MT7981 Daniel Golle
@ 2023-03-06 13:23 ` Linus Walleij
  2 siblings, 0 replies; 6+ messages in thread
From: Linus Walleij @ 2023-03-06 13:23 UTC (permalink / raw)
  To: arinc9.unal
  Cc: Sean Wang, Matthias Brugger, Arınç ÜNAL,
	linux-mediatek, linux-gpio, linux-kernel, linux-arm-kernel,
	Daniel Golle, erkin.bozoglu

On Sat, Feb 18, 2023 at 7:51 AM <arinc9.unal@gmail.com> wrote:

> From: Arınç ÜNAL <arinc.unal@arinc9.com>
>
> There are options missing from PINCTRL_MT7981 whilst being on every other
> pin controller. Add them.
>
> Signed-off-by: Arınç ÜNAL <arinc.unal@arinc9.com>

Patch applied for fixes.

Yours,
Linus Walleij

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

* Re: [PATCH 2/2] pinctrl: mediatek: fix naming inconsistency
  2023-02-18  6:51 ` [PATCH 2/2] pinctrl: mediatek: fix naming inconsistency arinc9.unal
  2023-02-18 17:32   ` Daniel Golle
@ 2023-03-06 13:23   ` Linus Walleij
  1 sibling, 0 replies; 6+ messages in thread
From: Linus Walleij @ 2023-03-06 13:23 UTC (permalink / raw)
  To: arinc9.unal
  Cc: Sean Wang, Matthias Brugger, Arınç ÜNAL,
	linux-mediatek, linux-gpio, linux-kernel, linux-arm-kernel,
	Daniel Golle, erkin.bozoglu

On Sat, Feb 18, 2023 at 7:51 AM <arinc9.unal@gmail.com> wrote:

> From: Arınç ÜNAL <arinc.unal@arinc9.com>
>
> Some options include "MediaTek", some "Mediatek". Rename all to "MediaTek"
> to address the naming inconsistency.
>
> Signed-off-by: Arınç ÜNAL <arinc.unal@arinc9.com>

Patch applied for fixes.

Yours,
Linus Walleij

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

end of thread, other threads:[~2023-03-06 13:23 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-02-18  6:51 [PATCH 1/2] pinctrl: mediatek: add missing options to PINCTRL_MT7981 arinc9.unal
2023-02-18  6:51 ` [PATCH 2/2] pinctrl: mediatek: fix naming inconsistency arinc9.unal
2023-02-18 17:32   ` Daniel Golle
2023-03-06 13:23   ` Linus Walleij
2023-02-18 17:31 ` [PATCH 1/2] pinctrl: mediatek: add missing options to PINCTRL_MT7981 Daniel Golle
2023-03-06 13:23 ` Linus Walleij

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).