Skip to content
Snippets Groups Projects
Commit e517db73 authored by Simon Glass's avatar Simon Glass Committed by Anatolij Gustschin
Browse files

cmd: Fix control bmp_display()


All paths should share the same return.

Reported-by: Coverity (CID:134903)

Signed-off-by: default avatarSimon Glass <sjg@chromium.org>
Reviewed-by: default avatarTom Rini <trini@konsulko.com>
parent a7c45ec4
No related branches found
No related tags found
No related merge requests found
...@@ -259,7 +259,6 @@ int bmp_display(ulong addr, int x, int y) ...@@ -259,7 +259,6 @@ int bmp_display(ulong addr, int x, int y)
ret = video_bmp_display(dev, addr, x, y, align); ret = video_bmp_display(dev, addr, x, y, align);
} }
} }
return ret ? CMD_RET_FAILURE : 0;
#elif defined(CONFIG_LCD) #elif defined(CONFIG_LCD)
ret = lcd_display_bitmap(addr, x, y); ret = lcd_display_bitmap(addr, x, y);
#elif defined(CONFIG_VIDEO) #elif defined(CONFIG_VIDEO)
...@@ -271,5 +270,5 @@ int bmp_display(ulong addr, int x, int y) ...@@ -271,5 +270,5 @@ int bmp_display(ulong addr, int x, int y)
if (bmp_alloc_addr) if (bmp_alloc_addr)
free(bmp_alloc_addr); free(bmp_alloc_addr);
return ret; return ret ? CMD_RET_FAILURE : 0;
} }
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment