On Thu, May 12, 2022 at 07:16:38PM +0300, Jani Nikula wrote:
> Need to use pipe_name(pipe) instead of pipe directly.
>
> Fixes: 1f31e35f2e88 ("drm/i915/audio: unify audio codec enable/disable debug logging")
> Cc: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx>
Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> ---
> drivers/gpu/drm/i915/display/intel_audio.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_audio.c b/drivers/gpu/drm/i915/display/intel_audio.c
> index 1c87bf66b092..f0f0dfce27ce 100644
> --- a/drivers/gpu/drm/i915/display/intel_audio.c
> +++ b/drivers/gpu/drm/i915/display/intel_audio.c
> @@ -827,7 +827,7 @@ void intel_audio_codec_enable(struct intel_encoder *encoder,
> drm_dbg_kms(&dev_priv->drm, "[CONNECTOR:%d:%s][ENCODER:%d:%s] Enable audio codec on pipe %c, %u bytes ELD\n",
> connector->base.id, connector->name,
> encoder->base.base.id, encoder->base.name,
> - pipe, drm_eld_size(connector->eld));
> + pipe_name(pipe), drm_eld_size(connector->eld));
>
> /* FIXME precompute the ELD in .compute_config() */
> if (!connector->eld[0])
> @@ -888,7 +888,7 @@ void intel_audio_codec_disable(struct intel_encoder *encoder,
>
> drm_dbg_kms(&dev_priv->drm, "[CONNECTOR:%d:%s][ENCODER:%d:%s] Disable audio codec on pipe %c\n",
> connector->base.id, connector->name,
> - encoder->base.base.id, encoder->base.name, pipe);
> + encoder->base.base.id, encoder->base.name, pipe_name(pipe));
>
> if (dev_priv->audio.funcs)
> dev_priv->audio.funcs->audio_codec_disable(encoder,
> --
> 2.30.2
--
Ville Syrjälä
Intel
