aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLinus Walleij <linus.walleij@linaro.org>2020-09-06 15:29:03 +0200
committerLinus Walleij <linus.walleij@linaro.org>2020-09-06 21:55:03 +0200
commitd2024cdf0f02b9bdad2adb8ce220e30c6424e247 (patch)
treef33982d7592453d6f70c2d1faf091e392e0881d0
parentdrm/panel: s6e63m0: Order enable/disable sequence (diff)
downloadlinux-dev-d2024cdf0f02b9bdad2adb8ce220e30c6424e247.tar.xz
linux-dev-d2024cdf0f02b9bdad2adb8ce220e30c6424e247.zip
drm/panel: s6e63m0: Fix up DRM_DEV* regression
Ooops the panel drivers stopped to use DRM_DEV* messages and we predictably create errors by merging code that still use it. Signed-off-by: Linus Walleij <linus.walleij@linaro.org> Acked-by: Sam Ravnborg <sam@ravnborg.org> Acked-by: David Airlie <airlied@linux.ie> Link: https://patchwork.freedesktop.org/patch/msgid/20200906132903.5739-1-linus.walleij@linaro.org
-rw-r--r--drivers/gpu/drm/panel/panel-samsung-s6e63m0-dsi.c22
-rw-r--r--drivers/gpu/drm/panel/panel-samsung-s6e63m0-spi.c8
-rw-r--r--drivers/gpu/drm/panel/panel-samsung-s6e63m0.c14
3 files changed, 17 insertions, 27 deletions
diff --git a/drivers/gpu/drm/panel/panel-samsung-s6e63m0-dsi.c b/drivers/gpu/drm/panel/panel-samsung-s6e63m0-dsi.c
index 2ec9e7900791..eec74c10ddda 100644
--- a/drivers/gpu/drm/panel/panel-samsung-s6e63m0-dsi.c
+++ b/drivers/gpu/drm/panel/panel-samsung-s6e63m0-dsi.c
@@ -23,11 +23,11 @@ static int s6e63m0_dsi_dcs_read(struct device *dev, const u8 cmd, u8 *data)
ret = mipi_dsi_dcs_read(dsi, cmd, data, 1);
if (ret < 0) {
- DRM_DEV_ERROR(dev, "could not read DCS CMD %02x\n", cmd);
+ dev_err(dev, "could not read DCS CMD %02x\n", cmd);
return ret;
}
- DRM_DEV_INFO(dev, "DSI read CMD %02x = %02x\n", cmd, *data);
+ dev_info(dev, "DSI read CMD %02x = %02x\n", cmd, *data);
return 0;
}
@@ -42,7 +42,7 @@ static int s6e63m0_dsi_dcs_write(struct device *dev, const u8 *data, size_t len)
int chunk;
int ret;
- DRM_DEV_INFO(dev, "DSI writing dcs seq: %*ph\n", (int)len, data);
+ dev_info(dev, "DSI writing dcs seq: %*ph\n", (int)len, data);
/* Pick out and skip past the DCS command */
cmd = *seqp;
@@ -56,9 +56,7 @@ static int s6e63m0_dsi_dcs_write(struct device *dev, const u8 *data, size_t len)
chunk = S6E63M0_DSI_MAX_CHUNK;
ret = mipi_dsi_dcs_write(dsi, cmd, seqp, chunk);
if (ret < 0) {
- DRM_DEV_ERROR(dev,
- "error sending DCS command seq cmd %02x\n",
- cmd);
+ dev_err(dev, "error sending DCS command seq cmd %02x\n", cmd);
return ret;
}
cmdwritten += chunk;
@@ -70,23 +68,19 @@ static int s6e63m0_dsi_dcs_write(struct device *dev, const u8 *data, size_t len)
chunk = S6E63M0_DSI_MAX_CHUNK;
ret = mipi_dsi_dcs_write(dsi, MCS_GLOBAL_PARAM, &cmdwritten, 1);
if (ret < 0) {
- DRM_DEV_ERROR(dev,
- "error sending CMD %02x global param %02x\n",
- cmd, cmdwritten);
+ dev_err(dev, "error sending CMD %02x global param %02x\n",
+ cmd, cmdwritten);
return ret;
}
ret = mipi_dsi_dcs_write(dsi, cmd, seqp, chunk);
if (ret < 0) {
- DRM_DEV_ERROR(dev,
- "error sending CMD %02x chunk\n",
- cmd);
+ dev_err(dev, "error sending CMD %02x chunk\n", cmd);
return ret;
}
cmdwritten += chunk;
seqp += chunk;
}
- DRM_DEV_INFO(dev, "sent command %02x %02x bytes\n",
- cmd, cmdwritten);
+ dev_info(dev, "sent command %02x %02x bytes\n", cmd, cmdwritten);
usleep_range(8000, 9000);
diff --git a/drivers/gpu/drm/panel/panel-samsung-s6e63m0-spi.c b/drivers/gpu/drm/panel/panel-samsung-s6e63m0-spi.c
index 3b1a2a3a44ea..d298d780220d 100644
--- a/drivers/gpu/drm/panel/panel-samsung-s6e63m0-spi.c
+++ b/drivers/gpu/drm/panel/panel-samsung-s6e63m0-spi.c
@@ -41,7 +41,7 @@ static int s6e63m0_spi_dcs_write(struct device *dev, const u8 *data, size_t len)
{
int ret = 0;
- DRM_DEV_DEBUG(dev, "SPI writing dcs seq: %*ph\n", (int)len, data);
+ dev_dbg(dev, "SPI writing dcs seq: %*ph\n", (int)len, data);
ret = s6e63m0_spi_write_word(dev, *data);
while (!ret && --len) {
@@ -50,8 +50,8 @@ static int s6e63m0_spi_dcs_write(struct device *dev, const u8 *data, size_t len)
}
if (ret) {
- DRM_DEV_ERROR(dev, "SPI error %d writing dcs seq: %*ph\n", ret,
- (int)len, data);
+ dev_err(dev, "SPI error %d writing dcs seq: %*ph\n", ret,
+ (int)len, data);
}
usleep_range(300, 310);
@@ -68,7 +68,7 @@ static int s6e63m0_spi_probe(struct spi_device *spi)
spi->mode = SPI_MODE_3;
ret = spi_setup(spi);
if (ret < 0) {
- DRM_DEV_ERROR(dev, "spi setup failed.\n");
+ dev_err(dev, "spi setup failed.\n");
return ret;
}
return s6e63m0_probe(dev, s6e63m0_spi_dcs_read, s6e63m0_spi_dcs_write,
diff --git a/drivers/gpu/drm/panel/panel-samsung-s6e63m0.c b/drivers/gpu/drm/panel/panel-samsung-s6e63m0.c
index c9472b2ae0cd..044294aafe27 100644
--- a/drivers/gpu/drm/panel/panel-samsung-s6e63m0.c
+++ b/drivers/gpu/drm/panel/panel-samsung-s6e63m0.c
@@ -179,28 +179,24 @@ static int s6e63m0_check_lcd_type(struct s6e63m0 *ctx)
ret = s6e63m0_clear_error(ctx);
if (ret) {
- DRM_DEV_ERROR(ctx->dev, "error checking LCD type (%d)\n",
- ret);
+ dev_err(ctx->dev, "error checking LCD type (%d)\n", ret);
ctx->lcd_type = 0x00;
return ret;
}
- DRM_DEV_INFO(ctx->dev, "MTP ID: %02x %02x %02x\n", id1, id2, id3);
+ dev_info(ctx->dev, "MTP ID: %02x %02x %02x\n", id1, id2, id3);
/* We attempt to detect what panel is mounted on the controller */
switch (id2) {
case S6E63M0_LCD_ID_VALUE_M2:
- DRM_DEV_INFO(ctx->dev,
- "detected LCD panel AMS397GE MIPI M2\n");
+ dev_info(ctx->dev, "detected LCD panel AMS397GE MIPI M2\n");
break;
case S6E63M0_LCD_ID_VALUE_SM2:
case S6E63M0_LCD_ID_VALUE_SM2_1:
- DRM_DEV_INFO(ctx->dev,
- "detected LCD panel AMS397GE MIPI SM2\n");
+ dev_info(ctx->dev, "detected LCD panel AMS397GE MIPI SM2\n");
break;
default:
- DRM_DEV_INFO(ctx->dev,
- "unknown LCD panel type %02x\n", id2);
+ dev_info(ctx->dev, "unknown LCD panel type %02x\n", id2);
break;
}