On Mon, Mar 11, 2024 at 10:29:45AM -0500, Lucas De Marchi wrote: > On Mon, Mar 11, 2024 at 11:18:03AM -0400, Rodrigo Vivi wrote: > > On Wed, Mar 06, 2024 at 11:36:41AM -0800, Lucas De Marchi wrote: > > > With no platform declaring graphics/media IP_VER(12, 50), > > > > this is not true. > > We still have > > > > #define XE_HPM_FEATURES \ > > .__runtime.media.ip.ver = 12, \ > > .__runtime.media.ip.rel = 50 > > <snip> > > > > -#define XE_HPM_FEATURES \ > > > - .__runtime.media.ip.ver = 12, \ > > > - .__runtime.media.ip.rel = 50 > > > - > > ^ being removed here since all the users, like below, are overriding it. > > > > #define DG2_FEATURES \ > > > XE_HP_FEATURES, \ > > > - XE_HPM_FEATURES, \ > > > DGFX_FEATURES, \ > > > + .__runtime.graphics.ip.ver = 12, \ > > > .__runtime.graphics.ip.rel = 55, \ > > > + .__runtime.media.ip.ver = 12, \ > > > .__runtime.media.ip.rel = 55, \ > > ^^ > > After applying until this patch: > > $ git grep -e "rel[[:space:]]*=" -- drivers/gpu/drm/i915/i915_pci.c > drivers/gpu/drm/i915/i915_pci.c: .__runtime.graphics.ip.rel = 10, > drivers/gpu/drm/i915/i915_pci.c: .__runtime.graphics.ip.rel = 55, \ > drivers/gpu/drm/i915/i915_pci.c: .__runtime.media.ip.rel = 55, \ > drivers/gpu/drm/i915/i915_pci.c: .__runtime.graphics.ip.rel = 60, > drivers/gpu/drm/i915/i915_pci.c: .__runtime.media.ip.rel = 60, > drivers/gpu/drm/i915/i915_pci.c: .__runtime.graphics.ip.rel = 70, > > should I reword anything in the commit message to make my intent > clearer? doh! sorry.. I read the first line of the commit message and stopped. perhaps we could do that HPM removal in a separate patch before this one? Reviewed-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> on the final result, whatever you decide to split or to rephrase the msg. > > thanks > Lucas De Marchi