Skip to contentSkip to conversation

regulator: Group TI TPSxxxxx regulators together

PATCH In review Axel Lin 3 participants 4 messages Feb 21, 2012 Voltage and current regulator framework

lore
Share & tools
lore

Axel Lin

author
#0 lore raw

PATCH 1/1 regulator: Group TI TPSxxxxx regulators together

Group TI TPSxxxxx regulators together and sort them in alphabetical order. This change makes it easier for users to find config options.

Cc Mike Rapoport <[email protected]>
Cc Cyril Chemparathy <[email protected]>
Cc Graeme Gregory <[email protected]>
Cc Laxman Dewangan <[email protected]>
---
 drivers/regulator/Kconfig  |   64 ++++++++++++++++++++++----------------------
 drivers/regulator/Makefile |    6 ++--
 2 files changed, 35 insertions(+), 35 deletions(-)
drivers/regulator/Kconfig +32 -32
index edcd9e2..343ab78 100644
@@ -258,6 +258,16 @@ config REGULATOR_TPS6105X
 	  It is a single boost converter primarily for white LEDs and
 	  audio amplifiers.
 
+config REGULATOR_TPS62360
+	tristate "TI TPS62360 Power Regulator"
+	depends on I2C
+	select REGMAP_I2C
+	help
+	  This driver supports TPS62360 voltage regulator chip. This
+	  regulator is meant for processor core supply. This chip is
+	  high-frequency synchronous step down dc-dc converter optimized
+	  for battery-powered portable applications.
+
 config REGULATOR_TPS65023
 	tristate "TI TPS65023 Power regulators"
 	depends on I2C
@@ -284,6 +294,28 @@ config REGULATOR_TPS65217
 	  voltage regulators. It supports software based voltage control
 	  for different voltage domains
 
+config REGULATOR_TPS6524X
+	tristate "TI TPS6524X Power regulators"
+	depends on SPI
+	help
+	  This driver supports TPS6524X voltage regulator chips. TPS6524X
+	  provides three step-down converters and two general-purpose LDO
+	  voltage regulators.  This device is interfaced using a customized
+	  serial interface currently supported on the sequencer serial
+	  port controller.
+
+config REGULATOR_TPS6586X
+	tristate "TI TPS6586X Power regulators"
+	depends on MFD_TPS6586X
+	help
+	  This driver supports TPS6586X voltage regulator chips.
+
+config REGULATOR_TPS65910
+	tristate "TI TPS65910/TPS65911 Power Regulators"
+	depends on MFD_TPS65910
+	help
+	  This driver supports TPS65910/TPS65911 voltage regulator chips.
+
 config REGULATOR_TPS65912
 	tristate "TI TPS65912 Power regulator"
 	depends on (MFD_TPS65912_I2C || MFD_TPS65912_SPI)
@@ -327,38 +359,6 @@ config REGULATOR_DB8500_PRCMU
 	  This driver supports the voltage domain regulators controlled by the
 	  DB8500 PRCMU
 
-config REGULATOR_TPS6586X
-	tristate "TI TPS6586X Power regulators"
-	depends on MFD_TPS6586X
-	help
-	  This driver supports TPS6586X voltage regulator chips.
-
-config REGULATOR_TPS6524X
-	tristate "TI TPS6524X Power regulators"
-	depends on SPI
-	help
-	  This driver supports TPS6524X voltage regulator chips. TPS6524X
-	  provides three step-down converters and two general-purpose LDO
-	  voltage regulators.  This device is interfaced using a customized
-	  serial interface currently supported on the sequencer serial
-	  port controller.
-
-config REGULATOR_TPS65910
-	tristate "TI TPS65910/TPS65911 Power Regulators"
-	depends on MFD_TPS65910
-	help
-	  This driver supports TPS65910/TPS65911 voltage regulator chips.
-
-config REGULATOR_TPS62360
-	tristate "TI TPS62360 Power Regulator"
-	depends on I2C
-	select REGMAP_I2C
-	help
-	  This driver supports TPS62360 voltage regulator chip. This
-	  regulator is meant for processor core supply. This chip is
-	  high-frequency synchronous step down dc-dc converter optimized
-	  for battery-powered portable applications.
-
 config REGULATOR_AAT2870
 	tristate "AnalogicTech AAT2870 Regulators"
 	depends on MFD_AAT2870_CORE
drivers/regulator/Makefile +3 -3
index 4cbf8c5..eb1920b 100644
@@ -28,7 +28,6 @@ obj-$(CONFIG_REGULATOR_WM831X) += wm831x-ldo.o
 obj-$(CONFIG_REGULATOR_WM8350) += wm8350-regulator.o
 obj-$(CONFIG_REGULATOR_WM8400) += wm8400-regulator.o
 obj-$(CONFIG_REGULATOR_WM8994) += wm8994-regulator.o
-obj-$(CONFIG_REGULATOR_TPS6586X) += tps6586x-regulator.o
 obj-$(CONFIG_REGULATOR_DA903X)	+= da903x.o
 obj-$(CONFIG_REGULATOR_DA9052)	+= da9052-regulator.o
 obj-$(CONFIG_REGULATOR_PCF50633) += pcf50633-regulator.o
@@ -38,18 +37,19 @@ obj-$(CONFIG_REGULATOR_MC13892) += mc13892-regulator.o
 obj-$(CONFIG_REGULATOR_MC13XXX_CORE) +=  mc13xxx-regulator-core.o
 obj-$(CONFIG_REGULATOR_AB3100) += ab3100.o
 obj-$(CONFIG_REGULATOR_TPS6105X) += tps6105x-regulator.o
+obj-$(CONFIG_REGULATOR_TPS62360) += tps62360-regulator.o
 obj-$(CONFIG_REGULATOR_TPS65023) += tps65023-regulator.o
 obj-$(CONFIG_REGULATOR_TPS6507X) += tps6507x-regulator.o
 obj-$(CONFIG_REGULATOR_TPS65217) += tps65217-regulator.o
 obj-$(CONFIG_REGULATOR_TPS6524X) += tps6524x-regulator.o
+obj-$(CONFIG_REGULATOR_TPS6586X) += tps6586x-regulator.o
+obj-$(CONFIG_REGULATOR_TPS65910) += tps65910-regulator.o
 obj-$(CONFIG_REGULATOR_TPS65912) += tps65912-regulator.o
 obj-$(CONFIG_REGULATOR_88PM8607) += 88pm8607.o
 obj-$(CONFIG_REGULATOR_ISL6271A) += isl6271a-regulator.o
 obj-$(CONFIG_REGULATOR_AB8500)	+= ab8500.o
 obj-$(CONFIG_REGULATOR_DBX500_PRCMU) += dbx500-prcmu.o
 obj-$(CONFIG_REGULATOR_DB8500_PRCMU) += db8500-prcmu.o
-obj-$(CONFIG_REGULATOR_TPS65910) += tps65910-regulator.o
-obj-$(CONFIG_REGULATOR_TPS62360) += tps62360-regulator.o
 obj-$(CONFIG_REGULATOR_AAT2870) += aat2870-regulator.o
 obj-$(CONFIG_REGULATOR_S5M8767) += s5m8767.o
 
Signature · 1 line
1.7.5.4
#1 lore raw
Quoted 9 lines · Axel Lin

Group TI TPSxxxxx regulators together and sort them in alphabetical order. This change makes it easier for users to find config options.

Cc Mike Rapoport<[email protected]>
Cc Cyril Chemparathy<[email protected]>
Cc Graeme Gregory<[email protected]>
Cc Laxman Dewangan<[email protected]>

---

Should we only arrange the tps regulators or all regulators in alphabetical order?
I will prefer to arrange all which inherently will arrange tps as all ti regulators start with tps..

author
#2 lore raw

2012/2/21 Laxman Dewangan <[email protected]>:

Quoted 12 lines · Axel Lin
Quoted 10 lines

Group TI TPSxxxxx regulators together and sort them in alphabetical order. This change makes it easier for users to find config options.

Cc Mike Rapoport<[email protected]>
Cc Cyril Chemparathy<[email protected]>
Cc Graeme Gregory<[email protected]>
Cc Laxman Dewangan<[email protected]>

---

Quoted 4 lines

Should we only arrange the tps regulators or all regulators in alphabetical order?
I will prefer to arrange all which inherently will arrange tps as all ti regulators start with tps..

I think it would be easier for review the patch if we can apply this one first, then an incremental patch to arrange all other regulator drivers. ( Most other regulator drivers are already grouped by similar drivers).

Regards,
Axel

#3 lore raw
In reply to Laxman Dewangan
Quoted 8 lines · Laxman Dewangan
Quoted 2 lines · Axel Lin

Group TI TPSxxxxx regulators together and sort them in alphabetical order. This change makes it easier for users to find config options.

Should we only arrange the tps regulators or all regulators in alphabetical order?
I will prefer to arrange all which inherently will arrange tps as all ti regulators start with tps..

We should just sort the whole list.