Re: [PATCH] drm/ast: fix missing break in switch statement for format->cpp[0] case 4

2020-06-10 Thread Thomas Zimmermann
Hi Am 10.06.20 um 13:58 schrieb Colin King: > From: Colin Ian King > > Currently the switch statement for format->cpp[0] value 4 assigns > color_index which is never read again and then falls through to the > default case and returns. This looks like a missing break statement > bug. Fix this by

[PATCH] drm/ast: fix missing break in switch statement for format->cpp[0] case 4

2020-06-10 Thread Colin King
From: Colin Ian King Currently the switch statement for format->cpp[0] value 4 assigns color_index which is never read again and then falls through to the default case and returns. This looks like a missing break statement bug. Fix this by adding a break statement. Addresses-Coverity: ("Unused v