Commit 4464bd82 authored by Luca Coelho's avatar Luca Coelho Committed by Jani Nikula
Browse files

drm/i915: remove noisy logs in intel_dp_dsc_get_output_bpp() 



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: default avatarLuca Coelho <luciano.coelho@intel.com>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220607074433.1202917-1-luca@coelho.fi
parent aec23025

drivers/gpu/drm/i915/display/intel_dp.c

+0 −5
Original line number Diff line number Diff line
@@ -657,7 +657,6 @@ static u16 intel_dp_dsc_get_output_bpp(struct drm_i915_private *i915, 
	 */

	bits_per_pixel = (link_clock * lane_count * 8) /

			 intel_dp_mode_to_fec_clock(mode_clock);

	drm_dbg_kms(&i915->drm, "Max link bpp: %u\n", bits_per_pixel);



	/* Small Joiner Check: output bpp <= joiner RAM (bits) / Horiz. width */

	max_bpp_small_joiner_ram = small_joiner_ram_size_bits(i915) /
@@ -666,9 +665,6 @@ static u16 intel_dp_dsc_get_output_bpp(struct drm_i915_private *i915, 
	if (bigjoiner)

		max_bpp_small_joiner_ram *= 2;



	drm_dbg_kms(&i915->drm, "Max small joiner bpp: %u\n",

		    max_bpp_small_joiner_ram);



	/*

	 * Greatest allowed DSC BPP = MIN (output BPP from available Link BW

	 * check, output bpp from small joiner RAM check)
@@ -680,7 +676,6 @@ static u16 intel_dp_dsc_get_output_bpp(struct drm_i915_private *i915, 
			i915->max_cdclk_freq * 48 /

			intel_dp_mode_to_fec_clock(mode_clock);



		drm_dbg_kms(&i915->drm, "Max big joiner bpp: %u\n", max_bpp_bigjoiner);

		bits_per_pixel = min(bits_per_pixel, max_bpp_bigjoiner);

	}