[Intel-gfx] [PATCH] drm/i915/bios: remove extra debug messages
Jani Nikula
jani.nikula at linux.intel.com
Mon Dec 9 09:58:57 UTC 2019
On Fri, 06 Dec 2019, Lucas De Marchi <lucas.demarchi at intel.com> wrote:
> Just like in 523e0cc89b83 ("drm/i915/tgl: allow DVI/HDMI on port A"),
> the port checks when reading the VBT can easily not match what the
> platform really exposes. However here we only have some additional debug
> messages that are not adding much value: in the previous debug message
> we already print everything we know about the VBT.
>
> Instead of keep fixing the possible port assignments according to the
> platform, just nuke the additional messages.
>
> Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
Cc: Ville
Acked-by: Jani Nikula <jani.nikula at intel.com>
> ---
> drivers/gpu/drm/i915/display/intel_bios.c | 16 ----------------
> 1 file changed, 16 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_bios.c b/drivers/gpu/drm/i915/display/intel_bios.c
> index 1aeecdd02293..e054e87eba8e 100644
> --- a/drivers/gpu/drm/i915/display/intel_bios.c
> +++ b/drivers/gpu/drm/i915/display/intel_bios.c
> @@ -1602,22 +1602,6 @@ static void parse_ddi_port(struct drm_i915_private *dev_priv,
> HAS_LSPCON(dev_priv) && child->lspcon,
> info->supports_typec_usb, info->supports_tbt);
>
> - if (is_edp && is_dvi)
> - DRM_DEBUG_KMS("Internal DP port %c is TMDS compatible\n",
> - port_name(port));
> - if (is_crt && port != PORT_E)
> - DRM_DEBUG_KMS("Port %c is analog\n", port_name(port));
> - if (is_crt && (is_dvi || is_dp))
> - DRM_DEBUG_KMS("Analog port %c is also DP or TMDS compatible\n",
> - port_name(port));
> - if (is_dvi && (port == PORT_A || port == PORT_E))
> - DRM_DEBUG_KMS("Port %c is TMDS compatible\n", port_name(port));
> - if (!is_dvi && !is_dp && !is_crt)
> - DRM_DEBUG_KMS("Port %c is not DP/TMDS/CRT compatible\n",
> - port_name(port));
> - if (is_edp && (port == PORT_B || port == PORT_C || port == PORT_E))
> - DRM_DEBUG_KMS("Port %c is internal DP\n", port_name(port));
> -
> if (is_dvi) {
> u8 ddc_pin;
--
Jani Nikula, Intel Open Source Graphics Center
More information about the Intel-gfx
mailing list