Skip to contentSkip to conversation

dt-bindings: Add vendor prefix for VXT Ltd

PATCH RESEND In review Fabio Estevam via dri-devel 2 participants 4 messages Feb 17, 2019

Subsystems (4)
lore
Share & tools
lore
#0 lore raw

PATCH RESEND 1/3 dt-bindings: Add vendor prefix for VXT Ltd

VXT Ltd is a manufacturer of projected capacitive touch panel and display solutions: http://www.vxt.com.tw/

---
 Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
 1 file changed, 1 insertion(+)
Documentation/devicetree/bindings/vendor-prefixes.txt +1 -0
index 3bbe3b8..16a2a97 100644
@@ -426,6 +426,7 @@ vivante	Vivante Corporation
 vocore VoCore Studio
 voipac	Voipac Technologies s.r.o.
 vot	Vision Optical Technology Co., Ltd.
+vxt	VXT Ltd
 wd	Western Digital Corp.
 wetek	WeTek Electronics, limited.
 wexler	Wexler
Signature · 6 lines
2.7.4

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel
#1 lore raw

PATCH RESEND 2/3 dt-bindings: Add VXT VL050-8048NT-C01 panel bindings

The VXT VL050-8048NT-C01 is a TFT LCD panel with a 800x480 resolution connected via 24 width parallel interface.

---
 .../devicetree/bindings/display/panel/vl050_8048nt_c01.txt   | 12 ++++++++++++
 1 file changed, 12 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/display/panel/vl050_8048nt_c01.txt
Documentation/devicetree/bindings/display/panel/vl050_8048nt_c01.txt new file +12 -0
new file mode 100644
index 0000000..b42bf06
@@ -0,0 +1,12 @@
+VXT 800x480 color TFT LCD panel
+
+Required properties:
+- compatible: should be "vxt,vl050-8048nt-c01"
+- power-supply: as specified in the base binding
+
+Optional properties:
+- backlight: as specified in the base binding
+- enable-gpios: as specified in the base binding
+
+This binding is compatible with the simple-panel binding, which is specified
+in simple-panel.txt in this directory.
Signature · 6 lines
2.7.4

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel
#2 lore raw

PATCH RESEND 3/3 drm/panel: simple: Add support for VXT VL050-8048NT-C01 panel

In reply to Fabio Estevam via dri-devel

Add support for the VXT VL050-8048NT-C01 800x480 panel to the panel-simple driver.

This panel is used on some boards manufactured by TechNexion, such as imx7d-pico.

---
 drivers/gpu/drm/panel/panel-simple.c | 27 +++++++++++++++++++++++++++
 1 file changed, 27 insertions(+)
drivers/gpu/drm/panel/panel-simple.c +27 -0
index 5fbee83..2e76e95 100644
@@ -2410,6 +2410,30 @@ static const struct panel_desc urt_umsh_8596md_parallel = {
 	.bus_format = MEDIA_BUS_FMT_RGB666_1X18,
 };
 
+static const struct drm_display_mode vl050_8048nt_c01_mode = {
+	.clock = 33333,
+	.hdisplay = 800,
+	.hsync_start = 800 + 210,
+	.hsync_end = 800 + 210 + 20,
+	.htotal = 800 + 210 + 20 + 46,
+	.vdisplay =  480,
+	.vsync_start = 480 + 22,
+	.vsync_end = 480 + 22 + 10,
+	.vtotal = 480 + 22 + 10 + 23,
+	.vrefresh = 60,
+};
+
+static const struct panel_desc vl050_8048nt_c01 = {
+	.modes = &vl050_8048nt_c01_mode,
+	.num_modes = 1,
+	.bpc = 8,
+	.size = {
+		.width = 120,
+		.height = 76,
+	},
+	.bus_format = MEDIA_BUS_FMT_RGB888_1X24,
+};
+
 static const struct drm_display_mode winstar_wf35ltiacd_mode = {
 	.clock = 6410,
 	.hdisplay = 320,
@@ -2725,6 +2749,9 @@ static const struct of_device_id platform_of_match[] = {
 		.compatible = "urt,umsh-8596md-20t",
 		.data = &urt_umsh_8596md_parallel,
 	}, {
+		.compatible = "vxt,vl050-8048nt-c01",
+		.data = &vl050_8048nt_c01,
+	}, {
 		.compatible = "winstar,wf35ltiacd",
 		.data = &winstar_wf35ltiacd,
 	}, {
Signature · 6 lines
2.7.4

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel
Reviewed-by on v1 3/3
#3 lore raw

Re: [PATCH RESEND 3/3] drm/panel: simple: Add support for VXT VL050-8048NT-C01 panel

Quoted 22 lines · Fabio Estevam

Add support for the VXT VL050-8048NT-C01 800x480 panel to the panel-simple driver.

This panel is used on some boards manufactured by TechNexion, such as imx7d-pico.

---
 drivers/gpu/drm/panel/panel-simple.c | 27 +++++++++++++++++++++++++++
 1 file changed, 27 insertions(+)
drivers/gpu/drm/panel/panel-simple.c +2 -0
index 5fbee83..2e76e95 100644
@@ -2410,6 +2410,30 @@ static const struct panel_desc urt_umsh_8596md_parallel = {
 	.bus_format = MEDIA_BUS_FMT_RGB666_1X18,
 };
 
+static const struct drm_display_mode vl050_8048nt_c01_mode = {
+	.clock = 33333,

drivers/gpu/drm/panel/panel-simple.c static const struct panel_desc urt_umsh_8596md_parallel = { · line 2424

Quoted 10 lines
drivers/gpu/drm/panel/panel-simple.c +10 -0
+	.hdisplay = 800,
+	.hsync_start = 800 + 210,
+	.hsync_end = 800 + 210 + 20,
+	.htotal = 800 + 210 + 20 + 46,
+	.vdisplay =  480,
+	.vsync_start = 480 + 22,
+	.vsync_end = 480 + 22 + 10,
+	.vtotal = 480 + 22 + 10 + 23,
+	.vrefresh = 60,
+};

No need for .flags?
Example: .flags = DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC,

drivers/gpu/drm/panel/panel-simple.c static const struct panel_desc urt_umsh_8596md_parallel = { · line 2434

Quoted 10 lines
drivers/gpu/drm/panel/panel-simple.c +10 -0
+
+static const struct panel_desc vl050_8048nt_c01 = {
+	.modes = &vl050_8048nt_c01_mode,
+	.num_modes = 1,
+	.bpc = 8,
+	.size = {
+		.width = 120,
+		.height = 76,
+	},
+	.bus_format = MEDIA_BUS_FMT_RGB888_1X24,

What about .bus_flags?
Example: .bus_flags = DRM_BUS_FLAG_PIXDATA_POSEDGE,

The examples are random from the panel-simple driver. I could not find any datasheet at http://www.vxt.com.tw
for the panel.

With my comments considered you can add:

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel