On Tue, 29 Jul 2025, "Kandpal, Suraj" <[email protected]> wrote: >> Subject: [PATCH] drm/i915/display: remove superfluous <linux/types.h> >> includes >> >> Commit f7a9dc796567 ("drm/i915/scaler: Use intel_display as argument to >> skl_scaler_max_src_size") added superfluous includes. Remove them. > > I think the commit message quoted here is wrong the correct one would be > Fixes: 3347b55f2c6c ("drm/i915/xe3lpd: Prune modes for YUV420")
Oops, my bad, sorry. Unfortunately it's already merged, making my mistake permanent. :/ > Also Fixes tag. I don't usually bother with Fixes: when there's no need to backport or anything. BR, Jani. > > Regards, > Suraj Kandpal > >> >> Cc: Suraj Kandpal <[email protected]> >> Cc: Ankit Nautiyal <[email protected]> >> Signed-off-by: Jani Nikula <[email protected]> >> --- >> drivers/gpu/drm/i915/display/intel_pfit.h | 2 -- >> drivers/gpu/drm/i915/display/skl_scaler.h | 2 -- >> 2 files changed, 4 deletions(-) >> >> diff --git a/drivers/gpu/drm/i915/display/intel_pfit.h >> b/drivers/gpu/drm/i915/display/intel_pfit.h >> index 2f075748a34e..c1bb0d1f344e 100644 >> --- a/drivers/gpu/drm/i915/display/intel_pfit.h >> +++ b/drivers/gpu/drm/i915/display/intel_pfit.h >> @@ -6,8 +6,6 @@ >> #ifndef __INTEL_PFIT_H__ >> #define __INTEL_PFIT_H__ >> >> -#include <linux/types.h> >> - >> enum drm_mode_status; >> struct drm_display_mode; >> struct drm_connector_state; >> diff --git a/drivers/gpu/drm/i915/display/skl_scaler.h >> b/drivers/gpu/drm/i915/display/skl_scaler.h >> index 692716dd7616..e3b35d2c13be 100644 >> --- a/drivers/gpu/drm/i915/display/skl_scaler.h >> +++ b/drivers/gpu/drm/i915/display/skl_scaler.h >> @@ -5,8 +5,6 @@ >> #ifndef INTEL_SCALER_H >> #define INTEL_SCALER_H >> >> -#include <linux/types.h> >> - >> enum drm_mode_status; >> struct drm_display_mode; >> struct intel_atomic_state; >> -- >> 2.39.5 > -- Jani Nikula, Intel
