The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/pinctrl/pinctrl-at91.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-at91.c b
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/pinctrl/pinctrl-imx6dl.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-imx6dl.c b
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/pinctrl/pinctrl-vf610.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-vf610.c b
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Viresh Kumar viresh.ku...@linaro.org
---
drivers/pinctrl/spear/pinctrl-plgpio.c |2 +-
1 file changed, 1 insertion(+), 1 deletion
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/pinctrl/pinctrl-imx6sl.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-imx6sl.c b
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/pinctrl/pinctrl-samsung.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-samsung.c
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/pinctrl/pinctrl-imx51.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-imx51.c b
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/pinctrl/pinctrl-imx6q.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-imx6q.c b
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/pinctrl/pinctrl-exynos5440.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/mfd/max8925-i2c.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mfd/max8925-i2c.c b/drivers/mfd
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/mfd/tps65217.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mfd/tps65217.c b/drivers/mfd
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/mfd/omap-usb-host.c |2 +-
drivers/mfd/omap-usb-tll.c |2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/mfd/88pm860x-core.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mfd/88pm860x-core.c b/drivers
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/mfd/ti_am335x_tscadc.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mfd/ti_am335x_tscadc.c b
On 26 September 2013 13:20, Jingoo Han wrote:
> Use devm_regulator_register() to make cleanup paths simpler,
> and remove unnecessary remove().
>
> Signed-off-by: Jingoo Han
Patch title looks to be corrupted?
--
With warm regards,
Sachin
--
To unsubscribe from this list: send the line
On 26 September 2013 07:36, Jingoo Han wrote:
> Use devm_regulator_register() to make cleanup paths simpler.
> @@ -349,8 +350,6 @@ static int gpio_regulator_remove(struct platform_device
> *pdev)
> {
> struct gpio_regulator_data *drvdata = platform_get_drvdata(pdev);
>
> -
Hi Jingoo,
On 26 September 2013 07:35, Jingoo Han wrote:
> Use devm_regulator_register() to make cleanup paths simpler.
> struct fixed_voltage_data *drvdata = platform_get_drvdata(pdev);
>
> - regulator_unregister(drvdata->dev);
> kfree(drvdata->desc.supply_name);
>
Hi Jingoo,
On 26 September 2013 07:35, Jingoo Han jg1@samsung.com wrote:
Use devm_regulator_register() to make cleanup paths simpler.
struct fixed_voltage_data *drvdata = platform_get_drvdata(pdev);
- regulator_unregister(drvdata-dev);
On 26 September 2013 07:36, Jingoo Han jg1@samsung.com wrote:
Use devm_regulator_register() to make cleanup paths simpler.
@@ -349,8 +350,6 @@ static int gpio_regulator_remove(struct platform_device
*pdev)
{
struct gpio_regulator_data *drvdata = platform_get_drvdata(pdev);
-
On 26 September 2013 13:20, Jingoo Han jg1@samsung.com wrote:
Use devm_regulator_register() to make cleanup paths simpler,
and remove unnecessary remove().
Signed-off-by: Jingoo Han jg1@samsung.com
Patch title looks to be corrupted?
--
With warm regards,
Sachin
--
To unsubscribe
Hi Lukasz,
On 25 September 2013 16:52, Lukasz Majewski wrote:
>
> static void exynos4x12_set_apll(unsigned int index)
> {
> - unsigned int tmp, pdiv;
> + unsigned int tmp, freq = apll_freq_4x12[index].freq;
nit: It is better to put the 'freq' assignment on a new line.
>
> -
On 25 September 2013 13:21, Jingoo Han wrote:
> Use devm_regulator_register() to make cleanup paths simpler.
> - for (i = 0; i < abx500_regulator.info_size; i++) {
> - struct ab8500_regulator_info *info = NULL;
> - info = _regulator.info[i];
> -
> -
On 25 September 2013 13:32, Jingoo Han wrote:
> Use devm_regulator_register() to make cleanup paths simpler.
> static int lp3972_i2c_remove(struct i2c_client *i2c)
> {
> struct lp3972 *lp3972 = i2c_get_clientdata(i2c);
> - int i;
>
> - for (i = 0; i <
On 25 September 2013 13:07, Wei Yongjun wrote:
> From: Wei Yongjun
>
> - devm_free_irq(>dev, twl6040->irq_ready, twl6040);
> - devm_free_irq(>dev, twl6040->irq_th, twl6040);
> regmap_del_irq_chip(twl6040->irq, twl6040->irq_data);
Now that the freeing of irq will happen after
On 25 September 2013 13:07, Wei Yongjun weiyj...@gmail.com wrote:
From: Wei Yongjun yongjun_...@trendmicro.com.cn
- devm_free_irq(client-dev, twl6040-irq_ready, twl6040);
- devm_free_irq(client-dev, twl6040-irq_th, twl6040);
regmap_del_irq_chip(twl6040-irq,
On 25 September 2013 13:32, Jingoo Han jg1@samsung.com wrote:
Use devm_regulator_register() to make cleanup paths simpler.
static int lp3972_i2c_remove(struct i2c_client *i2c)
{
struct lp3972 *lp3972 = i2c_get_clientdata(i2c);
- int i;
- for (i = 0; i
On 25 September 2013 13:21, Jingoo Han jg1@samsung.com wrote:
Use devm_regulator_register() to make cleanup paths simpler.
- for (i = 0; i abx500_regulator.info_size; i++) {
- struct ab8500_regulator_info *info = NULL;
- info = abx500_regulator.info[i];
Hi Lukasz,
On 25 September 2013 16:52, Lukasz Majewski l.majew...@samsung.com wrote:
static void exynos4x12_set_apll(unsigned int index)
{
- unsigned int tmp, pdiv;
+ unsigned int tmp, freq = apll_freq_4x12[index].freq;
nit: It is better to put the 'freq' assignment on a new
Hi Mateusz,
On 23 September 2013 15:44, Mateusz Krawczuk
wrote:
> It prevents from executing platform code, when booting from device tree.
>
> Signed-off-by: Mateusz Krawczuk
> Signed-off-by: Kyungmin Park
> ---
nit: Please format the patch title appropriately. In this case, it
should start
Hi Mateusz,
On 23 September 2013 15:44, Mateusz Krawczuk
m.krawc...@partner.samsung.com wrote:
It prevents from executing platform code, when booting from device tree.
Signed-off-by: Mateusz Krawczuk m.krawc...@partner.samsung.com
Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com
---
Hi Mateusz,
On 21 September 2013 20:30, Mateusz Krawczuk
wrote:
> This patch series add restoring previous vpll rate after driver offs stream
> or recives error.
> It also replace mxr_info, mxr_dbg, mxr_warn and mxr_err macro
> by generic solution.
It is a good practice to include revision
Hi Mateusz,
On 21 September 2013 20:30, Mateusz Krawczuk
m.krawc...@partner.samsung.com wrote:
This patch series add restoring previous vpll rate after driver offs stream
or recives error.
It also replace mxr_info, mxr_dbg, mxr_warn and mxr_err macro
by generic solution.
It is a good
Driver core sets driver data to NULL upon failure or remove.
Signed-off-by: Sachin Kamat
---
drivers/mfd/mc13xxx-i2c.c |1 -
drivers/mfd/rtsx_pcr.c|5 +
drivers/mfd/ti-ssp.c |1 -
3 files changed, 1 insertion(+), 6 deletions(-)
diff --git a/drivers/mfd/mc13xxx-i2c.c b
Driver core sets driver data to NULL upon failure or remove.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/mfd/mc13xxx-i2c.c |1 -
drivers/mfd/rtsx_pcr.c|5 +
drivers/mfd/ti-ssp.c |1 -
3 files changed, 1 insertion(+), 6 deletions(-)
diff --git
Driver core sets driver data to NULL upon failure or remove.
Signed-off-by: Sachin Kamat
---
drivers/regulator/tps6524x-regulator.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/regulator/tps6524x-regulator.c
b/drivers/regulator/tps6524x-regulator.c
index 62e8d28..8b9ee39
Driver core sets driver data to NULL upon failure or remove.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/tps6524x-regulator.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/regulator/tps6524x-regulator.c
b/drivers/regulator/tps6524x-regulator.c
index
Fixes the following error:
drivers/mfd/aat2870-core.c:296:20: error:
incompatible types in comparison expression (different type sizes)
Signed-off-by: Sachin Kamat
Cc: Jin Park
---
drivers/mfd/aat2870-core.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mfd
Fixes the following error:
drivers/mfd/aat2870-core.c:296:20: error:
incompatible types in comparison expression (different type sizes)
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Jin Park jinyou...@nvidia.com
---
drivers/mfd/aat2870-core.c |2 +-
1 file changed, 1 insertion
tegra_dma_prep_dma_cyclic is referenced only in this file.
Make it static.
Signed-off-by: Sachin Kamat
Cc: Laxman Dewangan
---
drivers/dma/tegra20-apb-dma.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/tegra20-apb-dma.c b/drivers/dma/tegra20-apb-dma.c
Check the return value of regulator_enable to silence the following
warning:
drivers/power/pm2301_charger.c:725:20: warning:
ignoring return value of ‘regulator_enable’, declared with
attribute warn_unused_result [-Wunused-result]
Signed-off-by: Sachin Kamat
Cc: Lee Jones
---
Compile tested
Each of the if-else blocks has a break statement.
Remove the additional one which is unreachable.
Signed-off-by: Sachin Kamat
Acked-by: Lee Jones
---
No changes since v1.
---
drivers/power/ab8500_charger.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/power/ab8500_charger.c b
pm2xxx_charger_die_therm_mngt is used only in this file.
Make it static.
Signed-off-by: Sachin Kamat
Acked-by: Lee Jones
---
No changes since v1.
---
drivers/power/pm2301_charger.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/pm2301_charger.c b/drivers
Check the return value of regulator_enable to silence the following
type of warnings:
drivers/power/ab8500_charger.c:1390:20: warning: ignoring return value
of ‘regulator_enable’, declared with attribute warn_unused_result
[-Wunused-result]
Signed-off-by: Sachin Kamat
Cc: Lee Jones
---
Compile
On 6 September 2013 12:32, Lee Jones wrote:
> On Fri, 06 Sep 2013, Sachin Kamat wrote:
>
>> Check the return value of regulator_enable to silence the following
>> type of warnings:
>> drivers/power/ab8500_charger.c:1390:20: warning: ignoring return value
>> o
Each of the if-else blocks has a break statement.
Remove the additional one which is unreachable.
Signed-off-by: Sachin Kamat
---
No changes since v1.
---
drivers/power/ab8500_charger.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/power/ab8500_charger.c b/drivers/power
pm2xxx_charger_die_therm_mngt is used only in this file.
Make it static.
Signed-off-by: Sachin Kamat
Acked-by: Lee Jones
---
No changes since v1.
---
drivers/power/pm2301_charger.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/pm2301_charger.c b/drivers
Check the return value of regulator_enable to silence the following
type of warnings:
drivers/power/ab8500_charger.c:1390:20: warning: ignoring return value
of ‘regulator_enable’, declared with attribute warn_unused_result
[-Wunused-result]
Signed-off-by: Sachin Kamat
Cc: Lee Jones
---
Compile
Check the return value of regulator_enable to silence the following
warning:
drivers/power/pm2301_charger.c:725:20: warning:
ignoring return value of ‘regulator_enable’, declared with
attribute warn_unused_result [-Wunused-result]
Signed-off-by: Sachin Kamat
Cc: Lee Jones
---
Compile tested
Check the return value of regulator_enable to silence the following
type of warnings:
drivers/power/ab8500_charger.c:1390:20: warning: ignoring return value
of ‘regulator_enable’, declared with attribute warn_unused_result
[-Wunused-result]
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc
Check the return value of regulator_enable to silence the following
warning:
drivers/power/pm2301_charger.c:725:20: warning:
ignoring return value of ‘regulator_enable’, declared with
attribute warn_unused_result [-Wunused-result]
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Lee Jones
Each of the if-else blocks has a break statement.
Remove the additional one which is unreachable.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
No changes since v1.
---
drivers/power/ab8500_charger.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/power/ab8500_charger.c b
pm2xxx_charger_die_therm_mngt is used only in this file.
Make it static.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Acked-by: Lee Jones lee.jo...@linaro.org
---
No changes since v1.
---
drivers/power/pm2301_charger.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
On 6 September 2013 12:32, Lee Jones lee.jo...@linaro.org wrote:
On Fri, 06 Sep 2013, Sachin Kamat wrote:
Check the return value of regulator_enable to silence the following
type of warnings:
drivers/power/ab8500_charger.c:1390:20: warning: ignoring return value
of ‘regulator_enable
Each of the if-else blocks has a break statement.
Remove the additional one which is unreachable.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Acked-by: Lee Jones lee.jo...@linaro.org
---
No changes since v1.
---
drivers/power/ab8500_charger.c |1 -
1 file changed, 1 deletion(-)
diff
pm2xxx_charger_die_therm_mngt is used only in this file.
Make it static.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Acked-by: Lee Jones lee.jo...@linaro.org
---
No changes since v1.
---
drivers/power/pm2301_charger.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Check the return value of regulator_enable to silence the following
type of warnings:
drivers/power/ab8500_charger.c:1390:20: warning: ignoring return value
of ‘regulator_enable’, declared with attribute warn_unused_result
[-Wunused-result]
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc
Check the return value of regulator_enable to silence the following
warning:
drivers/power/pm2301_charger.c:725:20: warning:
ignoring return value of ‘regulator_enable’, declared with
attribute warn_unused_result [-Wunused-result]
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Lee Jones
tegra_dma_prep_dma_cyclic is referenced only in this file.
Make it static.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Laxman Dewangan ldewan...@nvidia.com
---
drivers/dma/tegra20-apb-dma.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/tegra20-apb
Hi Axel,
On 5 September 2013 09:01, Axel Lin wrote:
> Commmit af40a94aba "regulator: palmas: Use devm_regulator_register" missed
> removing a regulator_unregister() call if palmas_extreg_init falis. Fix it.
>
> Signed-off-by: Axel Lin
I was about to do it. Thanks for doin
NULL assignment corrupts the error pointer and is not
necessary.
Reported-by: kbuild test robot
Signed-off-by: Sachin Kamat
---
drivers/regulator/max77686.c |1 -
drivers/regulator/max77693.c |1 -
drivers/regulator/max8997.c |1 -
3 files changed, 3 deletions(-)
diff --git
Changed automaticall -> automatically.
Signed-off-by: Sachin Kamat
---
This series is based on topic/devm branch of regulator tree.
---
drivers/regulator/core.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c
in
Hi Jingoo,
On 5 September 2013 08:11, Jingoo Han wrote:
> LP8555 is one of LP855x family device.
> This device needs pre_init_device() and post_init_device() driver structure.
> It's same as LP8557, so the device configuration code is shared with LP8557.
> Backlight outputs are generated from
On 4 September 2013 22:03, Nishanth Menon wrote:
> On 09/04/2013 11:26 AM, Sachin Kamat wrote:
> I accept that probe failure gives me the required info for fail, and
> the cleanup of log prints also make sense, just that the log refactor
> seems out of context to the specific c
Hi Nishanth,
On 4 September 2013 18:57, Nishanth Menon wrote:
> On 09/04/2013 01:31 AM, Sachin Kamat wrote:
>> -err:
>> - dev_err(dev, "%s: Failed to initialize(%d)\n", __func__, ret);
>> - return ret;
> here -> with this, the lazy non detailed
On 4 September 2013 20:02, Marek Vasut wrote:
> Dear Sachin Kamat,
>
>> devm_* simplifies the code.
>>
>> Signed-off-by: Sachin Kamat
>> ---
>> drivers/regulator/palmas-regulator.c | 38
>> @@ -1021,7 +1021,7 @@ static int palmas_regulators_probe(str
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/tps62360-regulator.c | 17 +
1 file changed, 1 insertion(+), 16 deletions(-)
diff --git a/drivers/regulator/tps62360-regulator.c
b/drivers/regulator/tps62360-regulator.c
index a490d5b..c4a5695
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/tps65090-regulator.c | 35 +---
1 file changed, 5 insertions(+), 30 deletions(-)
diff --git a/drivers/regulator/tps65090-regulator.c
b/drivers/regulator/tps65090-regulator.c
index
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/tps80031-regulator.c | 30 +-
1 file changed, 5 insertions(+), 25 deletions(-)
diff --git a/drivers/regulator/tps80031-regulator.c
b/drivers/regulator/tps80031-regulator.c
index 6511d0b
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/tps6507x-regulator.c | 23 +++
1 file changed, 3 insertions(+), 20 deletions(-)
diff --git a/drivers/regulator/tps6507x-regulator.c
b/drivers/regulator/tps6507x-regulator.c
index 4117ff5
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/tps65910-regulator.c | 23 +++
1 file changed, 3 insertions(+), 20 deletions(-)
diff --git a/drivers/regulator/tps65910-regulator.c
b/drivers/regulator/tps65910-regulator.c
index 45c1644
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/tps65023-regulator.c | 22 +++---
1 file changed, 3 insertions(+), 19 deletions(-)
diff --git a/drivers/regulator/tps65023-regulator.c
b/drivers/regulator/tps65023-regulator.c
index 9d053e2
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/tps6586x-regulator.c | 31 ++-
1 file changed, 6 insertions(+), 25 deletions(-)
diff --git a/drivers/regulator/tps6586x-regulator.c
b/drivers/regulator/tps6586x-regulator.c
index
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
This series is compile tested.
---
drivers/regulator/tps51632-regulator.c | 11 +--
1 file changed, 1 insertion(+), 10 deletions(-)
diff --git a/drivers/regulator/tps51632-regulator.c
b/drivers/regulator/tps51632
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/tps65217-regulator.c | 26 --
1 file changed, 4 insertions(+), 22 deletions(-)
diff --git a/drivers/regulator/tps65217-regulator.c
b/drivers/regulator/tps65217-regulator.c
index 2df4616
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/tps65912-regulator.c | 24
1 file changed, 4 insertions(+), 20 deletions(-)
diff --git a/drivers/regulator/tps65912-regulator.c
b/drivers/regulator/tps65912-regulator.c
index 17e994e
On 4 September 2013 15:11, Lee Jones wrote:
> On Mon, 02 Sep 2013, Sachin Kamat wrote:
>
>> Check the return value of regulator_enable to silence the following
>> type of warnings:
>> drivers/power/ab8500_charger.c:1390:20: warning: ignoring return value
>> o
Commit e398b51a ("regulator: s2mps11: Convert to devm_regulator_register()")
intended to do this conversion. However the actual conversion to devm_* got
missed out. Fix this.
Signed-off-by: Sachin Kamat
---
drivers/regulator/s2mps11.c |3 ++-
1 file changed, 2 insertions(+),
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/mc13783-regulator.c | 25 -
1 file changed, 4 insertions(+), 21 deletions(-)
diff --git a/drivers/regulator/mc13783-regulator.c
b/drivers/regulator/mc13783-regulator.c
index 5ff99d2
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/mc13892-regulator.c | 22 +++---
1 file changed, 3 insertions(+), 19 deletions(-)
diff --git a/drivers/regulator/mc13892-regulator.c
b/drivers/regulator/mc13892-regulator.c
index 1037e07..96c9f80
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/ti-abb-regulator.c | 82 ++
1 file changed, 23 insertions(+), 59 deletions(-)
diff --git a/drivers/regulator/ti-abb-regulator.c
b/drivers/regulator/ti-abb-regulator.c
index 3753ed0
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/isl6271a-regulator.c | 24
1 file changed, 4 insertions(+), 20 deletions(-)
diff --git a/drivers/regulator/isl6271a-regulator.c
b/drivers/regulator/isl6271a-regulator.c
index b99c49b
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/palmas-regulator.c | 38 ++
1 file changed, 11 insertions(+), 27 deletions(-)
diff --git a/drivers/regulator/palmas-regulator.c
b/drivers/regulator/palmas-regulator.c
index d0c8785
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/rc5t583-regulator.c | 22 ++
1 file changed, 2 insertions(+), 20 deletions(-)
diff --git a/drivers/regulator/rc5t583-regulator.c
b/drivers/regulator/rc5t583-regulator.c
index 5885b45..b58affb
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
This series is compile tested.
---
drivers/regulator/anatop-regulator.c |3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/regulator/anatop-regulator.c
b/drivers/regulator/anatop-regulator.c
index 0d4a8cc
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
This series is compile tested.
---
drivers/regulator/anatop-regulator.c |3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/regulator/anatop-regulator.c
b/drivers/regulator/anatop
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/isl6271a-regulator.c | 24
1 file changed, 4 insertions(+), 20 deletions(-)
diff --git a/drivers/regulator/isl6271a-regulator.c
b/drivers/regulator/isl6271a
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/palmas-regulator.c | 38 ++
1 file changed, 11 insertions(+), 27 deletions(-)
diff --git a/drivers/regulator/palmas-regulator.c
b/drivers/regulator/palmas
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/rc5t583-regulator.c | 22 ++
1 file changed, 2 insertions(+), 20 deletions(-)
diff --git a/drivers/regulator/rc5t583-regulator.c
b/drivers/regulator/rc5t583-regulator.c
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/ti-abb-regulator.c | 82 ++
1 file changed, 23 insertions(+), 59 deletions(-)
diff --git a/drivers/regulator/ti-abb-regulator.c
b/drivers/regulator/ti-abb
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/mc13892-regulator.c | 22 +++---
1 file changed, 3 insertions(+), 19 deletions(-)
diff --git a/drivers/regulator/mc13892-regulator.c
b/drivers/regulator/mc13892-regulator.c
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/mc13783-regulator.c | 25 -
1 file changed, 4 insertions(+), 21 deletions(-)
diff --git a/drivers/regulator/mc13783-regulator.c
b/drivers/regulator/mc13783
Commit e398b51a (regulator: s2mps11: Convert to devm_regulator_register())
intended to do this conversion. However the actual conversion to devm_* got
missed out. Fix this.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/s2mps11.c |3 ++-
1 file changed, 2
On 4 September 2013 15:11, Lee Jones lee.jo...@linaro.org wrote:
On Mon, 02 Sep 2013, Sachin Kamat wrote:
Check the return value of regulator_enable to silence the following
type of warnings:
drivers/power/ab8500_charger.c:1390:20: warning: ignoring return value
of ‘regulator_enable
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/tps65217-regulator.c | 26 --
1 file changed, 4 insertions(+), 22 deletions(-)
diff --git a/drivers/regulator/tps65217-regulator.c
b/drivers/regulator/tps65217
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/tps65912-regulator.c | 24
1 file changed, 4 insertions(+), 20 deletions(-)
diff --git a/drivers/regulator/tps65912-regulator.c
b/drivers/regulator/tps65912
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
This series is compile tested.
---
drivers/regulator/tps51632-regulator.c | 11 +--
1 file changed, 1 insertion(+), 10 deletions(-)
diff --git a/drivers/regulator/tps51632-regulator.c
b/drivers
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/tps6507x-regulator.c | 23 +++
1 file changed, 3 insertions(+), 20 deletions(-)
diff --git a/drivers/regulator/tps6507x-regulator.c
b/drivers/regulator/tps6507x
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/tps65910-regulator.c | 23 +++
1 file changed, 3 insertions(+), 20 deletions(-)
diff --git a/drivers/regulator/tps65910-regulator.c
b/drivers/regulator/tps65910
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/tps65023-regulator.c | 22 +++---
1 file changed, 3 insertions(+), 19 deletions(-)
diff --git a/drivers/regulator/tps65023-regulator.c
b/drivers/regulator/tps65023
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/tps80031-regulator.c | 30 +-
1 file changed, 5 insertions(+), 25 deletions(-)
diff --git a/drivers/regulator/tps80031-regulator.c
b/drivers/regulator/tps80031
701 - 800 of 1357 matches
Mail list logo