linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2 1/2] dt-bindings: display: Add BOE nv101wxmn51 panel binding
@ 2016-12-08  7:26 Caesar Wang
  2016-12-08  7:26 ` [PATCH v2 2/2] drm/panel: simple: Add support BOE nv101wxmn51 Caesar Wang
  0 siblings, 1 reply; 4+ messages in thread
From: Caesar Wang @ 2016-12-08  7:26 UTC (permalink / raw)
  To: thierry.reding
  Cc: airlied, dri-devel, linux-kernel, linux-rockchip, Caesar Wang

The BOE 10.1" NV101WXMN51 panel is an WXGA TFT LCD panel.

Signed-off-by: Caesar Wang <wxt@rock-chips.com>
---

Changes in v2: None

 .../devicetree/bindings/display/panel/boe,nv101wxmn51.txt          | 7 +++++++
 1 file changed, 7 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/display/panel/boe,nv101wxmn51.txt

diff --git a/Documentation/devicetree/bindings/display/panel/boe,nv101wxmn51.txt b/Documentation/devicetree/bindings/display/panel/boe,nv101wxmn51.txt
new file mode 100644
index 0000000..b258d6a
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/panel/boe,nv101wxmn51.txt
@@ -0,0 +1,7 @@
+BOE OPTOELECTRONICS TECHNOLOGY 10.1" WXGA TFT LCD panel
+
+Required properties:
+- compatible: should be "boe,nv101wxmn51"
+
+This binding is compatible with the simple-panel binding, which is specified
+in simple-panel.txt in this directory.
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH v2 2/2] drm/panel: simple: Add support BOE nv101wxmn51
  2016-12-08  7:26 [PATCH v2 1/2] dt-bindings: display: Add BOE nv101wxmn51 panel binding Caesar Wang
@ 2016-12-08  7:26 ` Caesar Wang
  2016-12-12 20:22   ` Stéphane Marchesin
  0 siblings, 1 reply; 4+ messages in thread
From: Caesar Wang @ 2016-12-08  7:26 UTC (permalink / raw)
  To: thierry.reding
  Cc: airlied, dri-devel, linux-kernel, linux-rockchip, Caesar Wang

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,
+	.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

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH v2 2/2] drm/panel: simple: Add support BOE nv101wxmn51
  2016-12-08  7:26 ` [PATCH v2 2/2] drm/panel: simple: Add support BOE nv101wxmn51 Caesar Wang
@ 2016-12-12 20:22   ` Stéphane Marchesin
  2016-12-14  3:01     ` Caesar Wang
  0 siblings, 1 reply; 4+ messages in thread
From: Stéphane Marchesin @ 2016-12-12 20:22 UTC (permalink / raw)
  To: Caesar Wang; +Cc: Thierry Reding, linux-rockchip, Linux Kernel list, dri-devel

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v2 2/2] drm/panel: simple: Add support BOE nv101wxmn51
  2016-12-12 20:22   ` Stéphane Marchesin
@ 2016-12-14  3:01     ` Caesar Wang
  0 siblings, 0 replies; 4+ messages in thread
From: Caesar Wang @ 2016-12-14  3:01 UTC (permalink / raw)
  To: Stéphane Marchesin
  Cc: Caesar Wang, linux-rockchip, Thierry Reding, Linux Kernel list,
	dri-devel

在 2016年12月13日 04:22, Stéphane Marchesin 写道:
> 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?

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-12-14  3:02 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-12-08  7:26 [PATCH v2 1/2] dt-bindings: display: Add BOE nv101wxmn51 panel binding Caesar Wang
2016-12-08  7:26 ` [PATCH v2 2/2] drm/panel: simple: Add support BOE nv101wxmn51 Caesar Wang
2016-12-12 20:22   ` Stéphane Marchesin
2016-12-14  3:01     ` Caesar Wang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).