All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 RESEND] regulator: tps65023: Remove num_voltages array
@ 2011-08-29 14:54 Axel Lin
  2011-08-30  0:19 ` Axel Lin
  0 siblings, 1 reply; 2+ messages in thread
From: Axel Lin @ 2011-08-29 14:54 UTC (permalink / raw)
  To: linux-kernel; +Cc: Anuj Aggarwal, Liam Girdwood, Mark Brown

We can get n_voltages for each regulator from table_len of struct tps_info.

Signed-off-by: Axel Lin <axel.lin@gmail.com>
Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
---
 drivers/regulator/tps65023-regulator.c |    6 +-----
 1 files changed, 1 insertions(+), 5 deletions(-)

diff --git a/drivers/regulator/tps65023-regulator.c b/drivers/regulator/tps65023-regulator.c
index 701a590..3d2a81a 100644
--- a/drivers/regulator/tps65023-regulator.c
+++ b/drivers/regulator/tps65023-regulator.c
@@ -106,10 +106,6 @@ static const u16 LDO2_VSEL_table[] = {
 	2500, 2800, 3000, 3300,
 };
 
-static unsigned int num_voltages[] = {ARRAY_SIZE(VDCDC1_VSEL_table),
-				0, 0, ARRAY_SIZE(LDO1_VSEL_table),
-				ARRAY_SIZE(LDO2_VSEL_table)};
-
 /* Regulator specific details */
 struct tps_info {
 	const char *name;
@@ -453,7 +449,7 @@ static int __devinit tps_65023_probe(struct i2c_client *client,
 
 		tps->desc[i].name = info->name;
 		tps->desc[i].id = i;
-		tps->desc[i].n_voltages = num_voltages[i];
+		tps->desc[i].n_voltages = info->table_len;
 		tps->desc[i].ops = (i > TPS65023_DCDC_3 ?
 					&tps65023_ldo_ops : &tps65023_dcdc_ops);
 		tps->desc[i].type = REGULATOR_VOLTAGE;
-- 
1.7.4.1




^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH v2 RESEND] regulator: tps65023: Remove num_voltages array
  2011-08-29 14:54 [PATCH v2 RESEND] regulator: tps65023: Remove num_voltages array Axel Lin
@ 2011-08-30  0:19 ` Axel Lin
  0 siblings, 0 replies; 2+ messages in thread
From: Axel Lin @ 2011-08-30  0:19 UTC (permalink / raw)
  To: linux-kernel; +Cc: Anuj Aggarwal, Liam Girdwood, Mark Brown

2011/8/29 Axel Lin <axel.lin@gmail.com>:
> We can get n_voltages for each regulator from table_len of struct tps_info.
>

hi Liam,
Please ignore this one.
I just found it is already include in
commit  1c3ede05d1 "regulator: tps65023: Setting correct core
regulator for tps65021"

Thanks,
Axel

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-08-30  0:20 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-08-29 14:54 [PATCH v2 RESEND] regulator: tps65023: Remove num_voltages array Axel Lin
2011-08-30  0:19 ` Axel Lin

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.