HFP/HBP/HSA/EOT_PACKET modes in Exynos DSI host specifies
0 = Enable and 1 = Disable.

The logic for checking these mode flags was correct before
the MIPI_DSI*_NO_* mode flag conversion.

Fix the MIPI_DSI*_NO_* mode flags handling.

Fixes: <0f3b68b66a6d> ("drm/dsi: Add _NO_ to MIPI_DSI_* flags disabling
features")
Reviewed-by: Nicolas Boichat <drink...@chromium.org>
Reported-by: Sébastien Szymanski <sebastien.szyman...@armadeus.com>
Signed-off-by: Jagan Teki <ja...@amarulasolutions.com>
---
Changes for v9:
- none

 drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c 
b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
index e5b1540c4ae4..50a2a9ca88a9 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
@@ -805,15 +805,15 @@ static int exynos_dsi_init_link(struct exynos_dsi *dsi)
                        reg |= DSIM_AUTO_MODE;
                if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO_HSE)
                        reg |= DSIM_HSE_MODE;
-               if (!(dsi->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HFP))
+               if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HFP)
                        reg |= DSIM_HFP_MODE;
-               if (!(dsi->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HBP))
+               if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HBP)
                        reg |= DSIM_HBP_MODE;
-               if (!(dsi->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HSA))
+               if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HSA)
                        reg |= DSIM_HSA_MODE;
        }
 
-       if (!(dsi->mode_flags & MIPI_DSI_MODE_NO_EOT_PACKET))
+       if (dsi->mode_flags & MIPI_DSI_MODE_NO_EOT_PACKET)
                reg |= DSIM_EOT_DISABLE;
 
        switch (dsi->format) {
-- 
2.25.1

Reply via email to