From mboxrd@z Thu Jan 1 00:00:00 1970 From: Krzysztof Kozlowski Subject: Re: [PATCH RESEND] ARM: exynos_defconfig: Enable options for display panel support Date: Mon, 05 Jan 2015 09:48:11 +0100 Message-ID: <1420447691.3918.2.camel@AMDC1943> References: <1420215089-2093-1-git-send-email-javier.martinez@collabora.co.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from mailout3.w1.samsung.com ([210.118.77.13]:31520 "EHLO mailout3.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753283AbbAEIsU (ORCPT ); Mon, 5 Jan 2015 03:48:20 -0500 In-reply-to: <1420215089-2093-1-git-send-email-javier.martinez@collabora.co.uk> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Javier Martinez Canillas Cc: Kukjin Kim , Doug Anderson , Kevin Hilman , Olof Johansson , linux-samsung-soc@vger.kernel.org, Vivek Gautam , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org On pi=C4=85, 2015-01-02 at 17:11 +0100, Javier Martinez Canillas wrote: > Many Exynos devices have a display panel. Most of them just have > a simple panel while others have more complex configurations that > requires an embedded DisplayPort (eDP) to LVDS bridges. >=20 > This patch enables the following features to be built in the kernel > image to support both setups: >=20 > - Direct Rendering Manager (DRM) > - DRM bridge registration and lookup framework > - Parade ps8622/ps8625 eDP/LVDS bridge > - NXP ptn3460 eDP/LVDS bridge > - Exynos Fully Interactive Mobile Display controller (FIMD) > - Panel registration and lookup framework > - Simple panels > - Backlight & LCD device support >=20 > Signed-off-by: Javier Martinez Canillas > Tested-by: Kevin Hilman > Signed-off-by: Kukjin Kim > --- >=20 > Hello Kukjin, >=20 > You dropped this patch since exynos drm was causing boot hangs on som= e > platforms but the fix for that issue is already in linux-next (commit= : > f1e9203 clk: samsung: Fix Exynos 5420 pinctrl setup and clock disable > failure due to domain being gated) so I think it makes sense to enabl= e > the display options again. I think these issues were unrelated to each other. DRM was disabled because of infinite deferred probing of DRM components. I saw Inki was sending some patches for it but I am not sure if they were merged. Best regards, Krzysztof >=20 > NOTE: Display panel is still not working since patch "arm: dts: Exyno= s5: > Use pmu_system_controller phandle for dp phy" is needed [0] but I thi= nk > we should enable display options to catch the regressions easier. >=20 > Best regards, > Javier >=20 > [0]: https://lkml.org/lkml/2014/11/24/10 >=20 > arch/arm/configs/exynos_defconfig | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) >=20 > diff --git a/arch/arm/configs/exynos_defconfig b/arch/arm/configs/exy= nos_defconfig > index 22beed3..27cdd52 100644 > --- a/arch/arm/configs/exynos_defconfig > +++ b/arch/arm/configs/exynos_defconfig > @@ -111,11 +111,26 @@ CONFIG_REGULATOR_S2MPA01=3Dy > CONFIG_REGULATOR_S2MPS11=3Dy > CONFIG_REGULATOR_S5M8767=3Dy > CONFIG_REGULATOR_TPS65090=3Dy > +CONFIG_DRM=3Dy > +CONFIG_DRM_BRIDGE=3Dy > +CONFIG_DRM_PTN3460=3Dy > +CONFIG_DRM_PS8622=3Dy > +CONFIG_DRM_EXYNOS=3Dy > +CONFIG_DRM_EXYNOS_FIMD=3Dy > +CONFIG_DRM_EXYNOS_DP=3Dy > +CONFIG_DRM_PANEL=3Dy > +CONFIG_DRM_PANEL_SIMPLE=3Dy > CONFIG_FB=3Dy > CONFIG_FB_MODE_HELPERS=3Dy > CONFIG_FB_SIMPLE=3Dy > CONFIG_EXYNOS_VIDEO=3Dy > CONFIG_EXYNOS_MIPI_DSI=3Dy > +CONFIG_BACKLIGHT_LCD_SUPPORT=3Dy > +CONFIG_LCD_CLASS_DEVICE=3Dy > +CONFIG_LCD_PLATFORM=3Dy > +CONFIG_BACKLIGHT_CLASS_DEVICE=3Dy > +CONFIG_BACKLIGHT_GENERIC=3Dy > +CONFIG_BACKLIGHT_PWM=3Dy > CONFIG_FRAMEBUFFER_CONSOLE=3Dy > CONFIG_FONTS=3Dy > CONFIG_FONT_7x14=3Dy From mboxrd@z Thu Jan 1 00:00:00 1970 From: k.kozlowski@samsung.com (Krzysztof Kozlowski) Date: Mon, 05 Jan 2015 09:48:11 +0100 Subject: [PATCH RESEND] ARM: exynos_defconfig: Enable options for display panel support In-Reply-To: <1420215089-2093-1-git-send-email-javier.martinez@collabora.co.uk> References: <1420215089-2093-1-git-send-email-javier.martinez@collabora.co.uk> Message-ID: <1420447691.3918.2.camel@AMDC1943> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On pi?, 2015-01-02 at 17:11 +0100, Javier Martinez Canillas wrote: > Many Exynos devices have a display panel. Most of them just have > a simple panel while others have more complex configurations that > requires an embedded DisplayPort (eDP) to LVDS bridges. > > This patch enables the following features to be built in the kernel > image to support both setups: > > - Direct Rendering Manager (DRM) > - DRM bridge registration and lookup framework > - Parade ps8622/ps8625 eDP/LVDS bridge > - NXP ptn3460 eDP/LVDS bridge > - Exynos Fully Interactive Mobile Display controller (FIMD) > - Panel registration and lookup framework > - Simple panels > - Backlight & LCD device support > > Signed-off-by: Javier Martinez Canillas > Tested-by: Kevin Hilman > Signed-off-by: Kukjin Kim > --- > > Hello Kukjin, > > You dropped this patch since exynos drm was causing boot hangs on some > platforms but the fix for that issue is already in linux-next (commit: > f1e9203 clk: samsung: Fix Exynos 5420 pinctrl setup and clock disable > failure due to domain being gated) so I think it makes sense to enable > the display options again. I think these issues were unrelated to each other. DRM was disabled because of infinite deferred probing of DRM components. I saw Inki was sending some patches for it but I am not sure if they were merged. Best regards, Krzysztof > > NOTE: Display panel is still not working since patch "arm: dts: Exynos5: > Use pmu_system_controller phandle for dp phy" is needed [0] but I think > we should enable display options to catch the regressions easier. > > Best regards, > Javier > > [0]: https://lkml.org/lkml/2014/11/24/10 > > arch/arm/configs/exynos_defconfig | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/arch/arm/configs/exynos_defconfig b/arch/arm/configs/exynos_defconfig > index 22beed3..27cdd52 100644 > --- a/arch/arm/configs/exynos_defconfig > +++ b/arch/arm/configs/exynos_defconfig > @@ -111,11 +111,26 @@ CONFIG_REGULATOR_S2MPA01=y > CONFIG_REGULATOR_S2MPS11=y > CONFIG_REGULATOR_S5M8767=y > CONFIG_REGULATOR_TPS65090=y > +CONFIG_DRM=y > +CONFIG_DRM_BRIDGE=y > +CONFIG_DRM_PTN3460=y > +CONFIG_DRM_PS8622=y > +CONFIG_DRM_EXYNOS=y > +CONFIG_DRM_EXYNOS_FIMD=y > +CONFIG_DRM_EXYNOS_DP=y > +CONFIG_DRM_PANEL=y > +CONFIG_DRM_PANEL_SIMPLE=y > CONFIG_FB=y > CONFIG_FB_MODE_HELPERS=y > CONFIG_FB_SIMPLE=y > CONFIG_EXYNOS_VIDEO=y > CONFIG_EXYNOS_MIPI_DSI=y > +CONFIG_BACKLIGHT_LCD_SUPPORT=y > +CONFIG_LCD_CLASS_DEVICE=y > +CONFIG_LCD_PLATFORM=y > +CONFIG_BACKLIGHT_CLASS_DEVICE=y > +CONFIG_BACKLIGHT_GENERIC=y > +CONFIG_BACKLIGHT_PWM=y > CONFIG_FRAMEBUFFER_CONSOLE=y > CONFIG_FONTS=y > CONFIG_FONT_7x14=y