[Intel-gfx] [PATCH] drm/i915: remove noisy logs in intel_dp_dsc_get_output_bpp()

Luca Coelho luca at coelho.fi
Tue Jun 7 08:07:46 UTC 2022


On Tue, 2022-06-07 at 11:05 +0300, Jani Nikula wrote:
> On Tue, 07 Jun 2022, Luca Coelho <luca at coelho.fi> wrote:
> > From: Luca Coelho <luciano.coelho at intel.com>
> > 
> > The intel_dp_dsc_get_output_bpp() function outputs two lines of
> > unconditional logs, which was okay when it was called only once.  But
> > now, we also call this function from intel_dp_mode_valid(), which is
> > in turn called for every mode we need to validate.  This causes a lot
> > of useless noise.
> > 
> > Remove the unconditional prints to avoid spamming the logs.  Also
> > remove one more print that is not unconditional, but is related.
> > 
> > Signed-off-by: Luca Coelho <luciano.coelho at intel.com>
> 
> Some of these might be useful on the compute config path, but meh.

Yeah, as we discussed offline, it was not worth refactoring it for the
config path.  If needed, we can add them back in a less noisy way.


> Reviewed-by: Jani Nikula <jani.nikula at intel.com>

Thanks!

--
Cheers,
Luca.


More information about the Intel-gfx mailing list