From b11fcc8e0c327c0722271a2aecea730382c5ceb5 Mon Sep 17 00:00:00 2001 From: Georgij Cernysiov Date: Tue, 10 May 2022 23:24:04 +0200 Subject: [PATCH] drivers: display: stm32_ltdc: fix set orientation API Refactors invalid switch into if statement. Fixes orientation set return value for normal display orientation. Signed-off-by: Georgij Cernysiov --- drivers/display/display_stm32_ltdc.c | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/drivers/display/display_stm32_ltdc.c b/drivers/display/display_stm32_ltdc.c index 9516b784291..937cccf6e64 100644 --- a/drivers/display/display_stm32_ltdc.c +++ b/drivers/display/display_stm32_ltdc.c @@ -129,16 +129,13 @@ static int stm32_ltdc_set_pixel_format(const struct device *dev, static int stm32_ltdc_set_orientation(const struct device *dev, const enum display_orientation orientation) { - int err; + ARG_UNUSED(dev); - switch (orientation) { - case DISPLAY_ORIENTATION_NORMAL: - err = 0; - default: - err = -ENOTSUP; + if (orientation == DISPLAY_ORIENTATION_NORMAL) { + return 0; } - return err; + return -ENOTSUP; } static void stm32_ltdc_get_capabilities(const struct device *dev,