Re: [PATCH 4/4] drm/xe: Use intel_display_gse_irq_handler()

["Ville Syrjälä" <ville.syrjala@linux.intel.com>]

On Mon, Nov 09, 2026 at 11:51:05AM +0000, sashiko-bot@kernel.org wrote:
> Thank you for your contribution! Sashiko AI review found 1 potential
> issue to consider:
> - [High] drm/xe: call to undeclared and undefined
>   intel_display_gse_irq_handler()
> --
> 
> commit da9b4f3b875bfbf33a72170de1fb2c018547566b
> Author: Ville Syrjälä <ville.syrjala@linux.intel.com>
> 
> drm/xe: Use intel_display_gse_irq_handler()
> 
> This commit replaces the direct call to intel_opregion_asle_intr() in
> xe_display_gse_irq_handler() with intel_display_gse_irq_handler().
> 
> > diff --git a/drivers/gpu/drm/xe/display/xe_display.c b/drivers/gpu/drm/xe/display/xe_display.c
> > index 21692408f9be0..0236f69a0f883 100644
> > --- a/drivers/gpu/drm/xe/display/xe_display.c
> > +++ b/drivers/gpu/drm/xe/display/xe_display.c
> > @@ -220,7 +220,7 @@ void xe_display_gse_irq_handler(struct xe_device *xe, u32 gu_misc_iir)
> >  		return;
> >  
> >  	if (gu_misc_iir & GU_MISC_GSE)
> > -		intel_opregion_asle_intr(display);
> > +		intel_display_gse_irq_handler(display);
> >  }
> 
> [Severity: High]
> Does this break the build when CONFIG_DRM_XE_DISPLAY is enabled?
> intel_display_gse_irq_handler() is neither declared nor defined anywhere
> in the tree, causing an implicit function declaration error during
> compilation and an undefined symbol at link time.

Your baseline is too old.

-- 
Ville Syrjälä
Intel