- 16 May, 2019 1 commit
-
-
Axel Lin authored
A NULL init_data once incremented will lead to oops, fix it. Fixes: f979c08f ("regulator: tps6507x: Convert to regulator core's simplified DT parsing code") Reported-by:
Sekhar Nori <nsekhar@ti.com> Signed-off-by:
Axel Lin <axel.lin@ingics.com> Tested-by:
Sekhar Nori <nsekhar@ti.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
- 03 May, 2019 7 commits
-
-
Jorge Ramirez-Ortiz authored
All the current clients of this API assume that 0 corresponds to a failure and non-zero to a pass therefore ignoring the need to handle a negative error code. This commit modifies the API to follow that standard since returning a negative (EINVAL) doesn't seem to provide enough value to justify the need to handle it. Signed-off-by:
Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
Convert Dialog Semiconductor DA9xxx regulator drivers to SPDX identifier. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
For fixed regulator, setting .n_voltages = 1 and .fixed_uV is enough, no need to set .min_uV and .list_volage. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
Just use rdev->desc instead. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
Slightly better readability by setting fixed_uV instead of min_uV. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
These regulator_ops variables and ab3100_regulator_desc array never need to be modified, make them const so compiler can put them to .rodata. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
Convert Powerventure Semiconductor PV88060/PV88080/PV88090 regulator drivers to SPDX identifier. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
- 02 May, 2019 3 commits
-
-
Axel Lin authored
Convert HiSilicon hi6xxx PMIC drivers to SPDX identifier. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
Signed-off-by:
Axel Lin <axel.lin@ingics.com> Acked-by:
Sudeep Holla <sudeep.holla@arm.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
The *regdev and *regmap can be replaced by local variables in probe(). Only desc of struct vexpress_regulator is really need, so just use struct regulator_desc directly and remove struct vexpress_regulator. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Reviewed-by:
Sudeep Holla <sudeep.holla@arm.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
- 26 Apr, 2019 10 commits
-
-
Axel Lin authored
Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
sky81452_reg_ops is never changed, make it const. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
All the fields in struct sy8106a are only used in sy8106a_i2c_probe(), so use local variables instead. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Jorge Ramirez-Ortiz authored
Temporary failures to get a regulator (EPROBE_DEFER) should be logged as debug information instead of errors. Signed-off-by:
Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
Convert MediaTek mt63xx PMIC drivers to SPDX identifier. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
The *regmap and *rdev can be replaced by local variables. The slew_rate is no longer used since commit dd7e71fb ("regulator: fan53555: use set_ramp_delay to set the ramp up slew rate"). Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
After converting to use simplified DT parsing, the struct ltc3589_regulator only has a desc member. So let's remove struct ltc3589_regulator and use struct regulator_desc instead. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
Use regulator core's simplified DT parsing code to simplify the driver implementation. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
- 17 Apr, 2019 7 commits
-
-
Axel Lin authored
Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
Signed-off-by:
Axel Lin <axel.lin@ingics.com> Acked-by:
Richard Fitzgerald <rf@opensource.cirrus.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Wei Yongjun authored
In case of error, the function of_iomap() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Fixes: 6cdae817 ("regulator: Add support for stm32 power regulators") Signed-off-by:
Wei Yongjun <weiyongjun1@huawei.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
kbuild test robot authored
Fixes: 6cdae817 ("regulator: Add support for stm32 power regulators") Signed-off-by:
kbuild test robot <lkp@intel.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
- 16 Apr, 2019 3 commits
-
-
Axel Lin authored
Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
These regulator_ops variables and tps80031_dcdc_voltages array never need to be modified, make them const so compiler can put them to .rodata. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
The ri-rdev is assigend but not used, so remove it. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
- 15 Apr, 2019 9 commits
-
-
Pascal PAILLET-LME authored
Add support for 1V1 1V8 USB3V3 power regulators. Signed-off-by:
Pascal Paillet <p.paillet@st.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Pascal PAILLET-LME authored
Document devicetree bindings for stm32mp1 pwr regulators. Signed-off-by:
Pascal Paillet <p.paillet@st.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
The struct voltage_map_desc is used for storing min/max/step fileds, and then setup the min_uV, uV_step and n_voltages fields for regulator_desc. Introduce MAX8998_LINEAR_REG/MAX8998_OTHERS_REG macro to setup these filds directly in regulator_desc to simplify the code. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
These regulator_ops variables never need to be modified, make them const so compiler can put them to .rodata. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
Current code is using devm_regulator_register() so we don't need to store *rdev for clean up, use a local variable instead. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
The regulator_ops is never changed, make it const. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
Use regulator_get_current_limit_regmap helper to save some code. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
Current code is using devm_regulator_register() so we don't need to save *regulator for clean up, use a local variable instead. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-
Axel Lin authored
These regulator_ops variables never need to be modified, make them const so compiler can put them to .rodata. Signed-off-by:
Axel Lin <axel.lin@ingics.com> Signed-off-by:
Mark Brown <broonie@kernel.org>
-