[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <575CC361.4080003@rock-chips.com>
Date: Sun, 12 Jun 2016 10:05:21 +0800
From: Yakir Yang <ykk@...k-chips.com>
To: Stéphane Marchesin <stephane.marchesin@...il.com>
Cc: David Airlie <airlied@...ux.ie>,
Thierry Reding <thierry.reding@...il.com>,
Rob Herring <robh+dt@...nel.org>,
Pawel Moll <pawel.moll@....com>,
Mark Rutland <mark.rutland@....com>,
Kumar Gala <galak@...eaurora.org>,
Ian Campbell <ijc+devicetree@...lion.org.uk>,
devicetree@...r.kernel.org,
Linux Kernel list <linux-kernel@...r.kernel.org>,
"dri-devel@...ts.freedesktop.org" <dri-devel@...ts.freedesktop.org>
Subject: Re: [PATCH v2 4/6] drm/panel: simple: Add support for Samsung
LSN122DL01-C01 2560x1600 panel
Stéphane,
On 06/10/2016 08:04 AM, Stéphane Marchesin wrote:
> On Wed, Jun 8, 2016 at 4:52 AM, Yakir Yang <ykk@...k-chips.com> wrote:
>> The Samsung LSN122DL01-C01 is an 12.2" 2560x1600 (WQXGA) TFT-LCD panel
>> connected using eDP interfaces.
>>
>> Signed-off-by: Yakir Yang <ykk@...k-chips.com>
>> ---
>> Changes in v2: None
>>
>> drivers/gpu/drm/panel/panel-simple.c | 25 +++++++++++++++++++++++++
>> 1 file changed, 25 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
>> index 41020e1..067a5c4 100644
>> --- a/drivers/gpu/drm/panel/panel-simple.c
>> +++ b/drivers/gpu/drm/panel/panel-simple.c
>> @@ -1246,6 +1246,28 @@ static const struct panel_desc qd43003c0_40 = {
>> .bus_format = MEDIA_BUS_FMT_RGB888_1X24,
>> };
>>
>> +static const struct drm_display_mode samsung_lsn122dl01_c01_mode = {
>> + .clock = 271560,
>> + .hdisplay = 2560,
>> + .hsync_start = 2560 + 48,
>> + .hsync_end = 2560 + 48 + 32,
>> + .htotal = 2560 + 48 + 32 + 80,
>> + .vdisplay = 1600,
>> + .vsync_start = 1600 + 2,
>> + .vsync_end = 1600 + 2 + 5,
>> + .vtotal = 1600 + 2 + 5 + 57,
>> + .vrefresh = 60,
>> +};
>> +
>> +static const struct panel_desc samsung_lsn122dl01_c01 = {
>> + .modes = &samsung_lsn122dl01_c01_mode,
>> + .num_modes = 1,
>> + .size = {
>> + .width = 2560,
>> + .height = 1600,
> These are meant to be the physical dimensions (same thing for the
> other patches btw).
Got it, so this should be the "active area" in panel book.
Display area 262.656(H) X 164.16(V) (12.2”diagonal)
Thanks,
-Yakir
> Stéphane
>
>> + },
>> +};
>> +
>> static const struct drm_display_mode samsung_ltn101nt05_mode = {
>> .clock = 54030,
>> .hdisplay = 1024,
>> @@ -1506,6 +1528,9 @@ static const struct of_device_id platform_of_match[] = {
>> .compatible = "qiaodian,qd43003c0-40",
>> .data = &qd43003c0_40,
>> }, {
>> + .compatible = "samsung,lsn122dl01-c01",
>> + .data = &samsung_lsn122dl01_c01,
>> + }, {
>> .compatible = "samsung,ltn101nt05",
>> .data = &samsung_ltn101nt05,
>> }, {
>> --
>> 1.9.1
>>
>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel@...ts.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
>
Powered by blists - more mailing lists