[PATCH 6/6] video: panel: himax-hx8394: use common mipi_dsi_dcs_write_seq macro

Julien Stephan <[email protected]> Thu, 06 Aug 2026 11:27:12 +0200
Newsgroups gmane.comp.boot-loaders.u-boot
Message-ID <20260806-add_mipi_dsi_write_seq_helper_macros-v1-6-6b533788d0ac__42779.7699275956$1786008467$gmane$org@baylibre.com>
The current driver defines dsi_dcs_write_seq as a wrapper around
mipi_dsi_dcs_write_buffer.

Now that mipi_dsi.h defines such macro, we can use the common one.

No functional changes intended.

Signed-off-by: Julien Stephan <[email protected]>
---
 drivers/video/himax-hx8394.c | 86 ++++++++++++++++++++------------------------
 1 file changed, 39 insertions(+), 47 deletions(-)

diff --git a/drivers/video/himax-hx8394.c b/drivers/video/himax-hx8394.c
index cb7f93e9c99..1107070bd1a 100644
--- a/drivers/video/himax-hx8394.c
+++ b/drivers/video/himax-hx8394.c
@@ -31,59 +31,51 @@ static const struct display_timing default_timing = {
 	.flags			= DISPLAY_FLAGS_VSYNC_LOW | DISPLAY_FLAGS_HSYNC_LOW,
 };
 
-#define dsi_dcs_write_seq(device, seq...) do {					\
-		static const u8 d[] = { seq };					\
-		int ret;							\
-		ret = mipi_dsi_dcs_write_buffer(device, d, ARRAY_SIZE(d));	\
-		if (ret < 0)							\
-			return ret;						\
-	} while (0)
-
 static int hx8394_init_sequence(struct udevice *dev)
 {
 	struct mipi_dsi_panel_plat *plat = dev_get_plat(dev);
 	struct mipi_dsi_device *device = plat->device;
 	int ret;
 
-	dsi_dcs_write_seq(device, 0xb9, 0xff, 0x83, 0x94);
-	dsi_dcs_write_seq(device, 0xb1, 0x48, 0x11, 0x71, 0x09, 0x32, 0x24,
-			  0x71, 0x31, 0x55, 0x30);
-	dsi_dcs_write_seq(device, 0xba, 0x63, 0x03, 0x68, 0x6b, 0xb2, 0xc0);
-	dsi_dcs_write_seq(device, 0xb2, 0x00, 0x80, 0x78, 0x0c, 0x07);
-	dsi_dcs_write_seq(device, 0xb4, 0x12, 0x63, 0x12, 0x63, 0x12, 0x63,
-			  0x01, 0x0c, 0x7c, 0x55, 0x00, 0x3f, 0x12, 0x6b, 0x12,
-			  0x6b, 0x12, 0x6b, 0x01, 0x0c, 0x7c);
-	dsi_dcs_write_seq(device, 0xd3, 0x00, 0x00, 0x00, 0x00, 0x3c, 0x1c,
-			  0x00, 0x00, 0x32, 0x10, 0x09, 0x00, 0x09, 0x32, 0x15,
-			  0xad, 0x05, 0xad, 0x32, 0x00, 0x00, 0x00, 0x00, 0x37,
-			  0x03, 0x0b, 0x0b, 0x37, 0x00, 0x00, 0x00, 0x0c, 0x40);
-	dsi_dcs_write_seq(device, 0xd5, 0x19, 0x19, 0x18, 0x18, 0x1b, 0x1b,
-			  0x1a, 0x1a, 0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06,
-			  0x07, 0x20, 0x21, 0x18, 0x18, 0x18, 0x18, 0x18, 0x18,
-			  0x18, 0x18, 0x18, 0x18, 0x18, 0x18, 0x24, 0x25, 0x18,
-			  0x18, 0x18, 0x18, 0x18, 0x18, 0x18, 0x18, 0x18, 0x18,
-			  0x18, 0x18);
-	dsi_dcs_write_seq(device, 0xd6, 0x18, 0x18, 0x19, 0x19, 0x1b, 0x1b,
-			  0x1a, 0x1a, 0x07, 0x06, 0x05, 0x04, 0x03, 0x02, 0x01,
-			  0x00, 0x25, 0x24, 0x18, 0x18, 0x18, 0x18, 0x18, 0x18,
-			  0x18, 0x18, 0x18, 0x18, 0x18, 0x18, 0x21, 0x20, 0x18,
-			  0x18, 0x18, 0x18, 0x18, 0x18, 0x18, 0x18, 0x18, 0x18,
-			  0x18, 0x18);
-	dsi_dcs_write_seq(device, 0xe0, 0x00, 0x04, 0x0c, 0x12, 0x14, 0x18,
-			  0x1a, 0x18, 0x31, 0x3f, 0x4d, 0x4c, 0x54, 0x65, 0x6b,
-			  0x70, 0x7f, 0x82, 0x7e, 0x8a, 0x99, 0x4a, 0x48, 0x49,
-			  0x4b, 0x4a, 0x4c, 0x4b, 0x7f, 0x00, 0x04, 0x0c, 0x11,
-			  0x13, 0x17, 0x1a, 0x18, 0x31, 0x3f, 0x4d, 0x4c, 0x54,
-			  0x65, 0x6b, 0x70, 0x7f, 0x82, 0x7e, 0x8a, 0x99, 0x4a,
-			  0x48, 0x49, 0x4b, 0x4a, 0x4c, 0x4b, 0x7f);
-	dsi_dcs_write_seq(device, 0xcc, 0x0b);
-	dsi_dcs_write_seq(device, 0xc0, 0x1f, 0x31);
-	dsi_dcs_write_seq(device, 0xb6, 0x7d, 0x7d);
-	dsi_dcs_write_seq(device, 0xd4, 0x02);
-	dsi_dcs_write_seq(device, 0xbd, 0x01);
-	dsi_dcs_write_seq(device, 0xb1, 0x00);
-	dsi_dcs_write_seq(device, 0xbd, 0x00);
-	dsi_dcs_write_seq(device, 0xc6, 0xed);
+	mipi_dsi_dcs_write_seq(device, 0xb9, 0xff, 0x83, 0x94);
+	mipi_dsi_dcs_write_seq(device, 0xb1, 0x48, 0x11, 0x71, 0x09, 0x32, 0x24,
+			       0x71, 0x31, 0x55, 0x30);
+	mipi_dsi_dcs_write_seq(device, 0xba, 0x63, 0x03, 0x68, 0x6b, 0xb2, 0xc0);
+	mipi_dsi_dcs_write_seq(device, 0xb2, 0x00, 0x80, 0x78, 0x0c, 0x07);
+	mipi_dsi_dcs_write_seq(device, 0xb4, 0x12, 0x63, 0x12, 0x63, 0x12, 0x63,
+			       0x01, 0x0c, 0x7c, 0x55, 0x00, 0x3f, 0x12, 0x6b, 0x12,
+			       0x6b, 0x12, 0x6b, 0x01, 0x0c, 0x7c);
+	mipi_dsi_dcs_write_seq(device, 0xd3, 0x00, 0x00, 0x00, 0x00, 0x3c, 0x1c,
+			       0x00, 0x00, 0x32, 0x10, 0x09, 0x00, 0x09, 0x32, 0x15,
+			       0xad, 0x05, 0xad, 0x32, 0x00, 0x00, 0x00, 0x00, 0x37,
+			       0x03, 0x0b, 0x0b, 0x37, 0x00, 0x00, 0x00, 0x0c, 0x40);
+	mipi_dsi_dcs_write_seq(device, 0xd5, 0x19, 0x19, 0x18, 0x18, 0x1b, 0x1b,
+			       0x1a, 0x1a, 0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06,
+			       0x07, 0x20, 0x21, 0x18, 0x18, 0x18, 0x18, 0x18, 0x18,
+			       0x18, 0x18, 0x18, 0x18, 0x18, 0x18, 0x24, 0x25, 0x18,
+			       0x18, 0x18, 0x18, 0x18, 0x18, 0x18, 0x18, 0x18, 0x18,
+			       0x18, 0x18);
+	mipi_dsi_dcs_write_seq(device, 0xd6, 0x18, 0x18, 0x19, 0x19, 0x1b, 0x1b,
+			       0x1a, 0x1a, 0x07, 0x06, 0x05, 0x04, 0x03, 0x02, 0x01,
+			       0x00, 0x25, 0x24, 0x18, 0x18, 0x18, 0x18, 0x18, 0x18,
+			       0x18, 0x18, 0x18, 0x18, 0x18, 0x18, 0x21, 0x20, 0x18,
+			       0x18, 0x18, 0x18, 0x18, 0x18, 0x18, 0x18, 0x18, 0x18,
+			       0x18, 0x18);
+	mipi_dsi_dcs_write_seq(device, 0xe0, 0x00, 0x04, 0x0c, 0x12, 0x14, 0x18,
+			       0x1a, 0x18, 0x31, 0x3f, 0x4d, 0x4c, 0x54, 0x65, 0x6b,
+			       0x70, 0x7f, 0x82, 0x7e, 0x8a, 0x99, 0x4a, 0x48, 0x49,
+			       0x4b, 0x4a, 0x4c, 0x4b, 0x7f, 0x00, 0x04, 0x0c, 0x11,
+			       0x13, 0x17, 0x1a, 0x18, 0x31, 0x3f, 0x4d, 0x4c, 0x54,
+			       0x65, 0x6b, 0x70, 0x7f, 0x82, 0x7e, 0x8a, 0x99, 0x4a,
+			       0x48, 0x49, 0x4b, 0x4a, 0x4c, 0x4b, 0x7f);
+	mipi_dsi_dcs_write_seq(device, 0xcc, 0x0b);
+	mipi_dsi_dcs_write_seq(device, 0xc0, 0x1f, 0x31);
+	mipi_dsi_dcs_write_seq(device, 0xb6, 0x7d, 0x7d);
+	mipi_dsi_dcs_write_seq(device, 0xd4, 0x02);
+	mipi_dsi_dcs_write_seq(device, 0xbd, 0x01);
+	mipi_dsi_dcs_write_seq(device, 0xb1, 0x00);
+	mipi_dsi_dcs_write_seq(device, 0xbd, 0x00);
+	mipi_dsi_dcs_write_seq(device, 0xc6, 0xed);
 
 	ret = mipi_dsi_dcs_exit_sleep_mode(device);
 	if (ret)

-- 
2.54.0