On Fri, 17 Dec 2021, Ville Syrjälä <ville.syrj...@linux.intel.com> wrote: > On Fri, Dec 17, 2021 at 08:02:55AM -0800, Harish Chegondi wrote: >> Check return pointer from intel_crtc_for_plane() before dereferencing >> it, as it can be NULL. > > Can't actually bu NULL. But I guess no real harm in having the > check if it shuts up some static analysis thing.
That's what I figured too, and better have it here than spread around wherever this is called. Reviewed-by: Jani Nikula <jani.nik...@intel.com> > >> >> v2: Moved the NULL check into intel_crtc_active(). >> >> Cc: Jani Nikula <jani.nik...@intel.com> >> Cc: Caz Yokoyama <caz.yokoy...@intel.com> >> Cc: Radhakrishna Sripada <radhakrishna.srip...@intel.com> >> Signed-off-by: Harish Chegondi <harish.chego...@intel.com> >> --- >> drivers/gpu/drm/i915/intel_pm.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/i915/intel_pm.c >> b/drivers/gpu/drm/i915/intel_pm.c >> index bdf97a8c9ef3..8b357ec35a4a 100644 >> --- a/drivers/gpu/drm/i915/intel_pm.c >> +++ b/drivers/gpu/drm/i915/intel_pm.c >> @@ -877,7 +877,7 @@ static bool intel_crtc_active(struct intel_crtc *crtc) >> * crtc->state->active once we have proper CRTC states wired up >> * for atomic. >> */ >> - return crtc->active && crtc->base.primary->state->fb && >> + return crtc && crtc->active && crtc->base.primary->state->fb && >> crtc->config->hw.adjusted_mode.crtc_clock; >> } >> >> -- >> 2.31.1 -- Jani Nikula, Intel Open Source Graphics Center
