[Intel-gfx] [PATCH] drm: i915: Fix audio issue on BXT

[Singh, Gaurav K]

On 3/5/2018 11:51 PM, Pandiyan, Dhinakaran wrote:
> On Thu, 2018-01-04 at 00:48 +0530, Gaurav K Singh wrote:
>> From: Gaurav Singh <gaurav.k.singh at intel.com>
>>
>> On Apollolake, with stress test warm reboot, audio card
>> was not getting enumerated after reboot. This was a
>> spurious issue happening on Apollolake. HW codec and
>> HD audio controller link was going out of sync for which
>> there was a fix in i915 driver but was not getting invoked
>> for BXT. Extending this fix to BXT as well.
>>
>> Tested on apollolake chromebook by stress test warm reboot
>> with 2500 iterations.
>>
>> Signed-off-by: Gaurav K Singh <gaurav.k.singh at intel.com>
> Might be worth adding
>
> Bspec: 21829
>
> to the commit message.
> Reviewed-by: Dhinakaran Pandiyan <dhinakaran.pandiyan at intel.com>
>
> Please rebase and send this patch to the list to CI it.

Hi DK,
Sure, i have added Bspec index and sent the updated patch.

https://patchwork.freedesktop.org/patch/215067/

With regards,
Gaurav
>
>
>> ---
>>   drivers/gpu/drm/i915/intel_audio.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c
>> index f1502a0188eb..c71c04e1c3f6 100644
>> --- a/drivers/gpu/drm/i915/intel_audio.c
>> +++ b/drivers/gpu/drm/i915/intel_audio.c
>> @@ -729,7 +729,7 @@ static void i915_audio_component_codec_wake_override(struct device *kdev,
>>   	struct drm_i915_private *dev_priv = kdev_to_i915(kdev);
>>   	u32 tmp;
>>   
>> -	if (!IS_GEN9_BC(dev_priv))
>> +	if (!IS_GEN9_BC(dev_priv) && !IS_BROXTON(dev_priv))
>>   		return;
>>   
>>   	i915_audio_component_get_power(kdev);