On Wed, 2026-03-04 at 16:40 +0200, Jani Nikula wrote: > Now that the step name is in runtime info, switch to using it instead of > intel_step_name(). > > The ** are only relevant for DMC, so make their use explicit. > > Signed-off-by: Jani Nikula <[email protected]> > ---
Reviewed-by: Luca Coelho <[email protected]> -- Cheers, Luca. > drivers/gpu/drm/i915/display/intel_dmc.c | 7 +++---- > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_dmc.c > b/drivers/gpu/drm/i915/display/intel_dmc.c > index 4e763f2f17c7..b250d41dc6a1 100644 > --- a/drivers/gpu/drm/i915/display/intel_dmc.c > +++ b/drivers/gpu/drm/i915/display/intel_dmc.c > @@ -39,7 +39,6 @@ > #include "intel_dmc.h" > #include "intel_dmc_regs.h" > #include "intel_flipq.h" > -#include "intel_step.h" > > /** > * DOC: DMC Firmware Support > @@ -420,10 +419,10 @@ bool intel_dmc_has_payload(struct intel_display > *display) > > static void initialize_stepping_info(struct intel_display *display, struct > stepping_info *si) > { > - const char *step_name = intel_step_name(INTEL_DISPLAY_STEP(display)); > + const char *step_name = DISPLAY_RUNTIME_INFO(display)->step_name; > > - si->stepping = step_name[0]; > - si->substepping = step_name[1]; > + si->stepping = step_name[0] ?: '*'; > + si->substepping = step_name[1] ?: '*'; > } > > static void gen9_set_dc_state_debugmask(struct intel_display *display)
