Commit 76544e4b authored by Colin Ian King's avatar Colin Ian King Committed by Rob Clark
Browse files

drm/msm/dp: Remove redundant initialization of variable bpp 



The variable bpp is being initialized with a value that is never
read, it is being updated later on in both paths of an if statement.
The assignment is redundant and can be removed.

Addresses-Coverity: ("Unused value")
Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
Reviewed-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Link: https://lore.kernel.org/r/20210929105458.209895-1-colin.king@canonical.com


Signed-off-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Signed-off-by: default avatarRob Clark <robdclark@chromium.org>
parent 9960f7a8

drivers/gpu/drm/msm/dp/dp_panel.c

+1 −1
Original line number Diff line number Diff line
@@ -234,7 +234,7 @@ u32 dp_panel_get_mode_bpp(struct dp_panel *dp_panel, 
		u32 mode_edid_bpp, u32 mode_pclk_khz)

{

	struct dp_panel_private *panel;

	u32 bpp = mode_edid_bpp;

	u32 bpp;



	if (!dp_panel || !mode_edid_bpp || !mode_pclk_khz) {

		DRM_ERROR("invalid input\n");