2023-02-18 06:51:29

by Arınç ÜNAL

Subject: [PATCH 1/2] pinctrl: mediatek: add missing options to PINCTRL_MT7981

From: Arınç ÜNAL <[email protected]>

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

Signed-off-by: Arınç ÜNAL <[email protected]>
---
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

2023-02-18 06:51:29

by Arınç ÜNAL

Subject: [PATCH 2/2] pinctrl: mediatek: fix naming inconsistency

From: Arınç ÜNAL <[email protected]>

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

Signed-off-by: Arınç ÜNAL <[email protected]>
---
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

2023-02-18 17:31:42

by Daniel Golle

Subject: Re: [PATCH 1/2] pinctrl: mediatek: add missing options to PINCTRL_MT7981

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

Acked-by: Daniel Golle <[email protected]>

> Signed-off-by: Arınç ÜNAL <[email protected]>
> ---
> 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
>

2023-02-18 17:32:39

by Daniel Golle

Subject: Re: [PATCH 2/2] pinctrl: mediatek: fix naming inconsistency

On Sat, Feb 18, 2023 at 09:51:07AM +0300, [email protected] wrote:
> From: Arınç ÜNAL <[email protected]>
>
> Some options include "MediaTek", some "Mediatek". Rename all to "MediaTek"
> to address the naming inconsistency.
>

Reviewed-by: Daniel Golle <[email protected]>

> Signed-off-by: Arınç ÜNAL <[email protected]>
> ---
> 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
>

2023-03-06 13:23:26

by Linus Walleij

Subject: Re: [PATCH 1/2] pinctrl: mediatek: add missing options to PINCTRL_MT7981

On Sat, Feb 18, 2023 at 7:51 AM <[email protected]> wrote:

> From: Arınç ÜNAL <[email protected]>
>
> There are options missing from PINCTRL_MT7981 whilst being on every other
> pin controller. Add them.
>
> Signed-off-by: Arınç ÜNAL <[email protected]>

Patch applied for fixes.

Yours,
Linus Walleij

2023-03-06 13:23:51

by Linus Walleij

Subject: Re: [PATCH 2/2] pinctrl: mediatek: fix naming inconsistency

On Sat, Feb 18, 2023 at 7:51 AM <[email protected]> wrote:

> From: Arınç ÜNAL <[email protected]>
>
> Some options include "MediaTek", some "Mediatek". Rename all to "MediaTek"
> to address the naming inconsistency.
>
> Signed-off-by: Arınç ÜNAL <[email protected]>

Patch applied for fixes.

Yours,
Linus Walleij