On Fri, 02 Dec 2016, "Yang, Libin" <libin.yang@xxxxxxxxx> wrote:
> Any comments? Thanks.
Both pushed to dinq, thanks for the patches.
Libin and/or Dhinakaran, please improve the documentation of struct
i915_audio_component_ops to describe the parameters of sync_audio_rate
and get_eld hooks. For example, there's nothing about pipe being -1 for
non-MST.
BR,
Jani.
>
> Regards,
> Libin
>
>
>>-----Original Message-----
>>From: Yang, Libin
>>Sent: Thursday, December 1, 2016 1:17 PM
>>To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx; jani.nikula@xxxxxxxxxxxxxxx;
>>ville.syrjala@xxxxxxxxxxxxxxx; Vetter, Daniel <daniel.vetter@xxxxxxxxx>;
>>Pandiyan, Dhinakaran <dhinakaran.pandiyan@xxxxxxxxx>; tiwai@xxxxxxx
>>Cc: Yang, Libin <libin.yang@xxxxxxxxx>
>>Subject: [PATCH 2/2] drm/i915/audio: extend audio sync rate support for DP
>>MST
>>
>>From: Libin Yang <libin.yang@xxxxxxxxx>
>>
>>Remove the type judgement in i915_audio_component_sync_audio_rate().
>>Audio rate sync is necessary for all i915 digital audio now.
>>
>>Signed-off-by: Libin Yang <libin.yang@xxxxxxxxx>
>>---
>> drivers/gpu/drm/i915/intel_audio.c | 4 +---
>> 1 file changed, 1 insertion(+), 3 deletions(-)
>>
>>diff --git a/drivers/gpu/drm/i915/intel_audio.c
>>b/drivers/gpu/drm/i915/intel_audio.c
>>index c8a1345..3bbc96c 100644
>>--- a/drivers/gpu/drm/i915/intel_audio.c
>>+++ b/drivers/gpu/drm/i915/intel_audio.c
>>@@ -805,9 +805,7 @@ static int
>>i915_audio_component_sync_audio_rate(struct device *kdev, int port,
>>
>> /* 1. get the pipe */
>> intel_encoder = get_saved_enc(dev_priv, port, pipe);
>>- if (!intel_encoder || !intel_encoder->base.crtc ||
>>- (intel_encoder->type != INTEL_OUTPUT_HDMI &&
>>- intel_encoder->type != INTEL_OUTPUT_DP)) {
>>+ if (!intel_encoder || !intel_encoder->base.crtc) {
>> DRM_DEBUG_KMS("Not valid for port %c\n",
>>port_name(port));
>> err = -ENODEV;
>> goto unlock;
>>--
>>2.7.4
>
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
