Add of_node_put to decrement the ref count.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/da9063-regulator.c |1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/da9063-regulator.c
b/drivers/regulator/da9063-regulator.c
index bfd7f2f795a6
Add of_node_put to decrement the ref count.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/as3722-regulator.c |1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/as3722-regulator.c
b/drivers/regulator/as3722-regulator.c
index 8b17d786cb71
linux/wait.h was included twice.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
include/linux/tty_ldisc.h |1 -
1 file changed, 1 deletion(-)
diff --git a/include/linux/tty_ldisc.h b/include/linux/tty_ldisc.h
index b8347c207cb8..add26da2faeb 100644
--- a/include/linux/tty_ldisc.h
Propagate the error value returned by the function instead.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/ti-abb-regulator.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/ti-abb-regulator.c
b/drivers/regulator/ti-abb
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/max8973-regulator.c |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/regulator/max8973-regulator.c
b/drivers/regulator/max8973-regulator.c
Propagate the error value returned by the function instead.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/wm8350-regulator.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/regulator/wm8350-regulator.c
b/drivers/regulator/wm8350
Propagate the error value returned by the function instead.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/lp3971.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/lp3971.c b/drivers/regulator/lp3971.c
index 3b1102b75071
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/act8865-regulator.c |5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/regulator/act8865-regulator.c
b/drivers/regulator/act8865-regulator.c
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/88pm800.c |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/regulator/88pm800.c b/drivers/regulator/88pm800.c
index d333f7eac106
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/arizona-ldo1.c |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/regulator/arizona-ldo1.c b/drivers/regulator/arizona-ldo1.c
index
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/as3711-regulator.c |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/regulator/as3711-regulator.c
b/drivers/regulator/as3711-regulator.c
Propagate the error value returned by the function instead.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/da9055-regulator.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/da9055-regulator.c
b/drivers/regulator/da9055
Propagate the error values returned by the function instead.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/da9063-regulator.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/regulator/da9063-regulator.c
b/drivers/regulator/da9063
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/arizona-micsupp.c |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/regulator/arizona-micsupp.c
b/drivers/regulator/arizona-micsupp.c
index
Propagate the error values returned by the function instead.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/fan53555.c |8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/regulator/fan53555.c b/drivers/regulator/fan53555.c
index
Propagate the error values returned by the function instead.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/lp872x.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/regulator/lp872x.c b/drivers/regulator/lp872x.c
index 2e4734ff79fc
Propagate the error value returned by the function instead.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/max8925-regulator.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/max8925-regulator.c
b/drivers/regulator/max8925
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/ti-abb-regulator.c | 11 +++
1 file changed, 3 insertions(+), 8 deletions(-)
diff --git a/drivers/regulator/ti-abb-regulator.c
b/drivers/regulator/ti-abb
On 18 February 2014 21:20, Fabio Estevam fabio.este...@freescale.com wrote:
Since commit d7857c42 (regulator: pfuze100: Use of_get_child_by_name) we get
the following probe failure:
pfuze100-regulator 1-0008: Full layer: 1, Metal layer: 0
pfuze100-regulator 1-0008: FAB: 0, FIN: 0
On 19 February 2014 05:58, SeongJae Park sj38.p...@gmail.com wrote:
On Wed, Feb 19, 2014 at 2:07 AM, Greg KH gre...@linuxfoundation.org wrote:
On Tue, Feb 18, 2014 at 08:23:25PM +0900, SeongJae Park wrote:
Signed-off-by: SeongJae Park sj38.p...@gmail.com
---
drivers/staging/android/binder.c
On 19 February 2014 07:42, Shawn Guo shawn@linaro.org wrote:
On Tue, Feb 18, 2014 at 09:58:52PM +0530, Sachin Kamat wrote:
On 18 February 2014 21:20, Fabio Estevam fabio.este...@freescale.com wrote:
Since commit d7857c42 (regulator: pfuze100: Use of_get_child_by_name) we
get
@linaro.org
Signed-off-by: Fabio Estevam fabio.este...@freescale.com
Thanks for the fix Fabio.
Acked-by: Sachin Kamat sachin.ka...@linaro.org
Mark,
There are a few more instances that should be fixed like this one. How would you
prefer to handle them - Resend the complete patch or send just
Hi Grant,
I observe the following boot failure with today's (next-20140220) linux-next
tree on Exynos based boards with the default exynos_defconfig.
Uncompressing Linux... done, booting the kernel.
[0.00] Booting Linux on physical CPU 0x900
[0.00] Linux version
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: S Twiss stwiss.opensou...@diasemi.com
---
drivers/regulator/da9210-regulator.c |5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/regulator/da9210-regulator.c
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Krystian Garbaciak krystian.garbac...@diasemi.com
---
drivers/regulator/da9063-regulator.c |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/regulator/da9063
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/fixed.c |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/regulator/fixed.c b/drivers/regulator/fixed.c
index 3c307d62fd31..c61f7e97e4f8
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Laxman Dewangan ldewan...@nvidia.com
---
drivers/regulator/tps62360-regulator.c |9 ++---
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/drivers/regulator/tps62360
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/wm831x-ldo.c | 12 +++-
1 file changed, 3 insertions(+), 9 deletions(-)
diff --git a/drivers/regulator/wm831x-ldo.c b/drivers/regulator/wm831x-ldo.c
index
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/wm831x-dcdc.c | 16
1 file changed, 4 insertions(+), 12 deletions(-)
diff --git a/drivers/regulator/wm831x-dcdc.c b/drivers/regulator/wm831x-dcdc.c
index
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/wm8994-regulator.c |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/regulator/wm8994-regulator.c
b/drivers/regulator/wm8994-regulator.c
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/wm831x-isink.c |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/regulator/wm831x-isink.c b/drivers/regulator/wm831x-isink.c
index
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Graeme Gregory g...@slimlogic.co.uk
---
drivers/regulator/tps65910-regulator.c | 21 +
1 file changed, 5 insertions(+), 16 deletions(-)
diff --git a/drivers/regulator
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Laxman Dewangan ldewan...@nvidia.com
---
drivers/regulator/tps80031-regulator.c |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/regulator/tps80031
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/tps6524x-regulator.c |5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/regulator/tps6524x-regulator.c
b/drivers/regulator/tps6524x
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Venu Byravarasu vbyravar...@nvidia.com
---
drivers/regulator/tps65090-regulator.c | 12 +++-
1 file changed, 3 insertions(+), 9 deletions(-)
diff --git a/drivers/regulator/tps65090
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Mike Rapoport m...@compulab.co.il
---
drivers/regulator/tps6586x-regulator.c |8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/regulator/tps6586x
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/tps6507x-regulator.c |8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/regulator/tps6507x-regulator.c
b/drivers/regulator/tps6507x
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Laxman Dewangan ldewan...@nvidia.com
---
drivers/regulator/tps51632-regulator.c |8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/regulator/tps51632
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Sangbeom Kim sbki...@samsung.com
---
drivers/regulator/s5m8767.c | 10 ++
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/drivers/regulator/s5m8767.c b/drivers
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/max8997.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/regulator/max8997.c b/drivers/regulator/max8997.c
index 10108f2941e3..f657d8db9c0c 100644
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Laxman Dewangan ldewan...@nvidia.com
---
drivers/regulator/rc5t583-regulator.c |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/regulator/rc5t583-regulator.c
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/max8952.c |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/regulator/max8952.c b/drivers/regulator/max8952.c
index 788e5ae2af1b
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/max8907-regulator.c |5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/regulator/max8907-regulator.c
b/drivers/regulator/max8907
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/max77686.c |2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/regulator/max77686.c b/drivers/regulator/max77686.c
index b411af00065d..de020adf0568 100644
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Sundar Iyer sundar.i...@stericsson.com
---
drivers/regulator/dbx500-prcmu.c | 10 ++
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/drivers/regulator/dbx500-prcmu.c b
kzalloc prints its own OOM message upon failure.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Yunfan Zhang yfzh...@marvell.com
---
drivers/regulator/fan53555.c |5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/regulator/fan53555.c b/drivers/regulator
Hi Kevin,
On 20 February 2014 15:42, Kevin Hao haoke...@gmail.com wrote:
On Thu, Feb 20, 2014 at 02:09:08PM +0530, Sachin Kamat wrote:
Hi Grant,
I observe the following boot failure with today's (next-20140220) linux-next
tree on Exynos based boards with the default exynos_defconfig.
Does
On 20 February 2014 16:25, Kevin Hao haoke...@gmail.com wrote:
On Thu, Feb 20, 2014 at 03:57:07PM +0530, Sachin Kamat wrote:
Hi Kevin,
On 20 February 2014 15:42, Kevin Hao haoke...@gmail.com wrote:
On Thu, Feb 20, 2014 at 02:09:08PM +0530, Sachin Kamat wrote:
Hi Grant,
I observe
Hi Krzysztof,
On 20 February 2014 20:49, Krzysztof Kozlowski k.kozlow...@samsung.com wrote:
This fixes bug introduced in 667a6b7a (regulator: max14577: Add missing
of_node_put). The DTS parsing function returned number of matched
regulators as success status which then was compared against 0
On 21 February 2014 16:12, Krzysztof Kozlowski k.kozlow...@samsung.com wrote:
On Fri, 2014-02-21 at 15:54 +0530, Sachin Kamat wrote:
Hi Krzysztof,
On 20 February 2014 20:49, Krzysztof Kozlowski k.kozlow...@samsung.com
wrote:
This fixes bug introduced in 667a6b7a (regulator: max14577: Add
On 13 February 2014 12:07, Sachin Kamat sachin.ka...@linaro.org wrote:
Added initial binding documentation for S2MPA01 MFD.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
Changes since v2:
* Re-worded as suggested by Mark Rutland
---
Documentation/devicetree/bindings/mfd/s2mpa01
.
Result was a probe fail after successful parsing the DTS:
max14577-regulator: probe of max14577-regulator failed with error 2
Signed-off-by: Krzysztof Kozlowski k.kozlow...@samsung.com
Cc: Sachin Kamat sachin.ka...@linaro.org
Looks good.
Reviewed-by: Sachin Kamat sachin.ka...@linaro.org
Hi Tarek,
Nothing in specific about the patch, but to just make you aware that
some clean up
patches haves been proposed which might cause conflicts with this patch. Please
see inline for details.
On 10 February 2014 15:38, Tarek Dakhran t.dakh...@samsung.com wrote:
EXYNOS5410 is SoC in
Hi Laszlo,
If you are considering re-spinning this patch based on Lee's comments, please
also consider my comments inline.
On 8 February 2014 17:03, Laszlo Papp lp...@kde.org wrote:
MAX6650/MAX6651 chip is a multi-function device with I2C busses. The
chip includes fan-speed regulators and
On 10 February 2014 17:51, Lee Jones lee.jo...@linaro.org wrote:
+#include linux/device.h
+#include linux/mfd/core.h
+#include linux/module.h
+#include linux/i2c.h
Please arrange these alphabetically.
Why?
1. It makes it easier to avoid adding duplicate includes.
2. Code looks more
On 10 February 2014 18:48, Laszlo Papp lp...@kde.org wrote:
On Mon, Feb 10, 2014 at 12:02 PM, Sachin Kamat sachin.ka...@linaro.org
wrote:
Hi Laszlo,
If you are considering re-spinning this patch based on Lee's comments, please
also consider my comments inline.
Hi, sure, thanks.
On 8
Hi Tarek,
On 11 February 2014 07:45, Tarek Dakhran t.dakh...@samsung.com wrote:
Due to implementation of exynos_boot_secondary function
only one secondary core boots on Exynos SoC's.
Even without this patch I could boot the secondary CPUs on Exynos4210, 4412 and
5250 based boards with the
] Brought up 4 CPUs
[4.255000] SMP: Total of 4 processors activated.
Regards,
Sachin
On Tue, Feb 11, 2014 at 3:17 PM, Sachin Kamat sachin.ka...@linaro.org
wrote:
Hi Tarek,
On 11 February 2014 07:45, Tarek Dakhran t.dakh...@samsung.com wrote:
Due to implementation of exynos_boot_secondary
On 11 February 2014 15:40, Laszlo Papp lp...@kde.org wrote:
[snip]
+
+struct max665x_dev {
+ struct device *dev;
+ struct mutex iolock;
+
+ struct i2c_client *i2c;
+ struct regmap *map;
+
+ int type;
Unnecessary extra lines above could be removed.
I
On 11 February 2014 17:09, Laszlo Papp lp...@kde.org wrote:
On Tue, Feb 11, 2014 at 11:14 AM, Sachin Kamat sachin.ka...@linaro.org
wrote:
On 11 February 2014 15:40, Laszlo Papp lp...@kde.org wrote:
[snip]
+
+struct max665x_dev {
+ struct device *dev;
+ struct mutex iolock
Hi Laszlo,
Sorry for missing out on a couple of points during my earlier review.
Please see inline.
On 12 February 2014 09:32, Laszlo Papp lp...@kde.org wrote:
MAX6650/MAX6651 chip is a multi-function device with I2C busses. The
chip includes fan-speed regulators and monitors, GPIO, and alarm.
stw481x_regmap_config is local to this file.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Linus Walleij linus.wall...@linaro.org
---
drivers/mfd/stw481x.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mfd/stw481x.c b/drivers/mfd/stw481x.c
index
Return directly to avoid redundant lines of code.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/mfd/wm8400-core.c | 22 +-
1 file changed, 5 insertions(+), 17 deletions(-)
diff --git a/drivers/mfd/wm8400-core.c b/drivers/mfd/wm8400-core.c
index
devm_regmap_init_i2c can fail. Check for it.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Linus Walleij linus.wall...@linaro.org
---
drivers/mfd/stw481x.c |6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/mfd/stw481x.c b/drivers/mfd/stw481x.c
index cc42f88586f6
Add this header explicitly for IS_ERR and friends.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/mfd/max14577.c |1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/mfd/max14577.c b/drivers/mfd/max14577.c
index 2ac2f2d7cea6..88cf5811ce3b 100644
--- a/drivers/mfd
On 9 January 2014 16:52, Sachin Kamat sachin.ka...@linaro.org wrote:
Added initial binding documentation for S2MPA01 MFD.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
* Re-organised as suggested by Mark Rutland.
---
Documentation/devicetree/bindings/mfd/s2mpa01.txt | 86
= {
};
#endif
-#if defined(CONFIG_CPUS_3C2443) || defined(CONFIG_CPU_S3C2416)
+#if defined(CONFIG_CPU_S3C2443) || defined(CONFIG_CPU_S3C2416)
This is definitely a typo and the current fix looks good.
Reviewed-by: Sachin Kamat sachin.ka...@linaro.org
--
With warm regards,
Sachin
On 12 February 2014 23:32, Mark Rutland mark.rutl...@arm.com wrote:
On Thu, Jan 09, 2014 at 11:22:33AM +, Sachin Kamat wrote:
Add support for S2MPA01 voltage and current regulator.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
* Addressed comments from Mark Brown
- Used
Hi Mark,
On 12 February 2014 23:40, Mark Rutland mark.rutl...@arm.com wrote:
Hi Sachin,
Apologies for the delay on this.
No problem :). Thank you for your review. Please see my comments inline.
On Thu, Jan 09, 2014 at 11:22:34AM +, Sachin Kamat wrote:
Added initial binding
Added initial binding documentation for S2MPA01 MFD.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
Changes since v2:
* Re-worded as suggested by Mark Rutland
---
Documentation/devicetree/bindings/mfd/s2mpa01.txt | 90 +
1 file changed, 90 insertions(+)
create
Add support for S2MPA01 voltage and current regulator.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Acked-by: Mark Brown broo...@linaro.org
---
Changes since v2:
* Added missing of_node_put
* Used of_get_child_by_name
---
drivers/regulator/Kconfig |7 +
drivers/regulator/Makefile
Add the necessary entries required for S2MPA01 multi-function
device. While at it also convert whitespaces to tabs in core.h.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Acked-by: Lee Jones lee.jo...@linaro.org
---
No change in v3 since v2.
---
drivers/mfd/sec-core.c | 45
-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 the FAQ at http
On 18 March 2014 18:09, Mark Brown broo...@kernel.org wrote:
On Tue, Mar 18, 2014 at 03:53:02PM +0530, Sachin Kamat wrote:
Silences the following type of error/warnings:
ERROR: that open brace { should be on the previous line
WARNING: Missing a blank line after declarations
Applied
/mfd/sec-core.c:342:45: warning: 'regmap_rtc' may be used
uninitialized in this function [-Wuninitialized]
Signed-off-by: Krzysztof Kozlowski k.kozlow...@samsung.com
From the point of suppressing this warning and considering how it is
done for S2MPS11,
Acked-by: Sachin Kamat sachin.ka
On 19 March 2014 14:29, Cho KyongHo pullip@samsung.com wrote:
On Tue, 18 Mar 2014 16:14:53 +0100, Tomasz Figa wrote:
On 18.03.2014 12:09, Cho KyongHo wrote:
On Fri, 14 Mar 2014 20:52:43 +0530, Sachin Kamat wrote:
Hi KyongHo,
On 14 March 2014 10:35, Cho KyongHo pullip@samsung.com
Move the }; to next line as is the general coding practice.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/regulator/max8660.c |3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/regulator/max8660.c b/drivers/regulator/max8660.c
index aabfb3b2407c
Hi Wei Yongjun,
On 7 May 2013 18:54, Wei Yongjun weiyj...@gmail.com wrote:
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return a negative error code from the error handling
case instead of 0, as done elsewhere in this function.
ipp_create_cmd_work() return ERR_PTR() on error and
init_data is a pointer. Use NULL instead of 0.
Silences the following sparse warning:
drivers/regulator/isl6271a-regulator.c:133:44:
warning: Using plain integer as NULL pointer
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Marek Vasut marek.va...@gmail.com
---
drivers/regulator
Since 'id' cannot take two values at the same time, the condition
should probably be an OR (||) instead of AND ().
Introduced by commit 28d1e8cd67 (regulator: palma: add ramp delay
support through regulator constraints).
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Laxman Dewangan
power_state_active_get is used only in this file. Make it static.
While at it also move this function definition inside the
CONFIG_REGULATOR_DEBUG macro as it is called only from within it.
This also avoids further build warning related to unused definition.
Signed-off-by: Sachin Kamat sachin.ka
On 10 June 2013 17:05, Tushar Behera tushar.beh...@linaro.org wrote:
Commit 75096579c3ac (lib: devres: Introduce devm_ioremap_resource())
introduced devm_ioremap_resource() and deprecated the use of
devm_request_and_ioremap().
Signed-off-by: Tushar Behera tushar.beh...@linaro.org
CC: Mark
On 10 June 2013 17:04, Tushar Behera tushar.beh...@linaro.org wrote:
Commit 75096579c3ac (lib: devres: Introduce devm_ioremap_resource())
introduced devm_ioremap_resource() and deprecated the use of
devm_request_and_ioremap().
Signed-off-by: Tushar Behera tushar.beh...@linaro.org
CC:
On 10 June 2013 17:05, Tushar Behera tushar.beh...@linaro.org wrote:
Commit 75096579c3ac (lib: devres: Introduce devm_ioremap_resource())
introduced devm_ioremap_resource() and deprecated the use of
devm_request_and_ioremap().
This patch does a little more than just conversion. May that change
On 10 June 2013 17:05, Tushar Behera tushar.beh...@linaro.org wrote:
Commit 75096579c3ac (lib: devres: Introduce devm_ioremap_resource())
introduced devm_ioremap_resource() and deprecated the use of
devm_request_and_ioremap().
Signed-off-by: Tushar Behera tushar.beh...@linaro.org
CC:
devm_* APIs are device managed and make code simpler.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Miguel Aguilar miguel.agui...@ridgerun.com
---
Patch is based on mfd-next tree and compile tested.
This driver gives following compilation error (not introduced by this patch):
drivers
devm_* APIs are device managed and make code simpler.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Miguel Aguilar miguel.agui...@ridgerun.com
---
CC'd Lee Jones
Patch is based on mfd-next tree and compile tested.
This driver gives following compilation error (not introduced
devm_regmap_init_i2c() is device managed and makes cleanup simpler.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
drivers/mfd/88pm860x-core.c |3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/mfd/88pm860x-core.c b/drivers/mfd/88pm860x-core.c
index ef6d82c
Hi Thierry,
On 12 June 2013 16:59, Thierry Reding thierry.red...@gmail.com wrote:
Some drivers don't set the .owner fields of the struct device_driver or
struct pwm_ops, which causes the module usage count to become wrong.
Recently a patch [1] was added to solve this issue of missing .owner
On 13 June 2013 03:01, Marc Kleine-Budde m...@pengutronix.de wrote:
Commit 75096579c3ac (lib: devres: Introduce devm_ioremap_resource())
introduced devm_ioremap_resource() and encourage to check its return value
with
IS_ERR(). This however leads to the following sparse warnings, as
On 13 June 2013 11:00, Tushar Behera tushar.beh...@linaro.org wrote:
On 06/10/2013 05:05 PM, Tushar Behera wrote:
Commit 75096579c3ac (lib: devres: Introduce devm_ioremap_resource())
introduced devm_ioremap_resource() and deprecated the use of
devm_request_and_ioremap().
Signed-off-by:
Hi Lee,
On 13 June 2013 16:19, Lee Jones lee.jo...@linaro.org wrote:
On Wed, 12 Jun 2013, Sachin Kamat wrote:
devm_* APIs are device managed and make code simpler.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Miguel Aguilar miguel.agui...@ridgerun.com
---
CC'd Lee Jones
Can
On 14 June 2013 12:36, Lee Jones lee.jo...@linaro.org wrote:
On Fri, 14 Jun 2013, Sachin Kamat wrote:
Hi Lee,
On 13 June 2013 16:19, Lee Jones lee.jo...@linaro.org wrote:
On Wed, 12 Jun 2013, Sachin Kamat wrote:
devm_* APIs are device managed and make code simpler.
Signed-off
devm_* APIs are device managed and make code simpler.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Miguel Aguilar miguel.agui...@ridgerun.com
---
Patch is based on mfd-next tree and compile tested.
This driver gives following compilation error (not introduced by this patch):
drivers
]
exynos_wkup_irq_ack(irqd);
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Doug Anderson diand...@chromium.org
---
drivers/pinctrl/pinctrl-exynos.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-exynos.c b/drivers
]
exynos_wkup_irq_ack(irqd);
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Doug Anderson diand...@chromium.org
---
Fixed the commit message.
---
drivers/pinctrl/pinctrl-exynos.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/pinctrl
On 17 June 2013 16:55, Linus Walleij linus.wall...@linaro.org wrote:
On Mon, Jun 17, 2013 at 11:38 AM, Sachin Kamat sachin.ka...@linaro.org
wrote:
'exynos_wkup_irq_ack' should be defined ahead of its call.
Fixes the following error:
drivers/pinctrl/pinctrl-exynos.c: In function
Hi Greg,
On 17 June 2013 23:07, Greg KH gre...@linuxfoundation.org wrote:
On Mon, Jun 17, 2013 at 03:47:41PM +0200, Pavel Machek wrote:
Hi!
This adds ability to bind uio driver to given open firmware device
using command line option. Thus, userspace driver can be developed and
used
Hi Samuel,
On 18 June 2013 14:10, Samuel Ortiz sa...@linux.intel.com wrote:
Hi Sachin,
On Wed, Jun 12, 2013 at 01:49:42PM +0530, Sachin Kamat wrote:
devm_* APIs are device managed and make code simpler.
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Cc: Miguel Aguilar miguel.agui
'spear310_o2p' is referenced only in this file.
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(-)
diff --git a/drivers/pinctrl/spear/pinctrl-plgpio.c
b
301 - 400 of 1357 matches
Mail list logo
