Messages in this thread | | | Subject | regulator: tps51632: Seems current code doesn't properly support dvfs_step_20mV case | From | Axel Lin <> | Date | Thu, 01 Nov 2012 14:48:41 +0800 |
| |
Hi Laxman,
While reading the tps51632 driver, I found there is a dvfs_step_20mV setting in platform data. But seems current code doesn't properly handle the case when dvfs_step_20mV is true.
I guess if pdata->dvfs_step_20mV is true, we need:
tps->desc.uV_step = TPS51632_VOLATGE_STEP_20mV; Fix TPS51632_VOLT_VSEL macro to support dvfs_step_20mV case. Also I'm wondering if either TPS51632_MAX_VSEL/TPS51632_MAX_VOLATGE or desc.n_voltages needs change for dvfs_step_20mV case.
Oh, I don't have the datasheet, so my understanding might be wrong.
Regards, Axel
| |