drm/i915: Fix checkpatch warns in cursor code
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Tue, 10 Nov 2020 17:56:24 +0000 (19:56 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Mon, 11 Jan 2021 17:38:25 +0000 (19:38 +0200)
Eliminate checkpatch warnings from intel_cursor.c:
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
WARNING: Possible repeated word: 'by'

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20201110175624.3524-2-ville.syrjala@linux.intel.com
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
drivers/gpu/drm/i915/display/intel_cursor.c

index 276d2bb..21fe4d2 100644 (file)
@@ -492,8 +492,8 @@ static void i9xx_update_cursor(struct intel_plane *plane,
        unsigned long irqflags;
 
        if (plane_state && plane_state->uapi.visible) {
-               unsigned width = drm_rect_width(&plane_state->uapi.dst);
-               unsigned height = drm_rect_height(&plane_state->uapi.dst);
+               int width = drm_rect_width(&plane_state->uapi.dst);
+               int height = drm_rect_height(&plane_state->uapi.dst);
 
                cntl = plane_state->ctl |
                        i9xx_cursor_ctl_crtc(crtc_state);
@@ -522,7 +522,7 @@ static void i9xx_update_cursor(struct intel_plane *plane,
         * cursor that doesn't appear to move, or even change
         * shape. Thus we always write CURBASE.
         *
-        * The other registers are armed by by the CURBASE write
+        * The other registers are armed by the CURBASE write
         * except when the plane is getting enabled at which time
         * the CURCNTR write arms the update.
         */