Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1752951AbcLLUXm (ORCPT <rfc822;w@1wt.eu>);
        Mon, 12 Dec 2016 15:23:42 -0500
Received: from mail-wm0-f66.google.com ([74.125.82.66]:36860 "EHLO
        mail-wm0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1751028AbcLLUXk (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Mon, 12 Dec 2016 15:23:40 -0500
MIME-Version: 1.0
In-Reply-To: <1481181996-27922-2-git-send-email-wxt@rock-chips.com>
References: <1481181996-27922-1-git-send-email-wxt@rock-chips.com> <1481181996-27922-2-git-send-email-wxt@rock-chips.com>
From: =?UTF-8?Q?St=C3=A9phane_Marchesin?= <stephane.marchesin@gmail.com>
Date: Mon, 12 Dec 2016 12:22:58 -0800
Message-ID: <CACP_E+K1_c2ryUOKgqOvj9oJp6EKUc+44J0AiFNAU_C=7WJFSw@mail.gmail.com>
Subject: Re: [PATCH v2 2/2] drm/panel: simple: Add support BOE nv101wxmn51
To: Caesar Wang <wxt@rock-chips.com>
Cc: Thierry Reding <thierry.reding@gmail.com>,
        linux-rockchip@lists.infradead.org,
        Linux Kernel list <linux-kernel@vger.kernel.org>,
        "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>
Content-Type: text/plain; charset=UTF-8
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id uBCKNpW2028326
Content-Length: 3326
Lines: 108

On Wed, Dec 7, 2016 at 11:26 PM, Caesar Wang <wxt@rock-chips.com> wrote:
> 10.1WXGA is a color active matrix TFT LCD module using amorphous silicon
> TFT's as an active switching devices. It can be supported by the
> simple-panel driver.
>
> Read the panel edid information;
>
> EDID MODE DETAILS
>                 name = <NULL>
>                 pixel_clock = 71900
>                 lvds_dual_channel = 0
>                 refresh = 0
>                 ha = 1280
>                 hbl = 160
>                 hso = 48
>                 hspw = 32
>                 hborder = 0
>                 va = 800
>                 vbl = 32
>                 vso = 3
>                 vspw = 5
>                 vborder = 0
>                 phsync = +
>                 pvsync = -
>                 x_mm = 0
>                 y_mm = 0
> drm_display_mode
>                 .hdisplay = 1280
>                 .hsync_start = 1328
>                 .hsync_end = 1360
>                 .htotal = 1440
>                 .vdisplay = 800
>                 .vsync_start = 803
>                 .vsync_end = 808
>                 .vtotal = 832
>
> Signed-off-by: Caesar Wang <wxt@rock-chips.com>
> ---
>
> Changes in v2:
> - fix the vsync_start and vsync_end from the edid.
> - change the commit.
>
>  drivers/gpu/drm/panel/panel-simple.c | 31 +++++++++++++++++++++++++++++++
>  1 file changed, 31 insertions(+)
>
> diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
> index 06aaf79..7c90f16 100644
> --- a/drivers/gpu/drm/panel/panel-simple.c
> +++ b/drivers/gpu/drm/panel/panel-simple.c
> @@ -668,6 +668,34 @@ static const struct panel_desc avic_tm070ddh03 = {
>         },
>  };
>
> +static const struct drm_display_mode boe_nv101wxmn51_mode = {
> +       .clock = 71900,
> +       .hdisplay = 1280,
> +       .hsync_start = 1280 + 48,
> +       .hsync_end = 1280 + 48 + 32,
> +       .htotal = 1280 + 48 + 32 + 80,
> +       .vdisplay = 800,
> +       .vsync_start = 800 + 3,
> +       .vsync_end = 800 + 3 + 5,
> +       .vtotal = 800 + 3 + 5 + 24,
> +       .vrefresh = 60,
> +};
> +
> +static const struct panel_desc boe_nv101wxmn51 = {
> +       .modes = &boe_nv101wxmn51_mode,
> +       .num_modes = 1,

There are two modes in the EDID (there is a downclock one). Can you
add both modes?

Stéphane

> +       .bpc = 8,
> +       .size = {
> +               .width = 217,
> +               .height = 136,
> +       },
> +       .delay = {
> +               .prepare = 210,
> +               .enable = 50,
> +               .unprepare = 160,
> +       },
> +};
> +
>  static const struct drm_display_mode chunghwa_claa070wp03xg_mode = {
>         .clock = 66770,
>         .hdisplay = 800,
> @@ -1748,6 +1776,9 @@ static const struct of_device_id platform_of_match[] = {
>                 .compatible = "avic,tm070ddh03",
>                 .data = &avic_tm070ddh03,
>         }, {
> +               .compatible = "boe,nv101wxmn51",
> +               .data = &boe_nv101wxmn51,
> +       }, {
>                 .compatible = "chunghwa,claa070wp03xg",
>                 .data = &chunghwa_claa070wp03xg,
>         }, {
> --
> 2.7.4
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel