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
>
>
next prev parent 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