[PATCH] rtc: Group Kconfig entries by vendor

[PATCH] rtc: Group Kconfig entries by vendor

[Soren Brinkmann]

The RTC entries are mostly grouped by vendor. Move the few outliers in
place.
Also, change the one occurrence of 'nxp' to 'NXP' to make all NXP
entries consistent.

Signed-off-by: Soren Brinkmann <soren.brinkmann@xilinx.com>
---
 drivers/rtc/Kconfig | 90 ++++++++++++++++++++++++++---------------------------
 1 file changed, 45 insertions(+), 45 deletions(-)

diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig
index 2a524244afec..ea7f43f12f69 100644
--- a/drivers/rtc/Kconfig
+++ b/drivers/rtc/Kconfig
@@ -325,16 +325,6 @@ config RTC_DRV_MAX77686
 	  This driver can also be built as a module. If so, the module
 	  will be called rtc-max77686.
 
-config RTC_DRV_RK808
-	tristate "Rockchip RK808 RTC"
-	depends on MFD_RK808
-	help
-	  If you say yes here you will get support for the
-	  RTC of RK808 PMIC.
-
-	  This driver can also be built as a module. If so, the module
-	  will be called rk808-rtc.
-
 config RTC_DRV_MAX77802
 	tristate "Maxim 77802 RTC"
 	depends on MFD_MAX77686
@@ -345,6 +335,16 @@ config RTC_DRV_MAX77802
 	  This driver can also be built as a module. If so, the module
 	  will be called rtc-max77802.
 
+config RTC_DRV_RK808
+	tristate "Rockchip RK808 RTC"
+	depends on MFD_RK808
+	help
+	  If you say yes here you will get support for the
+	  RTC of RK808 PMIC.
+
+	  This driver can also be built as a module. If so, the module
+	  will be called rk808-rtc.
+
 config RTC_DRV_RS5C372
 	tristate "Ricoh R2025S/D, RS5C372A/B, RV5C386, RV5C387A"
 	help
@@ -391,16 +391,6 @@ config RTC_DRV_X1205
 	  This driver can also be built as a module. If so, the module
 	  will be called rtc-x1205.
 
-config RTC_DRV_PALMAS
-	tristate "TI Palmas RTC driver"
-	depends on MFD_PALMAS
-	help
-	  If you say yes here you get support for the RTC of TI PALMA series PMIC
-	  chips.
-
-	  This driver can also be built as a module. If so, the module
-	  will be called rtc-palma.
-
 config RTC_DRV_PCF2127
 	tristate "NXP PCF2127"
 	help
@@ -419,6 +409,14 @@ config RTC_DRV_PCF8523
 	  This driver can also be built as a module. If so, the module
 	  will be called rtc-pcf8523.
 
+config RTC_DRV_PCF85063
+	tristate "NXP PCF85063"
+	help
+	  If you say yes here you get support for the PCF85063 RTC chip
+
+	  This driver can also be built as a module. If so, the module
+	  will be called rtc-pcf85063.
+
 config RTC_DRV_PCF8563
 	tristate "Philips PCF8563/Epson RTC8564"
 	help
@@ -429,14 +427,6 @@ config RTC_DRV_PCF8563
 	  This driver can also be built as a module. If so, the module
 	  will be called rtc-pcf8563.
 
-config RTC_DRV_PCF85063
-	tristate "nxp PCF85063"
-	help
-	  If you say yes here you get support for the PCF85063 RTC chip
-
-	  This driver can also be built as a module. If so, the module
-	  will be called rtc-pcf85063.
-
 config RTC_DRV_PCF8583
 	tristate "Philips PCF8583"
 	help
@@ -501,6 +491,16 @@ config RTC_DRV_TWL4030
 	  This driver can also be built as a module. If so, the module
 	  will be called rtc-twl.
 
+config RTC_DRV_PALMAS
+	tristate "TI Palmas RTC driver"
+	depends on MFD_PALMAS
+	help
+	  If you say yes here you get support for the RTC of TI PALMA series PMIC
+	  chips.
+
+	  This driver can also be built as a module. If so, the module
+	  will be called rtc-palma.
+
 config RTC_DRV_TPS6586X
 	tristate "TI TPS6586X RTC driver"
 	depends on MFD_TPS6586X
@@ -681,15 +681,6 @@ config RTC_DRV_DS1390
 	  This driver can also be built as a module. If so, the module
 	  will be called rtc-ds1390.
 
-config RTC_DRV_MAX6902
-	tristate "Maxim MAX6902"
-	help
-	  If you say yes here you will get support for the
-	  Maxim MAX6902 SPI RTC chip.
-
-	  This driver can also be built as a module. If so, the module
-	  will be called rtc-max6902.
-
 config RTC_DRV_R9701
 	tristate "Epson RTC-9701JE"
 	help
@@ -699,6 +690,14 @@ config RTC_DRV_R9701
 	  This driver can also be built as a module. If so, the module
 	  will be called rtc-r9701.
 
+config RTC_DRV_RX4581
+	tristate "Epson RX-4581"
+	help
+	  If you say yes here you will get support for the Epson RX-4581.
+
+	  This driver can also be built as a module. If so the module
+	  will be called rtc-rx4581.
+
 config RTC_DRV_RS5C348
 	tristate "Ricoh RS5C348A/B"
 	help
@@ -708,6 +707,15 @@ config RTC_DRV_RS5C348
 	  This driver can also be built as a module. If so, the module
 	  will be called rtc-rs5c348.
 
+config RTC_DRV_MAX6902
+	tristate "Maxim MAX6902"
+	help
+	  If you say yes here you will get support for the
+	  Maxim MAX6902 SPI RTC chip.
+
+	  This driver can also be built as a module. If so, the module
+	  will be called rtc-max6902.
+
 config RTC_DRV_DS3234
 	tristate "Maxim/Dallas DS3234"
 	help
@@ -726,14 +734,6 @@ config RTC_DRV_PCF2123
 	  This driver can also be built as a module. If so, the module
 	  will be called rtc-pcf2123.
 
-config RTC_DRV_RX4581
-	tristate "Epson RX-4581"
-	help
-	  If you say yes here you will get support for the Epson RX-4581.
-
-	  This driver can also be built as a module. If so the module
-	  will be called rtc-rx4581.
-
 config RTC_DRV_MCP795
 	tristate "Microchip MCP795"
 	help
-- 
2.6.2.3.ga463a5b

Re: [PATCH] rtc: Group Kconfig entries by vendor

[Alexandre Belloni]

On 03/11/2015 at 23:13:46 -0800, Soren Brinkmann wrote :
> The RTC entries are mostly grouped by vendor. Move the few outliers in
> place.
> Also, change the one occurrence of 'nxp' to 'NXP' to make all NXP
> entries consistent.
> 

Yeah, I've been contemplating reordering the Kconfig, the main issue
being that Philips and NXP is the same entity. And also, we have RTCs
manufactured by different vendors that use the same chip
(Abracon/Epson/Philips/Microcrystal) so I'm not sure what is clearer for
the end user :)

> Signed-off-by: Soren Brinkmann <soren.brinkmann@xilinx.com>
> ---
>  drivers/rtc/Kconfig | 90 ++++++++++++++++++++++++++---------------------------
>  1 file changed, 45 insertions(+), 45 deletions(-)
> 
> diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig
> index 2a524244afec..ea7f43f12f69 100644
> --- a/drivers/rtc/Kconfig
> +++ b/drivers/rtc/Kconfig
> @@ -325,16 +325,6 @@ config RTC_DRV_MAX77686
>  	  This driver can also be built as a module. If so, the module
>  	  will be called rtc-max77686.
>  
> -config RTC_DRV_RK808
> -	tristate "Rockchip RK808 RTC"
> -	depends on MFD_RK808
> -	help
> -	  If you say yes here you will get support for the
> -	  RTC of RK808 PMIC.
> -
> -	  This driver can also be built as a module. If so, the module
> -	  will be called rk808-rtc.
> -
>  config RTC_DRV_MAX77802
>  	tristate "Maxim 77802 RTC"
>  	depends on MFD_MAX77686
> @@ -345,6 +335,16 @@ config RTC_DRV_MAX77802
>  	  This driver can also be built as a module. If so, the module
>  	  will be called rtc-max77802.
>  
> +config RTC_DRV_RK808
> +	tristate "Rockchip RK808 RTC"
> +	depends on MFD_RK808
> +	help
> +	  If you say yes here you will get support for the
> +	  RTC of RK808 PMIC.
> +
> +	  This driver can also be built as a module. If so, the module
> +	  will be called rk808-rtc.
> +
>  config RTC_DRV_RS5C372
>  	tristate "Ricoh R2025S/D, RS5C372A/B, RV5C386, RV5C387A"
>  	help
> @@ -391,16 +391,6 @@ config RTC_DRV_X1205
>  	  This driver can also be built as a module. If so, the module
>  	  will be called rtc-x1205.
>  
> -config RTC_DRV_PALMAS
> -	tristate "TI Palmas RTC driver"
> -	depends on MFD_PALMAS
> -	help
> -	  If you say yes here you get support for the RTC of TI PALMA series PMIC
> -	  chips.
> -
> -	  This driver can also be built as a module. If so, the module
> -	  will be called rtc-palma.
> -
>  config RTC_DRV_PCF2127
>  	tristate "NXP PCF2127"
>  	help
> @@ -419,6 +409,14 @@ config RTC_DRV_PCF8523
>  	  This driver can also be built as a module. If so, the module
>  	  will be called rtc-pcf8523.
>  
> +config RTC_DRV_PCF85063
> +	tristate "NXP PCF85063"
> +	help
> +	  If you say yes here you get support for the PCF85063 RTC chip
> +
> +	  This driver can also be built as a module. If so, the module
> +	  will be called rtc-pcf85063.
> +
>  config RTC_DRV_PCF8563
>  	tristate "Philips PCF8563/Epson RTC8564"
>  	help
> @@ -429,14 +427,6 @@ config RTC_DRV_PCF8563
>  	  This driver can also be built as a module. If so, the module
>  	  will be called rtc-pcf8563.
>  
> -config RTC_DRV_PCF85063
> -	tristate "nxp PCF85063"
> -	help
> -	  If you say yes here you get support for the PCF85063 RTC chip
> -
> -	  This driver can also be built as a module. If so, the module
> -	  will be called rtc-pcf85063.
> -
>  config RTC_DRV_PCF8583
>  	tristate "Philips PCF8583"
>  	help
> @@ -501,6 +491,16 @@ config RTC_DRV_TWL4030
>  	  This driver can also be built as a module. If so, the module
>  	  will be called rtc-twl.
>  
> +config RTC_DRV_PALMAS
> +	tristate "TI Palmas RTC driver"
> +	depends on MFD_PALMAS
> +	help
> +	  If you say yes here you get support for the RTC of TI PALMA series PMIC
> +	  chips.
> +
> +	  This driver can also be built as a module. If so, the module
> +	  will be called rtc-palma.
> +
>  config RTC_DRV_TPS6586X
>  	tristate "TI TPS6586X RTC driver"
>  	depends on MFD_TPS6586X
> @@ -681,15 +681,6 @@ config RTC_DRV_DS1390
>  	  This driver can also be built as a module. If so, the module
>  	  will be called rtc-ds1390.
>  
> -config RTC_DRV_MAX6902
> -	tristate "Maxim MAX6902"
> -	help
> -	  If you say yes here you will get support for the
> -	  Maxim MAX6902 SPI RTC chip.
> -
> -	  This driver can also be built as a module. If so, the module
> -	  will be called rtc-max6902.
> -
>  config RTC_DRV_R9701
>  	tristate "Epson RTC-9701JE"
>  	help
> @@ -699,6 +690,14 @@ config RTC_DRV_R9701
>  	  This driver can also be built as a module. If so, the module
>  	  will be called rtc-r9701.
>  
> +config RTC_DRV_RX4581
> +	tristate "Epson RX-4581"
> +	help
> +	  If you say yes here you will get support for the Epson RX-4581.
> +
> +	  This driver can also be built as a module. If so the module
> +	  will be called rtc-rx4581.
> +
>  config RTC_DRV_RS5C348
>  	tristate "Ricoh RS5C348A/B"
>  	help
> @@ -708,6 +707,15 @@ config RTC_DRV_RS5C348
>  	  This driver can also be built as a module. If so, the module
>  	  will be called rtc-rs5c348.
>  
> +config RTC_DRV_MAX6902
> +	tristate "Maxim MAX6902"
> +	help
> +	  If you say yes here you will get support for the
> +	  Maxim MAX6902 SPI RTC chip.
> +
> +	  This driver can also be built as a module. If so, the module
> +	  will be called rtc-max6902.
> +
>  config RTC_DRV_DS3234
>  	tristate "Maxim/Dallas DS3234"
>  	help
> @@ -726,14 +734,6 @@ config RTC_DRV_PCF2123
>  	  This driver can also be built as a module. If so, the module
>  	  will be called rtc-pcf2123.
>  
> -config RTC_DRV_RX4581
> -	tristate "Epson RX-4581"
> -	help
> -	  If you say yes here you will get support for the Epson RX-4581.
> -
> -	  This driver can also be built as a module. If so the module
> -	  will be called rtc-rx4581.
> -
>  config RTC_DRV_MCP795
>  	tristate "Microchip MCP795"
>  	help
> -- 
> 2.6.2.3.ga463a5b
> 

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

Re: [PATCH] rtc: Group Kconfig entries by vendor

[Sören Brinkmann]

Hi Alexandre,

On Wed, 2015-11-04 at 10:25AM +0100, Alexandre Belloni wrote:
> On 03/11/2015 at 23:13:46 -0800, Soren Brinkmann wrote :
> > The RTC entries are mostly grouped by vendor. Move the few outliers in
> > place.
> > Also, change the one occurrence of 'nxp' to 'NXP' to make all NXP
> > entries consistent.
> > 
> 
> Yeah, I've been contemplating reordering the Kconfig, the main issue
> being that Philips and NXP is the same entity. And also, we have RTCs
> manufactured by different vendors that use the same chip
> (Abracon/Epson/Philips/Microcrystal) so I'm not sure what is clearer for
> the end user :)

Yeah, I wasn't sure if this is wanted or how far to go. It was mainly
triggered by the NXP devices that weren't grouped and differed in
capitalization. From there I took it a little further. Just let met know if
you want this somewhat different or not at all. It's just a suggestion
and I find it helpful if things are somewhat ordered (I was thinking go
full alphabetical sorting but didn't want to go that far without another
opinion). But I'm fine either way.

	Thanks,
	Sören

Re: [PATCH] rtc: Group Kconfig entries by vendor

[Alexandre Belloni]

On 03/11/2015 at 23:13:46 -0800, Soren Brinkmann wrote :
> The RTC entries are mostly grouped by vendor. Move the few outliers in
> place.
> Also, change the one occurrence of 'nxp' to 'NXP' to make all NXP
> entries consistent.
> 
> Signed-off-by: Soren Brinkmann <soren.brinkmann@xilinx.com>
> ---
>  drivers/rtc/Kconfig | 90 ++++++++++++++++++++++++++---------------------------
>  1 file changed, 45 insertions(+), 45 deletions(-)
> 
Applied, thanks.

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com