On Wed, 05 Jun 2024, Ville Syrjala <ville.syrjala@xxxxxxxxxxxxxxx> wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > No idea why the DDI clock gating print is done with drm_notice(). > Just use drm_dbg_kms() since no one is going to be interested in > this under normal circumstances. > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx> > --- > drivers/gpu/drm/i915/display/intel_ddi.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c > index 3c3fc53376ce..c76bbd13bb27 100644 > --- a/drivers/gpu/drm/i915/display/intel_ddi.c > +++ b/drivers/gpu/drm/i915/display/intel_ddi.c > @@ -2072,9 +2072,9 @@ void intel_ddi_sanitize_encoder_pll_mapping(struct intel_encoder *encoder) > !encoder->is_clock_enabled(encoder)) > return; > > - drm_notice(&i915->drm, > - "[ENCODER:%d:%s] is disabled/in DSI mode with an ungated DDI clock, gate it\n", > - encoder->base.base.id, encoder->base.name); > + drm_dbg_kms(&i915->drm, > + "[ENCODER:%d:%s] is disabled/in DSI mode with an ungated DDI clock, gate it\n", > + encoder->base.base.id, encoder->base.name); > > encoder->disable_clock(encoder); > } -- Jani Nikula, Intel
