| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20260131032321.6332-1-chintanlike@gmail.com>
Date: Fri, 30 Jan 2026 19:23:21 -0800
From: Chintan Patel <chintanlike@...il.com>
To: neil.armstrong@...aro.org
Cc: dianders@...omium.org,
jesszhan0024@...il.com,
dri-devel@...ts.freedesktop.org,
linux-kernel@...r.kernel.org,
Chintan Patel <chintanlike@...il.com>
Subject: [PATCH] drm/panel: jdi-lt070me05000: Use MIPI DSI multi functions
Convert to the non-deprecated mipi_dsi_*_multi() helpers per the TODO
list. These reduce boilerplate error checking while providing proper
error accumulation via mipi_dsi_multi_context.
Also use mipi_dsi_msleep()/mipi_dsi_usleep_range() macros for all
delays, which automatically skip on error.
Suggested-by: Doug Anderson <dianders@...omium.org>
Signed-off-by: Chintan Patel <chintanlike@...il.com>
---
.../gpu/drm/panel/panel-jdi-lt070me05000.c | 105 ++++--------------
1 file changed, 23 insertions(+), 82 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel-jdi-lt070me05000.c b/drivers/gpu/drm/panel/panel-jdi-lt070me05000.c
index 3513e5c4dd8c..50272364d9d4 100644
--- a/drivers/gpu/drm/panel/panel-jdi-lt070me05000.c
+++ b/drivers/gpu/drm/panel/panel-jdi-lt070me05000.c
@@ -48,34 +48,16 @@ static inline struct jdi_panel *to_jdi_panel(struct drm_panel *panel)
static int jdi_panel_init(struct jdi_panel *jdi)
{
struct mipi_dsi_device *dsi = jdi->dsi;
- struct device *dev = &jdi->dsi->dev;
- int ret;
+ struct mipi_dsi_multi_context dsi_ctx = { .dsi = dsi };
dsi->mode_flags |= MIPI_DSI_MODE_LPM;
- ret = mipi_dsi_dcs_soft_reset(dsi);
- if (ret < 0)
- return ret;
-
- usleep_range(10000, 20000);
-
- ret = mipi_dsi_dcs_set_pixel_format(dsi, MIPI_DCS_PIXEL_FMT_24BIT << 4);
- if (ret < 0) {
- dev_err(dev, "failed to set pixel format: %d\n", ret);
- return ret;
- }
-
- ret = mipi_dsi_dcs_set_column_address(dsi, 0, jdi->mode->hdisplay - 1);
- if (ret < 0) {
- dev_err(dev, "failed to set column address: %d\n", ret);
- return ret;
- }
+ mipi_dsi_dcs_soft_reset_multi(&dsi_ctx);
+ mipi_dsi_usleep_range(&dsi_ctx, 10000, 20000);
- ret = mipi_dsi_dcs_set_page_address(dsi, 0, jdi->mode->vdisplay - 1);
- if (ret < 0) {
- dev_err(dev, "failed to set page address: %d\n", ret);
- return ret;
- }
+ mipi_dsi_dcs_set_pixel_format_multi(&dsi_ctx, MIPI_DCS_PIXEL_FMT_24BIT << 4);
+ mipi_dsi_dcs_set_column_address_multi(&dsi_ctx, 0, jdi->mode->hdisplay - 1);
+ mipi_dsi_dcs_set_page_address_multi(&dsi_ctx, 0, jdi->mode->vdisplay - 1);
/*
* BIT(5) BCTRL = 1 Backlight Control Block On, Brightness registers
@@ -83,90 +65,49 @@ static int jdi_panel_init(struct jdi_panel *jdi)
* BIT(3) BL = 1 Backlight Control On
* BIT(2) DD = 0 Display Dimming is Off
*/
- ret = mipi_dsi_dcs_write(dsi, MIPI_DCS_WRITE_CONTROL_DISPLAY,
- (u8[]){ 0x24 }, 1);
- if (ret < 0) {
- dev_err(dev, "failed to write control display: %d\n", ret);
- return ret;
- }
+ mipi_dsi_dcs_write_seq_multi(&dsi_ctx, MIPI_DCS_WRITE_CONTROL_DISPLAY, 0x24);
/* CABC off */
- ret = mipi_dsi_dcs_write(dsi, MIPI_DCS_WRITE_POWER_SAVE,
- (u8[]){ 0x00 }, 1);
- if (ret < 0) {
- dev_err(dev, "failed to set cabc off: %d\n", ret);
- return ret;
- }
+ mipi_dsi_dcs_write_seq_multi(&dsi_ctx, MIPI_DCS_WRITE_POWER_SAVE, 0x00);
- ret = mipi_dsi_dcs_exit_sleep_mode(dsi);
- if (ret < 0) {
- dev_err(dev, "failed to set exit sleep mode: %d\n", ret);
- return ret;
- }
-
- msleep(120);
-
- ret = mipi_dsi_generic_write(dsi, (u8[]){0xB0, 0x00}, 2);
- if (ret < 0) {
- dev_err(dev, "failed to set mcap: %d\n", ret);
- return ret;
- }
+ mipi_dsi_dcs_exit_sleep_mode_multi(&dsi_ctx);
+ mipi_dsi_msleep(&dsi_ctx, 120);
- mdelay(10);
+ mipi_dsi_generic_write_multi(&dsi_ctx, (u8[]){0xB0, 0x00}, 2);
+ mipi_dsi_msleep(&dsi_ctx, 10);
/* Interface setting, video mode */
- ret = mipi_dsi_generic_write(dsi, (u8[])
- {0xB3, 0x26, 0x08, 0x00, 0x20, 0x00}, 6);
- if (ret < 0) {
- dev_err(dev, "failed to set display interface setting: %d\n"
- , ret);
- return ret;
- }
+ mipi_dsi_generic_write_multi(&dsi_ctx, (u8[]){0xB3, 0x26, 0x08, 0x00, 0x20, 0x00}, 6);
+ mipi_dsi_msleep(&dsi_ctx, 20);
- mdelay(20);
+ mipi_dsi_generic_write_multi(&dsi_ctx, (u8[]){0xB0, 0x03}, 2);
- ret = mipi_dsi_generic_write(dsi, (u8[]){0xB0, 0x03}, 2);
- if (ret < 0) {
- dev_err(dev, "failed to set default values for mcap: %d\n"
- , ret);
- return ret;
- }
-
- return 0;
+ return dsi_ctx.accum_err;
}
static int jdi_panel_on(struct jdi_panel *jdi)
{
struct mipi_dsi_device *dsi = jdi->dsi;
- struct device *dev = &jdi->dsi->dev;
- int ret;
+ struct mipi_dsi_multi_context dsi_ctx = { .dsi = dsi };
dsi->mode_flags |= MIPI_DSI_MODE_LPM;
- ret = mipi_dsi_dcs_set_display_on(dsi);
- if (ret < 0)
- dev_err(dev, "failed to set display on: %d\n", ret);
+ mipi_dsi_dcs_set_display_on_multi(&dsi_ctx);
- return ret;
+ return dsi_ctx.accum_err;
}
static void jdi_panel_off(struct jdi_panel *jdi)
{
struct mipi_dsi_device *dsi = jdi->dsi;
- struct device *dev = &jdi->dsi->dev;
- int ret;
+ struct mipi_dsi_multi_context dsi_ctx = { .dsi = dsi };
dsi->mode_flags &= ~MIPI_DSI_MODE_LPM;
- ret = mipi_dsi_dcs_set_display_off(dsi);
- if (ret < 0)
- dev_err(dev, "failed to set display off: %d\n", ret);
-
- ret = mipi_dsi_dcs_enter_sleep_mode(dsi);
- if (ret < 0)
- dev_err(dev, "failed to enter sleep mode: %d\n", ret);
+ mipi_dsi_dcs_set_display_off_multi(&dsi_ctx);
+ mipi_dsi_dcs_enter_sleep_mode_multi(&dsi_ctx);
- msleep(100);
+ mipi_dsi_msleep(&dsi_ctx, 100);
}
static int jdi_panel_disable(struct drm_panel *panel)
--
2.43.0
Powered by blists - more mailing lists