All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] dt-bindings: Add LG lp120up1 panel bindings
@ 2016-02-17  6:50 ` Jitao Shi
  0 siblings, 0 replies; 18+ messages in thread
From: Jitao Shi @ 2016-02-17  6:50 UTC (permalink / raw)
  To: Thierry Reding
  Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	David Airlie, Matthias Brugger, Jitao Shi, Thierry Reding,
	Ajay Kumar, Inki Dae, Rahul Sharma, Sean Paul, Vincent Palatin,
	Andy Yan, Philipp Zabel, Russell King, devicetree, linux-kernel,
	dri-devel, linux-arm-kernel, linux-mediatek, srv_heupstream,
	Sascha Hauer, yingjoe.chen, eddie.huang, cawa.cheng, bibby.hsieh,
	ck.hu, stonea168

Add documentation for lp120up1 panel

Signed-off-by: Jitao Shi <jitao.shi@mediatek.com>
---
 .../bindings/display/panel/lg,lp120up1.txt         |    7 +++++++
 1 file changed, 7 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt

diff --git a/Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt b/Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt
new file mode 100644
index 0000000..ff0b6c6
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt
@@ -0,0 +1,7 @@
+LG 12.0" (1920x1280 pixels) TFT LCD panel
+
+Required properties:
+- compatible: should be "lg,lp120up1"
+
+This binding is compatible with the simple-panel binding, which is specified
+in simple-panel.txt in this directory.
-- 
1.7.9.5

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

* [PATCH 1/2] dt-bindings: Add LG lp120up1 panel bindings
@ 2016-02-17  6:50 ` Jitao Shi
  0 siblings, 0 replies; 18+ messages in thread
From: Jitao Shi @ 2016-02-17  6:50 UTC (permalink / raw)
  To: Thierry Reding
  Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	David Airlie, Matthias Brugger, Jitao Shi, Thierry Reding,
	Ajay Kumar, Inki Dae, Rahul Sharma, Sean Paul, Vincent Palatin,
	Andy Yan, Philipp Zabel, Russell King,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	srv_heupstream-NuS5LvNUpcJWk0Htik3J/w, Sascha Hauer

Add documentation for lp120up1 panel

Signed-off-by: Jitao Shi <jitao.shi-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
---
 .../bindings/display/panel/lg,lp120up1.txt         |    7 +++++++
 1 file changed, 7 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt

diff --git a/Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt b/Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt
new file mode 100644
index 0000000..ff0b6c6
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt
@@ -0,0 +1,7 @@
+LG 12.0" (1920x1280 pixels) TFT LCD panel
+
+Required properties:
+- compatible: should be "lg,lp120up1"
+
+This binding is compatible with the simple-panel binding, which is specified
+in simple-panel.txt in this directory.
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* [PATCH 1/2] dt-bindings: Add LG lp120up1 panel bindings
@ 2016-02-17  6:50 ` Jitao Shi
  0 siblings, 0 replies; 18+ messages in thread
From: Jitao Shi @ 2016-02-17  6:50 UTC (permalink / raw)
  To: linux-arm-kernel

Add documentation for lp120up1 panel

Signed-off-by: Jitao Shi <jitao.shi@mediatek.com>
---
 .../bindings/display/panel/lg,lp120up1.txt         |    7 +++++++
 1 file changed, 7 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt

diff --git a/Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt b/Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt
new file mode 100644
index 0000000..ff0b6c6
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt
@@ -0,0 +1,7 @@
+LG 12.0" (1920x1280 pixels) TFT LCD panel
+
+Required properties:
+- compatible: should be "lg,lp120up1"
+
+This binding is compatible with the simple-panel binding, which is specified
+in simple-panel.txt in this directory.
-- 
1.7.9.5

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

* [PATCH 2/2] drm/panel: Add support for LG lp120up1 panel
  2016-02-17  6:50 ` Jitao Shi
  (?)
@ 2016-02-17  6:50   ` Jitao Shi
  -1 siblings, 0 replies; 18+ messages in thread
From: Jitao Shi @ 2016-02-17  6:50 UTC (permalink / raw)
  To: Thierry Reding
  Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	David Airlie, Matthias Brugger, Jitao Shi, Thierry Reding,
	Ajay Kumar, Inki Dae, Rahul Sharma, Sean Paul, Vincent Palatin,
	Andy Yan, Philipp Zabel, Russell King, devicetree, linux-kernel,
	dri-devel, linux-arm-kernel, linux-mediatek, srv_heupstream,
	Sascha Hauer, yingjoe.chen, eddie.huang, cawa.cheng, bibby.hsieh,
	ck.hu, stonea168

The LG lp120up1 panel is a 12.0" 1920x1280 panel,
which can be supported by the simple panel driver

Signed-off-by: Jitao Shi <jitao.shi@mediatek.com>
---
 drivers/gpu/drm/panel/panel-simple.c |   26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
index f88a631..2030c37 100644
--- a/drivers/gpu/drm/panel/panel-simple.c
+++ b/drivers/gpu/drm/panel/panel-simple.c
@@ -982,6 +982,29 @@ static const struct panel_desc lg_lb070wv8 = {
 	.bus_format = MEDIA_BUS_FMT_RGB888_1X7X4_SPWG,
 };
 
+static const struct drm_display_mode lg_lp120up1_mode = {
+	.clock = 162300,
+	.hdisplay = 1920,
+	.hsync_start = 1920 + 40,
+	.hsync_end = 1920 + 40 + 40,
+	.htotal = 1920 + 40 + 40+ 80,
+	.vdisplay = 1280,
+	.vsync_start = 1280 + 4,
+	.vsync_end = 1280 + 4 + 4,
+	.vtotal = 1280 + 4 + 4 + 12,
+	.vrefresh = 60,
+};
+
+static const struct panel_desc lg_lp120up1 = {
+	.modes = &lg_lp120up1_mode,
+	.num_modes = 1,
+	.bpc = 8,
+	.size = {
+		.width = 267,
+		.height = 183,
+	},
+};
+
 static const struct drm_display_mode lg_lp129qe_mode = {
 	.clock = 285250,
 	.hdisplay = 2560,
@@ -1256,6 +1279,9 @@ static const struct of_device_id platform_of_match[] = {
 		.compatible = "lg,lb070wv8",
 		.data = &lg_lb070wv8,
 	}, {
+		.compatible = "lg,lp120up1",
+		.data = &lg_lp120up1,
+	}, {
 		.compatible = "lg,lp129qe",
 		.data = &lg_lp129qe,
 	}, {
-- 
1.7.9.5

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

* [PATCH 2/2] drm/panel: Add support for LG lp120up1 panel
@ 2016-02-17  6:50   ` Jitao Shi
  0 siblings, 0 replies; 18+ messages in thread
From: Jitao Shi @ 2016-02-17  6:50 UTC (permalink / raw)
  To: Thierry Reding
  Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	David Airlie, Matthias Brugger, Jitao Shi, Thierry Reding,
	Ajay Kumar, Inki Dae, Rahul Sharma, Sean Paul, Vincent Palatin,
	Andy Yan, Philipp Zabel, Russell King, devicetree, linux-kernel,
	dri-devel, linux-arm-kernel, linux-mediatek, srv_heupstream,
	Sascha Hauer

The LG lp120up1 panel is a 12.0" 1920x1280 panel,
which can be supported by the simple panel driver

Signed-off-by: Jitao Shi <jitao.shi@mediatek.com>
---
 drivers/gpu/drm/panel/panel-simple.c |   26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
index f88a631..2030c37 100644
--- a/drivers/gpu/drm/panel/panel-simple.c
+++ b/drivers/gpu/drm/panel/panel-simple.c
@@ -982,6 +982,29 @@ static const struct panel_desc lg_lb070wv8 = {
 	.bus_format = MEDIA_BUS_FMT_RGB888_1X7X4_SPWG,
 };
 
+static const struct drm_display_mode lg_lp120up1_mode = {
+	.clock = 162300,
+	.hdisplay = 1920,
+	.hsync_start = 1920 + 40,
+	.hsync_end = 1920 + 40 + 40,
+	.htotal = 1920 + 40 + 40+ 80,
+	.vdisplay = 1280,
+	.vsync_start = 1280 + 4,
+	.vsync_end = 1280 + 4 + 4,
+	.vtotal = 1280 + 4 + 4 + 12,
+	.vrefresh = 60,
+};
+
+static const struct panel_desc lg_lp120up1 = {
+	.modes = &lg_lp120up1_mode,
+	.num_modes = 1,
+	.bpc = 8,
+	.size = {
+		.width = 267,
+		.height = 183,
+	},
+};
+
 static const struct drm_display_mode lg_lp129qe_mode = {
 	.clock = 285250,
 	.hdisplay = 2560,
@@ -1256,6 +1279,9 @@ static const struct of_device_id platform_of_match[] = {
 		.compatible = "lg,lb070wv8",
 		.data = &lg_lb070wv8,
 	}, {
+		.compatible = "lg,lp120up1",
+		.data = &lg_lp120up1,
+	}, {
 		.compatible = "lg,lp129qe",
 		.data = &lg_lp129qe,
 	}, {
-- 
1.7.9.5

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

* [PATCH 2/2] drm/panel: Add support for LG lp120up1 panel
@ 2016-02-17  6:50   ` Jitao Shi
  0 siblings, 0 replies; 18+ messages in thread
From: Jitao Shi @ 2016-02-17  6:50 UTC (permalink / raw)
  To: linux-arm-kernel

The LG lp120up1 panel is a 12.0" 1920x1280 panel,
which can be supported by the simple panel driver

Signed-off-by: Jitao Shi <jitao.shi@mediatek.com>
---
 drivers/gpu/drm/panel/panel-simple.c |   26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
index f88a631..2030c37 100644
--- a/drivers/gpu/drm/panel/panel-simple.c
+++ b/drivers/gpu/drm/panel/panel-simple.c
@@ -982,6 +982,29 @@ static const struct panel_desc lg_lb070wv8 = {
 	.bus_format = MEDIA_BUS_FMT_RGB888_1X7X4_SPWG,
 };
 
+static const struct drm_display_mode lg_lp120up1_mode = {
+	.clock = 162300,
+	.hdisplay = 1920,
+	.hsync_start = 1920 + 40,
+	.hsync_end = 1920 + 40 + 40,
+	.htotal = 1920 + 40 + 40+ 80,
+	.vdisplay = 1280,
+	.vsync_start = 1280 + 4,
+	.vsync_end = 1280 + 4 + 4,
+	.vtotal = 1280 + 4 + 4 + 12,
+	.vrefresh = 60,
+};
+
+static const struct panel_desc lg_lp120up1 = {
+	.modes = &lg_lp120up1_mode,
+	.num_modes = 1,
+	.bpc = 8,
+	.size = {
+		.width = 267,
+		.height = 183,
+	},
+};
+
 static const struct drm_display_mode lg_lp129qe_mode = {
 	.clock = 285250,
 	.hdisplay = 2560,
@@ -1256,6 +1279,9 @@ static const struct of_device_id platform_of_match[] = {
 		.compatible = "lg,lb070wv8",
 		.data = &lg_lb070wv8,
 	}, {
+		.compatible = "lg,lp120up1",
+		.data = &lg_lp120up1,
+	}, {
 		.compatible = "lg,lp129qe",
 		.data = &lg_lp129qe,
 	}, {
-- 
1.7.9.5

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

* Re: [PATCH 1/2] dt-bindings: Add LG lp120up1 panel bindings
  2016-02-17  6:50 ` Jitao Shi
  (?)
@ 2016-02-18 14:36   ` Rob Herring
  -1 siblings, 0 replies; 18+ messages in thread
From: Rob Herring @ 2016-02-18 14:36 UTC (permalink / raw)
  To: Jitao Shi
  Cc: Thierry Reding, Pawel Moll, Mark Rutland, Ian Campbell,
	Kumar Gala, David Airlie, Matthias Brugger, Thierry Reding,
	Ajay Kumar, Inki Dae, Rahul Sharma, Sean Paul, Vincent Palatin,
	Andy Yan, Philipp Zabel, Russell King, devicetree, linux-kernel,
	dri-devel, linux-arm-kernel, linux-mediatek, srv_heupstream,
	Sascha Hauer, yingjoe.chen, eddie.huang, cawa.cheng, bibby.hsieh,
	ck.hu, stonea168

On Wed, Feb 17, 2016 at 02:50:38PM +0800, Jitao Shi wrote:
> Add documentation for lp120up1 panel
> 
> Signed-off-by: Jitao Shi <jitao.shi@mediatek.com>
> ---
>  .../bindings/display/panel/lg,lp120up1.txt         |    7 +++++++
>  1 file changed, 7 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt

Acked-by: Rob Herring <robh@kernel.org>

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

* Re: [PATCH 1/2] dt-bindings: Add LG lp120up1 panel bindings
@ 2016-02-18 14:36   ` Rob Herring
  0 siblings, 0 replies; 18+ messages in thread
From: Rob Herring @ 2016-02-18 14:36 UTC (permalink / raw)
  To: Jitao Shi
  Cc: Thierry Reding, Pawel Moll, Mark Rutland, Ian Campbell,
	Kumar Gala, David Airlie, Matthias Brugger, Thierry Reding,
	Ajay Kumar, Inki Dae, Rahul Sharma, Sean Paul, Vincent Palatin,
	Andy Yan, Philipp Zabel, Russell King, devicetree, linux-kernel,
	dri-devel, linux-arm-kernel, linux-mediatek, srv_heupstream,
	Sascha Hauer, yingjoe.chen

On Wed, Feb 17, 2016 at 02:50:38PM +0800, Jitao Shi wrote:
> Add documentation for lp120up1 panel
> 
> Signed-off-by: Jitao Shi <jitao.shi@mediatek.com>
> ---
>  .../bindings/display/panel/lg,lp120up1.txt         |    7 +++++++
>  1 file changed, 7 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt

Acked-by: Rob Herring <robh@kernel.org>

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

* [PATCH 1/2] dt-bindings: Add LG lp120up1 panel bindings
@ 2016-02-18 14:36   ` Rob Herring
  0 siblings, 0 replies; 18+ messages in thread
From: Rob Herring @ 2016-02-18 14:36 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Feb 17, 2016 at 02:50:38PM +0800, Jitao Shi wrote:
> Add documentation for lp120up1 panel
> 
> Signed-off-by: Jitao Shi <jitao.shi@mediatek.com>
> ---
>  .../bindings/display/panel/lg,lp120up1.txt         |    7 +++++++
>  1 file changed, 7 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt

Acked-by: Rob Herring <robh@kernel.org>

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

* Re: [PATCH 2/2] drm/panel: Add support for LG lp120up1 panel
@ 2016-02-18 15:12     ` Thierry Reding
  0 siblings, 0 replies; 18+ messages in thread
From: Thierry Reding @ 2016-02-18 15:12 UTC (permalink / raw)
  To: Jitao Shi
  Cc: Thierry Reding, Rob Herring, Pawel Moll, Mark Rutland,
	Ian Campbell, Kumar Gala, David Airlie, Matthias Brugger,
	Ajay Kumar, Inki Dae, Rahul Sharma, Sean Paul, Vincent Palatin,
	Andy Yan, Philipp Zabel, Russell King, devicetree, linux-kernel,
	dri-devel, linux-arm-kernel, linux-mediatek, srv_heupstream,
	Sascha Hauer, yingjoe.chen, eddie.huang, cawa.cheng, bibby.hsieh,
	ck.hu, stonea168

[-- Attachment #1: Type: text/plain, Size: 381 bytes --]

On Wed, Feb 17, 2016 at 02:50:39PM +0800, Jitao Shi wrote:
> The LG lp120up1 panel is a 12.0" 1920x1280 panel,
> which can be supported by the simple panel driver

It'd be nice to mention the type of interface of the panel here. While
at it, please rewrite the commit message to make use of what width you
have (i.e. any line after the subject can be 72 characters wide).

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

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

* Re: [PATCH 2/2] drm/panel: Add support for LG lp120up1 panel
@ 2016-02-18 15:12     ` Thierry Reding
  0 siblings, 0 replies; 18+ messages in thread
From: Thierry Reding @ 2016-02-18 15:12 UTC (permalink / raw)
  To: Jitao Shi
  Cc: Thierry Reding, Rob Herring, Pawel Moll, Mark Rutland,
	Ian Campbell, Kumar Gala, David Airlie, Matthias Brugger,
	Ajay Kumar, Inki Dae, Rahul Sharma, Sean Paul, Vincent Palatin,
	Andy Yan, Philipp Zabel, Russell King,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	srv_heupstream-NuS5LvNUpcJWk0Htik3J/w, Sascha Hauer, yingjo

[-- Attachment #1: Type: text/plain, Size: 381 bytes --]

On Wed, Feb 17, 2016 at 02:50:39PM +0800, Jitao Shi wrote:
> The LG lp120up1 panel is a 12.0" 1920x1280 panel,
> which can be supported by the simple panel driver

It'd be nice to mention the type of interface of the panel here. While
at it, please rewrite the commit message to make use of what width you
have (i.e. any line after the subject can be 72 characters wide).

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

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

* [PATCH 2/2] drm/panel: Add support for LG lp120up1 panel
@ 2016-02-18 15:12     ` Thierry Reding
  0 siblings, 0 replies; 18+ messages in thread
From: Thierry Reding @ 2016-02-18 15:12 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Feb 17, 2016 at 02:50:39PM +0800, Jitao Shi wrote:
> The LG lp120up1 panel is a 12.0" 1920x1280 panel,
> which can be supported by the simple panel driver

It'd be nice to mention the type of interface of the panel here. While
at it, please rewrite the commit message to make use of what width you
have (i.e. any line after the subject can be 72 characters wide).

Thierry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160218/1b854aba/attachment.sig>

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

* Re: [PATCH 2/2] drm/panel: Add support for LG lp120up1 panel
@ 2016-02-22  2:12       ` Jitao Shi
  0 siblings, 0 replies; 18+ messages in thread
From: Jitao Shi @ 2016-02-22  2:12 UTC (permalink / raw)
  To: Thierry Reding
  Cc: Thierry Reding, Rob Herring, Pawel Moll, Mark Rutland,
	Ian Campbell, Kumar Gala, David Airlie, Matthias Brugger,
	Ajay Kumar, Inki Dae, Rahul Sharma, Sean Paul, Vincent Palatin,
	Andy Yan, Philipp Zabel, Russell King, devicetree, linux-kernel,
	dri-devel, linux-arm-kernel, linux-mediatek, srv_heupstream,
	Sascha Hauer, yingjoe.chen, eddie.huang, cawa.cheng, bibby.hsieh,
	ck.hu, stonea168

On Thu, 2016-02-18 at 16:12 +0100, Thierry Reding wrote:
> On Wed, Feb 17, 2016 at 02:50:39PM +0800, Jitao Shi wrote:
> > The LG lp120up1 panel is a 12.0" 1920x1280 panel,
> > which can be supported by the simple panel driver
> 
> It'd be nice to mention the type of interface of the panel here. While
> at it, please rewrite the commit message to make use of what width you
> have (i.e. any line after the subject can be 72 characters wide).
> 
> Thierry

Thanks for your review.
I'll fix it next patchset.

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

* Re: [PATCH 2/2] drm/panel: Add support for LG lp120up1 panel
@ 2016-02-22  2:12       ` Jitao Shi
  0 siblings, 0 replies; 18+ messages in thread
From: Jitao Shi @ 2016-02-22  2:12 UTC (permalink / raw)
  To: Thierry Reding
  Cc: Thierry Reding, Rob Herring, Pawel Moll, Mark Rutland,
	Ian Campbell, Kumar Gala, David Airlie, Matthias Brugger,
	Ajay Kumar, Inki Dae, Rahul Sharma, Sean Paul, Vincent Palatin,
	Andy Yan, Philipp Zabel, Russell King,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	srv_heupstream-NuS5LvNUpcJWk0Htik3J/w, Sascha Hauer,
	yingjoe.chen

On Thu, 2016-02-18 at 16:12 +0100, Thierry Reding wrote:
> On Wed, Feb 17, 2016 at 02:50:39PM +0800, Jitao Shi wrote:
> > The LG lp120up1 panel is a 12.0" 1920x1280 panel,
> > which can be supported by the simple panel driver
> 
> It'd be nice to mention the type of interface of the panel here. While
> at it, please rewrite the commit message to make use of what width you
> have (i.e. any line after the subject can be 72 characters wide).
> 
> Thierry

Thanks for your review.
I'll fix it next patchset.

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* [PATCH 2/2] drm/panel: Add support for LG lp120up1 panel
@ 2016-02-22  2:12       ` Jitao Shi
  0 siblings, 0 replies; 18+ messages in thread
From: Jitao Shi @ 2016-02-22  2:12 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, 2016-02-18 at 16:12 +0100, Thierry Reding wrote:
> On Wed, Feb 17, 2016 at 02:50:39PM +0800, Jitao Shi wrote:
> > The LG lp120up1 panel is a 12.0" 1920x1280 panel,
> > which can be supported by the simple panel driver
> 
> It'd be nice to mention the type of interface of the panel here. While
> at it, please rewrite the commit message to make use of what width you
> have (i.e. any line after the subject can be 72 characters wide).
> 
> Thierry

Thanks for your review.
I'll fix it next patchset.

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

* Re: [PATCH 1/2] dt-bindings: Add LG lp120up1 panel bindings
  2016-02-18 14:36   ` Rob Herring
  (?)
@ 2016-02-22  2:13     ` Jitao Shi
  -1 siblings, 0 replies; 18+ messages in thread
From: Jitao Shi @ 2016-02-22  2:13 UTC (permalink / raw)
  To: Rob Herring
  Cc: Thierry Reding, Pawel Moll, Mark Rutland, Ian Campbell,
	Kumar Gala, David Airlie, Matthias Brugger, Thierry Reding,
	Ajay Kumar, Inki Dae, Rahul Sharma, Sean Paul, Vincent Palatin,
	Andy Yan, Philipp Zabel, Russell King, devicetree, linux-kernel,
	dri-devel, linux-arm-kernel, linux-mediatek, srv_heupstream,
	Sascha Hauer, yingjoe.chen, eddie.huang, cawa.cheng, bibby.hsieh,
	ck.hu, stonea168

On Thu, 2016-02-18 at 08:36 -0600, Rob Herring wrote:
> On Wed, Feb 17, 2016 at 02:50:38PM +0800, Jitao Shi wrote:
> > Add documentation for lp120up1 panel
> > 
> > Signed-off-by: Jitao Shi <jitao.shi@mediatek.com>
> > ---
> >  .../bindings/display/panel/lg,lp120up1.txt         |    7 +++++++
> >  1 file changed, 7 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt
> 
> Acked-by: Rob Herring <robh@kernel.org>

Thanks for your "acked"

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

* Re: [PATCH 1/2] dt-bindings: Add LG lp120up1 panel bindings
@ 2016-02-22  2:13     ` Jitao Shi
  0 siblings, 0 replies; 18+ messages in thread
From: Jitao Shi @ 2016-02-22  2:13 UTC (permalink / raw)
  To: Rob Herring
  Cc: Thierry Reding, Pawel Moll, Mark Rutland, Ian Campbell,
	Kumar Gala, David Airlie, Matthias Brugger, Thierry Reding,
	Ajay Kumar, Inki Dae, Rahul Sharma, Sean Paul, Vincent Palatin,
	Andy Yan, Philipp Zabel, Russell King,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	srv_heupstream-NuS5LvNUpcJWk0Htik3J/w, Sascha Hauer, yingjoe.che

On Thu, 2016-02-18 at 08:36 -0600, Rob Herring wrote:
> On Wed, Feb 17, 2016 at 02:50:38PM +0800, Jitao Shi wrote:
> > Add documentation for lp120up1 panel
> > 
> > Signed-off-by: Jitao Shi <jitao.shi-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
> > ---
> >  .../bindings/display/panel/lg,lp120up1.txt         |    7 +++++++
> >  1 file changed, 7 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt
> 
> Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>

Thanks for your "acked"

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* [PATCH 1/2] dt-bindings: Add LG lp120up1 panel bindings
@ 2016-02-22  2:13     ` Jitao Shi
  0 siblings, 0 replies; 18+ messages in thread
From: Jitao Shi @ 2016-02-22  2:13 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, 2016-02-18 at 08:36 -0600, Rob Herring wrote:
> On Wed, Feb 17, 2016 at 02:50:38PM +0800, Jitao Shi wrote:
> > Add documentation for lp120up1 panel
> > 
> > Signed-off-by: Jitao Shi <jitao.shi@mediatek.com>
> > ---
> >  .../bindings/display/panel/lg,lp120up1.txt         |    7 +++++++
> >  1 file changed, 7 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/display/panel/lg,lp120up1.txt
> 
> Acked-by: Rob Herring <robh@kernel.org>

Thanks for your "acked"

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

end of thread, other threads:[~2016-02-22  2:13 UTC | newest]

Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-17  6:50 [PATCH 1/2] dt-bindings: Add LG lp120up1 panel bindings Jitao Shi
2016-02-17  6:50 ` Jitao Shi
2016-02-17  6:50 ` Jitao Shi
2016-02-17  6:50 ` [PATCH 2/2] drm/panel: Add support for LG lp120up1 panel Jitao Shi
2016-02-17  6:50   ` Jitao Shi
2016-02-17  6:50   ` Jitao Shi
2016-02-18 15:12   ` Thierry Reding
2016-02-18 15:12     ` Thierry Reding
2016-02-18 15:12     ` Thierry Reding
2016-02-22  2:12     ` Jitao Shi
2016-02-22  2:12       ` Jitao Shi
2016-02-22  2:12       ` Jitao Shi
2016-02-18 14:36 ` [PATCH 1/2] dt-bindings: Add LG lp120up1 panel bindings Rob Herring
2016-02-18 14:36   ` Rob Herring
2016-02-18 14:36   ` Rob Herring
2016-02-22  2:13   ` Jitao Shi
2016-02-22  2:13     ` Jitao Shi
2016-02-22  2:13     ` Jitao Shi

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.