Commit da85f0aa authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Neil Armstrong

drm/panel: ili9341: Use predefined error codes

In one case the -1 is returned which is quite confusing code for
the wrong device ID, in another the ret is returning instead of
plain 0 that also confusing as readed may ask the possible meaning
of positive codes, which are never the case there. Convert both
to use explicit predefined error codes to make it clear what's going
on there.

Fixes: 5a042273 ("drm/panel: Add ilitek ili9341 panel driver")
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: default avatarNeil Armstrong <neil.armstrong@linaro.org>
Reviewed-by: default avatarSui Jingfeng <sui.jingfeng@linux.dev>
Link: https://lore.kernel.org/r/20240425142706.2440113-4-andriy.shevchenko@linux.intel.comSigned-off-by: default avatarNeil Armstrong <neil.armstrong@linaro.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20240425142706.2440113-4-andriy.shevchenko@linux.intel.com
parent 740fc1e0
...@@ -422,7 +422,7 @@ static int ili9341_dpi_prepare(struct drm_panel *panel) ...@@ -422,7 +422,7 @@ static int ili9341_dpi_prepare(struct drm_panel *panel)
ili9341_dpi_init(ili); ili9341_dpi_init(ili);
return ret; return 0;
} }
static int ili9341_dpi_enable(struct drm_panel *panel) static int ili9341_dpi_enable(struct drm_panel *panel)
...@@ -726,7 +726,7 @@ static int ili9341_probe(struct spi_device *spi) ...@@ -726,7 +726,7 @@ static int ili9341_probe(struct spi_device *spi)
else if (!strcmp(id->name, "yx240qv29")) else if (!strcmp(id->name, "yx240qv29"))
return ili9341_dbi_probe(spi, dc, reset); return ili9341_dbi_probe(spi, dc, reset);
return -1; return -ENODEV;
} }
static void ili9341_remove(struct spi_device *spi) static void ili9341_remove(struct spi_device *spi)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment