[PATCH v2 2/2] drm/panel: Support for LG lp120up1 panel with eDP input

2016-03-02 Thread Thierry Reding
On Mon, Feb 22, 2016 at 07:01:44PM +0800, Jitao Shi wrote: > The LG lp120up1 TFT LCD panel with eDP interface is a 12.0" 1920x1280 > panel, which can be supported by the simple panel driver > > Signed-off-by: Jitao Shi > --- > Changes since v1: > - Add eDP panel type in comment msg > - Fixed co

[PATCH v2 2/2] drm/panel: Support for LG lp120up1 panel with eDP input

2016-02-23 Thread Thierry Reding
On Tue, Feb 23, 2016 at 10:45:01AM +0530, sanjeev sharma wrote: > Hello Thierry, > > I am not agree here and how it would be redundant ? The panel being a separate device means that it needs a compatible value and the compatible value implies a specific video timing. It would be redundant, theref

[PATCH v2 2/2] drm/panel: Support for LG lp120up1 panel with eDP input

2016-02-23 Thread sanjeev sharma
Hello Thierry, I am not agree here and how it would be redundant ? please see lvds interface, where panel timing parameter is defined in the Device Tree. If there are multiple display panel then we can create a separate dtsi file and include in main Board dts file. I believe that, It is simpler w

[PATCH v2 2/2] drm/panel: Support for LG lp120up1 panel with eDP input

2016-02-22 Thread Jitao Shi
The LG lp120up1 TFT LCD panel with eDP interface is a 12.0" 1920x1280 panel, which can be supported by the simple panel driver Signed-off-by: Jitao Shi --- Changes since v1: - Add eDP panel type in comment msg - Fixed comment msg with 72 characters width --- drivers/gpu/drm/panel/panel-simple.

[PATCH v2 2/2] drm/panel: Support for LG lp120up1 panel with eDP input

2016-02-22 Thread sanjeev sharma
Hello Jitao, Can't we add this panel information in device tree file instead inside the device driver ? Regards Sanjeev Sharma On Mon, Feb 22, 2016 at 4:31 PM, Jitao Shi wrote: > The LG lp120up1 TFT LCD panel with eDP interface is a 12.0" 1920x1280 > panel, which can be supported by the simple

[PATCH v2 2/2] drm/panel: Support for LG lp120up1 panel with eDP input

2016-02-22 Thread Thierry Reding
On Mon, Feb 22, 2016 at 04:42:54PM +0530, sanjeev sharma wrote: > Hello Jitao, > > Can't we add this panel information in device tree file instead inside the > device driver ? We could, but that would be redundant. Panels need to be represented by a specific compatible string anyway, and that com