public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Yakir Yang <ykk@rock-chips.com>
To: "Stéphane Marchesin" <stephane.marchesin@gmail.com>
Cc: David Airlie <airlied@linux.ie>,
	Thierry Reding <thierry.reding@gmail.com>,
	Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>,
	Mark Rutland <mark.rutland@arm.com>,
	Kumar Gala <galak@codeaurora.org>,
	Ian Campbell <ijc+devicetree@hellion.org.uk>,
	devicetree@vger.kernel.org,
	Linux Kernel list <linux-kernel@vger.kernel.org>,
	"dri-devel@lists.freedesktop.org"
	<dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH v2 4/6] drm/panel: simple: Add support for Samsung LSN122DL01-C01 2560x1600 panel
Date: Sun, 12 Jun 2016 10:05:21 +0800	[thread overview]
Message-ID: <575CC361.4080003@rock-chips.com> (raw)
In-Reply-To: <CACP_E+KFDF2VoXo4pRkAmd9wTcy7a6NTK9YpaPbtMD8=2f7e7Q@mail.gmail.com>

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@rock-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@rock-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@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
>

  reply	other threads:[~2016-06-12  2:05 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-08 11:52 [PATCH v2 1/6] dt-bindings: add LG LP097QX1-SPA1 panel binding Yakir Yang
2016-06-08 11:52 ` [PATCH v2 2/6] drm/panel: simple: Add support for LG LP097QX1-SPA1 2048x1536 panel Yakir Yang
2016-06-08 12:20   ` Emil Velikov
2016-06-10 13:26     ` Thierry Reding
2016-06-12  2:09     ` Yakir Yang
2016-06-08 11:52 ` [PATCH v2 3/6] dt-bindings: add Samsung LSN122DL01-C01 panel binding Yakir Yang
2016-06-08 11:52 ` [PATCH v2 4/6] drm/panel: simple: Add support for Samsung LSN122DL01-C01 2560x1600 panel Yakir Yang
2016-06-10  0:04   ` Stéphane Marchesin
2016-06-12  2:05     ` Yakir Yang [this message]
2016-06-08 11:52 ` [PATCH v2 5/6] dt-bindings: add Sharp LQ123P1JX31 panel binding Yakir Yang
2016-06-08 16:31   ` Doug Anderson
2016-06-10 13:42   ` Rob Herring
2016-06-10 16:40     ` Doug Anderson
2016-06-10 17:54       ` Rob Herring
2016-06-10 18:08         ` Doug Anderson
2016-06-12  2:14       ` Yakir Yang
2016-06-08 11:52 ` [PATCH v2 6/6] drm/panel: simple: Add support for Sharp LQ123P1JX31 2400x1600 panel Yakir Yang
2016-06-08 16:35   ` Doug Anderson
2016-06-10 16:11     ` Doug Anderson
2016-06-12  2:12       ` Yakir Yang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=575CC361.4080003@rock-chips.com \
    --to=ykk@rock-chips.com \
    --cc=airlied@linux.ie \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=galak@codeaurora.org \
    --cc=ijc+devicetree@hellion.org.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=pawel.moll@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=stephane.marchesin@gmail.com \
    --cc=thierry.reding@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox