Re: [PATCH] video: exynos_mipi_dsim: Remove unused variable

+ Tomi Hi Olof, On 15 November 2013 02:39, Olof Johansson wrote: > commit 7e0be9f9f7cba3356f75b86737dbe3a005da067e ('video: exynos_mipi_dsim: > Use the generic PHY driver') resulted in a warning about an unused > variable: > > drivers/video/exynos/exynos_mipi_dsi.c:144:26: warning: unused

Re: [PATCH] video: exynos_mipi_dsim: Remove unused variable

+ Tomi Hi Olof, On 15 November 2013 02:39, Olof Johansson o...@lixom.net wrote: commit 7e0be9f9f7cba3356f75b86737dbe3a005da067e ('video: exynos_mipi_dsim: Use the generic PHY driver') resulted in a warning about an unused variable: drivers/video/exynos/exynos_mipi_dsi.c:144:26: warning:

Re: [PATCH 1/1] Documentation/mfd: Fix typos in s2mps11.txt

On 8 November 2013 16:50, Sachin Kamat wrote: > On 8 November 2013 15:58, Lee Jones wrote: >> As this is only a trivial patch, I'll take it out the hands of the >> already overloaded DT maintainers. >> >>> Fixed trivial typos to improve readability. >&

[PATCH 1/1] Documentation/mfd: Update s2mps11.txt

Updated the number of LDOs and BUCKs as per the user manual. Fixed trivial typos to improve readability. Signed-off-by: Sachin Kamat --- Documentation/devicetree/bindings/mfd/s2mps11.txt | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/Documentation/devicetree

Re: [PATCH 1/1] Documentation/mfd: Fix typos in s2mps11.txt

On 8 November 2013 15:58, Lee Jones wrote: > As this is only a trivial patch, I'll take it out the hands of the > already overloaded DT maintainers. > >> Fixed trivial typos to improve readability. >> >> Signed-off-by: Sachin Kamat >> --- >> Documentat

Re: [PATCH 1/1] Documentation/mfd: Fix typos in s2mps11.txt

On 8 November 2013 15:58, Lee Jones lee.jo...@linaro.org wrote: As this is only a trivial patch, I'll take it out the hands of the already overloaded DT maintainers. Fixed trivial typos to improve readability. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- Documentation

[PATCH 1/1] Documentation/mfd: Update s2mps11.txt

Updated the number of LDOs and BUCKs as per the user manual. Fixed trivial typos to improve readability. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- Documentation/devicetree/bindings/mfd/s2mps11.txt | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git

Re: [PATCH 1/1] Documentation/mfd: Fix typos in s2mps11.txt

On 8 November 2013 16:50, Sachin Kamat sachin.ka...@linaro.org wrote: On 8 November 2013 15:58, Lee Jones lee.jo...@linaro.org wrote: As this is only a trivial patch, I'll take it out the hands of the already overloaded DT maintainers. Fixed trivial typos to improve readability. Signed-off

Re: Boot failure on Origen with next-20131107

Hi Tomasz, On 7 November 2013 16:59, Tomasz Figa wrote: > Hi Sachin, > > On Thursday 07 of November 2013 16:25:15 Sachin Kamat wrote: >> Hi, >> >> Today's linux next fails to boot Origen (Exynos4210) board. However, >> Arndale (Exynos5250) board boots fine. Git

Re: [PATCH 1/2] of: irq: Fix interrupt-map entry matching

On 5 November 2013 20:51, Tomasz Figa wrote: > This patch fixes interrupt-map entry matching code to properly match all > specifier cells with interrupt map entries. > > Signed-off-by: Tomasz Figa Tested on Origen and Arndale boards. Works fine. Tested-by: Sachin Kamat -- With

Boot failure on Origen with next-20131107

Hi, Today's linux next fails to boot Origen (Exynos4210) board. However, Arndale (Exynos5250) board boots fine. Git bisect points to the following as bad commit: 2361613206e66ce59cc0e08efa8d98ec15b84ed1 is the first bad commit commit 2361613206e66ce59cc0e08efa8d98ec15b84ed1 Author: Grant Likely

Boot failure on Origen with next-20131107

Hi, Today's linux next fails to boot Origen (Exynos4210) board. However, Arndale (Exynos5250) board boots fine. Git bisect points to the following as bad commit: 2361613206e66ce59cc0e08efa8d98ec15b84ed1 is the first bad commit commit 2361613206e66ce59cc0e08efa8d98ec15b84ed1 Author: Grant Likely

Re: [PATCH 1/2] of: irq: Fix interrupt-map entry matching

-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://www.tux.org

Re: Boot failure on Origen with next-20131107

Hi Tomasz, On 7 November 2013 16:59, Tomasz Figa t.f...@samsung.com wrote: Hi Sachin, On Thursday 07 of November 2013 16:25:15 Sachin Kamat wrote: Hi, Today's linux next fails to boot Origen (Exynos4210) board. However, Arndale (Exynos5250) board boots fine. Git bisect points

[PATCH 1/1] Documentation/mfd: Fix typos in s2mps11.txt

Fixed trivial typos to improve readability. Signed-off-by: Sachin Kamat --- Documentation/devicetree/bindings/mfd/s2mps11.txt |9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/Documentation/devicetree/bindings/mfd/s2mps11.txt b/Documentation/devicetree/bindings

Re: [PATCH] pwm: samsung: memory leak bugfix in pwm_samsung_free

Hi Sangjung, On 5 November 2013 08:32, Sangjung Woo wrote: > There is certainly a kind of memory leak since allocated data (i.e. > chip_data) is set NULL before freeing it by calling devm_kfree(). This > patch fixes the memory leak bug by modifying its call order. > > Signed-off-by: Sangjung Woo

Re: [PATCH] pwm: samsung: memory leak bugfix in pwm_samsung_free

Hi Sangjung, On 5 November 2013 08:32, Sangjung Woo sangjung@samsung.com wrote: There is certainly a kind of memory leak since allocated data (i.e. chip_data) is set NULL before freeing it by calling devm_kfree(). This patch fixes the memory leak bug by modifying its call order.

[PATCH 1/1] Documentation/mfd: Fix typos in s2mps11.txt

Fixed trivial typos to improve readability. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- Documentation/devicetree/bindings/mfd/s2mps11.txt |9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/Documentation/devicetree/bindings/mfd/s2mps11.txt b/Documentation

Re: [PATCH 1/1] drivers: phy: Fix memory leak

On 31 October 2013 14:16, Kishon Vijay Abraham I wrote: > Hi, > > On Friday 25 October 2013 10:56 AM, Sachin Kamat wrote: >> 'phy' was not being freed upon error in one of the cases. >> Adjust the 'goto's to fix this. > > This patch looks good. I'll send this once -r

Re: [PATCH 1/1] drivers: phy: Fix memory leak

On 31 October 2013 14:16, Kishon Vijay Abraham I kis...@ti.com wrote: Hi, On Friday 25 October 2013 10:56 AM, Sachin Kamat wrote: 'phy' was not being freed upon error in one of the cases. Adjust the 'goto's to fix this. This patch looks good. I'll send this once -rc1 is tagged? Sure

[PATCH 1/1] drivers: phy: Fix memory leak

'phy' was not being freed upon error in one of the cases. Adjust the 'goto's to fix this. Signed-off-by: Sachin Kamat --- drivers/phy/phy-core.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/phy/phy-core.c b/drivers/phy/phy-core.c index 03cf8fb

[PATCH 1/1] drivers: phy: Fix memory leak

'phy' was not being freed upon error in one of the cases. Adjust the 'goto's to fix this. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/phy/phy-core.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/phy/phy-core.c b/drivers/phy/phy

[PATCH 1/1] pinctrl: mvebu: remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/pinctrl/mvebu/pinctrl-armada-370.c |2 +- drivers/pinctrl/mvebu/pinctrl-armada-xp.c |2 +- drivers/pinctrl/mvebu/pinctrl-dove.c |2

Re: [PATCH v3 1/4] ab8500-charger: Check return value of regulator_enable

On 6 September 2013 15:08, 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

Re: [PATCH v3 1/4] ab8500-charger: Check return value of regulator_enable

On 6 September 2013 15:08, 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

[PATCH 1/1] pinctrl: mvebu: remove redundant of_match_ptr

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/mvebu/pinctrl-armada-370.c |2 +- drivers/pinctrl/mvebu/pinctrl-armada-xp.c |2 +- drivers/pinctrl/mvebu/pinctrl

[PATCH 1/1] mfd: max77693: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid build breakage in the future. Signed-off-by: Sachin Kamat Cc: SangYoung, Son --- drivers/mfd/max77693.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/max77693.c b/drivers/mfd/max77693.c index 4d81ed2

[PATCH 1/1] mfd: max77693: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid build breakage in the future. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org Cc: SangYoung, Son hello@samsung.com --- drivers/mfd/max77693.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd

[PATCH 4/9] mfd: max8997: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat --- drivers/mfd/max8997.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/max8997.c b/drivers/mfd/max8997.c index cee098c..791aea3 100644 --- a/drivers

[PATCH 2/9] mfd: max77686: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat --- drivers/mfd/max77686.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/max77686.c b/drivers/mfd/max77686.c index 522be67..34520cb 100644

[PATCH 3/9] mfd: max8907: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat --- drivers/mfd/max8907.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/max8907.c b/drivers/mfd/max8907.c index e9b1c93..3bbfedc 100644 --- a/drivers

[PATCH 6/9] mfd: tps6507x: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat --- drivers/mfd/tps6507x.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/tps6507x.c b/drivers/mfd/tps6507x.c index 5ad4b77..a081b92 100644

[PATCH 8/9] mfd: tps65910: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat --- drivers/mfd/tps65910.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/tps65910.c b/drivers/mfd/tps65910.c index 78ffe6e..c0f608e 100644

[PATCH 1/9] mfd: as3711: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat --- drivers/mfd/as3711.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/as3711.c b/drivers/mfd/as3711.c index abd3ab7..ec684fc 100644 --- a/drivers/mfd

[PATCH 5/9] mfd: sec-core: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat Cc: Sangbeom Kim --- drivers/mfd/sec-core.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/sec-core.c b/drivers/mfd/sec-core.c index f530e4b..34c18fb

[PATCH 7/9] mfd: tps6586x: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat --- drivers/mfd/tps6586x.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/tps6586x.c b/drivers/mfd/tps6586x.c index 68906b1..ee61fd7 100644

[PATCH 9/9] mfd: arizona: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat --- drivers/mfd/arizona-i2c.c |1 + drivers/mfd/arizona-spi.c |1 + 2 files changed, 2 insertions(+) diff --git a/drivers/mfd/arizona-i2c.c b/drivers/mfd/arizona

[PATCH 7/9] mfd: tps6586x: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/mfd/tps6586x.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/tps6586x.c b/drivers/mfd/tps6586x.c index 68906b1

[PATCH 1/9] mfd: as3711: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/mfd/as3711.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/as3711.c b/drivers/mfd/as3711.c index abd3ab7..ec684fc

[PATCH 5/9] mfd: sec-core: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org Cc: Sangbeom Kim sbki...@samsung.com --- drivers/mfd/sec-core.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/sec-core.c b

[PATCH 9/9] mfd: arizona: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/mfd/arizona-i2c.c |1 + drivers/mfd/arizona-spi.c |1 + 2 files changed, 2 insertions(+) diff --git a/drivers/mfd/arizona-i2c.c

[PATCH 8/9] mfd: tps65910: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/mfd/tps65910.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/tps65910.c b/drivers/mfd/tps65910.c index 78ffe6e

[PATCH 6/9] mfd: tps6507x: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/mfd/tps6507x.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/tps6507x.c b/drivers/mfd/tps6507x.c index 5ad4b77

[PATCH 3/9] mfd: max8907: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/mfd/max8907.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/max8907.c b/drivers/mfd/max8907.c index e9b1c93

[PATCH 4/9] mfd: max8997: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/mfd/max8997.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/max8997.c b/drivers/mfd/max8997.c index cee098c

[PATCH 2/9] mfd: max77686: Include linux/of.h header

'of_match_ptr' is defined in linux/of.h. Include it explicitly to avoid breakage in the future. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/mfd/max77686.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/max77686.c b/drivers/mfd/max77686.c index 522be67

[PATCH 3/6] mfd: 88pm860x: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/mfd/88pm860x-core.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/88pm860x-core.c b/drivers/mfd/88pm860x-core.c

[PATCH 2/6] mfd: tps65217: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/mfd/tps65217.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/tps65217.c b/drivers/mfd/tps65217.c index b8f4864

[PATCH 4/6] mfd: ti_am335x_tscadc: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/mfd/ti_am335x_tscadc.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/ti_am335x_tscadc.c b/drivers/mfd

[PATCH Resend 1/6] mfd: max8925: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/mfd/max8925-i2c.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/max8925-i2c.c b/drivers/mfd/max8925-i2c.c index

[PATCH 5/6] mfd: omap-usb: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/mfd/omap-usb-host.c |2 +- drivers/mfd/omap-usb-tll.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mfd/omap-usb

[PATCH 6/6] mfd: palmas: Remove redundant of_match_ptr

'of_palmas_match_tbl' is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat Cc: Graeme Gregory --- drivers/mfd/palmas.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/palmas.c b/drivers/mfd/palmas.c index ae91478..d280d78

[PATCH 6/6] mfd: palmas: Remove redundant of_match_ptr

'of_palmas_match_tbl' is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org Cc: Graeme Gregory g...@slimlogic.co.uk --- drivers/mfd/palmas.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/palmas.c b

[PATCH Resend 1/6] mfd: max8925: Remove redundant of_match_ptr

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

[PATCH 5/6] mfd: omap-usb: Remove redundant of_match_ptr

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

[PATCH 3/6] mfd: 88pm860x: Remove redundant of_match_ptr

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

[PATCH 2/6] mfd: tps65217: Remove redundant of_match_ptr

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

[PATCH 4/6] mfd: ti_am335x_tscadc: Remove redundant of_match_ptr

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

Re: [PATCH 3/3] ARM: dts: exynos4x12: Add maudio/gps_alive power domain for exynos4x12

On 10 October 2013 06:00, Chanwoo Choi wrote: > This patch maudio/gps_alive power domain to exynos4x12.dtsi. > > Signed-off-by: Chanwoo Choi > Signed-off-by: Kyungmin Park > --- > arch/arm/boot/dts/exynos4x12.dtsi | 10 ++ > 1 file changed, 10 insertions(+) > > diff --git

Re: [PATCH 3/3] ARM: dts: exynos4x12: Add maudio/gps_alive power domain for exynos4x12

On 10 October 2013 06:00, Chanwoo Choi cw00.c...@samsung.com wrote: This patch maudio/gps_alive power domain to exynos4x12.dtsi. Signed-off-by: Chanwoo Choi cw00.c...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- arch/arm/boot/dts/exynos4x12.dtsi | 10 ++ 1

Re: [PATCH 11/11] pinctrl: plgpio: Remove redundant of_match_ptr

On 8 October 2013 13:36, Linus Walleij wrote: > On Sat, Sep 28, 2013 at 2:08 PM, Sachin Kamat wrote: > >> The data structure of_match_ptr() protects is always compiled in. >> Hence of_match_ptr() is not needed. >> >> Signed-off-by: Sachin Kamat >>

Re: [PATCH 11/11] pinctrl: plgpio: Remove redundant of_match_ptr

On 8 October 2013 13:36, Linus Walleij linus.wall...@linaro.org wrote: On Sat, Sep 28, 2013 at 2:08 PM, Sachin Kamat sachin.ka...@linaro.org wrote: 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

[PATCH 2/2] regulator: gpio: Remove redundant break

'break' after goto has no effect. Remove it. Signed-off-by: Sachin Kamat --- drivers/regulator/gpio-regulator.c |1 - 1 file changed, 1 deletion(-) diff --git a/drivers/regulator/gpio-regulator.c b/drivers/regulator/gpio-regulator.c index 98a98ff..04406a9 100644 --- a/drivers/regulator

[PATCH 1/2] regulator: wm8994: Remove redundant break

'break' after return has no effect. Remove it. Signed-off-by: Sachin Kamat --- drivers/regulator/wm8994-regulator.c |1 - 1 file changed, 1 deletion(-) diff --git a/drivers/regulator/wm8994-regulator.c b/drivers/regulator/wm8994-regulator.c index 71c5911..ffc1b2f 100644 --- a/drivers

[PATCH 1/2] regulator: wm8994: Remove redundant break

'break' after return has no effect. Remove it. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/regulator/wm8994-regulator.c |1 - 1 file changed, 1 deletion(-) diff --git a/drivers/regulator/wm8994-regulator.c b/drivers/regulator/wm8994-regulator.c index 71c5911..ffc1b2f

[PATCH 2/2] regulator: gpio: Remove redundant break

'break' after goto has no effect. Remove it. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/regulator/gpio-regulator.c |1 - 1 file changed, 1 deletion(-) diff --git a/drivers/regulator/gpio-regulator.c b/drivers/regulator/gpio-regulator.c index 98a98ff..04406a9 100644

Re: [PATCH 11/11] pinctrl: plgpio: Remove redundant of_match_ptr

On 3 October 2013 12:42, Viresh Kumar wrote: > On 3 October 2013 12:40, Sachin Kamat wrote: >> Do you really want a new one. If so, please let me know how you would >> like this to be re-worded. I have already used this for others and >> thought it was clear enough :) &

Re: [PATCH 11/11] pinctrl: plgpio: Remove redundant of_match_ptr

On 3 October 2013 12:37, Viresh Kumar wrote: > On 3 October 2013 12:36, Sachin Kamat wrote: >> On 3 October 2013 10:07, Viresh Kumar wrote: >>> On 28 September 2013 17:38, Sachin Kamat wrote: > Yeah I will Ack this.. but was waiting for your next version with fixed log.

Re: [PATCH 11/11] pinctrl: plgpio: Remove redundant of_match_ptr

On 3 October 2013 10:07, Viresh Kumar wrote: > On 28 September 2013 17:38, Sachin Kamat wrote: >> The data structure of_match_ptr() protects is always compiled in. > > of_match_ptr() isn't a data structure.. Yes you are right. of_match_ptr() is a macro. The data structure her

Re: [PATCH 11/11] pinctrl: plgpio: Remove redundant of_match_ptr

On 3 October 2013 10:07, Viresh Kumar viresh.ku...@linaro.org wrote: On 28 September 2013 17:38, Sachin Kamat sachin.ka...@linaro.org wrote: The data structure of_match_ptr() protects is always compiled in. of_match_ptr() isn't a data structure.. Yes you are right. of_match_ptr() is a macro

Re: [PATCH 11/11] pinctrl: plgpio: Remove redundant of_match_ptr

On 3 October 2013 12:37, Viresh Kumar viresh.ku...@linaro.org wrote: On 3 October 2013 12:36, Sachin Kamat sachin.ka...@linaro.org wrote: On 3 October 2013 10:07, Viresh Kumar viresh.ku...@linaro.org wrote: On 28 September 2013 17:38, Sachin Kamat sachin.ka...@linaro.org wrote: Yeah I

Re: [PATCH 11/11] pinctrl: plgpio: Remove redundant of_match_ptr

On 3 October 2013 12:42, Viresh Kumar viresh.ku...@linaro.org wrote: On 3 October 2013 12:40, Sachin Kamat sachin.ka...@linaro.org wrote: Do you really want a new one. If so, please let me know how you would like this to be re-worded. I have already used this for others and thought

Re: [PATCH 1/2] regulator: ti-abb: Remove redundant of_match_ptr

On 30 September 2013 16:31, Mark Brown wrote: > On Mon, Sep 30, 2013 at 02:27:13PM +0530, Sachin Kamat wrote: > > Note that one of the things these macros are supposed to do (but don't > yet IIRC) is allow for dead code elimination to discard references to > things automatical

[PATCH 1/1] drivers/pps/clients/pps-gpio.c: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/pps/clients/pps-gpio.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pps/clients/pps-gpio.c b/drivers/pps/clients/pps

[PATCH 2/2] regulator: twl: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/regulator/twl-regulator.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/regulator/twl-regulator.c b/drivers/regulator

[PATCH 1/2] regulator: ti-abb: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- 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

[PATCH 2/2] regulator: twl: Remove redundant of_match_ptr

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/regulator/twl-regulator.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/regulator/twl-regulator.c

[PATCH 1/2] regulator: ti-abb: Remove redundant of_match_ptr

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/regulator/ti-abb-regulator.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/regulator/ti-abb

[PATCH 1/1] drivers/pps/clients/pps-gpio.c: Remove redundant of_match_ptr

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/pps/clients/pps-gpio.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pps/clients/pps-gpio.c b

Re: [PATCH 1/2] regulator: ti-abb: Remove redundant of_match_ptr

On 30 September 2013 16:31, Mark Brown broo...@kernel.org wrote: On Mon, Sep 30, 2013 at 02:27:13PM +0530, Sachin Kamat wrote: Note that one of the things these macros are supposed to do (but don't yet IIRC) is allow for dead code elimination to discard references to things automatically even

[PATCH 3/5] mfd: 88pm860x: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/mfd/88pm860x-core.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/88pm860x-core.c b/drivers/mfd/88pm860x-core.c

[PATCH 4/5] mfd: ti_am335x_tscadc: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/mfd/ti_am335x_tscadc.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/ti_am335x_tscadc.c b/drivers/mfd

[PATCH 2/5] mfd: tps65217: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/mfd/tps65217.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/tps65217.c b/drivers/mfd/tps65217.c index b8f4864

[PATCH 5/5] mfd: omap-usb: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/mfd/omap-usb-host.c |2 +- drivers/mfd/omap-usb-tll.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mfd/omap-usb

[PATCH 1/5] mfd: max8925: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/mfd/max8925-i2c.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/max8925-i2c.c b/drivers/mfd/max8925-i2c.c index

[PATCH 02/11] pinctrl: exynos5440: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/pinctrl/pinctrl-exynos5440.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/pinctrl-exynos5440.c b/drivers

[PATCH 04/11] pinctrl: imx51: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/pinctrl/pinctrl-imx51.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/pinctrl-imx51.c b/drivers/pinctrl/pinctrl

[PATCH 07/11] pinctrl: imx6q: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/pinctrl/pinctrl-imx6q.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/pinctrl-imx6q.c b/drivers/pinctrl/pinctrl

[PATCH 10/11] pinctrl: imx6sl: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/pinctrl/pinctrl-imx6sl.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/pinctrl-imx6sl.c b/drivers/pinctrl

[PATCH 08/11] pinctrl: samsung: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/pinctrl/pinctrl-samsung.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/pinctrl-samsung.c b/drivers/pinctrl

[PATCH 06/11] pinctrl: imx6dl: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/pinctrl/pinctrl-imx6dl.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/pinctrl-imx6dl.c b/drivers/pinctrl

[PATCH 09/11] pinctrl: vf610: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/pinctrl/pinctrl-vf610.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/pinctrl-vf610.c b/drivers/pinctrl/pinctrl

[PATCH 11/11] pinctrl: plgpio: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat Cc: Viresh Kumar --- drivers/pinctrl/spear/pinctrl-plgpio.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/spear/pinctrl

[PATCH 03/11] pinctrl: imx35: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/pinctrl/pinctrl-imx35.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/pinctrl-imx35.c b/drivers/pinctrl/pinctrl

[PATCH 05/11] pinctrl: imx53: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/pinctrl/pinctrl-imx53.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/pinctrl-imx53.c b/drivers/pinctrl/pinctrl

[PATCH 01/11] pinctrl: at91: Remove redundant of_match_ptr

The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat --- drivers/pinctrl/pinctrl-at91.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/pinctrl/pinctrl-at91.c b/drivers/pinctrl

[PATCH 1/1] mfd: palmas: Remove redundant of_match_ptr

'of_palmas_match_tbl' is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat Cc: Graeme Gregory --- drivers/mfd/palmas.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/palmas.c b/drivers/mfd/palmas.c index 135afab..e6c8aa0

[PATCH 1/1] mfd: palmas: Remove redundant of_match_ptr

'of_palmas_match_tbl' is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org Cc: Graeme Gregory g...@slimlogic.co.uk --- drivers/mfd/palmas.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/palmas.c b

[PATCH 03/11] pinctrl: imx35: Remove redundant of_match_ptr

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-imx35.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/pinctrl-imx35.c b

[PATCH 05/11] pinctrl: imx53: Remove redundant of_match_ptr

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-imx53.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/pinctrl-imx53.c b

<    2   3   4   5   6   7   8   9   10   11   >