Fix ERROR: devm_regmap_init_i2c [drivers/regulator/isl9305.ko] undefined!
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/Kconfig b/drivers/regulator/Kconfig
index 2b98a34..2419fe4 100644
2014-09-10 11:50 GMT+08:00 Guodong Xu guodong...@linaro.org:
These of_node_get() were added to balance refcount decrements inside of
of_find_node_by_name().
See: commit c92f5dd2c42f (regulator: Add missing of_node_put())
However of_find_node_by_name() was then replaced by
2014-09-10 12:20 GMT+08:00 Axel Lin axel@ingics.com:
2014-09-10 11:50 GMT+08:00 Guodong Xu guodong...@linaro.org:
These of_node_get() were added to balance refcount decrements inside of
of_find_node_by_name().
See: commit c92f5dd2c42f (regulator: Add missing of_node_put())
However
2014-09-06 21:49 GMT+08:00 Mark Brown :
> On Sat, Sep 06, 2014 at 08:59:36PM +0800, Axel Lin wrote:
>> Signed-off-by: Axel Lin
>
> Applied, thanks. Please always send incremental patches.
Thanks, and sorry that I have limited internet access while traveling.
I did not realize th
Signed-off-by: Axel Lin
---
drivers/regulator/bcm590xx-regulator.c | 8
1 file changed, 8 deletions(-)
diff --git a/drivers/regulator/bcm590xx-regulator.c
b/drivers/regulator/bcm590xx-regulator.c
index 5d1fd6f..fe6ac69 100644
--- a/drivers/regulator/bcm590xx-regulator.c
+++ b/drivers
, so remove it.
_min_uV and _max_uV are not required, all required settings are set in
REGULATOR_LINEAR_RANGE macro.
Signed-off-by: Axel Lin
---
drivers/regulator/as3711-regulator.c | 61 +---
1 file changed, 28 insertions(+), 33 deletions(-)
diff --git a/drivers
, so remove it.
_min_uV and _max_uV are not required, all required settings are set in
REGULATOR_LINEAR_RANGE macro.
Signed-off-by: Axel Lin
---
drivers/regulator/as3711-regulator.c | 61 +---
1 file changed, 28 insertions(+), 33 deletions(-)
diff --git a/drivers
, so remove it.
_min_uV and _max_uV are not required, all required settings are set in
REGULATOR_LINEAR_RANGE macro.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/as3711-regulator.c | 61 +---
1 file changed, 28 insertions(+), 33 deletions(-)
diff
, so remove it.
_min_uV and _max_uV are not required, all required settings are set in
REGULATOR_LINEAR_RANGE macro.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/as3711-regulator.c | 61 +---
1 file changed, 28 insertions(+), 33 deletions(-)
diff
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/bcm590xx-regulator.c | 8
1 file changed, 8 deletions(-)
diff --git a/drivers/regulator/bcm590xx-regulator.c
b/drivers/regulator/bcm590xx-regulator.c
index 5d1fd6f..fe6ac69 100644
--- a/drivers/regulator/bcm590xx
2014-09-06 21:49 GMT+08:00 Mark Brown broo...@kernel.org:
On Sat, Sep 06, 2014 at 08:59:36PM +0800, Axel Lin wrote:
Signed-off-by: Axel Lin axel@ingics.com
Applied, thanks. Please always send incremental patches.
Thanks, and sorry that I have limited internet access while traveling.
I
Signed-off-by: Axel Lin
---
drivers/regulator/hi6421-regulator.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/regulator/hi6421-regulator.c
b/drivers/regulator/hi6421-regulator.c
index e389920..156d0d1 100644
--- a/drivers/regulator/hi6421-regulator.c
+++ b
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/hi6421-regulator.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/regulator/hi6421-regulator.c
b/drivers/regulator/hi6421-regulator.c
index e389920..156d0d1 100644
--- a/drivers/regulator
Signed-off-by: Axel Lin
---
drivers/regulator/max77802.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/regulator/max77802.c b/drivers/regulator/max77802.c
index 967e109..d89792b 100644
--- a/drivers/regulator/max77802.c
+++ b/drivers/regulator/max77802.c
Signed-off-by: Axel Lin
---
drivers/regulator/hi6421-regulator.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/hi6421-regulator.c
b/drivers/regulator/hi6421-regulator.c
index e389920..a8c362c 100644
--- a/drivers/regulator/hi6421-regulator.c
+++ b/drivers
Also move da9211_i2c_id and da9211_dt_ids close to the user for better
readability.
Signed-off-by: Axel Lin
---
drivers/regulator/da9211-regulator.c | 29 +++--
1 file changed, 15 insertions(+), 14 deletions(-)
diff --git a/drivers/regulator/da9211-regulator.c
b
Also move da9211_i2c_id and da9211_dt_ids close to the user for better
readability.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/da9211-regulator.c | 29 +++--
1 file changed, 15 insertions(+), 14 deletions(-)
diff --git a/drivers/regulator/da9211
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/hi6421-regulator.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/hi6421-regulator.c
b/drivers/regulator/hi6421-regulator.c
index e389920..a8c362c 100644
--- a/drivers/regulator/hi6421
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/max77802.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/regulator/max77802.c b/drivers/regulator/max77802.c
index 967e109..d89792b 100644
--- a/drivers/regulator/max77802.c
+++ b/drivers/regulator
Signed-off-by: Axel Lin
---
drivers/regulator/da9211-regulator.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/da9211-regulator.c
b/drivers/regulator/da9211-regulator.c
index 9722728..044c36c 100644
--- a/drivers/regulator/da9211-regulator.c
+++ b/drivers/regulator
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/da9211-regulator.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/da9211-regulator.c
b/drivers/regulator/da9211-regulator.c
index 9722728..044c36c 100644
--- a/drivers/regulator/da9211-regulator.c
+++ b
2014-09-02 10:11 GMT+08:00 Greg Kroah-Hartman :
> On Tue, Sep 02, 2014 at 09:47:37AM +0800, Axel Lin wrote:
>> 2014-08-03 10:36 GMT+08:00 Axel Lin :
>> > This driver uses GENERIC_IRQ_CHIP, so it needs to select GENERIC_IRQ_CHIP
>> > to
>> > avoid build error
2014-08-03 10:36 GMT+08:00 Axel Lin :
> This driver uses GENERIC_IRQ_CHIP, so it needs to select GENERIC_IRQ_CHIP to
> avoid build error.
>
> Fixes below build errors:
> ERROR: "irq_alloc_domain_generic_chips" [drivers/gpu/ipu-v3/imx-ipu-v3.ko]
> undefined!
> ERROR:
The min_sel is 0, max_sel is 15, so n_voltages should be 16.
Signed-off-by: Axel Lin
---
drivers/regulator/rk808-regulator.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/rk808-regulator.c
b/drivers/regulator/rk808-regulator.c
index 0d11df1..0da8867
The min_sel is 0, max_sel is 15, so n_voltages should be 16.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/rk808-regulator.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/rk808-regulator.c
b/drivers/regulator/rk808-regulator.c
index
2014-08-03 10:36 GMT+08:00 Axel Lin axel@ingics.com:
This driver uses GENERIC_IRQ_CHIP, so it needs to select GENERIC_IRQ_CHIP to
avoid build error.
Fixes below build errors:
ERROR: irq_alloc_domain_generic_chips [drivers/gpu/ipu-v3/imx-ipu-v3.ko]
undefined!
ERROR: irq_gc_mask_clr_bit
2014-09-02 10:11 GMT+08:00 Greg Kroah-Hartman gre...@linuxfoundation.org:
On Tue, Sep 02, 2014 at 09:47:37AM +0800, Axel Lin wrote:
2014-08-03 10:36 GMT+08:00 Axel Lin axel@ingics.com:
This driver uses GENERIC_IRQ_CHIP, so it needs to select GENERIC_IRQ_CHIP
to
avoid build error
The memory allocated in rk808_regulator_probe() needs to be freed when the
module is unloaded. Thus pass >dev rather than >dev to
devm_kzalloc.
Signed-off-by: Axel Lin
---
drivers/regulator/rk808-regulator.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/reg
The memory allocated in rk808_regulator_probe() needs to be freed when the
module is unloaded. Thus pass pdev-dev rather than client-dev to
devm_kzalloc.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/rk808-regulator.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
The valid_modes_mask and *dev are not used in this driver, remove them.
Current code uses devm_regulator_register, so we don't need *regulator in
hi6421_regulator_info. Use a local variable instead.
Also removes a few unnecessary inclusion of header files.
Signed-off-by: Axel Lin
---
drivers
r due
to const qualifier for ops"). The build error was fixed by commit 39f5460d7f9c
("regulator: core: add const to regulator_ops and fix build error in mc13892").
Signed-off-by: Axel Lin
---
include/linux/regulator/driver.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff -
to const qualifier for ops). The build error was fixed by commit 39f5460d7f9c
(regulator: core: add const to regulator_ops and fix build error in mc13892).
Signed-off-by: Axel Lin axel@ingics.com
---
include/linux/regulator/driver.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
The valid_modes_mask and *dev are not used in this driver, remove them.
Current code uses devm_regulator_register, so we don't need *regulator in
hi6421_regulator_info. Use a local variable instead.
Also removes a few unnecessary inclusion of header files.
Signed-off-by: Axel Lin axel
2014-08-18 22:10 GMT+08:00 Mark Brown :
> On Mon, Aug 18, 2014 at 10:34:08AM +0800, Axel Lin wrote:
>> Drop const qualifier for ops of struct regulator_desc.
>> Allow regulator drivers to update ops before registering regulator.
>
> Applied, thanks.
Hi Mark,
I found this co
Both num_regulators and *rdev[MAX77802_REG_MAX] are not used, remove them.
Signed-off-by: Axel Lin
---
drivers/regulator/max77802.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/regulator/max77802.c b/drivers/regulator/max77802.c
index 5f022f8..ad1caa9 100644
--- a/drivers
Both num_regulators and *rdev[MAX77802_REG_MAX] are not used, remove them.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/max77802.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/regulator/max77802.c b/drivers/regulator/max77802.c
index 5f022f8..ad1caa9 100644
2014-08-18 22:10 GMT+08:00 Mark Brown broo...@kernel.org:
On Mon, Aug 18, 2014 at 10:34:08AM +0800, Axel Lin wrote:
Drop const qualifier for ops of struct regulator_desc.
Allow regulator drivers to update ops before registering regulator.
Applied, thanks.
Hi Mark,
I found this commit
: *** [drivers] Error 2
Reported-by: Stephen Rothwell
Signed-off-by: Axel Lin
---
include/linux/regulator/driver.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/regulator/driver.h b/include/linux/regulator/driver.h
index efe058f..3abda75 100644
--- a/include/linux
Signed-off-by: Axel Lin
---
drivers/regulator/da9211-regulator.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/regulator/da9211-regulator.c
b/drivers/regulator/da9211-regulator.c
index ccc2e36..a26f1d2 100644
--- a/drivers/regulator/da9211-regulator.c
+++ b/drivers/regulator
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/da9211-regulator.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/regulator/da9211-regulator.c
b/drivers/regulator/da9211-regulator.c
index ccc2e36..a26f1d2 100644
--- a/drivers/regulator/da9211-regulator.c
+++ b
: *** [drivers] Error 2
Reported-by: Stephen Rothwell s...@canb.auug.org.au
Signed-off-by: Axel Lin axel@ingics.com
---
include/linux/regulator/driver.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/regulator/driver.h b/include/linux/regulator/driver.h
index efe058f
2014-08-13 5:55 GMT+08:00 Mark Brown :
> On Tue, Aug 12, 2014 at 09:37:09AM +0800, Axel Lin wrote:
>> Fix below build error when !CONFIG_REGULATOR.
>>
>> CC drivers/gpu/drm/msm/hdmi/hdmi.o
>> drivers/gpu/drm/msm/hdmi/hdmi.c: In function 'hdmi_init':
>> driv
variable for regmap_config in da9052-spi driver,
so the settings in spi driver won't impact the settings in i2c driver.
Also makes da9052_regmap_config const to avoid similar issue.
Signed-off-by: Axel Lin
---
Hi Adam and Steve,
Any chance to test this patch?
Thanks,
Axel
drivers/mfd/da9052
a local variable for regmap_config in da9052-spi driver,
so the settings in spi driver won't impact the settings in i2c driver.
Also makes da9052_regmap_config const to avoid similar issue.
Signed-off-by: Axel Lin axel@ingics.com
---
Hi Adam and Steve,
Any chance to test this patch?
Thanks
2014-08-13 5:55 GMT+08:00 Mark Brown broo...@kernel.org:
On Tue, Aug 12, 2014 at 09:37:09AM +0800, Axel Lin wrote:
Fix below build error when !CONFIG_REGULATOR.
CC drivers/gpu/drm/msm/hdmi/hdmi.o
drivers/gpu/drm/msm/hdmi/hdmi.c: In function 'hdmi_init':
drivers/gpu/drm/msm/hdmi/hdmi.c
So remove the I2C_FUNC_SMBUS_BYTE_DATA functionality check in the driver code.
Signed-off-by: Axel Lin
---
drivers/mfd/da9052-i2c.c | 7 ---
1 file changed, 7 deletions(-)
diff --git a/drivers/mfd/da9052-i2c.c b/drivers/mfd/da9052-i2c.c
index 6da8ec8..ec39287 100644
--- a/drivers/mfd/da9052-i2c.c
+++ b/drivers/mfd/da
functionality check in the driver code.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/mfd/da9052-i2c.c | 7 ---
1 file changed, 7 deletions(-)
diff --git a/drivers/mfd/da9052-i2c.c b/drivers/mfd/da9052-i2c.c
index 6da8ec8..ec39287 100644
--- a/drivers/mfd/da9052-i2c.c
+++ b/drivers
So remove the I2C_FUNC_SMBUS_BYTE_DATA functionality check in the driver code.
Signed-off-by: Axel Lin
---
drivers/regulator/tps65023-regulator.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/regulator/tps65023-regulator.c
b/drivers/regulator/tps65023-regulator.c
index 3ef67a8..7380af8 100644
--- a/drivers
]
Also makes regulator_get_exclusive and devm_regulator_get_exclusive stub
functions return error pointer.
Signed-off-by: Axel Lin
---
include/linux/regulator/consumer.h | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/include/linux/regulator/consumer.h
b/include/linux
]
Also makes regulator_get_exclusive and devm_regulator_get_exclusive stub
functions return error pointer.
Signed-off-by: Axel Lin axel@ingics.com
---
include/linux/regulator/consumer.h | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/include/linux/regulator/consumer.h
functionality check in the driver code.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/tps65023-regulator.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/regulator/tps65023-regulator.c
b/drivers/regulator/tps65023-regulator.c
index 3ef67a8..7380af8 100644
]
Signed-off-by: Axel Lin
---
include/linux/regulator/consumer.h | 6 ++
1 file changed, 6 insertions(+)
diff --git a/include/linux/regulator/consumer.h
b/include/linux/regulator/consumer.h
index f8a8733..150d2f0 100644
--- a/include/linux/regulator/consumer.h
+++ b/include/linux/regulator
]
Signed-off-by: Axel Lin axel@ingics.com
---
include/linux/regulator/consumer.h | 6 ++
1 file changed, 6 insertions(+)
diff --git a/include/linux/regulator/consumer.h
b/include/linux/regulator/consumer.h
index f8a8733..150d2f0 100644
--- a/include/linux/regulator/consumer.h
+++ b/include
Signed-off-by: Axel Lin
---
drivers/gpu/ipu-v3/ipu-common.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/ipu-v3/ipu-common.c b/drivers/gpu/ipu-v3/ipu-common.c
index 04e7b2e..e340beb 100644
--- a/drivers/gpu/ipu-v3/ipu-common.c
+++ b/drivers/gpu/ipu-v3/ipu
ip" [drivers/gpu/ipu-v3/imx-ipu-v3.ko]
undefined!
make[1]: *** [__modpost] Error 1
make: *** [modules] Error 2
Signed-off-by: Axel Lin
---
drivers/gpu/ipu-v3/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/ipu-v3/Kconfig b/drivers/gpu/ipu-v3/Kconfig
index 2f228a2..0
]: *** [__modpost] Error 1
make: *** [modules] Error 2
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/gpu/ipu-v3/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/ipu-v3/Kconfig b/drivers/gpu/ipu-v3/Kconfig
index 2f228a2..01864a5 100644
--- a/drivers/gpu/ipu-v3/Kconfig
+++ b
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/gpu/ipu-v3/ipu-common.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/ipu-v3/ipu-common.c b/drivers/gpu/ipu-v3/ipu-common.c
index 04e7b2e..e340beb 100644
--- a/drivers/gpu/ipu-v3/ipu-common.c
+++ b
This driver also supports S2MPU02 now, thus update module description and
Kconfig accordingly.
Signed-off-by: Axel Lin
---
drivers/regulator/Kconfig | 4 ++--
drivers/regulator/s2mps11.c | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/regulator/Kconfig b/drivers
This driver also supports S2MPU02 now, thus update module description and
Kconfig accordingly.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/Kconfig | 4 ++--
drivers/regulator/s2mps11.c | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers
Current code uses devm_regulator_register(), so the resource management code
will ensure that the resource is freed.
Signed-off-by: Axel Lin
---
drivers/regulator/da9211-regulator.c | 28
1 file changed, 4 insertions(+), 24 deletions(-)
diff --git a/drivers
Current code uses devm_regulator_register(), so the resource management code
will ensure that the resource is freed.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/da9211-regulator.c | 28
1 file changed, 4 insertions(+), 24 deletions(-)
diff
ested-by: Wenyou Yang
For this serial, Reviewed-by: Axel Lin
patch 3/4 was not CCed to me, but I have checked the code on LKML mail list.
Regards,
Axel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.o
2014-07-05 23:08 GMT+08:00 Axel Lin :
> 2014-07-05 21:20 GMT+08:00 Beniamino Galvani :
>> act8865_pdata_from_dt() populates the array pdata->regulators with all
>> the regulators and then assigns the field init_data only for the ones
>> actually found in the DT.
>>
2014-07-05 21:20 GMT+08:00 Beniamino Galvani :
> act8865_pdata_from_dt() populates the array pdata->regulators with all
> the regulators and then assigns the field init_data only for the ones
> actually found in the DT.
>
> The patch changes the value assigned to pdata->num_regulators to match
>
2014-07-05 21:20 GMT+08:00 Beniamino Galvani b.galv...@gmail.com:
act8865_pdata_from_dt() populates the array pdata-regulators with all
the regulators and then assigns the field init_data only for the ones
actually found in the DT.
The patch changes the value assigned to pdata-num_regulators
2014-07-05 23:08 GMT+08:00 Axel Lin axel@ingics.com:
2014-07-05 21:20 GMT+08:00 Beniamino Galvani b.galv...@gmail.com:
act8865_pdata_from_dt() populates the array pdata-regulators with all
the regulators and then assigns the field init_data only for the ones
actually found in the DT
Tested-by: Wenyou Yang wenyou.y...@atmel.com
For this serial, Reviewed-by: Axel Lin axel@ingics.com
patch 3/4 was not CCed to me, but I have checked the code on LKML mail list.
Regards,
Axel
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message
Silence below build warning:
CC [M] drivers/regulator/act8865-regulator.o
drivers/regulator/act8865-regulator.c: In function ‘act8865_pmic_probe’:
drivers/regulator/act8865-regulator.c:321: warning: ‘of_node’ may be used
uninitialized in this function
Signed-off-by: Axel Lin
---
drivers
Simply use ret variable instead.
Also remove unneeded initialize for ret variable.
Signed-off-by: Axel Lin
---
drivers/regulator/act8865-regulator.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/regulator/act8865-regulator.c
b/drivers/regulator/act8865
Simply use ret variable instead.
Also remove unneeded initialize for ret variable.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/act8865-regulator.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/regulator/act8865-regulator.c
b/drivers
Silence below build warning:
CC [M] drivers/regulator/act8865-regulator.o
drivers/regulator/act8865-regulator.c: In function ‘act8865_pmic_probe’:
drivers/regulator/act8865-regulator.c:321: warning: ‘of_node’ may be used
uninitialized in this function
Signed-off-by: Axel Lin axel
Signed-off-by: Axel Lin
---
drivers/regulator/max8952.c | 34 ++
1 file changed, 10 insertions(+), 24 deletions(-)
diff --git a/drivers/regulator/max8952.c b/drivers/regulator/max8952.c
index c2792f0..f7f9efc 100644
--- a/drivers/regulator/max8952.c
+++ b/drivers
Signed-off-by: Axel Lin
---
drivers/regulator/twl-regulator.c | 13 +
1 file changed, 1 insertion(+), 12 deletions(-)
diff --git a/drivers/regulator/twl-regulator.c
b/drivers/regulator/twl-regulator.c
index fed28ab..0b4f866 100644
--- a/drivers/regulator/twl-regulator.c
+++ b
Use devm_clk_register() to simplify the code by removing twl6040_clk_remove().
Signed-off-by: Axel Lin
Acked-by: Peter Ujfalusi
---
This patch was sent on https://lkml.org/lkml/2014/5/18/75 with Peter's Ack.
drivers/clk/clk-twl6040.c | 12 +---
1 file changed, 1 insertion(+), 11
The devm_clk_unregister() in .probe error case is not necessary as it will be
automatically called when probe fails.
Signed-off-by: Axel Lin
Reviewed-by: Krzysztof Kozlowski
---
This patch was sent on https://lkml.org/lkml/2014/5/18/96 with Krzysztof's
review.
drivers/clk/clk-s2mps11.c | 14
The devm_clk_unregister() in .probe error case is not necessary as it will be
automatically called when probe fails.
Signed-off-by: Axel Lin axel@ingics.com
Reviewed-by: Krzysztof Kozlowski k.kozlow...@samsung.com
---
This patch was sent on https://lkml.org/lkml/2014/5/18/96 with Krzysztof's
Use devm_clk_register() to simplify the code by removing twl6040_clk_remove().
Signed-off-by: Axel Lin axel@ingics.com
Acked-by: Peter Ujfalusi peter.ujfal...@ti.com
---
This patch was sent on https://lkml.org/lkml/2014/5/18/75 with Peter's Ack.
drivers/clk/clk-twl6040.c | 12 +---
1
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/twl-regulator.c | 13 +
1 file changed, 1 insertion(+), 12 deletions(-)
diff --git a/drivers/regulator/twl-regulator.c
b/drivers/regulator/twl-regulator.c
index fed28ab..0b4f866 100644
--- a/drivers/regulator/twl
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/max8952.c | 34 ++
1 file changed, 10 insertions(+), 24 deletions(-)
diff --git a/drivers/regulator/max8952.c b/drivers/regulator/max8952.c
index c2792f0..f7f9efc 100644
--- a/drivers/regulator
CONFIG_REGULATOR_AB8500_DEBUG is always not defined.
ab8500_regulator_debug_init() is not called at all now,
ab8500_regulator_debug_exit() simply return 0, thus remove them.
Signed-off-by: Axel Lin
---
drivers/regulator/ab8500.c | 13 -
include/linux/regulator/ab8500.h | 14
Now this is a DT-only driver because non-devicetree probe path is removed,
so merge ab8500_regulator_of_probe() into ab8500_regulator_probe().
Signed-off-by: Axel Lin
---
drivers/regulator/ab8500.c | 31 ---
1 file changed, 12 insertions(+), 19 deletions(-)
diff
Now this is a DT-only driver because non-devicetree probe path is removed,
so merge ab8500_regulator_of_probe() into ab8500_regulator_probe().
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/ab8500.c | 31 ---
1 file changed, 12 insertions(+), 19
CONFIG_REGULATOR_AB8500_DEBUG is always not defined.
ab8500_regulator_debug_init() is not called at all now,
ab8500_regulator_debug_exit() simply return 0, thus remove them.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/ab8500.c | 13 -
include/linux
of_find_node_by_name() walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name() instead.
Signed-off-by: Axel Lin
---
drivers/regulator/ltc3589.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/ltc3589.c b/drivers
The voltage tables for dvm/ldo4/sw2 are actually linear mapping.
Thus convert dvm/ldo4/sw2 to use regulator_list_voltage_linear.
Signed-off-by: Axel Lin
---
drivers/regulator/tps6586x-regulator.c | 129 +
1 file changed, 68 insertions(+), 61 deletions(-)
diff
The voltage tables for dvm/ldo4/sw2 are actually linear mapping.
Thus convert dvm/ldo4/sw2 to use regulator_list_voltage_linear.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/tps6586x-regulator.c | 129 +
1 file changed, 68 insertions(+), 61
of_find_node_by_name() walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name() instead.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/ltc3589.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator
Signed-off-by: Axel Lin
---
drivers/regulator/tps6586x-regulator.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/regulator/tps6586x-regulator.c
b/drivers/regulator/tps6586x-regulator.c
index e045b7f..0a3bb3a 100644
--- a/drivers/regulator/tps6586x-regulator.c
+++ b/drivers
This driver is using devm_regulator_register() so we don't need to save the
pointer for cleanup.
Signed-off-by: Axel Lin
---
drivers/regulator/pfuze100-regulator.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/regulator/pfuze100-regulator.c
b/drivers
ulator.
Signed-off-by: Axel Lin
---
drivers/regulator/ltc3589.c | 15 ++-
1 file changed, 2 insertions(+), 13 deletions(-)
diff --git a/drivers/regulator/ltc3589.c b/drivers/regulator/ltc3589.c
index fef64ee..110a99e 100644
--- a/drivers/regulator/ltc3589.c
+++ b/drivers/regulator/ltc
.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/ltc3589.c | 15 ++-
1 file changed, 2 insertions(+), 13 deletions(-)
diff --git a/drivers/regulator/ltc3589.c b/drivers/regulator/ltc3589.c
index fef64ee..110a99e 100644
--- a/drivers/regulator/ltc3589.c
+++ b/drivers
This driver is using devm_regulator_register() so we don't need to save the
pointer for cleanup.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/pfuze100-regulator.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/regulator/pfuze100
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/tps6586x-regulator.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/regulator/tps6586x-regulator.c
b/drivers/regulator/tps6586x-regulator.c
index e045b7f..0a3bb3a 100644
--- a/drivers/regulator/tps6586x
Make this driver depend on I2C and select REGMAP_I2C to fix build failure.
Also allows this driver to be built as module.
Reported-by: Stephen Rothwell
Signed-off-by: Axel Lin
---
drivers/regulator/Kconfig | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/regulator
Current code has .enable_reg and .enable_mask settings, but the implementation
for corresponding callbacks are missing. Fix it.
Signed-off-by: Axel Lin
---
Hi Robin,
Can you review and test this patch?
Thanks,
Axel
drivers/regulator/pfuze100-regulator.c | 3 +++
1 file changed, 3 insertions
Current code has .enable_reg and .enable_mask settings, but the implementation
for corresponding callbacks are missing. Fix it.
Signed-off-by: Axel Lin axel@ingics.com
---
Hi Robin,
Can you review and test this patch?
Thanks,
Axel
drivers/regulator/pfuze100-regulator.c | 3 +++
1 file
Make this driver depend on I2C and select REGMAP_I2C to fix build failure.
Also allows this driver to be built as module.
Reported-by: Stephen Rothwell s...@canb.auug.org.au
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/Kconfig | 4 +++-
1 file changed, 3 insertions(+), 1
Use map_voltage_linear_range() if list_voltage_linear_range() is in use and
nothing is set.
Signed-off-by: Axel Lin
---
drivers/regulator/core.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c
index b97ffd2..7399cbf 100644
Use map_voltage_linear_range() if list_voltage_linear_range() is in use and
nothing is set.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/core.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c
index b97ffd2..7399cbf
Use regulator_set_voltage_time_sel() instead of open-coded.
Signed-off-by: Axel Lin
---
Hi Keerthy,
This patch was sent on:
https://lkml.org/lkml/2014/2/18/190
I'd appreciate if you can review and test it.
Thanks,
Axel
drivers/regulator/tps65218-regulator.c | 37
701 - 800 of 2364 matches
Mail list logo
