The purpose of this mask is to simplify writing to the lxcr
register and not to forget any fields.

Signed-off-by: Yannick Fertre <[email protected]>
---
 drivers/gpu/drm/stm/ltdc.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
index cb4f08673a64..3876346a1303 100644
--- a/drivers/gpu/drm/stm/ltdc.c
+++ b/drivers/gpu/drm/stm/ltdc.c
@@ -187,6 +187,7 @@
 #define LXCR_COLKEN    BIT(1)          /* Color Keying Enable */
 #define LXCR_CLUTEN    BIT(4)          /* Color Look-Up Table ENable */
 #define LXCR_HMEN      BIT(8)          /* Horizontal Mirroring ENable */
+#define LXCR_MASK (LXCR_LEN | LXCR_COLKEN | LXCR_CLUTEN | LXCR_HMEN)
 
 #define LXWHPCR_WHSTPOS        GENMASK(11, 0)  /* Window Horizontal StarT 
POSition */
 #define LXWHPCR_WHSPPOS        GENMASK(27, 16) /* Window Horizontal StoP 
POSition */
@@ -806,8 +807,7 @@ static void ltdc_crtc_atomic_disable(struct drm_crtc *crtc,
 
        /* Disable all layers */
        for (layer_index = 0; layer_index < ldev->caps.nb_layers; layer_index++)
-               regmap_write_bits(ldev->regmap, LTDC_L1CR + layer_index * 
LAY_OFS,
-                                 LXCR_CLUTEN | LXCR_LEN, 0);
+               regmap_write_bits(ldev->regmap, LTDC_L1CR + layer_index * 
LAY_OFS, LXCR_MASK, 0);
 
        /* disable IRQ */
        regmap_clear_bits(ldev->regmap, LTDC_IER, IER_FUWIE | IER_FUEIE | 
IER_RRIE | IER_TERRIE);
@@ -1474,7 +1474,7 @@ static void ltdc_plane_atomic_update(struct drm_plane 
*plane,
        if (newstate->rotation & DRM_MODE_REFLECT_X)
                val |= LXCR_HMEN;
 
-       regmap_write_bits(ldev->regmap, LTDC_L1CR + lofs, LXCR_LEN | 
LXCR_CLUTEN | LXCR_HMEN, val);
+       regmap_write_bits(ldev->regmap, LTDC_L1CR + lofs, LXCR_MASK, val);
 
        /* Commit shadow registers = update plane at next vblank */
        if (ldev->caps.plane_reg_shadow)
@@ -1512,7 +1512,7 @@ static void ltdc_plane_atomic_disable(struct drm_plane 
*plane,
        u32 lofs = plane->index * LAY_OFS;
 
        /* Disable layer */
-       regmap_write_bits(ldev->regmap, LTDC_L1CR + lofs, LXCR_LEN | 
LXCR_CLUTEN |  LXCR_HMEN, 0);
+       regmap_write_bits(ldev->regmap, LTDC_L1CR + lofs, LXCR_MASK, 0);
 
        /* Reset the layer transparency to hide any related background color */
        regmap_write_bits(ldev->regmap, LTDC_L1CACR + lofs, LXCACR_CONSTA, 
0x00);
-- 
2.34.1

Reply via email to