Skip to content

Commit 9f26907

Browse files
heminhongjnikula
authored andcommitted
drm/i915: correct the input parameter on _intel_dsb_commit()
Current, the dewake_scanline variable is defined as unsigned int, an unsigned int variable that is always greater than or equal to 0. when _intel_dsb_commit function is called by intel_dsb_commit function, the dewake_scanline variable may have an int value. So the dewake_scanline variable is necessary to defined as an int. Fixes: f83b94d ("drm/i915/dsb: Use DEwake to combat PkgC latency") Reported-by: kernel test robot <[email protected]> Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/ Cc: Ville Syrjälä <[email protected]> Cc: Uma Shankar <[email protected]> Signed-off-by: heminhong <[email protected]> Reviewed-by: Jani Nikula <[email protected]> Signed-off-by: Jani Nikula <[email protected]> Link: https://patchwork.freedesktop.org/patch/msgid/[email protected] (cherry picked from commit ef32c3c) Signed-off-by: Jani Nikula <[email protected]>
1 parent dd7eb65 commit 9f26907

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/gpu/drm/i915/display/intel_dsb.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -340,7 +340,7 @@ static int intel_dsb_dewake_scanline(const struct intel_crtc_state *crtc_state)
340340
}
341341

342342
static void _intel_dsb_commit(struct intel_dsb *dsb, u32 ctrl,
343-
unsigned int dewake_scanline)
343+
int dewake_scanline)
344344
{
345345
struct intel_crtc *crtc = dsb->crtc;
346346
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);

0 commit comments

Comments
 (0)