linux-toradex.git/drivers/regulator/88pm800.c, branch v4.4.136 Linux kernel for Apalis and Colibri modules Merge branch 'fix/88pm800' of git://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator into regulator-88pm800 2015-07-17T11:42:48+00:00 Mark Brown broonie@kernel.org 2015-07-17T11:42:48+00:00 bf1fc23c80e61695e8e899354d17549a4946a9d0 Conflicts: drivers/regulator/88pm800.c 
Conflicts:
	drivers/regulator/88pm800.c
regulator: 88pm800: Use regulator_nodes/of_match in the descriptor 2015-07-16T20:32:58+00:00 Vaibhav Hiremath vaibhav.hiremath@linaro.org 2015-07-16T18:16:56+00:00 fa26e4d2b3cf1c84f20fbc003501bb028f682d92 This patch is add regulator_nodes/of_match in the regulator descriptor for using information from DT instead of specific codes. With this patch, driver gets simplified, - No need to maintain "struct of_regulator_match" table and call of_regulator_match() fn. - No need for pm800_regulator_dt_init() fn, as it was only used for of_regulator_match(). - probe() fn got simplified around regulator_config and regulator_desc initialization. Signed-off-by: Vaibhav Hiremath <vaibhav.hiremath@linaro.org> Signed-off-by: Mark Brown <broonie@kernel.org> 
This patch is add regulator_nodes/of_match in the regulator
descriptor for using information from DT instead of specific codes.

With this patch, driver gets simplified,

  - No need to maintain "struct of_regulator_match" table
    and call of_regulator_match() fn.
  - No need for pm800_regulator_dt_init() fn, as it was only
    used for of_regulator_match().
  - probe() fn got simplified around regulator_config and regulator_desc
    initialization.

Signed-off-by: Vaibhav Hiremath <vaibhav.hiremath@linaro.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
regulator: 88pm800: Update driver to use devm_regulator_register fn 2015-07-16T20:32:58+00:00 Vaibhav Hiremath vaibhav.hiremath@linaro.org 2015-07-16T18:16:55+00:00 a07d94a54b93d94d8cb990ffe018c595cfb94662 This patch replaces standard regulator_register with devm_regulator_register() fn, as using devm_regulator_register() fn simplifies the driver return/exit path. As part of this update, patch also cleanups up all unnecessary changes which is result of this patch - - Remove _remove() fn, as devm_ variant takes care of it. - Remove pm800_regulators.regulators[] field, as it was only needed during cleanup, so we no longer need this. This also saved some amount of memory. Signed-off-by: Vaibhav Hiremath <vaibhav.hiremath@linaro.org> Signed-off-by: Mark Brown <broonie@kernel.org> 
This patch replaces standard regulator_register with
devm_regulator_register() fn, as using devm_regulator_register() fn
simplifies the driver return/exit path.

As part of this update, patch also cleanups up all unnecessary changes
which is result of this patch -

  - Remove _remove() fn, as devm_ variant takes care of it.
  - Remove pm800_regulators.regulators[] field, as it was only
    needed during cleanup, so we no longer need this.
    This also saved some amount of memory.

Signed-off-by: Vaibhav Hiremath <vaibhav.hiremath@linaro.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
regulator: 88pm800: Fix indentation of assignments of data structures 2015-07-16T20:32:58+00:00 Vaibhav Hiremath vaibhav.hiremath@linaro.org 2015-07-16T18:16:54+00:00 4d45c70bfb3ffe55fad1d1681feaec2094f4da3b This patch makes code more clean from readability point of view, make all assignments of LDO, BUCk and regulator_ops structure at the same indentation. Signed-off-by: Vaibhav Hiremath <vaibhav.hiremath@linaro.org> Signed-off-by: Mark Brown <broonie@kernel.org> 
This patch makes code more clean from readability point of view,
make all assignments of LDO, BUCk and regulator_ops structure
at the same indentation.

Signed-off-by: Vaibhav Hiremath <vaibhav.hiremath@linaro.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
regulator: 88pm800: fix LDO vsel_mask value 2015-07-09T19:04:25+00:00 Yi Zhang yizhang@marvell.com 2015-07-09T12:41:31+00:00 322dfa6402ec08da1ca182ac0061a1cf2c5c3101 As per datasheet, Except LDO2, all other LDO's use bit [3:0] for VOUT select. Current code uses wrong mask value of 0x1f, So this patch fixes it to use 0xf. Signed-off-by: Yi Zhang <yizhang@marvell.com> [vaibhav.hiremath@linaro.org: Updated changelog with more detailed description] Signed-off-by: Vaibhav Hiremath <vaibhav.hiremath@linaro.org> Signed-off-by: Mark Brown <broonie@kernel.org> 
As per datasheet,
Except LDO2, all other LDO's use bit [3:0] for VOUT select.

Current code uses wrong mask value of 0x1f, So this patch
fixes it to use 0xf.

Signed-off-by: Yi Zhang <yizhang@marvell.com>
[vaibhav.hiremath@linaro.org: Updated changelog with more detailed description]
Signed-off-by: Vaibhav Hiremath <vaibhav.hiremath@linaro.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
regulator: drop owner assignment from platform_drivers 2014-10-20T14:21:29+00:00 Wolfram Sang wsa@the-dreams.de 2014-10-20T14:21:29+00:00 6c794b2654555d6af7572e3a8f13b2b39126df0c A platform_driver does not need to set an owner, it will be populated by the driver core. Signed-off-by: Wolfram Sang <wsa@the-dreams.de> 
A platform_driver does not need to set an owner, it will be populated by the
driver core.

Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
regulator: 88pm800: remove duplicate PM800_BUCK3 define 2014-06-09T19:58:39+00:00 Dan Carpenter dan.carpenter@oracle.com 2014-06-09T15:09:52+00:00 ce44beb7c0b12d33ce88b396aeaea0ba4ca1bbfc The PM800_BUCK3 define is cut and pasted twice so we can remove the second instance. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Mark Brown <broonie@linaro.org> 
The PM800_BUCK3 define is cut and pasted twice so we can remove the
second instance.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mark Brown <broonie@linaro.org>
regulator: 88pm800: Remove redundant error message 2014-02-19T16:34:38+00:00 Sachin Kamat sachin.kamat@linaro.org 2014-02-18T10:41:07+00:00 001f004ba71f5b405e67b52ecf29322dbfbcc47b kzalloc prints its own OOM message upon failure. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Mark Brown <broonie@linaro.org> 
kzalloc prints its own OOM message upon failure.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Mark Brown <broonie@linaro.org>
regulator: Add REGULATOR_LINEAR_RANGE macro 2013-10-11T11:49:16+00:00 Axel Lin axel.lin@ingics.com 2013-10-11T01:32:18+00:00 8828bae464b129abed95b748263f1ab53bdc5755 Add REGULATOR_LINEAR_RANGE macro and convert regulator drivers to use it. Signed-off-by: Axel Lin <axel.lin@ingics.com> Signed-off-by: Mark Brown <broonie@linaro.org> 
Add REGULATOR_LINEAR_RANGE macro and convert regulator drivers to use it.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Mark Brown <broonie@linaro.org>
regulator: Remove max_uV from struct regulator_linear_range 2013-10-11T11:49:12+00:00 Axel Lin axel.lin@ingics.com 2013-10-11T01:30:24+00:00 e277e656804c85a0729d4fd8cdd3c8ab3e6b3b86 linear ranges means each range has linear voltage settings. So we can calculate max_uV for each linear range in regulator core rather than set the max_uV field in drivers. Signed-off-by: Axel Lin <axel.lin@ingics.com> Signed-off-by: Mark Brown <broonie@linaro.org> 
linear ranges means each range has linear voltage settings.
So we can calculate max_uV for each linear range in regulator core rather than
set the max_uV field in drivers.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Mark Brown <broonie@linaro.org>