[GIT PULL] ARM: defconfig: Improvements for 4.3

2015-07-16 Thread Krzysztof Kozlowski
Dear Kukjin, Few defconfig related changes. Description along with a tag. You can find them also on the lists with my reviewed-by. Best regards, Krzysztof The following changes since commit 1c4c7159ed2468f3ac4ce5a7f08d79663d381a93: Merge tag 'ext4_for_linus_stable' of git://git.kernel.org/

Re: [PATCH v2 4/4] mfd: max77686: Split out regulator part from the DT binding

2015-07-16 Thread Krzysztof Kozlowski
2015-07-17 15:46 GMT+09:00 Javier Martinez Canillas : > Hello Krzysztof, > > On 07/17/2015 08:42 AM, Krzysztof Kozlowski wrote: >> 2015-07-17 15:29 GMT+09:00 Javier Martinez Canillas : >>> The Maxim MAX77686 PMIC is a multi-function device with regulators, >>> clocks and a RTC. The DT bindings for

Re: [PATCH v2 4/4] mfd: max77686: Split out regulator part from the DT binding

2015-07-16 Thread Javier Martinez Canillas
Hello Krzysztof, On 07/17/2015 08:42 AM, Krzysztof Kozlowski wrote: > 2015-07-17 15:29 GMT+09:00 Javier Martinez Canillas : >> The Maxim MAX77686 PMIC is a multi-function device with regulators, >> clocks and a RTC. The DT bindings for the clocks are in a separate >> file but the bindings for the

Re: [PATCH v2 4/4] mfd: max77686: Split out regulator part from the DT binding

2015-07-16 Thread Krzysztof Kozlowski
2015-07-17 15:29 GMT+09:00 Javier Martinez Canillas : > The Maxim MAX77686 PMIC is a multi-function device with regulators, > clocks and a RTC. The DT bindings for the clocks are in a separate > file but the bindings for the regulators are inside the mfd part. > > To make it consistent with the clo

Re: [PATCH v2 3/4] mfd: Add DT binding for Maxim MAX77802 IC

2015-07-16 Thread Krzysztof Kozlowski
2015-07-17 15:29 GMT+09:00 Javier Martinez Canillas : > The MAX77802 is a chip that contains regulators, 2 32kHz clocks, > a RTC and an I2C interface to program the individual components. > > The are already DT bindings for the regulators and clocks and > these reference to a bindings/mfd/max77802.

[PATCH v2 3/4] mfd: Add DT binding for Maxim MAX77802 IC

2015-07-16 Thread Javier Martinez Canillas
The MAX77802 is a chip that contains regulators, 2 32kHz clocks, a RTC and an I2C interface to program the individual components. The are already DT bindings for the regulators and clocks and these reference to a bindings/mfd/max77802.txt file, that didn't exist, for the details about the PMIC. S

[PATCH v2 0/4] mfd: Improve DT binding docs for max77686 and max77802

2015-07-16 Thread Javier Martinez Canillas
Hello Lee, This series contains some improvements for the Device Tree bindings of the Maxim MAX77686 and MAX77802 multi-function devices. This is the second version of the series that addresses issues pointed out by Krzysztof Kozlowski and Sergei Shtylyov. Patch #1 changes the max77686 binding t

[PATCH v2 2/4] mfd: max77686: Use a generic name for the PMIC node in the example

2015-07-16 Thread Javier Martinez Canillas
The ePAR standard says that: "the name of a node should be somewhat generic, reflecting the function of the device and not its precise programming model." So, change the max77686 binding document example to use a generic node name instead of using the chip's name. Suggested-by: Sergei Shtylyov S

[PATCH v2 4/4] mfd: max77686: Split out regulator part from the DT binding

2015-07-16 Thread Javier Martinez Canillas
The Maxim MAX77686 PMIC is a multi-function device with regulators, clocks and a RTC. The DT bindings for the clocks are in a separate file but the bindings for the regulators are inside the mfd part. To make it consistent with the clocks portion of the binding and because is more natural to look

[PATCH v2 1/4] mfd: max77686: Don't suggest in binding to use a deprecated property

2015-07-16 Thread Javier Martinez Canillas
The regulator-compatible property from the regulator DT binding was deprecated. But the max77686 DT binding doc still suggest to use it instead of the regulator node name's which is the correct approach. Signed-off-by: Javier Martinez Canillas Reviewed-by: Krzysztof Kozlowski --- Changes in v2

Re: [PATCH] i2c: s3c2410: remove unused variable

2015-07-16 Thread Wolfram Sang
On Thu, Jul 16, 2015 at 10:36:16PM +0200, Laurent Navet wrote: > ret is assigned zero and just used to return. remove it. > Found by using coccinelle. > > Signed-off-by: Laurent Navet Why not make it void? signature.asc Description: Digital signature

Re: [PATCH v2 06/13] irqchip: kill off set_irq_flags usage

2015-07-16 Thread Russell King - ARM Linux
On Thu, Jul 16, 2015 at 09:32:20PM +0200, Robert Jarzmik wrote: > For PXA I must admit I don't yet know. I know lubbock has an UCB1400, but I > don't have it working yet, so I can't foresee the problems yet. The UCB1400 is an AC'97 device which is quite different from its predecessors, and is not

[PATCH] i2c: s3c2410: remove unused variable

2015-07-16 Thread Laurent Navet
ret is assigned zero and just used to return. remove it. Found by using coccinelle. Signed-off-by: Laurent Navet --- drivers/i2c/busses/i2c-s3c2410.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/i2c/busses/i2c-s3c2410.c b/drivers/i2c/busses/i2c-s3c2410.c index 50

Re: [PATCH v2 06/13] irqchip: kill off set_irq_flags usage

2015-07-16 Thread Robert Jarzmik
Russell King - ARM Linux writes: >> drivers/net/ethernet/smsc/smc911x.c >> drivers/net/ethernet/smsc/smc9194.c >> drivers/net/ethernet/smsc/smc91x.c >> >> Those might still be, but on the DT based boards the probing should be >> completely irrelevant > > SA11x0 stuff uses smc91x.c PXA uses it al

Re: [PATCH v2 00/23] drm/exynos: atomic improvements + exynos_encoder removal

2015-07-16 Thread Gustavo Padovan
2015-07-14 Joonyoung Shim : > On 07/06/2015 11:20 PM, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > Hi, > > > > This set improves exynos in a number of ways. The first five patches are > > general clean up/fixes. > > > > Patches 06 to 12 are improvements on top of the newly added atomi

[PATCH v3 5/9] drm/exynos: pass struct exynos_drm_plane in update/enable

2015-07-16 Thread Gustavo Padovan
From: Gustavo Padovan We already have the plane pointer in before calling .update_plane() or disable_plane() so pass it directly to those calls avoiding a new conversion from zpos to struct exynos_drm_plane. v2: don't remove check for suspended in FIMD (comment by Joonyoung) Signed-off-by: Gust

[PATCH v3 7/9] drm/exynos: remove unused fields from struct exynos_drm_plane

2015-07-16 Thread Gustavo Padovan
From: Gustavo Padovan Now after the move to use drm_plane_state directly struct drm_plane_state has many unused fields, along with others that weren't used before the plane state change. Thus remove them all. Signed-off-by: Gustavo Padovan Reviewed-by: Joonyoung Shim --- drivers/gpu/drm/exyno

[PATCH v3 9/9] drm/exynos: return return value of exynos_crtc->enable_vblank

2015-07-16 Thread Gustavo Padovan
From: Gustavo Padovan Instead of blindly ignore the return value of enable_vblank return it to the upper DRM layer for error handling. Suggested-by: Joonyoung Shim Signed-off-by: Gustavo Padovan Reviewed-by: Joonyoung Shim --- drivers/gpu/drm/exynos/exynos_drm_crtc.c | 2 +- 1 file changed,

[PATCH v3 8/9] drm/exynos: unify exynos_drm_plane names with drm core

2015-07-16 Thread Gustavo Padovan
From: Gustavo Padovan Rename crtc_{widht,height} to crtc_{w,h} and src_{width,height} to src_{w,h} to make it similar to the atomic state names. Signed-off-by: Gustavo Padovan Reviewed-by: Joonyoung Shim --- drivers/gpu/drm/exynos/exynos5433_drm_decon.c | 10 +- drivers/gpu/drm/exynos

[PATCH v3 4/9] drm/exynos: rename win_commit/disable to atomic-like names

2015-07-16 Thread Gustavo Padovan
From: Gustavo Padovan Rename win_commit() helper to update_plane() and win_disable() to disable_plane(). Signed-off-by: Gustavo Padovan Reviewed-by: Joonyoung Shim --- drivers/gpu/drm/exynos/exynos5433_drm_decon.c | 10 +- drivers/gpu/drm/exynos/exynos7_drm_decon.c| 10 +-

[PATCH v3 6/9] drm/exynos: use drm atomic state directly

2015-07-16 Thread Gustavo Padovan
From: Gustavo Padovan For some fields the use of struct exynos_drm_plane filled with data from the plane state just creates a source of duplicated information and overhead. Here we change the crtc drivers to access the plane state directly simplifying the code by not relying on a exynos internal

[PATCH v3 3/9] drm/exynos: remove duplicated check for suspend

2015-07-16 Thread Gustavo Padovan
From: Gustavo Padovan The same check is placed twice in fimd/decon_update_plane(), remove one of them. Signed-off-by: Gustavo Padovan Reviewed-by: Joonyoung Shim --- drivers/gpu/drm/exynos/exynos7_drm_decon.c | 3 --- drivers/gpu/drm/exynos/exynos_drm_fimd.c | 3 --- 2 files changed, 6 dele

[PATCH v3 2/9] drm/exynos: use KMS version of DRM vblanks functions

2015-07-16 Thread Gustavo Padovan
From: Gustavo Padovan Get rid of legacy DRM vblank function that are less clear to use. The new ones basically requires only the crtc as parameters. It also clean ups exynos_drm_crtc_finish_pageflip() parameters as a consequence. Signed-off-by: Gustavo Padovan Reviewed-by: Joonyoung Shim ---

[PATCH v3 1/9] drm/exynos: pass the correct pipe number

2015-07-16 Thread Gustavo Padovan
From: Gustavo Padovan Instead of giving -1 to as arg to drm_send_vblank_event() pass the correct pipe number to it. Signed-off-by: Gustavo Padovan Reviewed-by: Joonyoung Shim --- drivers/gpu/drm/exynos/exynos_drm_crtc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/driv

Re: [PATCH v2 06/13] irqchip: kill off set_irq_flags usage

2015-07-16 Thread Russell King - ARM Linux
On Sun, Jul 12, 2015 at 06:43:56PM +0200, Thomas Gleixner wrote: > The probe function was added in the initial implementation of the > driver (2006), so it predates device tree. > > drivers/net/appletalk/ltpc.c > drivers/net/arcnet/com20020-isa.c > drivers/net/arcnet/com90io.c > drivers/net/arcnet

Re: [PATCHv7 14/15] cec: s5p-cec: Add s5p-cec driver

2015-07-16 Thread Hans Verkuil
Marek, Kamil, On 06/29/15 12:14, Hans Verkuil wrote: > From: Kamil Debski > > Add CEC interface driver present in the Samsung Exynos range of > SoCs. > > The following files were based on work by SangPil Moon: > - exynos_hdmi_cec.h > - exynos_hdmi_cecctl.c > > Signed-off-by: Kamil Debski >

Re: [PATCH v2 06/13] irqchip: kill off set_irq_flags usage

2015-07-16 Thread Thomas Gleixner
On Mon, 13 Jul 2015, Rob Herring wrote: > On Sun, Jul 12, 2015 at 11:43 AM, Thomas Gleixner wrote: > > So in most of the irqchip drivers, this is irrelevant. > > Agreed, but that's a separate series I think. I'm trying not to change > behavior with this series. Are you proposing I do something di

Re: [patch v2 2/2] pinctrl: samsung: remove "out of memory" messages

2015-07-16 Thread Linus Walleij
On Mon, Jun 22, 2015 at 5:13 PM, Dan Carpenter wrote: > Checkpatch.pl complains about these: > > WARNING: Possible unnecessary 'out of memory' message > > The messages use a little extra RAM and they add a few extra lines of > code. We're probably never going to hit these out of memory situation

Re: [patch v2 1/2] pinctrl: samsung: don't truncate the last char

2015-07-16 Thread Linus Walleij
On Mon, Jun 22, 2015 at 5:12 PM, Dan Carpenter wrote: > We were allocating enough space because sizeof("-grp") and > sizeof("-mux") are both equal to 5 but in the snprintf() we only allowed > for 4 characters so the last 'p' and 'x' characters were truncated. > > The allocate and sprintf can be d

Re: [PATCH v6 1/3] clk: samsung: exynos3250: Add cpu clock configuration data and instaniate cpu clock

2015-07-16 Thread Krzysztof Kozlowski
2015-07-02 9:42 GMT+09:00 Chanwoo Choi : > This patch add CPU clock configuration data and instantiate the CPU clock type > for Exynos3250 to support Samsung specific cpu-clock type. > > Cc: Sylwester Nawrocki > Cc: Tomasz Figa > Signed-off-by: Chanwoo Choi > Acked-by: Kyungmin Park > Reviewed-