Re: [PATCH RESEND v2 2/2] regulator: TPS65910: VDD1/2 voltage selector count

2011-11-23 Thread Girdwood, Liam
Hi Afzal, On 23 November 2011 05:31, Mohammed, Afzal af...@ti.com wrote: Hi Liam, On Tue, Nov 08, 2011 at 21:26:39, Mark Brown wrote: On Tue, Nov 08, 2011 at 06:54:10PM +0530, Afzal Mohammed wrote: Count of selector voltage is required for regulator_set_voltage to work via

RE: [PATCH RESEND v2 2/2] regulator: TPS65910: VDD1/2 voltage selector count

2011-11-22 Thread Mohammed, Afzal
Hi Liam, On Tue, Nov 08, 2011 at 21:26:39, Mark Brown wrote: On Tue, Nov 08, 2011 at 06:54:10PM +0530, Afzal Mohammed wrote: Count of selector voltage is required for regulator_set_voltage to work via set_voltage_sel. VDD1/2 currently have it as zero, so regulator_set_voltage won't work

[PATCH RESEND v2 2/2] regulator: TPS65910: VDD1/2 voltage selector count

2011-11-08 Thread Afzal Mohammed
Count of selector voltage is required for regulator_set_voltage to work via set_voltage_sel. VDD1/2 currently have it as zero, so regulator_set_voltage won't work for VDD1/2. Update count (n_voltages) for VDD1/2. Output Voltage = (step value * 12.5 mV + 562.5 mV) * gain With above expr, number

Re: [PATCH RESEND v2 2/2] regulator: TPS65910: VDD1/2 voltage selector count

2011-11-08 Thread Mark Brown
On Tue, Nov 08, 2011 at 06:54:10PM +0530, Afzal Mohammed wrote: Count of selector voltage is required for regulator_set_voltage to work via set_voltage_sel. VDD1/2 currently have it as zero, so regulator_set_voltage won't work for VDD1/2. Update count (n_voltages) for VDD1/2. Acked-by: Mark