Re: [PATCH] drm/i915: remove noisy logs in intel_dp_dsc_get_output_bpp()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2022-06-07 at 11:05 +0300, Jani Nikula wrote:
> On Tue, 07 Jun 2022, Luca Coelho <luca@xxxxxxxxx> wrote:
> > From: Luca Coelho <luciano.coelho@xxxxxxxxx>
> > 
> > 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@xxxxxxxxx>
> 
> 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@xxxxxxxxx>

Thanks!

--
Cheers,
Luca.




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux