From 8256494319229b33abfd3bc416d2272e6a8fb7c6 Mon Sep 17 00:00:00 2001 From: Flavio Ipiranga Date: Thu, 30 Sep 2021 15:18:28 -0300 Subject: [PATCH] Fix: display blinking issue Brightness parameter was being updated more than once and more than one way inside dsi_panel_update_backlight function. This parameter should be set just once and only in one way inside dsi_panel_update_backlight function. Change-Id: I8090e04034fa2fb2c4590b643df0a8a303759129 Reviewed-on: https://gerrit.mot.com/2081098 SME-Granted: SME Approvals Granted SLTApproved: Slta Waiver Tested-by: Jira Key Reviewed-by: Rafael Ortolan Reviewed-by: Shuo Yan Reviewed-by: Ling Jin Submit-Approved: Jira Key --- msm/dsi/dsi_panel.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/msm/dsi/dsi_panel.c b/msm/dsi/dsi_panel.c index 8f72008cb4a3..a7c32f41754c 100755 --- a/msm/dsi/dsi_panel.c +++ b/msm/dsi/dsi_panel.c @@ -832,10 +832,7 @@ static int dsi_panel_update_backlight(struct dsi_panel *panel, if (panel->bl_config.bl_dcs_subtype == 0xc2) rc = dsi_panel_dcs_set_display_brightness_c2(dsi, bl_lvl); - else - rc = mipi_dsi_dcs_set_display_brightness(dsi, bl_lvl); - - if (bl->bl_2bytes_enable) + else if (bl->bl_2bytes_enable) rc = mipi_dsi_dcs_set_display_brightness_2bytes(dsi, bl_lvl); else rc = mipi_dsi_dcs_set_display_brightness(dsi, bl_lvl);