Re: [PATCH] clk: samsung: exynos7: Add required clock tree for UFS

2015-08-31 Thread Alim Akhtar
Hello, On Sat, Aug 29, 2015 at 2:23 PM, Krzysztof Kozlowski wrote: > 2015-08-28 18:28 GMT+09:00 Alim Akhtar : >> Adding required mux/div/gate clocks for UFS controller >> present on Exynos7. >> >> Signed-off-by: Alim Akhtar

Re: [PATCH 07/14] exynos/fimg2d: add g2d_validate_xyz() functions

2015-08-31 Thread Emil Velikov
On 31 August 2015 at 14:18, Inki Dae wrote: > On 2015년 08월 24일 23:14, Tobias Jakobi wrote: >> The G2D headers define a number of modes through enums >> (like e.g. color, select, repeat, etc.). >> >> This introduces g2d_validate_select_mode() and >> g2d_validate_blending_op()

Re: [PATCH 04/14] exynos/fimg2d: check buffer space in g2d_solid_fill()

2015-08-31 Thread Tobias Jakobi
Hello! Inki Dae wrote: > On 2015년 08월 24일 23:13, Tobias Jakobi wrote: >> The amount of commands (regular and GEM) doesn't depend >> on the input here. >> >> Signed-off-by: Tobias Jakobi >> --- >> exynos/exynos_fimg2d.c | 3 +++ >> 1 file changed, 3 insertions(+)

Re: [PATCH 07/14] exynos/fimg2d: add g2d_validate_xyz() functions

2015-08-31 Thread Tobias Jakobi
Hello, Emil Velikov wrote: > On 31 August 2015 at 14:18, Inki Dae wrote: >> On 2015년 08월 24일 23:14, Tobias Jakobi wrote: >>> The G2D headers define a number of modes through enums >>> (like e.g. color, select, repeat, etc.). >>> >>> This introduces

Re: [PATCH 04/14] exynos/fimg2d: check buffer space in g2d_solid_fill()

2015-08-31 Thread Emil Velikov
On 31 August 2015 at 20:27, Tobias Jakobi wrote: > Hello! > > Inki Dae wrote: >> On 2015년 08월 24일 23:13, Tobias Jakobi wrote: >>> +if (g2d_check_space(ctx, 7, 1)) >>> +return -ENOSPC; >> >> You can make 3 and 4 patches to one. These should be same

Re: [PATCH] rtc: Fix module autoload for rtc-{ab8500,max8997,s5m} drivers

2015-08-31 Thread Alexandre Belloni
On 27/08/2015 at 12:34:32 +0200, Javier Martinez Canillas wrote : > These platform drivers have a platform device ID table but the module > alias information is not created so module autoloading will not work. > > Signed-off-by: Javier Martinez Canillas > > --- > >

Re: [PATCH] drivers: i2c: exynos5: irq spinlock rt-safe

2015-08-31 Thread Anders Roxell
On 2015-08-25 23:51, Thomas Gleixner wrote: > On Tue, 25 Aug 2015, Anders Roxell wrote: > > > The exynos5_i2c_message_start enables interrupts while holding the i2c > > lock which is sought by the irq handler. If an IRQ is received before > > this lock is released then a deadlock occurs. > >

Re: [PATCH] ARM: EXYNOS: reset KFC cores when cpu is up

2015-08-31 Thread Kevin Hilman
Chanho Park writes: > The cpu booting of exynos5422 has been still broken since we discussed > it in last year[1]. This patch is inspired from odroid xu3 > code(Actually, it was from samsung exynos vendor kernel)[2]. This weird > reset code was founded exynos5420 octa cores

Re: [PATCH] ARM: EXYNOS: reset KFC cores when cpu is up

2015-08-31 Thread Javier Martinez Canillas
[adding Krzysztof Kozlowski to cc list] Hello Kevin, On 09/01/2015 12:11 AM, Kevin Hilman wrote: > Chanho Park writes: > >> The cpu booting of exynos5422 has been still broken since we discussed >> it in last year[1]. This patch is inspired from odroid xu3 >>

Re: [PATCH] ARM: EXYNOS: reset KFC cores when cpu is up

2015-08-31 Thread Krzysztof Kozlowski
On 31.08.2015 23:25, Chanho Park wrote: > The cpu booting of exynos5422 has been still broken since we discussed s/cpu/CPU/ (here and in other places) > it in last year[1]. This patch is inspired from odroid xu3 s/odroid xu3/Odroid XU3/ > code(Actually, it was from samsung exynos vendor

Re: [PATCH] ARM: EXYNOS: reset KFC cores when cpu is up

2015-08-31 Thread Krzysztof Kozlowski
On 01.09.2015 07:46, Javier Martinez Canillas wrote: > > [adding Krzysztof Kozlowski to cc list] > > Hello Kevin, > > On 09/01/2015 12:11 AM, Kevin Hilman wrote: >> Chanho Park writes: >> >>> The cpu booting of exynos5422 has been still broken since we discussed >>> it in

Re: [PATCH 10/14] exynos/fimg2d: remove default case from g2d_get_blend_op()

2015-08-31 Thread Inki Dae
On 2015년 09월 01일 03:53, Emil Velikov wrote: > On 31 August 2015 at 14:25, Inki Dae wrote: >> On 2015년 08월 24일 23:14, Tobias Jakobi wrote: >>> We now validate the blending mode via g2d_validate_mode() >>> prior to feeding it to g2d_get_blend_op(). >>> >>> Signed-off-by:

Re: [PATCH 04/14] exynos/fimg2d: check buffer space in g2d_solid_fill()

2015-08-31 Thread Inki Dae
On 2015년 09월 01일 04:57, Emil Velikov wrote: > On 31 August 2015 at 20:27, Tobias Jakobi > wrote: >> Hello! >> >> Inki Dae wrote: >>> On 2015년 08월 24일 23:13, Tobias Jakobi wrote: > +if (g2d_check_space(ctx, 7, 1)) +return -ENOSPC; >>> >>>

Re: [PATCH 00/14] drm/exynos: rewrite fimg2d error handling

2015-08-31 Thread Inki Dae
On 2015년 08월 28일 05:46, Tobias Jakobi wrote: > Hey Emil, > > > Emil Velikov wrote: >> Hi Tobias, >> >> On 24 August 2015 at 15:13, Tobias Jakobi >> wrote: >>> Hello, >>> >>> during the discussion about the last patchset touching the >>> fimg2d code, it became

Re: [PATCH v2] spi: bitbang: Replace spinlock by mutex

2015-08-31 Thread Mark Brown
On Mon, Aug 31, 2015 at 03:54:13PM +0800, Nicolas Boichat wrote: > On Mon, Aug 17, 2015 at 11:52 AM, Nicolas Boichat > wrote: > Any update on this patch? It still applies on for-next but happy to > resend if necessary (patchwork: >

Re: [PATCH v2] spi: bitbang: Replace spinlock by mutex

2015-08-31 Thread Nicolas Boichat
Hi Mark, On Mon, Aug 17, 2015 at 11:52 AM, Nicolas Boichat wrote: > chipselect (in the case of spi-gpio: spi_gpio_chipselect, which > calls gpiod_set_raw_value_cansleep) can sleep, so we should not > hold a spinlock while calling it from spi_bitbang_setup. > > This issue

[PATCH v4 02/16] drm: exynos/dp: convert to drm bridge mode

2015-08-31 Thread Yakir Yang
In order to move exynos dp code to bridge directory, we need to convert driver drm bridge mode first. As dp driver already have a ptn3460 bridge, so we need to move ptn bridge to the next bridge of dp bridge. Signed-off-by: Yakir Yang --- Changes in v4: - Take Romain

[PATCH v4 04/16] drm: bridge: analogix/dp: remove duplicate configuration of link rate and link count

2015-08-31 Thread Yakir Yang
link_rate and lane_count already configured in analogix_dp_set_link_train(), so we don't need to config those repeatly after training finished, just remove them out. Beside Display Port 1.2 already support 5.4Gbps link rate, the maximum sets would change from {1.62Gbps, 2.7Gbps} to {1.62Gbps,

[PATCH v4 05/16] drm: bridge: analogix/dp: dynamic parse sync_pol & interlace & colorimetry

2015-08-31 Thread Yakir Yang
Both hsync/vsync polarity and interlace mode can be parsed from drm display mode, and dynamic_range and ycbcr_coeff can be judge by the video code, same to color space and color depth can be parsed from EDID. But presumably Exynos still relaies on the DT properties, so take good use of

[PATCH v4 01/16] drm: exynos/dp: fix code style

2015-08-31 Thread Yakir Yang
After run "checkpatch.pl -f --subjective" command, I see there are lots of alignment problem in exynos_dp driver, so let just fix them. - Take Romain suggest, rebase on linux-next branch Signed-off-by: Yakir Yang --- Changes in v4: None Changes in v3: None Changes in v2: -

Re: [PATCH 05/14] exynos/fimg2d: check buffer space in g2d_copy()

2015-08-31 Thread Inki Dae
On 2015년 08월 24일 23:14, Tobias Jakobi wrote: > Move the parameter validation before buffer space checking > so that we can exit early if it fails. > Also don't reset the G2D context anymore in this situation > (since the buffers are not partially submitted). > > Signed-off-by: Tobias Jakobi

Re: [PATCH 10/14] exynos/fimg2d: remove default case from g2d_get_blend_op()

2015-08-31 Thread Inki Dae
On 2015년 08월 24일 23:14, Tobias Jakobi wrote: > We now validate the blending mode via g2d_validate_mode() > prior to feeding it to g2d_get_blend_op(). > > Signed-off-by: Tobias Jakobi > --- > exynos/exynos_fimg2d.c | 5 - > 1 file changed, 5 deletions(-) > >

Re: [PATCH 07/14] exynos/fimg2d: add g2d_validate_xyz() functions

2015-08-31 Thread Inki Dae
On 2015년 08월 24일 23:14, Tobias Jakobi wrote: > The G2D headers define a number of modes through enums > (like e.g. color, select, repeat, etc.). > > This introduces g2d_validate_select_mode() and > g2d_validate_blending_op() which validate a > select mode or blending operation respectively. > >

[PATCH] ARM: EXYNOS: reset KFC cores when cpu is up

2015-08-31 Thread Chanho Park
The cpu booting of exynos5422 has been still broken since we discussed it in last year[1]. This patch is inspired from odroid xu3 code(Actually, it was from samsung exynos vendor kernel)[2]. This weird reset code was founded exynos5420 octa cores series SoCs and only required for the first boot

Re: [PATCH 04/14] exynos/fimg2d: check buffer space in g2d_solid_fill()

2015-08-31 Thread Inki Dae
On 2015년 08월 24일 23:13, Tobias Jakobi wrote: > The amount of commands (regular and GEM) doesn't depend > on the input here. > > Signed-off-by: Tobias Jakobi > --- > exynos/exynos_fimg2d.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git

Re: [PATCH 08/14] exynos/fimg2d: check buffer space in g2d_blend()

2015-08-31 Thread Inki Dae
On 2015년 08월 24일 23:14, Tobias Jakobi wrote: > Move parameter validation to the top and also validate > the select mode of the source image and the requested > blending operation before starting command submission. > > Signed-off-by: Tobias Jakobi > --- >

Re: [PATCH 09/14] exynos/fimg2d: check buffer space in g2d_scale_and_blend()

2015-08-31 Thread Inki Dae
On 2015년 08월 24일 23:14, Tobias Jakobi wrote: > Apply the same transformation as in g2d_blend(). > > Signed-off-by: Tobias Jakobi > --- > exynos/exynos_fimg2d.c | 67 > +- > 1 file changed, 39 insertions(+), 28