2015-12-02 22:03:55

by Alexandre Belloni

[permalink] [raw]
Subject: [PATCH v4 1/2] of: Add vendor prefix for QiaoDian Xianshi

Use "qiaodian" as the vendor prefix for QiaoDian Xianshi Corporation in
device tree compatible strings.

Signed-off-by: Alexandre Belloni <[email protected]>
Acked-by: Rob Herring <[email protected]>
---
Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index 55df1d444e9f..8fd423dc096b 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -180,6 +180,7 @@ qca Qualcomm Atheros, Inc.
qcom Qualcomm Technologies, Inc
qemu QEMU, a generic and open source machine emulator and virtualizer
qi Qi Hardware
+qiaodian QiaoDian XianShi Corporation
qnap QNAP Systems, Inc.
radxa Radxa
raidsonic RaidSonic Technology GmbH
--
2.5.0


2015-12-02 22:03:53

by Alexandre Belloni

[permalink] [raw]
Subject: [PATCH v4 2/2] drm: panel: simple: add QiaoDian qd43003c0-40

From: Josh Wu <[email protected]>

The QiaoDian Xianshi QD43003C0-40 is a 4"3 TFT LCD panel.

Timings from the OTA5180A document, ver 0.9, section
10.1.1:
http://www.orientdisplay.com/pdf/OTA5180A.pdf

Signed-off-by: Josh Wu <[email protected]>
Signed-off-by: Alexandre Belloni <[email protected]>
Acked-by: Rob Herring <[email protected]>
---
Changes in v4:
- rebased on v4.4-rc1

.../display/panel/qiaodian,qd43003c0-40.txt | 7 ++++++
drivers/gpu/drm/panel/panel-simple.c | 27 ++++++++++++++++++++++
2 files changed, 34 insertions(+)
create mode 100644 Documentation/devicetree/bindings/display/panel/qiaodian,qd43003c0-40.txt

diff --git a/Documentation/devicetree/bindings/display/panel/qiaodian,qd43003c0-40.txt b/Documentation/devicetree/bindings/display/panel/qiaodian,qd43003c0-40.txt
new file mode 100644
index 000000000000..0fbdab89ac3d
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/panel/qiaodian,qd43003c0-40.txt
@@ -0,0 +1,7 @@
+QiaoDian XianShi Corporation 4"3 TFT LCD panel
+
+Required properties:
+- compatible: should be "qiaodian,qd43003c0-40"
+
+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 f97b73ec4713..c93ffa615005 100644
--- a/drivers/gpu/drm/panel/panel-simple.c
+++ b/drivers/gpu/drm/panel/panel-simple.c
@@ -1027,6 +1027,30 @@ static const struct panel_desc ortustech_com43h4m85ulc = {
.bus_format = MEDIA_BUS_FMT_RGB888_1X24,
};

+static const struct drm_display_mode qd43003c0_40_mode = {
+ .clock = 9000,
+ .hdisplay = 480,
+ .hsync_start = 480 + 8,
+ .hsync_end = 480 + 8 + 4,
+ .htotal = 480 + 8 + 4 + 39,
+ .vdisplay = 272,
+ .vsync_start = 272 + 4,
+ .vsync_end = 272 + 4 + 10,
+ .vtotal = 272 + 4 + 10 + 2,
+ .vrefresh = 60,
+};
+
+static const struct panel_desc qd43003c0_40 = {
+ .modes = &qd43003c0_40_mode,
+ .num_modes = 1,
+ .bpc = 8,
+ .size = {
+ .width = 95,
+ .height = 53,
+ },
+ .bus_format = MEDIA_BUS_FMT_RGB888_1X24,
+};
+
static const struct drm_display_mode samsung_ltn101nt05_mode = {
.clock = 54030,
.hdisplay = 1024,
@@ -1182,6 +1206,9 @@ static const struct of_device_id platform_of_match[] = {
.compatible = "ortustech,com43h4m85ulc",
.data = &ortustech_com43h4m85ulc,
}, {
+ .compatible = "qiaodian,qd43003c0-40",
+ .data = &qd43003c0_40,
+ }, {
.compatible = "samsung,ltn101nt05",
.data = &samsung_ltn101nt05,
}, {
--
2.5.0

2015-12-15 13:43:27

by Nicolas Ferre

[permalink] [raw]
Subject: Re: [PATCH v4 2/2] drm: panel: simple: add QiaoDian qd43003c0-40

Le 02/12/2015 23:03, Alexandre Belloni a ?crit :
> From: Josh Wu <[email protected]>
>
> The QiaoDian Xianshi QD43003C0-40 is a 4"3 TFT LCD panel.
>
> Timings from the OTA5180A document, ver 0.9, section
> 10.1.1:
> http://www.orientdisplay.com/pdf/OTA5180A.pdf
>
> Signed-off-by: Josh Wu <[email protected]>
> Signed-off-by: Alexandre Belloni <[email protected]>
> Acked-by: Rob Herring <[email protected]>

If it can speed-up things:
Acked-by: Nicolas Ferre <[email protected]>

Thierry, it seems that this patch keeps on sitting aside for quite some
time, and Alexandre only rebased it lately so it can be easier to take
(so this v4).

Can we see this trivial patch move forward so that we can at least see
it appearing in linux-next and use our screen with Mainline?

Bye,

> ---
> Changes in v4:
> - rebased on v4.4-rc1
>
> .../display/panel/qiaodian,qd43003c0-40.txt | 7 ++++++
> drivers/gpu/drm/panel/panel-simple.c | 27 ++++++++++++++++++++++
> 2 files changed, 34 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/display/panel/qiaodian,qd43003c0-40.txt
>
> diff --git a/Documentation/devicetree/bindings/display/panel/qiaodian,qd43003c0-40.txt b/Documentation/devicetree/bindings/display/panel/qiaodian,qd43003c0-40.txt
> new file mode 100644
> index 000000000000..0fbdab89ac3d
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/display/panel/qiaodian,qd43003c0-40.txt
> @@ -0,0 +1,7 @@
> +QiaoDian XianShi Corporation 4"3 TFT LCD panel
> +
> +Required properties:
> +- compatible: should be "qiaodian,qd43003c0-40"
> +
> +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 f97b73ec4713..c93ffa615005 100644
> --- a/drivers/gpu/drm/panel/panel-simple.c
> +++ b/drivers/gpu/drm/panel/panel-simple.c
> @@ -1027,6 +1027,30 @@ static const struct panel_desc ortustech_com43h4m85ulc = {
> .bus_format = MEDIA_BUS_FMT_RGB888_1X24,
> };
>
> +static const struct drm_display_mode qd43003c0_40_mode = {
> + .clock = 9000,
> + .hdisplay = 480,
> + .hsync_start = 480 + 8,
> + .hsync_end = 480 + 8 + 4,
> + .htotal = 480 + 8 + 4 + 39,
> + .vdisplay = 272,
> + .vsync_start = 272 + 4,
> + .vsync_end = 272 + 4 + 10,
> + .vtotal = 272 + 4 + 10 + 2,
> + .vrefresh = 60,
> +};
> +
> +static const struct panel_desc qd43003c0_40 = {
> + .modes = &qd43003c0_40_mode,
> + .num_modes = 1,
> + .bpc = 8,
> + .size = {
> + .width = 95,
> + .height = 53,
> + },
> + .bus_format = MEDIA_BUS_FMT_RGB888_1X24,
> +};
> +
> static const struct drm_display_mode samsung_ltn101nt05_mode = {
> .clock = 54030,
> .hdisplay = 1024,
> @@ -1182,6 +1206,9 @@ static const struct of_device_id platform_of_match[] = {
> .compatible = "ortustech,com43h4m85ulc",
> .data = &ortustech_com43h4m85ulc,
> }, {
> + .compatible = "qiaodian,qd43003c0-40",
> + .data = &qd43003c0_40,
> + }, {
> .compatible = "samsung,ltn101nt05",
> .data = &samsung_ltn101nt05,
> }, {
>


--
Nicolas Ferre

2015-12-16 17:14:10

by Thierry Reding

[permalink] [raw]
Subject: Re: [PATCH v4 2/2] drm: panel: simple: add QiaoDian qd43003c0-40

On Tue, Dec 15, 2015 at 02:43:47PM +0100, Nicolas Ferre wrote:
> Le 02/12/2015 23:03, Alexandre Belloni a écrit :
> > From: Josh Wu <[email protected]>
> >
> > The QiaoDian Xianshi QD43003C0-40 is a 4"3 TFT LCD panel.
> >
> > Timings from the OTA5180A document, ver 0.9, section
> > 10.1.1:
> > http://www.orientdisplay.com/pdf/OTA5180A.pdf
> >
> > Signed-off-by: Josh Wu <[email protected]>
> > Signed-off-by: Alexandre Belloni <[email protected]>
> > Acked-by: Rob Herring <[email protected]>
>
> If it can speed-up things:
> Acked-by: Nicolas Ferre <[email protected]>
>
> Thierry, it seems that this patch keeps on sitting aside for quite some
> time, and Alexandre only rebased it lately so it can be easier to take
> (so this v4).
>
> Can we see this trivial patch move forward so that we can at least see
> it appearing in linux-next and use our screen with Mainline?

Please make sure to Cc: [email protected] on patches that
apply under drivers/gpu/drm to make sure that patchwork picks them up. I
recommend also sending the corresponding vendor prefix patches to that
list, which makes it a whole lot easier to track.

Also, please use "drm/panel: " as the prefix (instead of "drm: panel: ")
to make it easier for me to pick up things. I was about to send out a
pull request for drm/panel when I noticed that there was this patch. The
v4 wasn't Cc: dri-devel, so I ended up applying v3 from patchwork. From
the changelog it seems like that's okay, though.

Thierry


Attachments:
(No filename) (1.49 kB)
signature.asc (819.00 B)
Download all attachments

2015-12-16 20:23:09

by Alexandre Belloni

[permalink] [raw]
Subject: Re: [PATCH v4 2/2] drm: panel: simple: add QiaoDian qd43003c0-40

Hi,

On 16/12/2015 at 18:14:06 +0100, Thierry Reding wrote :
> Also, please use "drm/panel: " as the prefix (instead of "drm: panel: ")
> to make it easier for me to pick up things. I was about to send out a
> pull request for drm/panel when I noticed that there was this patch. The
> v4 wasn't Cc: dri-devel, so I ended up applying v3 from patchwork. From
> the changelog it seems like that's okay, though.
>

I think v3 didn't have the documentation at the correct place, this
should be:
Documentation/devicetree/bindings/display/panel/qiaodian,qd43003c0-40.txt



--
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com