[PATCH v3] drm/panel: Add support for EDT ETM0700G0DH6 and ET070080DH6 panels

[Thierry Reding]

On Thu, May 15, 2014 at 09:54:07AM -0600, Stephen Warren wrote:
> On 05/15/2014 04:54 AM, Thierry Reding wrote:
> > On Thu, May 15, 2014 at 12:25:47PM +0200, Philipp Zabel wrote:
> >> The EDT ETM0700G0DH6 and ET070080DH6 are 7" 800x480 panels,
> >> which can be supported by the simple panel driver.
> >>
> >> Signed-off-by: Philipp Zabel 
> >> ---
> >> Changes since v2:
> >>  - Added device tree binding documentation. Do we really want to add one 
> >> little
> >>file for each panel?
> > 
> > I guess we could move all of the compatible values into simple-panel.txt
> > but I don't see a need for that yet.
> 
> If they aren't added there, then I believe checkpatch will complain
> about patches that start to use the new compatible values, since they
> won't be documented.

Added where? simple-panel.txt or in separate files per compatible?

Thierry
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: 

[PATCH v3] drm/panel: Add support for EDT ETM0700G0DH6 and ET070080DH6 panels

[Stephen Warren]

On 05/15/2014 03:41 PM, Thierry Reding wrote:
> On Thu, May 15, 2014 at 09:54:07AM -0600, Stephen Warren wrote:
>> On 05/15/2014 04:54 AM, Thierry Reding wrote:
>>> On Thu, May 15, 2014 at 12:25:47PM +0200, Philipp Zabel wrote:
 The EDT ETM0700G0DH6 and ET070080DH6 are 7" 800x480 panels,
 which can be supported by the simple panel driver.

 Signed-off-by: Philipp Zabel 
 ---
 Changes since v2:
  - Added device tree binding documentation. Do we really want to add one 
 little
file for each panel?
>>>
>>> I guess we could move all of the compatible values into simple-panel.txt
>>> but I don't see a need for that yet.
>>
>> If they aren't added there, then I believe checkpatch will complain
>> about patches that start to use the new compatible values, since they
>> won't be documented.
> 
> Added where? simple-panel.txt or in separate files per compatible?

I assume that checkpatch simply checks all files in
Documentation/devicetree/bindings. By "there", I meant "somewhere in
that directory".

Perhaps I misinterpreted the email I was replying to; I thought you'd
meant that we didn't need to document it yet, but it looks like you were
simply discussing where to document it. Sorry for the noise.

[PATCH v3] drm/panel: Add support for EDT ETM0700G0DH6 and ET070080DH6 panels

[Thierry Reding]

On Thu, May 15, 2014 at 12:25:47PM +0200, Philipp Zabel wrote:
> The EDT ETM0700G0DH6 and ET070080DH6 are 7" 800x480 panels,
> which can be supported by the simple panel driver.
> 
> Signed-off-by: Philipp Zabel 
> ---
> Changes since v2:
>  - Added device tree binding documentation. Do we really want to add one 
> little
>file for each panel?

I guess we could move all of the compatible values into simple-panel.txt
but I don't see a need for that yet.

> ---
>  .../devicetree/bindings/panel/edt,et070080dh6.txt  | 10 
>  .../devicetree/bindings/panel/edt,etm0700g0dh6.txt | 10 
>  drivers/gpu/drm/panel/panel-simple.c   | 29 
> ++
>  3 files changed, 49 insertions(+)
>  create mode 100644 
> Documentation/devicetree/bindings/panel/edt,et070080dh6.txt
>  create mode 100644 
> Documentation/devicetree/bindings/panel/edt,etm0700g0dh6.txt

Applied, thanks.

Thierry
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: 

[PATCH v3] drm/panel: Add support for EDT ETM0700G0DH6 and ET070080DH6 panels

[Philipp Zabel]

The EDT ETM0700G0DH6 and ET070080DH6 are 7" 800x480 panels,
which can be supported by the simple panel driver.

Signed-off-by: Philipp Zabel 
---
Changes since v2:
 - Added device tree binding documentation. Do we really want to add one little
   file for each panel?
---
 .../devicetree/bindings/panel/edt,et070080dh6.txt  | 10 
 .../devicetree/bindings/panel/edt,etm0700g0dh6.txt | 10 
 drivers/gpu/drm/panel/panel-simple.c   | 29 ++
 3 files changed, 49 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/panel/edt,et070080dh6.txt
 create mode 100644 Documentation/devicetree/bindings/panel/edt,etm0700g0dh6.txt

diff --git a/Documentation/devicetree/bindings/panel/edt,et070080dh6.txt 
b/Documentation/devicetree/bindings/panel/edt,et070080dh6.txt
new file mode 100644
index 000..20cb38e
--- /dev/null
+++ b/Documentation/devicetree/bindings/panel/edt,et070080dh6.txt
@@ -0,0 +1,10 @@
+Emerging Display Technology Corp. ET070080DH6 7.0" WVGA TFT LCD panel
+
+Required properties:
+- compatible: should be "edt,et070080dh6"
+
+This panel is the same as ETM0700G0DH6 except for the touchscreen.
+ET070080DH6 is the model with resistive touch.
+
+This binding is compatible with the simple-panel binding, which is specified
+in simple-panel.txt in this directory.
diff --git a/Documentation/devicetree/bindings/panel/edt,etm0700g0dh6.txt 
b/Documentation/devicetree/bindings/panel/edt,etm0700g0dh6.txt
new file mode 100644
index 000..ee4b180
--- /dev/null
+++ b/Documentation/devicetree/bindings/panel/edt,etm0700g0dh6.txt
@@ -0,0 +1,10 @@
+Emerging Display Technology Corp. ETM0700G0DH6 7.0" WVGA TFT LCD panel
+
+Required properties:
+- compatible: should be "edt,etm0700g0dh6"
+
+This panel is the same as ET070080DH6 except for the touchscreen.
+ETM0700G0DH6 is the model with capacitive multitouch.
+
+This binding is compatible with the simple-panel binding, which is specified
+in simple-panel.txt in this directory.
diff --git a/drivers/gpu/drm/panel/panel-simple.c 
b/drivers/gpu/drm/panel/panel-simple.c
index 309f29e..72da48c 100644
--- a/drivers/gpu/drm/panel/panel-simple.c
+++ b/drivers/gpu/drm/panel/panel-simple.c
@@ -328,6 +328,29 @@ static const struct panel_desc chunghwa_claa101wb01 = {
},
 };

+static const struct drm_display_mode edt_etm0700g0dh6_mode = {
+   .clock = 33260,
+   .hdisplay = 800,
+   .hsync_start = 800 + 40,
+   .hsync_end = 800 + 40 + 128,
+   .htotal = 800 + 40 + 128 + 88,
+   .vdisplay = 480,
+   .vsync_start = 480 + 10,
+   .vsync_end = 480 + 10 + 2,
+   .vtotal = 480 + 10 + 2 + 33,
+   .vrefresh = 60,
+   .flags = DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC,
+};
+
+static const struct panel_desc edt_etm0700g0dh6 = {
+   .modes = _etm0700g0dh6_mode,
+   .num_modes = 1,
+   .size = {
+   .width = 152,
+   .height = 91,
+   },
+};
+
 static const struct drm_display_mode lg_lp129qe_mode = {
.clock = 285250,
.hdisplay = 2560,
@@ -383,6 +406,12 @@ static const struct of_device_id platform_of_match[] = {
.compatible = "chunghwa,claa101wb01",
.data = _claa101wb01
}, {
+   .compatible = "edt,et070080dh6",
+   .data = _etm0700g0dh6,
+   }, {
+   .compatible = "edt,etm0700g0dh6",
+   .data = _etm0700g0dh6,
+   }, {
.compatible = "lg,lp129qe",
.data = _lp129qe,
}, {
-- 
2.0.0.rc0

[PATCH v3] drm/panel: Add support for EDT ETM0700G0DH6 and ET070080DH6 panels

[Stephen Warren]

On 05/15/2014 04:54 AM, Thierry Reding wrote:
> On Thu, May 15, 2014 at 12:25:47PM +0200, Philipp Zabel wrote:
>> The EDT ETM0700G0DH6 and ET070080DH6 are 7" 800x480 panels,
>> which can be supported by the simple panel driver.
>>
>> Signed-off-by: Philipp Zabel 
>> ---
>> Changes since v2:
>>  - Added device tree binding documentation. Do we really want to add one 
>> little
>>file for each panel?
> 
> I guess we could move all of the compatible values into simple-panel.txt
> but I don't see a need for that yet.

If they aren't added there, then I believe checkpatch will complain
about patches that start to use the new compatible values, since they
won't be documented.