Re: [PATCH v2 2/2] drm/panel: simple: Add support BOE nv101wxmn51

[Caesar Wang]

å 2016å12æ13æ 04:22, StÃphane Marchesin åé:
> On Wed, Dec 7, 2016 at 11:26 PM, Caesar Wang <wxt@xxxxxxxxxxxxxx> 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@xxxxxxxxxxxxxx>
> > ---
> >
> > 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?

Yup, I will add them for next version.
Thanks.

-Caesar
> 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@xxxxxxxxxxxxxxxxxxxxx
> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> _______________________________________________
> Linux-rockchip mailing list
> Linux-rockchip@xxxxxxxxxxxxxxxxxxx
> http://lists.infradead.org/mailman/listinfo/linux-rockchip