Re: [PATCH v4 14/16] drm: bridge: analogix/dp: try force hpd after plug in lookup failed

On Thu, Sep 03, 2015 at 11:04:40AM +0200, Thierry Reding wrote: > Conversely, if the panel isn't capable of generating an HPD signal, then > I don't think it would be appropriate to make it a DT property. It would > be better to hard-code it in the driver, lest someone forget to set the > property

Re: [PATCH] drm/exynos: add cursor plane support

On 2015년 09월 04일 16:19, Daniel Vetter wrote: > On Fri, Sep 04, 2015 at 01:05:35PM +0900, Inki Dae wrote: >> Hi Gustavo, >> >> I had already a review but I didn't give any comment to you. Sorry about >> that. This patch looks good to me but one thing isn't clear. Below is my >> comment. >> >> >> On

[PATCH 05/11] clk: samsung: exynos7: Corrects CMU_PERIC0 clocks names

This patch rename CMU_PERIC0 clocks names to match with user manual. And also adds missing gate clock for aclk_peric0_66. Signed-off-by: Alim Akhtar --- drivers/clk/samsung/clk-exynos7.c | 12 include/dt-bindings/clock/exynos7-clk.h |3 ++- 2

[PATCH 06/11] clk: samsung: exynos7: Corrects CMU_PERIC1 clocks names

This patch rename CMU_PERIC1 clocks names to match with user manual. And also adds missing gate clock for aclk_peric1_66. Signed-off-by: Alim Akhtar --- drivers/clk/samsung/clk-exynos7.c | 38 ---

[PATCH 07/11] clk: samsung: exynos7: Corrects CMU_PERIS clocks names

This patch rename CMU_PERIS clocks names to match with user manual. And also adds missing gate clock for aclk_peris_66. Signed-off-by: Alim Akhtar --- drivers/clk/samsung/clk-exynos7.c |7 +-- include/dt-bindings/clock/exynos7-clk.h |3 ++- 2 files

[PATCH 08/11] clk: samsung: exynos7: Corrects CMU_FSYS0 clocks names

This patch rename CMU_FSYS0 clocks names to match with user manual. And also adds missing gate clock for aclk_fsys0_200. Signed-off-by: Alim Akhtar --- drivers/clk/samsung/clk-exynos7.c | 24 ++-- include/dt-bindings/clock/exynos7-clk.h |

[PATCH 09/11] clk: samsung: exynos7: Corrects CMU_FSYS1 clocks names

This patch rename CMU_FSYS1 clocks names to match with user manual. And also adds missing gate clock for aclk_fsys1_200. Signed-off-by: Alim Akhtar --- drivers/clk/samsung/clk-exynos7.c | 16 ++-- include/dt-bindings/clock/exynos7-clk.h |3 ++- 2

[PATCH 10/11] clk: samsung: exynos7: Add missing fixed_clks to cmu_info

FSYS0 fixed clocks are not added to fsys0_cmu_info, this make some of the usb clocks as orphan. This fixes the same. Signed-off-by: Alim Akhtar --- drivers/clk/samsung/clk-exynos7.c |2 ++ 1 file changed, 2 insertions(+) diff --git

[PATCH 04/11] clk: samsung: exynos7: Corrects CMU_CCORE clocks names

This patch rename CMU_CCROE clocks names to match with user manual. And also adds missing gate clock for aclk_ccore_133. Signed-off-by: Alim Akhtar --- drivers/clk/samsung/clk-exynos7.c |8 ++-- include/dt-bindings/clock/exynos7-clk.h |3 ++- 2 files

[PATCH 02/11] clk: samsung: exynos7: Corrects CMU_TOP0 clocks names

This patch rename CMU_TOP0 clocks names to match with user manual. Signed-off-by: Alim Akhtar --- drivers/clk/samsung/clk-exynos7.c | 67 - 1 file changed, 37 insertions(+), 30 deletions(-) diff --git

[PATCH 03/11] clk: samsung: exynos7: Corrects CMU_TOP1 clocks names

This patch rename CMU_TOP1 clocks names to match with user manual. Signed-off-by: Alim Akhtar --- drivers/clk/samsung/clk-exynos7.c | 50 - 1 file changed, 27 insertions(+), 23 deletions(-) diff --git

Re: [PATCH v4 14/16] drm: bridge: analogix/dp: try force hpd after plug in lookup failed

On Thu, Sep 03, 2015 at 04:55:59PM -0500, Rob Herring wrote: > On Thu, Sep 3, 2015 at 3:47 AM, Thierry Reding wrote: > > On Wed, Sep 02, 2015 at 03:17:57PM -0500, Rob Herring wrote: > > [...] > >> Are there any eDP panels which don't have EDID and need panel details in > >>

[PATCH 00/11] clk: samsung: exynos7: Cleanup of clock file

Currently there are lots of ambiguity between clock names in clock file and in user manual, which leads to lots of confusion for the reviewers. This series attempts to cleanup the exynos7 clock file as per user manual naming convention. This also adds some of the missing Gate clocks which are

[PATCH 01/11] clk: samsung: exynos7: Change the CMU_TOPC block clock name

Corrects the CMU_TOPC block clock name as per user manual. This also adds few of the missing gate clocks of topc block. This does not change any functionalies. Signed-off-by: Alim Akhtar --- drivers/clk/samsung/clk-exynos7.c | 90 +++

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

Adding required mux/div/gate clocks for UFS controller present on Exynos7. Signed-off-by: Alim Akhtar --- drivers/clk/samsung/clk-exynos7.c | 116 +++ include/dt-bindings/clock/exynos7-clk.h | 24 ++- 2 files changed, 138

Re: [PATCH v4 03/16] drm: bridge: analogix/dp: split exynos dp driver to bridge dir

On Tue, Sep 1, 2015 at 3:46 PM, Heiko Stuebner wrote: > Am Dienstag, 1. September 2015, 13:49:58 schrieb Yakir Yang: >> Split the dp core driver from exynos directory to bridge >> directory, and rename the core driver to analogix_dp_*, >> leave the platform code to

[PATCH 03/10] drm/exynos: add pm_runtime to HDMI

From: Gustavo Padovan Let pm_runtime handle the enabling/disabling of the device with proper refcnt instead of rely on specific flags to track the enabled state. Signed-off-by: Gustavo Padovan ---

[PATCH 01/10] drm/exynos: do not start enabling DP at bind() phase

From: Gustavo Padovan The DP device will be properly enabled at the enable() call just after the bind call finishes. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/exynos/exynos_dp_core.c | 4 1 file changed, 4

[PATCH 02/10] drm/exynos: add pm_runtime to DP

From: Gustavo Padovan Let pm_runtime handle the enabling/disabling of the device with proper refcnt instead of rely on specific flags to track the enabled state. Signed-off-by: Gustavo Padovan ---

[PATCH 00/10] drm/exynos: add pm_runtime support

From: Gustavo Padovan Hi, This series adds proper runtime PM suport to CRTCs and Encoders, so now instead of relying on 'suspended' or 'enabled' flags to track when the CRTC or Encoder is enabled we let the pm_runtime subsystem do it for us and remove all the

[PATCH 04/10] drm/exynos: add pm_runtime to Mixer

From: Gustavo Padovan Let pm_runtime handle the enabling/disabling of the device with proper refcnt instead of rely on specific flags to track the enabled state. Signed-off-by: Gustavo Padovan ---

[PATCH 06/10] drm/exynos: Remove exynos_crtc commit() callback

From: Gustavo Padovan This callback is no longer used by any of the exynos_crtc drivers, remove it. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/exynos/exynos_drm_crtc.c | 10 --

[PATCH 07/10] drm/exynos: add pm_runtime to FIMD

From: Gustavo Padovan Let pm_runtime handle the enabling/disabling of the device with proper refcnt instead of rely on specific flags to track the enabled state. Signed-off-by: Gustavo Padovan ---

[PATCH 05/10] drm/exynos: remove exynos_crtc commit() callback

From: Gustavo Padovan It turns out that .commit() was never executed, because at the time .mode_set_nofb() called it ctx->suspended was still false and .commit() would return. It removes the callback from FIMD DECON 7 and DECON 5433. Signed-off-by: Gustavo

[PATCH 08/10] drm/exynos: Enable DP clock directly from FIMD

From: Gustavo Padovan Instead of having a .clock_enable callback enable the dp clock directly from FIMD. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/exynos/exynos_dp_core.c | 13 ---

[PATCH 09/10] drm/exynos: add pm_runtime to DECON 5433

From: Gustavo Padovan Let pm_runtime handle the enabling/disabling of the device with proper refcnt instead of rely on specific flags to track the enabled state. Signed-off-by: Gustavo Padovan ---

[PATCH 10/10] drm/exynos: add pm_runtime to DECON 7

From: Gustavo Padovan Let pm_runtime handle the enabling/disabling of the device with proper refcnt instead of rely on specific flags to track the enabled state. Signed-off-by: Gustavo Padovan ---

Re: [PATCH v4 14/16] drm: bridge: analogix/dp: try force hpd after plug in lookup failed

On Wed, Sep 2, 2015 at 11:27 PM, Yakir Yang wrote: > Hi Rob, > > 在 09/03/2015 04:17 AM, Rob Herring 写道: >> >> On Tue, Sep 1, 2015 at 1:14 AM, Yakir Yang wrote: >>> >>> Some edp screen do not have hpd signal, so we can't just return >>> failed when hpd

Re: [PATCH] drm/exynos: add cursor plane support

Hi Inki, 2015-09-04 Inki Dae : > Hi Gustavo, > > I had already a review but I didn't give any comment to you. Sorry about > that. This patch looks good to me but one thing isn't clear. Below is my > comment. > > > On 2015년 09월 04일 05:14, Gustavo Padovan wrote: > > From:

Re: [PATCH] arm:irqchip: IRQCHIP_DECLARE macro is now accessible

On Monday, July 13, 2015 11:20:35 PM Shawn Guo wrote: > On Tue, Jul 07, 2015 at 04:02:53PM -0400, Joel Porquet wrote: > > The IRQCHIP_DECLARE macro migrated to 'include/linux/irqchip.h', making it > > globally accessible. > > > > See commit 91e20b5040c67c51aad88cf87db4305c5bd7f79d > > ("irqchip:

Re: [PATCH v4 03/16] drm: bridge: analogix/dp: split exynos dp driver to bridge dir

Am Freitag, 4. September 2015, 16:06:02 schrieb Rob Herring: > On Tue, Sep 1, 2015 at 3:46 PM, Heiko Stuebner wrote: > > Am Dienstag, 1. September 2015, 13:49:58 schrieb Yakir Yang: > >> Split the dp core driver from exynos directory to bridge > >> directory, and rename the core

[PATCH 2/2] drm/exynos: add cursor plane support

From: Gustavo Padovan Set one of the planes for each crtc driver as a cursor plane enabled window managers to fully work on exynos. Signed-off-by: Gustavo Padovan --- v2: use the top window for cursor on each crtc ---

[PATCH 1/2] drm/exynos: add global macro for the default primary plane

From: Gustavo Padovan Define DEFAULT_WIN as zero to help set the primary plane on all CRTCs. Some CRTCs were defining a variable to store the default window, but that is not necessary as the default (primary) window is always the window zero. Signed-off-by:

Re: [PATCH] drm/exynos: add cursor plane support

On Fri, Sep 04, 2015 at 01:05:35PM +0900, Inki Dae wrote: > Hi Gustavo, > > I had already a review but I didn't give any comment to you. Sorry about > that. This patch looks good to me but one thing isn't clear. Below is my > comment. > > > On 2015년 09월 04일 05:14, Gustavo Padovan wrote: > >

Re: [PATCHv7] ARM: exynos_defconfig: Enable LEDS for Odroid-XU3/XU4

On 04.09.2015 15:20, Anand Moon wrote: > Hi All, > > On 4 September 2015 at 05:56, Javier Martinez Canillas > wrote: >> Hello Krzysztof, >> >> On 09/04/2015 01:55 AM, Krzysztof Kozlowski wrote: >>> On 04.09.2015 03:11, Anand Moon wrote: Earlier design of the LED for

Re: [PATCHv7] ARM: exynos_defconfig: Enable LEDS for Odroid-XU3/XU4

Hi All, On 4 September 2015 at 05:56, Javier Martinez Canillas wrote: > Hello Krzysztof, > > On 09/04/2015 01:55 AM, Krzysztof Kozlowski wrote: >> On 04.09.2015 03:11, Anand Moon wrote: >>> Earlier design of the LED for Odroid XU3 was using gpio-leds >>> Now It was change

Re: [PATCH] ARM: exynos_defconfig: Make S3C2410_WATCHDOG as loadable module

On 03.09.2015 14:20, Anand Moon wrote: > Hi Krzysztof > > On 3 September 2015 at 10:32, Krzysztof Kozlowski > wrote: >> On 03.09.2015 13:10, Anand Moon wrote: >>> S3C2410_WATCHDOG watchdog drivers should not be loaded automatically, >>> but only if a watchdog daemon is

Re: [PATCHv7] ARM: exynos_defconfig: Enable LEDS for Odroid-XU3/XU4

Hi Krzysztof, On 4 September 2015 at 12:18, Krzysztof Kozlowski wrote: > On 04.09.2015 15:20, Anand Moon wrote: >> Hi All, >> >> On 4 September 2015 at 05:56, Javier Martinez Canillas >> wrote: >>> Hello Krzysztof, >>> >>> On 09/04/2015 01:55 AM,

[PATCH V3 3/6] PM / OPP: Prefix exported opp routines with dev_pm_opp_

That's the naming convention followed in most of opp core, but few routines didn't follow this, fix them. Reviewed-by: Stephen Boyd Signed-off-by: Viresh Kumar --- arch/arm/mach-imx/mach-imx6q.c | 2 +- drivers/base/power/opp.c

[PATCH V3 2/6] PM / OPP: Rename opp init/free table routines

free-table routines are opposite of init-table ones, and must be named to make that clear. Opposite of 'init' is 'exit', but those doesn't suit really well. Replace 'init' with 'add' and 'free' with 'remove'. Reported-by: Pavel Machek Reviewed-by: Stephen Boyd