2014-10-31 23:42:15

by Javier Martinez Canillas

[permalink] [raw]
Subject: Re: [PATCH v2 1/1] ARM: exynos_defconfig: Enable options for display panel support

Hello Kukjin,

On Mon, Aug 25, 2014 at 10:45 AM, Javier Martinez Canillas
<[email protected]> 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 suport 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 <[email protected]>
> Tested-by: Kevin Hilman <[email protected]>
> ---

This patch is needed to have display working on many Exynos boards.

$subject enables the config option for the ps8622/ps8625 eDP/LVDS
bridge driver but this has not landed in mainline yet. Ajay will
re-spin a new revision of his series that adds this driver though,
after addressing some issues pointed out on a previous version.

The Kconfig symbol name (DRM_PS8622) won't change though so maybe is
not crazy to pick the patch as is since CONFIG_DRM_PS8622 will have to
be enabled later when the bridge driver lands. Or do you want me to
re-spin $subject removing that option?

Best regards,
Javier


2014-11-08 09:46:53

by Kukjin Kim

[permalink] [raw]
Subject: RE: [PATCH v2 1/1] ARM: exynos_defconfig: Enable options for display panel support

Javier Martinez Canillas wrote:
>
> Hello Kukjin,
>
Hi,

> On Mon, Aug 25, 2014 at 10:45 AM, Javier Martinez Canillas
> <[email protected]> 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 suport 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 <[email protected]>
> > Tested-by: Kevin Hilman <[email protected]>
> > ---
>
> This patch is needed to have display working on many Exynos boards.
>
OK, I'll apply this into samsung tree in this weekend.

Thanks for your gentle reminder.

- Kukjin

> $subject enables the config option for the ps8622/ps8625 eDP/LVDS
> bridge driver but this has not landed in mainline yet. Ajay will
> re-spin a new revision of his series that adds this driver though,
> after addressing some issues pointed out on a previous version.
>
> The Kconfig symbol name (DRM_PS8622) won't change though so maybe is
> not crazy to pick the patch as is since CONFIG_DRM_PS8622 will have to
> be enabled later when the bridge driver lands. Or do you want me to
> re-spin $subject removing that option?
>
> Best regards,
> Javier