From: Maxime Ripard <[email protected]>

Unlike what is currently being done, the ACCESS_CTRL bit documentation asks
that this bit should be set before modifying any register. The code in the
BSP also does this, so make sure we do this as well.

Signed-off-by: Maxime Ripard <[email protected]>
Signed-off-by: Paul Kocialkowski <[email protected]>
---
 drivers/gpu/drm/sun4i/sun4i_frontend.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/sun4i/sun4i_frontend.c 
b/drivers/gpu/drm/sun4i/sun4i_frontend.c
index d69d426c3014..c80338368992 100644
--- a/drivers/gpu/drm/sun4i/sun4i_frontend.c
+++ b/drivers/gpu/drm/sun4i/sun4i_frontend.c
@@ -74,6 +74,10 @@ static void sun4i_frontend_scaler_init(struct sun4i_frontend 
*frontend)
 {
        int i;
 
+       regmap_write_bits(frontend->regs, SUN4I_FRONTEND_FRM_CTRL_REG,
+                         SUN4I_FRONTEND_FRM_CTRL_COEF_ACCESS_CTRL,
+                         SUN4I_FRONTEND_FRM_CTRL_COEF_ACCESS_CTRL);
+
        for (i = 0; i < 32; i++) {
                regmap_write(frontend->regs, 
SUN4I_FRONTEND_CH0_HORZCOEF0_REG(i),
                             sun4i_frontend_horz_coef[2 * i]);
@@ -89,9 +93,6 @@ static void sun4i_frontend_scaler_init(struct sun4i_frontend 
*frontend)
                             sun4i_frontend_vert_coef[i]);
        }
 
-       regmap_update_bits(frontend->regs, SUN4I_FRONTEND_FRM_CTRL_REG,
-                          SUN4I_FRONTEND_FRM_CTRL_COEF_ACCESS_CTRL,
-                          SUN4I_FRONTEND_FRM_CTRL_COEF_ACCESS_CTRL);
 }
 
 int sun4i_frontend_init(struct sun4i_frontend *frontend)
-- 
2.19.2

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to