devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
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
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
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
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
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
On 4 September 2013 20:02, Marek Vasut ma...@denx.de wrote:
Dear Sachin Kamat,
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/palmas-regulator.c | 38
@@ -1021,7 +1021,7 @@ static int palmas_regulators_probe(struct
platform_device
Hi Nishanth,
On 4 September 2013 18:57, Nishanth Menon n...@ti.com 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 prints end with a generic
fail case
On 4 September 2013 22:03, Nishanth Menon n...@ti.com 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 change
Hi Jingoo,
On 5 September 2013 08:11, Jingoo Han jg1@samsung.com 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
Changed automaticall - automatically.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
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
NULL assignment corrupts the error pointer and is not
necessary.
Reported-by: kbuild test robot fengguang...@intel.com
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/max77686.c |1 -
drivers/regulator/max77693.c |1 -
drivers/regulator/max8997.c |1 -
3
it. Thanks for doing :)
Acked-by: Sachin Kamat sachin.ka...@linaro.org
--
With warm regards,
Sachin
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/max8660.c | 29 ++---
1 file changed, 6 insertions(+), 23 deletions(-)
diff --git a/drivers/regulator/max8660.c b/drivers/regulator/max8660.c
index d428ef9..a057342 100644
--- a/drivers
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/max1586.c | 26 +-
1 file changed, 5 insertions(+), 21 deletions(-)
diff --git a/drivers/regulator/max1586.c b/drivers/regulator/max1586.c
index 54af610..3abf76a 100644
--- a/drivers
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/max8649.c | 14 ++
1 file changed, 2 insertions(+), 12 deletions(-)
diff --git a/drivers/regulator/max8649.c b/drivers/regulator/max8649.c
index db6c9be..b2a808c 100644
--- a/drivers/regulator/max8649.c
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/max8998.c | 35 ---
1 file changed, 8 insertions(+), 27 deletions(-)
diff --git a/drivers/regulator/max8998.c b/drivers/regulator/max8998.c
index a4c53b2..ae3f065 100644
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/max77693.c | 28
1 file changed, 4 insertions(+), 24 deletions(-)
diff --git a/drivers/regulator/max77693.c b/drivers/regulator/max77693.c
index ce4b96c..2054ae1 100644
--- a/drivers
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/max8997.c | 38 ++
1 file changed, 10 insertions(+), 28 deletions(-)
diff --git a/drivers/regulator/max8997.c b/drivers/regulator/max8997.c
index df20069..059e8ed 100644
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/max8973-regulator.c | 11 +--
1 file changed, 1 insertion(+), 10 deletions(-)
diff --git a/drivers/regulator/max8973-regulator.c
b/drivers/regulator/max8973-regulator.c
index 0c5195a..04de63d 100644
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
drivers/regulator/max8907-regulator.c | 23 +++
1 file changed, 3 insertions(+), 20 deletions(-)
diff --git a/drivers/regulator/max8907-regulator.c
b/drivers/regulator/max8907-regulator.c
index 4568c15..0c5fe6c
devm_* simplifies the code.
Signed-off-by: Sachin Kamat
---
This series is compile tested.
---
drivers/regulator/max77686.c | 22 +++---
1 file changed, 3 insertions(+), 19 deletions(-)
diff --git a/drivers/regulator/max77686.c b/drivers/regulator/max77686.c
index f563057
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
This series is compile tested.
---
drivers/regulator/max77686.c | 22 +++---
1 file changed, 3 insertions(+), 19 deletions(-)
diff --git a/drivers/regulator/max77686.c b/drivers/regulator
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/max8973-regulator.c | 11 +--
1 file changed, 1 insertion(+), 10 deletions(-)
diff --git a/drivers/regulator/max8973-regulator.c
b/drivers/regulator/max8973-regulator.c
index 0c5195a
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/max8907-regulator.c | 23 +++
1 file changed, 3 insertions(+), 20 deletions(-)
diff --git a/drivers/regulator/max8907-regulator.c
b/drivers/regulator/max8907-regulator.c
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/max8997.c | 38 ++
1 file changed, 10 insertions(+), 28 deletions(-)
diff --git a/drivers/regulator/max8997.c b/drivers/regulator/max8997.c
index df20069
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/max8998.c | 35 ---
1 file changed, 8 insertions(+), 27 deletions(-)
diff --git a/drivers/regulator/max8998.c b/drivers/regulator/max8998.c
index a4c53b2
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/max77693.c | 28
1 file changed, 4 insertions(+), 24 deletions(-)
diff --git a/drivers/regulator/max77693.c b/drivers/regulator/max77693.c
index ce4b96c..2054ae1
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/max8660.c | 29 ++---
1 file changed, 6 insertions(+), 23 deletions(-)
diff --git a/drivers/regulator/max8660.c b/drivers/regulator/max8660.c
index d428ef9..a057342
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/max1586.c | 26 +-
1 file changed, 5 insertions(+), 21 deletions(-)
diff --git a/drivers/regulator/max1586.c b/drivers/regulator/max1586.c
index 54af610..3abf76a
devm_* simplifies the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/max8649.c | 14 ++
1 file changed, 2 insertions(+), 12 deletions(-)
diff --git a/drivers/regulator/max8649.c b/drivers/regulator/max8649.c
index db6c9be..b2a808c 100644
pm2xxx_charger_die_therm_mngt is used only in this file.
Make it static.
Signed-off-by: Sachin Kamat
Cc: Lee Jones
---
drivers/power/pm2301_charger.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/pm2301_charger.c b/drivers/power/pm2301_charger.c
index
Each of the if-else blocks has a break statement.
Remove the additional one which is unreachable.
Signed-off-by: Sachin Kamat
---
drivers/power/ab8500_charger.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/power/ab8500_charger.c b/drivers/power/ab8500_charger.c
index fec0546
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
---
drivers
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
---
drivers/power
Return directly if memory allocation fails. There is no need
of dma_free_coherent().
Signed-off-by: Sachin Kamat
Cc: Saeed Bishara
---
drivers/dma/mv_xor.c |6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/dma/mv_xor.c b/drivers/dma/mv_xor.c
index 856ea77
'const' was added twice.
Signed-off-by: Sachin Kamat
---
drivers/dma/ste_dma40.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/ste_dma40.c b/drivers/dma/ste_dma40.c
index 7bbcabb..df0a606 100644
--- a/drivers/dma/ste_dma40.c
+++ b/drivers/dma/ste_dma40.c
kcalloc can return NULL. Check the pointer before dereferencing.
Signed-off-by: Sachin Kamat
---
drivers/dma/ste_dma40.c |3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/dma/ste_dma40.c b/drivers/dma/ste_dma40.c
index df0a606..82d2b97 100644
--- a/drivers/dma/ste_dma40.c
+++ b
kfree on a NULL pointer is a no-op. Null pointer check is
not necessary.
Signed-off-by: Sachin Kamat
Cc: Sascha Hauer
---
drivers/dma/imx-dma.c |6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/dma/imx-dma.c b/drivers/dma/imx-dma.c
index ff2aab9..78f8ca5
kfree on a NULL pointer is a no-op. Null pointer check is
not necessary.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Sascha Hauer s.ha...@pengutronix.de
---
drivers/dma/imx-dma.c |6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/dma/imx-dma.c b
kcalloc can return NULL. Check the pointer before dereferencing.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/dma/ste_dma40.c |3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/dma/ste_dma40.c b/drivers/dma/ste_dma40.c
index df0a606..82d2b97 100644
--- a/drivers
'const' was added twice.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/dma/ste_dma40.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/ste_dma40.c b/drivers/dma/ste_dma40.c
index 7bbcabb..df0a606 100644
--- a/drivers/dma/ste_dma40.c
+++ b
Return directly if memory allocation fails. There is no need
of dma_free_coherent().
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Saeed Bishara sa...@marvell.com
---
drivers/dma/mv_xor.c |6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/dma/mv_xor.c
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
---
drivers/power/ab8500_charger.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/power/ab8500_charger.c b/drivers/power
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
pm2xxx_charger_die_therm_mngt is used only in this file.
Make it static.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Lee Jones lee.jo...@linaro.org
---
drivers/power/pm2301_charger.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/pm2301_charger.c
Local variables referenced only in this file are made static.
Signed-off-by: Sachin Kamat
---
drivers/mfd/ab8500-debugfs.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/mfd/ab8500-debugfs.c b/drivers/mfd/ab8500-debugfs.c
index 7d1f1b0..a0afa64 100644
Silences the following warning:
drivers/mfd/db8500-prcmu.c:2322:25: warning:
non-ANSI function declaration of function 'prcmu_ac_sleep_req'
Signed-off-by: Sachin Kamat
---
drivers/mfd/db8500-prcmu.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/mfd/db8500
clk_mgt is used only in this file. Make it static.
Signed-off-by: Sachin Kamat
---
drivers/mfd/db8500-prcmu.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/mfd/db8500-prcmu.c b/drivers/mfd/db8500-prcmu.c
index 8685872..53f371d 100644
--- a/drivers/mfd/db8500
usbhs_driver_name is used only in this file. Make it static.
Signed-off-by: Sachin Kamat
---
drivers/mfd/omap-usb-host.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/mfd/omap-usb-host.c b/drivers/mfd/omap-usb-host.c
index d2b8e71..29ee54d 100644
module_pci_driver removes some boilerplate and makes the code
simple.
Signed-off-by: Sachin Kamat
---
drivers/mfd/timberdale.c | 29 +
1 files changed, 1 insertions(+), 28 deletions(-)
diff --git a/drivers/mfd/timberdale.c b/drivers/mfd/timberdale.c
index 57dd89d
Local variables referenced only in this file are made static.
Signed-off-by: Sachin Kamat
---
drivers/mfd/timberdale.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/mfd/timberdale.c b/drivers/mfd/timberdale.c
index 0c1fcbc..4c9c149 100644
--- a/drivers/mfd
break after goto is unreachable. Delete it.
Signed-off-by: Sachin Kamat
---
drivers/mfd/timberdale.c |1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/drivers/mfd/timberdale.c b/drivers/mfd/timberdale.c
index 4c9c149..57dd89d 100644
--- a/drivers/mfd/timberdale.c
+++ b
Local variables referenced only in this file are made static.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/mfd/timberdale.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/mfd/timberdale.c b/drivers/mfd/timberdale.c
index 0c1fcbc..4c9c149
break after goto is unreachable. Delete it.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/mfd/timberdale.c |1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/drivers/mfd/timberdale.c b/drivers/mfd/timberdale.c
index 4c9c149..57dd89d 100644
--- a/drivers
module_pci_driver removes some boilerplate and makes the code
simple.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/mfd/timberdale.c | 29 +
1 files changed, 1 insertions(+), 28 deletions(-)
diff --git a/drivers/mfd/timberdale.c b/drivers/mfd
usbhs_driver_name is used only in this file. Make it static.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/mfd/omap-usb-host.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/mfd/omap-usb-host.c b/drivers/mfd/omap-usb-host.c
index d2b8e71
Silences the following warning:
drivers/mfd/db8500-prcmu.c:2322:25: warning:
non-ANSI function declaration of function 'prcmu_ac_sleep_req'
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/mfd/db8500-prcmu.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git
clk_mgt is used only in this file. Make it static.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/mfd/db8500-prcmu.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/mfd/db8500-prcmu.c b/drivers/mfd/db8500-prcmu.c
index 8685872..53f371d 100644
Local variables referenced only in this file are made static.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/mfd/ab8500-debugfs.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/mfd/ab8500-debugfs.c b/drivers/mfd/ab8500-debugfs.c
index
[+cc Russell]
On 22 August 2013 21:45, Bjorn Helgaas wrote:
> [+cc linux-kernel]
>
> On Thu, Aug 22, 2013 at 1:44 AM, Sachin Kamat wrote:
>> __initdata should be placed between the variable name and equal
>> sign for the variable to be placed in the intended sect
[+cc Russell]
On 22 August 2013 21:45, Bjorn Helgaas bhelg...@google.com wrote:
[+cc linux-kernel]
On Thu, Aug 22, 2013 at 1:44 AM, Sachin Kamat sachin.ka...@linaro.org wrote:
__initdata should be placed between the variable name and equal
sign for the variable to be placed in the intended
Hi Jingoo,
On 12 August 2013 14:26, Jingoo Han wrote:
> This patch adds support for Message Signaled Interrupt in the
> Exynops PCIe diver using Synopsys designware PCIe core IP.
s/Exynops PCIe diver/Exynos PCIe driver
>
> Signed-off-by: Siva Reddy Kallam
> Signed-off-by: Srikanth T Shivanand
Hi Jingoo,
On 12 August 2013 14:26, Jingoo Han jg1@samsung.com wrote:
This patch adds support for Message Signaled Interrupt in the
Exynops PCIe diver using Synopsys designware PCIe core IP.
s/Exynops PCIe diver/Exynos PCIe driver
Signed-off-by: Siva Reddy Kallam siva.kal...@samsung.com
Hi Dongjin,
On 5 August 2013 23:48, Dongjin Kim wrote:
> This patch adds device nodes for USBPHY to Exynos4x12.
>
> CC: Sachin Kamat
> Signed-off-by: Dongjin Kim
> ---
> arch/arm/boot/dts/exynos4x12.dtsi | 18 ++
> 1 file changed, 18 insertions(+)
>
Hi Dongjin,
On 5 August 2013 23:48, Dongjin Kim tobet...@gmail.com wrote:
This patch adds device nodes for USBPHY to Exynos4x12.
CC: Sachin Kamat sachin.ka...@linaro.org
Signed-off-by: Dongjin Kim tobet...@gmail.com
---
arch/arm/boot/dts/exynos4x12.dtsi | 18 ++
1 file
Fixes the following warning:
WARNING: space prohibited before semicolon
Signed-off-by: Sachin Kamat
---
drivers/extcon/extcon-max77693.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/extcon/extcon-max77693.c b/drivers/extcon/extcon-max77693.c
index b56bdaa
Fixes the following warning:
WARNING: space prohibited before semicolon
Signed-off-by: Sachin Kamat
---
drivers/extcon/extcon-max8997.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/extcon/extcon-max8997.c b/drivers/extcon/extcon-max8997.c
index 67d6738..e55ec38
Fixes an incomplete comment introduced by commit 9c2ba270
("extcon: arizona: Simplify HPDET based identification").
Signed-off-by: Sachin Kamat
---
drivers/extcon/extcon-arizona.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/extcon/extcon-arizona.c
Fixes an incomplete comment introduced by commit 9c2ba270
(extcon: arizona: Simplify HPDET based identification).
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/extcon/extcon-arizona.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/extcon/extcon
Fixes the following warning:
WARNING: space prohibited before semicolon
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/extcon/extcon-max8997.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/extcon/extcon-max8997.c b/drivers/extcon/extcon-max8997.c
Fixes the following warning:
WARNING: space prohibited before semicolon
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/extcon/extcon-max77693.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/extcon/extcon-max77693.c b/drivers/extcon/extcon
On 31 July 2013 22:09, Viresh Kumar wrote:
> On 31 July 2013 22:06, Sachin Kamat wrote:
>> Hi Viresh,
>>
>> On 31 July 2013 19:49, Viresh Kumar wrote:
>>> CPUFreq core does following at multiple places:
>>> module_{get|put}(cpufreq_driver->owner))
On 31 July 2013 22:09, Viresh Kumar viresh.ku...@linaro.org wrote:
On 31 July 2013 22:06, Sachin Kamat sachin.ka...@linaro.org wrote:
Hi Viresh,
On 31 July 2013 19:49, Viresh Kumar viresh.ku...@linaro.org wrote:
CPUFreq core does following at multiple places:
module_{get|put}(cpufreq_driver
On 2 August 2013 11:20, Jingoo Han wrote:
> 'mmc_callback_data' and 'vtg' are pointers.
> Fix the following sparse warning:
>
> drivers/mfd/menelaus.c:445:43: warning: Using plain integer as NULL pointer
> drivers/mfd/menelaus.c:469:20: warning: Using plain integer as NULL pointer
>
>
On 2 August 2013 11:20, Jingoo Han jg1@samsung.com wrote:
'mmc_callback_data' and 'vtg' are pointers.
Fix the following sparse warning:
drivers/mfd/menelaus.c:445:43: warning: Using plain integer as NULL pointer
drivers/mfd/menelaus.c:469:20: warning: Using plain integer as NULL pointer
Hi Viresh,
On 31 July 2013 19:49, Viresh Kumar wrote:
> CPUFreq core does following at multiple places:
> module_{get|put}(cpufreq_driver->owner)).
>
> This is done to make sure module doesn't get unloaded if it is currently in
> use.
> This will work only if the .owner field of cpufreq driver
Hi Viresh,
On 31 July 2013 19:49, Viresh Kumar viresh.ku...@linaro.org wrote:
CPUFreq core does following at multiple places:
module_{get|put}(cpufreq_driver-owner)).
This is done to make sure module doesn't get unloaded if it is currently in
use.
This will work only if the .owner field of
*map should be tested for NULL instead of map as kmalloc pointer
is assigned to it. This also fixes a potential null pointer dereference
bug later in the code.
Signed-off-by: Sachin Kamat
Cc: Maxime Ripard
---
drivers/pinctrl/pinctrl-sunxi.c |2 +-
1 file changed, 1 insertion(+), 1
On 29 July 2013 13:27, Cho KyongHo wrote:
>> -Original Message-
>> From: Cho KyongHo [mailto:pullip@samsung.com]
>> Sent: Monday, July 29, 2013 4:20 PM
>>
>> > -Original Message-
>> > From: Sachin Kamat [mailto:sachin.ka...@linaro.org
Hi KyongHo,
On 26 July 2013 16:58, Cho KyongHo wrote:
> Signed-off-by: Cho KyongHo
> ---
> .../bindings/iommu/samsung,exynos4210-sysmmu.txt | 103 +++
> arch/arm/boot/dts/exynos4.dtsi | 122
> arch/arm/boot/dts/exynos4210.dtsi | 25 ++
>
Hi KyongHo,
On 26 July 2013 16:58, Cho KyongHo pullip@samsung.com wrote:
Signed-off-by: Cho KyongHo pullip@samsung.com
---
.../bindings/iommu/samsung,exynos4210-sysmmu.txt | 103 +++
arch/arm/boot/dts/exynos4.dtsi | 122
On 29 July 2013 13:27, Cho KyongHo pullip@samsung.com wrote:
-Original Message-
From: Cho KyongHo [mailto:pullip@samsung.com]
Sent: Monday, July 29, 2013 4:20 PM
-Original Message-
From: Sachin Kamat [mailto:sachin.ka...@linaro.org]
Sent: Monday, July 29, 2013 3
*map should be tested for NULL instead of map as kmalloc pointer
is assigned to it. This also fixes a potential null pointer dereference
bug later in the code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Maxime Ripard maxime.rip...@free-electrons.com
---
drivers/pinctrl/pinctrl
Symbols used only in this file are made static.
Signed-off-by: Sachin Kamat
---
drivers/pinctrl/pinctrl-st.c |8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-st.c b/drivers/pinctrl/pinctrl-st.c
index 6246d7a..2a10a31 100644
--- a/drivers
devm_request_and_ioremap is deprecated. Use devm_ioremap_resource
instead.
Signed-off-by: Sachin Kamat
---
drivers/pinctrl/pinctrl-st.c |8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-st.c b/drivers/pinctrl/pinctrl-st.c
index 04d4506
devm_request_and_ioremap is deprecated. Use devm_ioremap_resource
instead.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/pinctrl/pinctrl-st.c |8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-st.c b/drivers/pinctrl/pinctrl
Symbols used only in this file are made static.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/pinctrl/pinctrl-st.c |8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-st.c b/drivers/pinctrl/pinctrl-st.c
index 6246d7a..2a10a31
Hi Antonios,
On 25 July 2013 21:04, Antonios Motakis
wrote:
> IOMMU groups are expected by certain users of the IOMMU API,
> e.g. VFIO. Since each device is behind its own System MMU, we
> can allocate a new IOMMU group for each device.
>
> This patch depends on Cho KyongHo's patch series titled
Hi Antonios,
On 25 July 2013 21:04, Antonios Motakis
a.mota...@virtualopensystems.com wrote:
IOMMU groups are expected by certain users of the IOMMU API,
e.g. VFIO. Since each device is behind its own System MMU, we
can allocate a new IOMMU group for each device.
This patch depends on Cho
On 24 July 2013 11:29, Dongjin Kim wrote:
> Thanks Sachin,
>
> On Wed, Jul 24, 2013 at 2:12 PM, Sachin Kamat wrote:
>> Hi Dongjin,
>>
>> On 23 July 2013 23:01, Dongjin Kim wrote:
>>> This patch adds device nodes for USBPHY to Exynos4x12.
>>>
>>
On 24 July 2013 11:29, Dongjin Kim tobet...@gmail.com wrote:
Thanks Sachin,
On Wed, Jul 24, 2013 at 2:12 PM, Sachin Kamat sachin.ka...@linaro.org wrote:
Hi Dongjin,
On 23 July 2013 23:01, Dongjin Kim tobet...@gmail.com wrote:
This patch adds device nodes for USBPHY to Exynos4x12.
Signed
Hi Dongjin,
On 23 July 2013 23:01, Dongjin Kim wrote:
> This patch adds device nodes for USBPHY to Exynos4x12.
>
> Signed-off-by: Dongjin Kim
> ---
> arch/arm/boot/dts/exynos4x12.dtsi | 18 ++
> 1 file changed, 18 insertions(+)
>
> diff --git
On 23 July 2013 23:02, Dongjin Kim wrote:
> This patch adds EHCI and OHCI host device nodes for Exynos4.
>
> Signed-off-by: Dongjin Kim
> ---
> arch/arm/boot/dts/exynos4.dtsi | 20
> 1 file changed, 20 insertions(+)
>
> diff --git a/arch/arm/boot/dts/exynos4.dtsi
On 23 July 2013 23:02, Dongjin Kim tobet...@gmail.com wrote:
This patch adds EHCI and OHCI host device nodes for Exynos4.
Signed-off-by: Dongjin Kim tobet...@gmail.com
---
arch/arm/boot/dts/exynos4.dtsi | 20
1 file changed, 20 insertions(+)
diff --git
Hi Dongjin,
On 23 July 2013 23:01, Dongjin Kim tobet...@gmail.com wrote:
This patch adds device nodes for USBPHY to Exynos4x12.
Signed-off-by: Dongjin Kim tobet...@gmail.com
---
arch/arm/boot/dts/exynos4x12.dtsi | 18 ++
1 file changed, 18 insertions(+)
diff --git
I think the patch title itself has a typo :)
s/doucmentations/documentations
On 17 July 2013 21:59, Masanari Iida wrote:
> Correct typo (double words) in documentations.
>
> Signed-off-by: Masanari Iida
> ---
> Documentation/arm/OMAP/omap_pm | 2 +-
>
I think the patch title itself has a typo :)
s/doucmentations/documentations
On 17 July 2013 21:59, Masanari Iida standby2...@gmail.com wrote:
Correct typo (double words) in documentations.
Signed-off-by: Masanari Iida standby2...@gmail.com
---
Documentation/arm/OMAP/omap_pm
On 17 July 2013 16:29, Mark Brown wrote:
> From: Mark Brown
>
> As the comment says exynos4_clk_register_fixed_ext() was only used by
> non-DT platforms. Since Exynos is now DT only it is no longer referenced
> and can be removed.
>
> Signed-off-by: Mark Brown
Similar patch has already been
On 17 July 2013 16:29, Mark Brown wrote:
> From: Mark Brown
>
> This avoids polluting the global namespace and silences lots of sparse
> warnings. The change pushes us over 80 columns a lot but I couldn't see
> a tasteful place to break the lines as-is.
>
> Signed-off-by: Mark Brown
I have
On 17 July 2013 16:29, Mark Brown broo...@kernel.org wrote:
From: Mark Brown broo...@linaro.org
This avoids polluting the global namespace and silences lots of sparse
warnings. The change pushes us over 80 columns a lot but I couldn't see
a tasteful place to break the lines as-is.
On 17 July 2013 16:29, Mark Brown broo...@kernel.org wrote:
From: Mark Brown broo...@linaro.org
As the comment says exynos4_clk_register_fixed_ext() was only used by
non-DT platforms. Since Exynos is now DT only it is no longer referenced
and can be removed.
Signed-off-by: Mark Brown
801 - 900 of 1357 matches
Mail list logo