drm/i915: Use u8 consistently for active_planes bitmask
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Fri, 14 May 2021 12:57:40 +0000 (15:57 +0300)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Wed, 22 Sep 2021 13:48:05 +0000 (16:48 +0300)
Be consistent in that active_planes bitmask fits in a u8.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210514125751.17075-4-ville.syrjala@linux.intel.com
Reviewed-by: Stanislav Lisovskiy <stanislav.lisovskiy@intel.com>
drivers/gpu/drm/i915/intel_pm.c

index cfc41f8fa74a743f85e0938149bf3627b36a8aba..9452f5af070d68dc0cb57708d68d96045f73310d 100644 (file)
@@ -1708,7 +1708,7 @@ static int vlv_compute_fifo(struct intel_crtc_state *crtc_state)
        const struct g4x_pipe_wm *raw =
                &crtc_state->wm.vlv.raw[VLV_WM_LEVEL_PM2];
        struct vlv_fifo_state *fifo_state = &crtc_state->wm.vlv.fifo_state;
-       unsigned int active_planes = crtc_state->active_planes & ~BIT(PLANE_CURSOR);
+       u8 active_planes = crtc_state->active_planes & ~BIT(PLANE_CURSOR);
        int num_active_planes = hweight8(active_planes);
        const int fifo_size = 511;
        int fifo_extra, fifo_left = fifo_size;
@@ -1900,8 +1900,8 @@ static int vlv_compute_pipe_wm(struct intel_atomic_state *state,
        struct vlv_wm_state *wm_state = &crtc_state->wm.vlv.optimal;
        const struct vlv_fifo_state *fifo_state =
                &crtc_state->wm.vlv.fifo_state;
-       int num_active_planes = hweight8(crtc_state->active_planes &
-                                        ~BIT(PLANE_CURSOR));
+       u8 active_planes = crtc_state->active_planes & ~BIT(PLANE_CURSOR);
+       int num_active_planes = hweight8(active_planes);
        bool needs_modeset = drm_atomic_crtc_needs_modeset(&crtc_state->uapi);
        const struct intel_plane_state *old_plane_state;
        const struct intel_plane_state *new_plane_state;