RE: [PATCH v3] pwm: tidyup PWM menu for Renesas

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Morimoto-San,

Thanks for the patch.

> -----Original Message-----
> From: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>
> Subject: [PATCH v3] pwm: tidyup PWM menu for Renesas
> 
> Because current PWM Kconfig is sorting by symbol name, it looks strange ordering in menuconfig.
> 
> =>	[ ]   Renesas R-Car PWM support
> =>	[ ]   Renesas TPU PWM support
> 	[ ]   Rockchip PWM support
> =>	[ ]   Renesas RZ/G2L MTU3a PWM Timer support
> 
> Let's use common CONFIG_PWM_RENESAS_xxx symbol name for Renesas, and sort it.
> 
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>
> ---
> v2 -> v3
> 	- tidyup sorting
> 
>  arch/arm/configs/multi_v7_defconfig |  2 +-  arch/arm/configs/shmobile_defconfig |  2 +-
>  arch/arm64/configs/defconfig        |  4 ++--
>  drivers/pwm/Kconfig                 | 24 ++++++++++++------------
>  drivers/pwm/Makefile                |  4 ++--
>  5 files changed, 18 insertions(+), 18 deletions(-)
> 
> diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
> index 37e3baa33b678..8439b24f63534 100644
> --- a/arch/arm/configs/multi_v7_defconfig
> +++ b/arch/arm/configs/multi_v7_defconfig
> @@ -1203,7 +1203,7 @@ CONFIG_PWM_BCM2835=y  CONFIG_PWM_BRCMSTB=m  CONFIG_PWM_FSL_FTM=m
> CONFIG_PWM_MESON=m -CONFIG_PWM_RCAR=m
> +CONFIG_PWM_RENESAS_RCAR=m
>  CONFIG_PWM_RENESAS_TPU=y
>  CONFIG_PWM_ROCKCHIP=m
>  CONFIG_PWM_SAMSUNG=m
> diff --git a/arch/arm/configs/shmobile_defconfig b/arch/arm/configs/shmobile_defconfig
> index fd28f3176c6ba..e4e69b381af15 100644
> --- a/arch/arm/configs/shmobile_defconfig
> +++ b/arch/arm/configs/shmobile_defconfig
> @@ -216,7 +216,7 @@ CONFIG_ARCH_SH73A0=y  CONFIG_IIO=y  CONFIG_AK8975=y  CONFIG_PWM=y -
> CONFIG_PWM_RCAR=y
> +CONFIG_PWM_RENESAS_RCAR=y
>  CONFIG_PWM_RENESAS_TPU=y
>  CONFIG_PHY_RCAR_GEN2=y
>  CONFIG_PHY_RCAR_GEN3_USB2=y
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig index
> 1f25423de3833..1a5d30a1bc481 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -1537,10 +1537,10 @@ CONFIG_PWM_IMX27=m  CONFIG_PWM_MESON=m  CONFIG_PWM_MTK_DISP=m
> CONFIG_PWM_MEDIATEK=m -CONFIG_PWM_RCAR=m
> +CONFIG_PWM_RENESAS_RCAR=m
> +CONFIG_PWM_RENESAS_RZ_MTU3=m
>  CONFIG_PWM_RENESAS_TPU=m
>  CONFIG_PWM_ROCKCHIP=y
> -CONFIG_PWM_RZ_MTU3=m
>  CONFIG_PWM_SAMSUNG=y
>  CONFIG_PWM_SL28CPLD=m
>  CONFIG_PWM_SUN4I=m
> diff --git a/drivers/pwm/Kconfig b/drivers/pwm/Kconfig index 0915c1e7df16d..a1000b48886d8 100644
> --- a/drivers/pwm/Kconfig
> +++ b/drivers/pwm/Kconfig
> @@ -510,7 +510,7 @@ config PWM_RASPBERRYPI_POE
>  	  Enable Raspberry Pi firmware controller PWM bus used to control the
>  	  official RPI PoE hat
> 
> -config PWM_RCAR
> +config PWM_RENESAS_RCAR
>  	tristate "Renesas R-Car PWM support"
>  	depends on ARCH_RENESAS || COMPILE_TEST
>  	depends on HAS_IOMEM
> @@ -521,6 +521,17 @@ config PWM_RCAR
>  	  To compile this driver as a module, choose M here: the module
>  	  will be called pwm-rcar.
> 
> +config PWM_RENESAS_RZ_MTU3
> +	tristate "Renesas RZ/G2L MTU3a PWM Timer support"
> +	depends on RZ_MTU3
> +	depends on HAS_IOMEM
> +	help
> +	  This driver exposes the MTU3a PWM Timer controller found in Renesas
> +	  RZ/G2L like chips through the PWM API.
> +
> +	  To compile this driver as a module, choose M here: the module
> +	  will be called pwm-rz-mtu3.
> +
>  config PWM_RENESAS_TPU
>  	tristate "Renesas TPU PWM support"
>  	depends on ARCH_RENESAS || COMPILE_TEST @@ -540,17 +551,6 @@ config PWM_ROCKCHIP
>  	  Generic PWM framework driver for the PWM controller found on
>  	  Rockchip SoCs.
> 
> -config PWM_RZ_MTU3
> -	tristate "Renesas RZ/G2L MTU3a PWM Timer support"
> -	depends on RZ_MTU3
> -	depends on HAS_IOMEM
> -	help
> -	  This driver exposes the MTU3a PWM Timer controller found in Renesas
> -	  RZ/G2L like chips through the PWM API.
> -
> -	  To compile this driver as a module, choose M here: the module
> -	  will be called pwm-rz-mtu3.
> -
>  config PWM_SAMSUNG
>  	tristate "Samsung PWM support"
>  	depends on PLAT_SAMSUNG || ARCH_S5PV210 || ARCH_EXYNOS || COMPILE_TEST diff --git
> a/drivers/pwm/Makefile b/drivers/pwm/Makefile index 9081e0c0e9e09..a44e5c0ff135e 100644
> --- a/drivers/pwm/Makefile
> +++ b/drivers/pwm/Makefile
> @@ -46,10 +46,10 @@ obj-$(CONFIG_PWM_OMAP_DMTIMER)	+= pwm-omap-dmtimer.o
>  obj-$(CONFIG_PWM_PCA9685)	+= pwm-pca9685.o
>  obj-$(CONFIG_PWM_PXA)		+= pwm-pxa.o
>  obj-$(CONFIG_PWM_RASPBERRYPI_POE)	+= pwm-raspberrypi-poe.o
> -obj-$(CONFIG_PWM_RCAR)		+= pwm-rcar.o
> +obj-$(CONFIG_PWM_RENESAS_RCAR)	+= pwm-rcar.o
> +obj-$(CONFIG_PWM_RENESAS_RZ_MTU3)	+= pwm-rz-mtu3.o
>  obj-$(CONFIG_PWM_RENESAS_TPU)	+= pwm-renesas-tpu.o
>  obj-$(CONFIG_PWM_ROCKCHIP)	+= pwm-rockchip.o
> -obj-$(CONFIG_PWM_RZ_MTU3)	+= pwm-rz-mtu3.o

Do we need to tidy up this as well [1]?

CONFIG_PWM_RZG2L_GPT-> CONFIG_PWM_RENESAS_RZG2L_GPT


[1] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?h=next-20250509&id=5ad7de623853570bd981be069ed5ce55540bedb8

Cheers,
Biju

>  obj-$(CONFIG_PWM_SAMSUNG)	+= pwm-samsung.o
>  obj-$(CONFIG_PWM_SIFIVE)	+= pwm-sifive.o
>  obj-$(CONFIG_PWM_SL28CPLD)	+= pwm-sl28cpld.o
> --
> 2.43.0
> 






[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux