]> git.baikalelectronics.ru Git - kernel.git/commit
drm/msm/mdp5: Do not multiply vclk line count by 100
authorMarijn Suijten <marijn.suijten@somainline.org>
Tue, 6 Apr 2021 21:47:25 +0000 (23:47 +0200)
committerRob Clark <robdclark@chromium.org>
Fri, 9 Apr 2021 19:02:35 +0000 (12:02 -0700)
commit127bbcd5c75d7ea4dcb80a5c57e19f7cb73472a3
tree6ce180300116a76009638ad8508905736ab468bd
parent7fa6eafacd4ce19a7e5624ba9e3a7aa9c1652b39
drm/msm/mdp5: Do not multiply vclk line count by 100

Neither vtotal nor drm_mode_vrefresh contain a value that is
premultiplied by 100 making the x100 variable name incorrect and
resulting in vclks_line to become 100 times larger than it is supposed
to be.  The hardware counts 100 clockticks too many before tearcheck,
leading to severe panel issues on at least the Sony Xperia lineup.

This is likely an artifact from the original MDSS DSI panel driver where
the calculation [1] corrected for a premultiplied reference framerate by
100 [2].  It does not appear that the above values were ever
premultiplied in the history of the DRM MDP5 driver.

With this change applied the value written to the SYNC_CONFIG_VSYNC
register is now identical to downstream kernels.

[1]: https://source.codeaurora.org/quic/la/kernel/msm-3.18/tree/drivers/video/msm/mdss/mdss_mdp_intf_cmd.c?h=LA.UM.8.6.c26-02400-89xx.0#n288
[2]: https://source.codeaurora.org/quic/la/kernel/msm-3.18/tree/drivers/video/msm/mdss/mdss_dsi_panel.c?h=LA.UM.8.6.c26-02400-89xx.0#n1648

Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org>
Signed-off-by: Marijn Suijten <marijn.suijten@somainline.org>
Link: https://lore.kernel.org/r/20210406214726.131534-3-marijn.suijten@somainline.org
Signed-off-by: Rob Clark <robdclark@chromium.org>
drivers/gpu/drm/msm/disp/mdp5/mdp5_cmd_encoder.c