19 Mar, 2012
3 commits
17 Mar, 2012
1 commit
-
of_parse_phandle() returns NULL either if the property name
itself does not exist or if it (exists and) does not
reference a valid phandle.
Giving out a warn like the one below (that the property references
an invalid phandle) can be confusing when the property itself
does not exist in the node.
Fix it with a more sensible message and make it a dev_dbg instead
of a dev_warn.Reported-by: Tomi Valkeinen
Signed-off-by: Rajendra Nayak
Signed-off-by: Mark Brown
14 Mar, 2012
12 commits
-
Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
All the callback functions implementation for DCDCx and LDOx are very similar,
I think it is ok to use one tps6507x_pmic_ops for all LDOs and DCDCs.
This refactor removes a couple of duplicated code.Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
Check s5m8767->buck[2|3|4]_gpiodvs status in s5m8767_get_voltage_register
and return correct register accordingly.Signed-off-by: Axel Lin
Acked-by: Sangbeom Kim
Signed-off-by: Mark Brown -
If we need to ensure only one of the buck[2|3|4]_gpiodvs can be specificed,
check them earlier.Signed-off-by: Axel Lin
Acked-by: Sangbeom Kim
Signed-off-by: Mark Brown -
Settling time is require when there is dcdc rail's voltage change.
Returning proper delay time for dcdc voltage change to settle down
the output voltage to new value.Signed-off-by: Laxman Dewangan
Signed-off-by: Mark Brown -
Anatop is an integrated regulator inside i.MX6 SoC.
There are 3 digital regulators which controls PU, CORE (ARM), and SOC.
And 3 analog regulators which controls 1P1, 2P5, 3P0 (USB).
This patch adds the Anatop regulator driver.Signed-off-by: Nancy Chen
Signed-off-by: Ying-Chun Liu (PaulLiu)
Acked-by: Shawn Guo
Reviewed-by: Axel Lin
Signed-off-by: Mark Brown -
Call tps65912_get_sel_register instead of duplicating the same code.
Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
commit 42b5ef "regulator: tps65912: Use simple equations to get register address"
uses tps65912_get_sel_register to replace tps65912_get_dcdc_sel_register
and tps65912_get_ldo_sel_register.Now tps65912_set_voltage_dcdc_sel and tps65912_set_voltage_ldo_sel has exactly
the same implementation. Merge them to tps65912_set_voltage_sel function.Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
There is settling time for each rails when it is switched to
ON. Implementing enable time for returning proper settling time
of regulator rails when it is enabled.
Filling the on-time for each rail as per tps65910/tps65911
datasheets.Signed-off-by: Laxman Dewangan
Signed-off-by: Mark Brown -
Use DIV_ROUND_UP macro for better readability.
Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
The address of ctrl and sel registers can be calculated by simple equations.
This patch simplifies the implementation in tps65912_get_ctrl_register and
implements tps65912_get_sel_register to replace tps65912_get_dcdc_sel_register
and tps65912_get_ldo_sel_register.Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
We actually clear LDO_ST_ON_BIT for standby mode in tps65910_set_mode.
Fix the logic in tps65910_get_mode.Supply state (EEPROM bits):
ST[1:0] = 00 : Off
ST[1:0] = 01 : On high power (ACTIVE)
ST[1:0] = 10 : Off
ST[1:0] = 11 : On low power (SLEEP)Signed-off-by: Axel Lin
Acked-by: Laxman Dewangan
Signed-off-by: Mark Brown
12 Mar, 2012
24 commits
-
Most callback functions implementation for tps65217_pmic_ldo234_ops and
tps65217_pmic_dcdc_ops are the same except the rid range checking.This patch uses tps65217_pmic_ops for all DCDCx, LDO2, LDO3, and LDO4.
And rework tps65217_pmic_set_voltage to make it can be called for
DCDCx, LDO2, LDO3, and LDO4.Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
DIV_ROUND_CLOSEST performs the computation (x + d/2)/d with better readability.
Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
Then we can use the regulator id as array index to access the array.
This change makes the code simpler.Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
This function finds the smallest voltage that falls within the specified range,
and then returns the selector. This rename makes the intention more clear.Also remove unneeded local variables min_vol and max_vol in s5m8767_set_voltage
and s5m8767_set_voltage_buck.Signed-off-by: Axel Lin
Acked-by: Sangbeom Kim
Signed-off-by: Mark Brown -
Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
This change improves readability.
Signed-off-by: Axel Lin
Acked-by: Laxman Dewangan
Signed-off-by: Mark Brown -
In the s5m8767_set_voltage_time_sel function, divisor unit is wrong.
ramp_delay is usec unit. So 1000 should be multiplied.Signed-off-by: Sangbeom Kim
Signed-off-by: Mark Brown -
Signed-off-by: Thierry Reding
Signed-off-by: Mark Brown -
Keep the rails OFF in sleep mode only when the rails are
controlled by external sleep control.
The devices tps65910 and tps65911, both has the sleep input.
The tps65911's sleep input is not same as tps65910's EN3 and hence
taking care of SLEEP input as separate external sleep control input.Signed-off-by: Laxman Dewangan
Signed-off-by: Mark Brown -
Call tps65912_list_voltage_dcdc instead of duplicating the same code.
Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
Use DIV_ROUND_UP macro for better readability.
Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
This looks like a mistakenly committed debug logging.
Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
Signed-off-by: Axel Lin
Acked-by: Sangbeom Kim
Signed-off-by: Mark Brown -
Use DIV_ROUND_UP macro for better readability.
Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
Use DIV_ROUND_UP macro for better readability.
Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
If used as a dummy voltage provider the fixed regulator should
not set n_voltages to make the core accept the device.Signed-off-by: Sascha Hauer
Signed-off-by: Mark Brown -
Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
Signed-off-by: Axel Lin
Acked-by: Sonic Zhang
Signed-off-by: Mark Brown -
The reason we add *pdev in struct aat2870_regulator is to use it
to get a pointer to struct aat2870_data.
Save a pointer to struct aat2870_data instead of pdev in struct
aat2870_regulator, this change makes the intention more clear.Signed-off-by: Axel Lin
Acked-by: Jin Park
Signed-off-by: Mark Brown -
Signed-off-by: Axel Lin
Acked-by: Marek Vasut
Signed-off-by: Mark Brown -
Use rdev_get_id() directly.
Signed-off-by: Axel Lin
Signed-off-by: Mark Brown -
Use rdev_get_id() directly.
Signed-off-by: Axel Lin
Signed-off-by: Mark Brown