* [PATCH] regulator: tps6507x: Avoid duplicating the same mapping table for LDO2 and VDCDCx
@ 2012-06-05 0:58 Axel Lin
2012-06-05 10:21 ` Mark Brown
0 siblings, 1 reply; 2+ messages in thread
From: Axel Lin @ 2012-06-05 0:58 UTC (permalink / raw)
To: linux-kernel; +Cc: Mark Brown, Liam Girdwood, Anuj Aggarwal
The voltage mapping table for LDO2 is exactly the same as VDCDCx.
Thus we can use one mapping table for both LDO2 and VDCDCx instead of
duplicating the mapping table.
Signed-off-by: Axel Lin <axel.lin@gmail.com>
---
drivers/regulator/tps6507x-regulator.c | 20 ++------------------
1 file changed, 2 insertions(+), 18 deletions(-)
diff --git a/drivers/regulator/tps6507x-regulator.c b/drivers/regulator/tps6507x-regulator.c
index eed6678..07d01cc 100644
--- a/drivers/regulator/tps6507x-regulator.c
+++ b/drivers/regulator/tps6507x-regulator.c
@@ -70,24 +70,8 @@ static const unsigned int LDO1_VSEL_table[] = {
2800000, 3000000, 3100000, 3300000,
};
-static const unsigned int LDO2_VSEL_table[] = {
- 725000, 750000, 775000, 800000,
- 825000, 850000, 875000, 900000,
- 925000, 950000, 975000, 1000000,
- 1025000, 1050000, 1075000, 1100000,
- 1125000, 1150000, 1175000, 1200000,
- 1225000, 1250000, 1275000, 1300000,
- 1325000, 1350000, 1375000, 1400000,
- 1425000, 1450000, 1475000, 1500000,
- 1550000, 1600000, 1650000, 1700000,
- 1750000, 1800000, 1850000, 1900000,
- 1950000, 2000000, 2050000, 2100000,
- 2150000, 2200000, 2250000, 2300000,
- 2350000, 2400000, 2450000, 2500000,
- 2550000, 2600000, 2650000, 2700000,
- 2750000, 2800000, 2850000, 2900000,
- 3000000, 3100000, 3200000, 3300000,
-};
+/* The voltage mapping table for LDO2 is the same as VDCDCx */
+#define LDO2_VSEL_table VDCDCx_VSEL_table
struct tps_info {
const char *name;
--
1.7.9.5
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] regulator: tps6507x: Avoid duplicating the same mapping table for LDO2 and VDCDCx
2012-06-05 0:58 [PATCH] regulator: tps6507x: Avoid duplicating the same mapping table for LDO2 and VDCDCx Axel Lin
@ 2012-06-05 10:21 ` Mark Brown
0 siblings, 0 replies; 2+ messages in thread
From: Mark Brown @ 2012-06-05 10:21 UTC (permalink / raw)
To: Axel Lin; +Cc: linux-kernel, Liam Girdwood, Anuj Aggarwal
[-- Attachment #1: Type: text/plain, Size: 249 bytes --]
On Tue, Jun 05, 2012 at 08:58:13AM +0800, Axel Lin wrote:
> The voltage mapping table for LDO2 is exactly the same as VDCDCx.
> Thus we can use one mapping table for both LDO2 and VDCDCx instead of
> duplicating the mapping table.
Applied, thanks.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-06-05 10:22 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-06-05 0:58 [PATCH] regulator: tps6507x: Avoid duplicating the same mapping table for LDO2 and VDCDCx Axel Lin
2012-06-05 10:21 ` Mark Brown
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox