* [PATCH] drm/i915/audio: Fix audio detection issue on GLK
@ 2018-04-17 7:18 Gaurav K Singh
2018-04-17 8:00 ` ✗ Fi.CI.BAT: failure for drm/i915/audio: Fix audio detection issue on GLK (rev2) Patchwork
` (4 more replies)
0 siblings, 5 replies; 19+ messages in thread
From: Gaurav K Singh @ 2018-04-17 7:18 UTC (permalink / raw)
To: intel-gfx; +Cc: jani.nikula, dhinakaran.pandiyan
On Geminilake, sometimes audio card is not getting
detected after reboot. This is a spurious issue happening on
Geminilake. 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 GLK. Extending this fix to GLK as well.
Tested by Du,Wenkai on GLK board.
Bspec: 21829
v2: Instead of checking GEN9_BC, BXT and GLK macros, use IS_GEN9 macro
Signed-off-by: Gaurav K Singh <gaurav.k.singh@intel.com>
Reviewed-by: Jani Nikula <jani.nikula@intel.com>
---
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 656f6c931341..3ea566f99450 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) && !IS_BROXTON(dev_priv))
+ if (!IS_GEN9(dev_priv))
return;
i915_audio_component_get_power(kdev);
--
1.9.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 19+ messages in thread
* ✗ Fi.CI.BAT: failure for drm/i915/audio: Fix audio detection issue on GLK (rev2)
2018-04-17 7:18 [PATCH] drm/i915/audio: Fix audio detection issue on GLK Gaurav K Singh
@ 2018-04-17 8:00 ` Patchwork
2018-04-17 8:05 ` Martin Peres
2018-04-17 8:02 ` [PATCH] drm/i915/audio: Fix audio detection issue on GLK Jani Nikula
` (3 subsequent siblings)
4 siblings, 1 reply; 19+ messages in thread
From: Patchwork @ 2018-04-17 8:00 UTC (permalink / raw)
To: Gaurav K Singh; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/audio: Fix audio detection issue on GLK (rev2)
URL : https://patchwork.freedesktop.org/series/41334/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4058 -> Patchwork_8701 =
== Summary - FAILURE ==
Serious unknown changes coming with Patchwork_8701 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_8701, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://patchwork.freedesktop.org/api/1.0/series/41334/revisions/2/mbox/
== Possible new issues ==
Here are the unknown changes that may have been introduced in Patchwork_8701:
=== IGT changes ===
==== Possible regressions ====
igt@gem_exec_suspend@basic-s3:
fi-ivb-3520m: PASS -> DMESG-WARN
==== Warnings ====
igt@prime_vgem@basic-fence-flip:
fi-cnl-y3: SKIP -> PASS
== Known issues ==
Here are the changes found in Patchwork_8701 that come from known issues:
=== IGT changes ===
==== Possible fixes ====
igt@debugfs_test@read_all_entries:
fi-snb-2520m: INCOMPLETE (fdo#103713) -> PASS
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
fi-bxt-dsi: INCOMPLETE (fdo#103927) -> PASS
fdo#103713 https://bugs.freedesktop.org/show_bug.cgi?id=103713
fdo#103927 https://bugs.freedesktop.org/show_bug.cgi?id=103927
== Participating hosts (35 -> 33) ==
Additional (1): fi-glk-1
Missing (3): fi-ilk-m540 fi-cnl-psr fi-skl-6700hq
== Build changes ==
* Linux: CI_DRM_4058 -> Patchwork_8701
CI_DRM_4058: 241d827c86078c4709c00251d22ea8f7554e3e36 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4432: 8b77704db49167f7ebfd1c470d9c129d3b862cb6 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_8701: 55226f577b074d07d717a4894646ead57dfd4cf2 @ git://anongit.freedesktop.org/gfx-ci/linux
piglit_4432: 93b35926a150e318439d2505901288594b3548f5 @ git://anongit.freedesktop.org/piglit
== Linux commits ==
55226f577b07 drm/i915/audio: Fix audio detection issue on GLK
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_8701/issues.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] drm/i915/audio: Fix audio detection issue on GLK
2018-04-17 7:18 [PATCH] drm/i915/audio: Fix audio detection issue on GLK Gaurav K Singh
2018-04-17 8:00 ` ✗ Fi.CI.BAT: failure for drm/i915/audio: Fix audio detection issue on GLK (rev2) Patchwork
@ 2018-04-17 8:02 ` Jani Nikula
2018-04-17 9:18 ` ✓ Fi.CI.BAT: success for drm/i915/audio: Fix audio detection issue on GLK (rev2) Patchwork
` (2 subsequent siblings)
4 siblings, 0 replies; 19+ messages in thread
From: Jani Nikula @ 2018-04-17 8:02 UTC (permalink / raw)
To: Gaurav K Singh, intel-gfx; +Cc: dhinakaran.pandiyan
On Tue, 17 Apr 2018, Gaurav K Singh <gaurav.k.singh@intel.com> wrote:
> On Geminilake, sometimes audio card is not getting
> detected after reboot. This is a spurious issue happening on
> Geminilake. 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 GLK. Extending this fix to GLK as well.
>
> Tested by Du,Wenkai on GLK board.
>
> Bspec: 21829
>
> v2: Instead of checking GEN9_BC, BXT and GLK macros, use IS_GEN9 macro
>
> Signed-off-by: Gaurav K Singh <gaurav.k.singh@intel.com>
> Reviewed-by: Jani Nikula <jani.nikula@intel.com>
That Reviewed-by is jumping to conclusions I'm afraid. Giving review
comments does not mean Reviewed-by. Only explicit Reviewed-by reply
does.
I can only say that this is the right approach *if* glk is to be covered
here, but I have no data to actually say if that's the right thing to
do.
BR,
Jani.
> ---
> 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 656f6c931341..3ea566f99450 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) && !IS_BROXTON(dev_priv))
> + if (!IS_GEN9(dev_priv))
> return;
>
> i915_audio_component_get_power(kdev);
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: ✗ Fi.CI.BAT: failure for drm/i915/audio: Fix audio detection issue on GLK (rev2)
2018-04-17 8:00 ` ✗ Fi.CI.BAT: failure for drm/i915/audio: Fix audio detection issue on GLK (rev2) Patchwork
@ 2018-04-17 8:05 ` Martin Peres
0 siblings, 0 replies; 19+ messages in thread
From: Martin Peres @ 2018-04-17 8:05 UTC (permalink / raw)
To: intel-gfx, Patchwork, Gaurav K Singh
On 17/04/18 11:00, Patchwork wrote:
> == Series Details ==
>
> Series: drm/i915/audio: Fix audio detection issue on GLK (rev2)
> URL : https://patchwork.freedesktop.org/series/41334/
> State : failure
>
> == Summary ==
>
> = CI Bug Log - changes from CI_DRM_4058 -> Patchwork_8701 =
>
> == Summary - FAILURE ==
>
> Serious unknown changes coming with Patchwork_8701 absolutely need to be
> verified manually.
>
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_8701, please notify your bug team to allow them
> to document this new failure mode, which will reduce false positives in CI.
>
> External URL: https://patchwork.freedesktop.org/api/1.0/series/41334/revisions/2/mbox/
>
> == Possible new issues ==
>
> Here are the unknown changes that may have been introduced in Patchwork_8701:
>
> === IGT changes ===
>
> ==== Possible regressions ====
>
> igt@gem_exec_suspend@basic-s3:
> fi-ivb-3520m: PASS -> DMESG-WARN
https://bugs.freedesktop.org/show_bug.cgi?id=106084
I re-queued the testing to get the shards.
>
>
> ==== Warnings ====
>
> igt@prime_vgem@basic-fence-flip:
> fi-cnl-y3: SKIP -> PASS
>
>
> == Known issues ==
>
> Here are the changes found in Patchwork_8701 that come from known issues:
>
> === IGT changes ===
>
> ==== Possible fixes ====
>
> igt@debugfs_test@read_all_entries:
> fi-snb-2520m: INCOMPLETE (fdo#103713) -> PASS
>
> igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
> fi-bxt-dsi: INCOMPLETE (fdo#103927) -> PASS
>
>
> fdo#103713 https://bugs.freedesktop.org/show_bug.cgi?id=103713
> fdo#103927 https://bugs.freedesktop.org/show_bug.cgi?id=103927
>
>
> == Participating hosts (35 -> 33) ==
>
> Additional (1): fi-glk-1
> Missing (3): fi-ilk-m540 fi-cnl-psr fi-skl-6700hq
>
>
> == Build changes ==
>
> * Linux: CI_DRM_4058 -> Patchwork_8701
>
> CI_DRM_4058: 241d827c86078c4709c00251d22ea8f7554e3e36 @ git://anongit.freedesktop.org/gfx-ci/linux
> IGT_4432: 8b77704db49167f7ebfd1c470d9c129d3b862cb6 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> Patchwork_8701: 55226f577b074d07d717a4894646ead57dfd4cf2 @ git://anongit.freedesktop.org/gfx-ci/linux
> piglit_4432: 93b35926a150e318439d2505901288594b3548f5 @ git://anongit.freedesktop.org/piglit
>
>
> == Linux commits ==
>
> 55226f577b07 drm/i915/audio: Fix audio detection issue on GLK
>
> == Logs ==
>
> For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_8701/issues.html
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 19+ messages in thread
* ✓ Fi.CI.BAT: success for drm/i915/audio: Fix audio detection issue on GLK (rev2)
2018-04-17 7:18 [PATCH] drm/i915/audio: Fix audio detection issue on GLK Gaurav K Singh
2018-04-17 8:00 ` ✗ Fi.CI.BAT: failure for drm/i915/audio: Fix audio detection issue on GLK (rev2) Patchwork
2018-04-17 8:02 ` [PATCH] drm/i915/audio: Fix audio detection issue on GLK Jani Nikula
@ 2018-04-17 9:18 ` Patchwork
2018-04-17 10:05 ` ✗ Fi.CI.IGT: failure " Patchwork
2018-04-17 18:00 ` [PATCH] drm/i915/audio: Fix audio detection issue on GLK Kumar, Abhay
4 siblings, 0 replies; 19+ messages in thread
From: Patchwork @ 2018-04-17 9:18 UTC (permalink / raw)
To: Gaurav K Singh; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/audio: Fix audio detection issue on GLK (rev2)
URL : https://patchwork.freedesktop.org/series/41334/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4058 -> Patchwork_8702 =
== Summary - WARNING ==
Minor unknown changes coming with Patchwork_8702 need to be verified
manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_8702, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://patchwork.freedesktop.org/api/1.0/series/41334/revisions/2/mbox/
== Possible new issues ==
Here are the unknown changes that may have been introduced in Patchwork_8702:
=== IGT changes ===
==== Warnings ====
igt@prime_vgem@basic-fence-flip:
fi-cnl-y3: SKIP -> PASS
== Known issues ==
Here are the changes found in Patchwork_8702 that come from known issues:
=== IGT changes ===
==== Issues hit ====
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
fi-ivb-3520m: PASS -> DMESG-WARN (fdo#106084)
igt@vgem_basic@dmabuf-fence-before:
fi-glk-1: NOTRUN -> INCOMPLETE (k.org#198133, fdo#103359)
==== Possible fixes ====
igt@debugfs_test@read_all_entries:
fi-snb-2520m: INCOMPLETE (fdo#103713) -> PASS
igt@gem_mmap_gtt@basic-small-bo-tiledx:
fi-gdg-551: FAIL (fdo#102575) -> PASS
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
fi-bxt-dsi: INCOMPLETE (fdo#103927) -> PASS
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c:
fi-ivb-3520m: DMESG-WARN (fdo#106084) -> PASS
fdo#102575 https://bugs.freedesktop.org/show_bug.cgi?id=102575
fdo#103359 https://bugs.freedesktop.org/show_bug.cgi?id=103359
fdo#103713 https://bugs.freedesktop.org/show_bug.cgi?id=103713
fdo#103927 https://bugs.freedesktop.org/show_bug.cgi?id=103927
fdo#106084 https://bugs.freedesktop.org/show_bug.cgi?id=106084
k.org#198133 https://bugzilla.kernel.org/show_bug.cgi?id=198133
== Participating hosts (35 -> 33) ==
Additional (1): fi-glk-1
Missing (3): fi-ilk-m540 fi-cnl-psr fi-skl-6700hq
== Build changes ==
* Linux: CI_DRM_4058 -> Patchwork_8702
CI_DRM_4058: 241d827c86078c4709c00251d22ea8f7554e3e36 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4432: 8b77704db49167f7ebfd1c470d9c129d3b862cb6 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_8702: 6dad30273262e306bac397d83d45f603da5607f0 @ git://anongit.freedesktop.org/gfx-ci/linux
piglit_4432: 93b35926a150e318439d2505901288594b3548f5 @ git://anongit.freedesktop.org/piglit
== Linux commits ==
6dad30273262 drm/i915/audio: Fix audio detection issue on GLK
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_8702/issues.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 19+ messages in thread
* ✗ Fi.CI.IGT: failure for drm/i915/audio: Fix audio detection issue on GLK (rev2)
2018-04-17 7:18 [PATCH] drm/i915/audio: Fix audio detection issue on GLK Gaurav K Singh
` (2 preceding siblings ...)
2018-04-17 9:18 ` ✓ Fi.CI.BAT: success for drm/i915/audio: Fix audio detection issue on GLK (rev2) Patchwork
@ 2018-04-17 10:05 ` Patchwork
2018-04-17 18:00 ` [PATCH] drm/i915/audio: Fix audio detection issue on GLK Kumar, Abhay
4 siblings, 0 replies; 19+ messages in thread
From: Patchwork @ 2018-04-17 10:05 UTC (permalink / raw)
To: Gaurav K Singh; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/audio: Fix audio detection issue on GLK (rev2)
URL : https://patchwork.freedesktop.org/series/41334/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4058_full -> Patchwork_8702_full =
== Summary - FAILURE ==
Serious unknown changes coming with Patchwork_8702_full absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_8702_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://patchwork.freedesktop.org/api/1.0/series/41334/revisions/2/mbox/
== Possible new issues ==
Here are the unknown changes that may have been introduced in Patchwork_8702_full:
=== IGT changes ===
==== Possible regressions ====
igt@kms_flip@flip-vs-absolute-wf_vblank-interruptible:
shard-apl: PASS -> FAIL
==== Warnings ====
igt@gem_mocs_settings@mocs-rc6-ctx-render:
shard-kbl: SKIP -> PASS
igt@kms_cursor_legacy@flip-vs-cursor-crc-legacy:
shard-snb: PASS -> SKIP +1
== Known issues ==
Here are the changes found in Patchwork_8702_full that come from known issues:
=== IGT changes ===
==== Issues hit ====
igt@gem_exec_store@cachelines-bsd:
shard-hsw: PASS -> FAIL (fdo#100007)
igt@kms_setmode@basic:
shard-apl: PASS -> FAIL (fdo#99912)
==== Possible fixes ====
igt@kms_flip@2x-flip-vs-expired-vblank-interruptible:
shard-hsw: FAIL (fdo#105189) -> PASS
igt@kms_flip@blocking-wf_vblank:
shard-hsw: FAIL (fdo#103928) -> PASS
fdo#100007 https://bugs.freedesktop.org/show_bug.cgi?id=100007
fdo#103928 https://bugs.freedesktop.org/show_bug.cgi?id=103928
fdo#105189 https://bugs.freedesktop.org/show_bug.cgi?id=105189
fdo#99912 https://bugs.freedesktop.org/show_bug.cgi?id=99912
== Participating hosts (6 -> 4) ==
Missing (2): shard-glk shard-glkb
== Build changes ==
* Linux: CI_DRM_4058 -> Patchwork_8702
CI_DRM_4058: 241d827c86078c4709c00251d22ea8f7554e3e36 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4432: 8b77704db49167f7ebfd1c470d9c129d3b862cb6 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_8702: 6dad30273262e306bac397d83d45f603da5607f0 @ git://anongit.freedesktop.org/gfx-ci/linux
piglit_4432: 93b35926a150e318439d2505901288594b3548f5 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_8702/shards.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] drm/i915/audio: Fix audio detection issue on GLK
2018-04-17 7:18 [PATCH] drm/i915/audio: Fix audio detection issue on GLK Gaurav K Singh
` (3 preceding siblings ...)
2018-04-17 10:05 ` ✗ Fi.CI.IGT: failure " Patchwork
@ 2018-04-17 18:00 ` Kumar, Abhay
4 siblings, 0 replies; 19+ messages in thread
From: Kumar, Abhay @ 2018-04-17 18:00 UTC (permalink / raw)
To: Gaurav K Singh, intel-gfx; +Cc: Nikula, Jani, Pandiyan, Dhinakaran
On 4/17/2018 12:18 AM, Gaurav K Singh wrote:
> On Geminilake, sometimes audio card is not getting
> detected after reboot. This is a spurious issue happening on
> Geminilake. 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 GLK. Extending this fix to GLK as well.
>
> Tested by Du,Wenkai on GLK board.
>
> Bspec: 21829
>
> v2: Instead of checking GEN9_BC, BXT and GLK macros, use IS_GEN9 macro
>
> Signed-off-by: Gaurav K Singh <gaurav.k.singh@intel.com>
> Reviewed-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Abhay Kumar <abhay.Kumar@intel.com>
> ---
> 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 656f6c931341..3ea566f99450 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) && !IS_BROXTON(dev_priv))
> + if (!IS_GEN9(dev_priv))
> return;
>
> i915_audio_component_get_power(kdev);
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] drm/i915/audio: Fix audio detection issue on GLK
2018-04-18 6:04 ` Singh, Gaurav K
@ 2018-04-18 8:07 ` Jani Nikula
0 siblings, 0 replies; 19+ messages in thread
From: Jani Nikula @ 2018-04-18 8:07 UTC (permalink / raw)
To: Singh, Gaurav K, Du, Wenkai, intel-gfx; +Cc: dhinakaran.pandiyan
On Wed, 18 Apr 2018, "Singh, Gaurav K" <gaurav.k.singh@intel.com> wrote:
> On 4/17/2018 11:58 PM, Du,Wenkai wrote:
>>
>> On 4/17/2018 11:22 AM, Gaurav K Singh wrote:
>>> On Geminilake, sometimes audio card is not getting
>>> detected after reboot. This is a spurious issue happening on
>>> Geminilake. 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 GLK. Extending this fix to GLK as well.
>>>
>>> Tested by Du,Wenkai on GLK board.
>>>
>>> Bspec: 21829
>>>
>>> v2: Instead of checking GEN9_BC, BXT and GLK macros, use IS_GEN9
>>> macro (Jani N)
>>>
>>> Signed-off-by: Gaurav K Singh <gaurav.k.singh@intel.com>
>>> Reviewed-by: Abhay Kumar <abhay.Kumar@intel.com>
>> Reviewed-by: Wenkai Du <wenkai.du@intel.com>
>> Tested-by: Wenkai Du <wenkai.du@intel.com>
>>
>>
>> Regards,
>> Wenkai
> Hi Jani,
>
> Could you please help in merging this patch to unblock audio.
Pushed, thanks for the patch and review.
Added Cc: stable, with a dependency on b651bd2a3ae3 ("drm/i915/audio:
Fix audio enumeration issue on BXT") to be backported as well.
BR,
Jani.
>
> 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 656f6c931341..3ea566f99450 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) && !IS_BROXTON(dev_priv))
>>> + if (!IS_GEN9(dev_priv))
>>> return;
>>> i915_audio_component_get_power(kdev);
>
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] drm/i915/audio: Fix audio detection issue on GLK
2018-04-17 18:28 ` Du,Wenkai
@ 2018-04-18 6:04 ` Singh, Gaurav K
2018-04-18 8:07 ` Jani Nikula
0 siblings, 1 reply; 19+ messages in thread
From: Singh, Gaurav K @ 2018-04-18 6:04 UTC (permalink / raw)
To: Du,Wenkai, intel-gfx; +Cc: jani.nikula, dhinakaran.pandiyan
On 4/17/2018 11:58 PM, Du,Wenkai wrote:
>
> On 4/17/2018 11:22 AM, Gaurav K Singh wrote:
>> On Geminilake, sometimes audio card is not getting
>> detected after reboot. This is a spurious issue happening on
>> Geminilake. 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 GLK. Extending this fix to GLK as well.
>>
>> Tested by Du,Wenkai on GLK board.
>>
>> Bspec: 21829
>>
>> v2: Instead of checking GEN9_BC, BXT and GLK macros, use IS_GEN9
>> macro (Jani N)
>>
>> Signed-off-by: Gaurav K Singh <gaurav.k.singh@intel.com>
>> Reviewed-by: Abhay Kumar <abhay.Kumar@intel.com>
> Reviewed-by: Wenkai Du <wenkai.du@intel.com>
> Tested-by: Wenkai Du <wenkai.du@intel.com>
>
>
> Regards,
> Wenkai
Hi Jani,
Could you please help in merging this patch to unblock audio.
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 656f6c931341..3ea566f99450 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) && !IS_BROXTON(dev_priv))
>> + if (!IS_GEN9(dev_priv))
>> return;
>> i915_audio_component_get_power(kdev);
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] drm/i915/audio: Fix audio detection issue on GLK
2018-04-17 18:22 Gaurav K Singh
@ 2018-04-17 18:28 ` Du,Wenkai
2018-04-18 6:04 ` Singh, Gaurav K
0 siblings, 1 reply; 19+ messages in thread
From: Du,Wenkai @ 2018-04-17 18:28 UTC (permalink / raw)
To: Gaurav K Singh, intel-gfx; +Cc: jani.nikula, dhinakaran.pandiyan
On 4/17/2018 11:22 AM, Gaurav K Singh wrote:
> On Geminilake, sometimes audio card is not getting
> detected after reboot. This is a spurious issue happening on
> Geminilake. 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 GLK. Extending this fix to GLK as well.
>
> Tested by Du,Wenkai on GLK board.
>
> Bspec: 21829
>
> v2: Instead of checking GEN9_BC, BXT and GLK macros, use IS_GEN9 macro (Jani N)
>
> Signed-off-by: Gaurav K Singh <gaurav.k.singh@intel.com>
> Reviewed-by: Abhay Kumar <abhay.Kumar@intel.com>
Reviewed-by: Wenkai Du <wenkai.du@intel.com>
Tested-by: Wenkai Du <wenkai.du@intel.com>
Regards,
Wenkai
> ---
> 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 656f6c931341..3ea566f99450 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) && !IS_BROXTON(dev_priv))
> + if (!IS_GEN9(dev_priv))
> return;
>
> i915_audio_component_get_power(kdev);
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH] drm/i915/audio: Fix audio detection issue on GLK
@ 2018-04-17 18:22 Gaurav K Singh
2018-04-17 18:28 ` Du,Wenkai
0 siblings, 1 reply; 19+ messages in thread
From: Gaurav K Singh @ 2018-04-17 18:22 UTC (permalink / raw)
To: intel-gfx; +Cc: jani.nikula, dhinakaran.pandiyan
On Geminilake, sometimes audio card is not getting
detected after reboot. This is a spurious issue happening on
Geminilake. 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 GLK. Extending this fix to GLK as well.
Tested by Du,Wenkai on GLK board.
Bspec: 21829
v2: Instead of checking GEN9_BC, BXT and GLK macros, use IS_GEN9 macro (Jani N)
Signed-off-by: Gaurav K Singh <gaurav.k.singh@intel.com>
Reviewed-by: Abhay Kumar <abhay.Kumar@intel.com>
---
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 656f6c931341..3ea566f99450 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) && !IS_BROXTON(dev_priv))
+ if (!IS_GEN9(dev_priv))
return;
i915_audio_component_get_power(kdev);
--
1.9.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH] drm/i915/audio: Fix audio detection issue on GLK
2018-04-10 11:02 ` Jani Nikula
@ 2018-04-11 19:27 ` Singh, Gaurav K
0 siblings, 0 replies; 19+ messages in thread
From: Singh, Gaurav K @ 2018-04-11 19:27 UTC (permalink / raw)
To: Jani Nikula, Kumar, Abhay, Pandiyan, Dhinakaran; +Cc: intel-gfx, Vivi, Rodrigo
On 4/10/2018 4:32 PM, Jani Nikula wrote:
> On Mon, 09 Apr 2018, "Kumar, Abhay" <abhay.kumar@intel.com> wrote:
>> On 4/9/2018 4:20 PM, Pandiyan, Dhinakaran wrote:
>>>
>>> On Mon, 2018-04-09 at 12:18 -0700, Kumar, Abhay wrote:
>>>> On 4/9/2018 12:10 PM, Rodrigo Vivi wrote:
>>>>> On Mon, Apr 09, 2018 at 05:07:31PM +0300, Jani Nikula wrote:
>>>>>> On Sun, 08 Apr 2018, Gaurav K Singh <gaurav.k.singh@intel.com> wrote:
>>>>>>> On Geminilake, sometimes audio card is not getting
>>>>>>> detected after reboot. This is a spurious issue happening on
>>>>>>> Geminilake. 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 GLK. Extending this fix to GLK as well.
>>>>>>>
>>>>>>> Tested by Du,Wenkai on GLK board.
>>>>>>>
>>>>>>> Signed-off-by: Gaurav K Singh <gaurav.k.singh@intel.com>
>>>>>>> ---
>>>>>>> drivers/gpu/drm/i915/intel_audio.c | 3 ++-
>>>>>>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>>>>>>
>>>>>>> diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c
>>>>>>> index 656f6c931341..73b1e0b96f88 100644
>>>>>>> --- a/drivers/gpu/drm/i915/intel_audio.c
>>>>>>> +++ b/drivers/gpu/drm/i915/intel_audio.c
>>>>>>> @@ -729,7 +729,8 @@ 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) && !IS_BROXTON(dev_priv))
>>>>>>> + if (!IS_GEN9_BC(dev_priv) && !IS_BROXTON(dev_priv) &&
>>>>>>> + !IS_GEMINILAKE(dev_priv))
>>>>>> That could be written as
>>>>>>
>>>>>> if (!IS_GEN9_BC(dev_priv) && !IS_GEN9_LP(dev_priv))
>>>>>>
>>>>>> which in turn could just be written as
>>>>>>
>>>>>> if (!IS_GEN9(dev_priv))
>>>>>>
>>>>>> ...but since GLK has gen 10 display, so I'm wondering if the same issue
>>>>>> will be present in gen 10 too, and whether this should just become
>>>>>>
>>>>>> if (INTEL_GEN(dev_priv) < 9)
>>>>> +1. I opened here to exactly add same comment.
>>>> I am checking with DINQ and without this patch for GLK it can enumerate
>>>> HDA codec. Ofcourse after cdclk fix.
>>> How about the other way around? i.e., does codec enumeration work this
>>> patch but without the cdclk change?
>> Nop. with DINQ we need to have cdclk change to make Codec detection
>> work. With or without this patch.
> Basically what you're saying is that this patch is not needed? Gaurav,
> can you check with the CDCLK patch [1] if that fixes the issue for you?
>
> BR,
> Jani.
Jani,
Our team tried my patch (not including Abhay's patch) which fixed the
issue. But will also try Abhay's patch(this time without my patch), Will
update accordingly.
With regards,
Gaurav
>
>
> PS. The CDCLK patch is not enough to fix the issue completely (probe
> without display outputs will still choose a low CDCLK) but that's work
> in progress.
>
>
> [1] http://patchwork.freedesktop.org/patch/msgid/1508968932-32208-1-git-send-email-abhay.kumar@intel.com
>
>
>>>
>>>
>>>>>> BR,
>>>>>> Jani.
>>>>>>
>>>>>>
>>>>>>
>>>>>>> return;
>>>>>>>
>>>>>>> i915_audio_component_get_power(kdev);
>>>>>> --
>>>>>> Jani Nikula, Intel Open Source Technology Center
>>>>>> _______________________________________________
>>>>>> Intel-gfx mailing list
>>>>>> Intel-gfx@lists.freedesktop.org
>>>>>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>>>>> _______________________________________________
>>>>> Intel-gfx mailing list
>>>>> Intel-gfx@lists.freedesktop.org
>>>>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>>>> _______________________________________________
>>>> Intel-gfx mailing list
>>>> Intel-gfx@lists.freedesktop.org
>>>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] drm/i915/audio: Fix audio detection issue on GLK
2018-04-09 23:13 ` Kumar, Abhay
@ 2018-04-10 11:02 ` Jani Nikula
2018-04-11 19:27 ` Singh, Gaurav K
0 siblings, 1 reply; 19+ messages in thread
From: Jani Nikula @ 2018-04-10 11:02 UTC (permalink / raw)
To: Kumar, Abhay, Pandiyan, Dhinakaran; +Cc: intel-gfx, Vivi, Rodrigo
On Mon, 09 Apr 2018, "Kumar, Abhay" <abhay.kumar@intel.com> wrote:
> On 4/9/2018 4:20 PM, Pandiyan, Dhinakaran wrote:
>>
>>
>> On Mon, 2018-04-09 at 12:18 -0700, Kumar, Abhay wrote:
>>> On 4/9/2018 12:10 PM, Rodrigo Vivi wrote:
>>>> On Mon, Apr 09, 2018 at 05:07:31PM +0300, Jani Nikula wrote:
>>>>> On Sun, 08 Apr 2018, Gaurav K Singh <gaurav.k.singh@intel.com> wrote:
>>>>>> On Geminilake, sometimes audio card is not getting
>>>>>> detected after reboot. This is a spurious issue happening on
>>>>>> Geminilake. 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 GLK. Extending this fix to GLK as well.
>>>>>>
>>>>>> Tested by Du,Wenkai on GLK board.
>>>>>>
>>>>>> Signed-off-by: Gaurav K Singh <gaurav.k.singh@intel.com>
>>>>>> ---
>>>>>> drivers/gpu/drm/i915/intel_audio.c | 3 ++-
>>>>>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>>>>>
>>>>>> diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c
>>>>>> index 656f6c931341..73b1e0b96f88 100644
>>>>>> --- a/drivers/gpu/drm/i915/intel_audio.c
>>>>>> +++ b/drivers/gpu/drm/i915/intel_audio.c
>>>>>> @@ -729,7 +729,8 @@ 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) && !IS_BROXTON(dev_priv))
>>>>>> + if (!IS_GEN9_BC(dev_priv) && !IS_BROXTON(dev_priv) &&
>>>>>> + !IS_GEMINILAKE(dev_priv))
>>>>> That could be written as
>>>>>
>>>>> if (!IS_GEN9_BC(dev_priv) && !IS_GEN9_LP(dev_priv))
>>>>>
>>>>> which in turn could just be written as
>>>>>
>>>>> if (!IS_GEN9(dev_priv))
>>>>>
>>>>> ...but since GLK has gen 10 display, so I'm wondering if the same issue
>>>>> will be present in gen 10 too, and whether this should just become
>>>>>
>>>>> if (INTEL_GEN(dev_priv) < 9)
>>>> +1. I opened here to exactly add same comment.
>>> I am checking with DINQ and without this patch for GLK it can enumerate
>>> HDA codec. Ofcourse after cdclk fix.
>> How about the other way around? i.e., does codec enumeration work this
>> patch but without the cdclk change?
> Nop. with DINQ we need to have cdclk change to make Codec detection
> work. With or without this patch.
Basically what you're saying is that this patch is not needed? Gaurav,
can you check with the CDCLK patch [1] if that fixes the issue for you?
BR,
Jani.
PS. The CDCLK patch is not enough to fix the issue completely (probe
without display outputs will still choose a low CDCLK) but that's work
in progress.
[1] http://patchwork.freedesktop.org/patch/msgid/1508968932-32208-1-git-send-email-abhay.kumar@intel.com
>
>>
>>
>>
>>>>> BR,
>>>>> Jani.
>>>>>
>>>>>
>>>>>
>>>>>> return;
>>>>>>
>>>>>> i915_audio_component_get_power(kdev);
>>>>> --
>>>>> Jani Nikula, Intel Open Source Technology Center
>>>>> _______________________________________________
>>>>> Intel-gfx mailing list
>>>>> Intel-gfx@lists.freedesktop.org
>>>>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>>>> _______________________________________________
>>>> Intel-gfx mailing list
>>>> Intel-gfx@lists.freedesktop.org
>>>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>>> _______________________________________________
>>> Intel-gfx mailing list
>>> Intel-gfx@lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] drm/i915/audio: Fix audio detection issue on GLK
2018-04-09 19:18 ` Kumar, Abhay
@ 2018-04-09 23:20 ` Dhinakaran Pandiyan
2018-04-09 23:13 ` Kumar, Abhay
0 siblings, 1 reply; 19+ messages in thread
From: Dhinakaran Pandiyan @ 2018-04-09 23:20 UTC (permalink / raw)
To: Kumar, Abhay; +Cc: intel-gfx, Rodrigo Vivi
On Mon, 2018-04-09 at 12:18 -0700, Kumar, Abhay wrote:
>
> On 4/9/2018 12:10 PM, Rodrigo Vivi wrote:
> > On Mon, Apr 09, 2018 at 05:07:31PM +0300, Jani Nikula wrote:
> >> On Sun, 08 Apr 2018, Gaurav K Singh <gaurav.k.singh@intel.com> wrote:
> >>> On Geminilake, sometimes audio card is not getting
> >>> detected after reboot. This is a spurious issue happening on
> >>> Geminilake. 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 GLK. Extending this fix to GLK as well.
> >>>
> >>> Tested by Du,Wenkai on GLK board.
> >>>
> >>> Signed-off-by: Gaurav K Singh <gaurav.k.singh@intel.com>
> >>> ---
> >>> drivers/gpu/drm/i915/intel_audio.c | 3 ++-
> >>> 1 file changed, 2 insertions(+), 1 deletion(-)
> >>>
> >>> diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c
> >>> index 656f6c931341..73b1e0b96f88 100644
> >>> --- a/drivers/gpu/drm/i915/intel_audio.c
> >>> +++ b/drivers/gpu/drm/i915/intel_audio.c
> >>> @@ -729,7 +729,8 @@ 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) && !IS_BROXTON(dev_priv))
> >>> + if (!IS_GEN9_BC(dev_priv) && !IS_BROXTON(dev_priv) &&
> >>> + !IS_GEMINILAKE(dev_priv))
> >> That could be written as
> >>
> >> if (!IS_GEN9_BC(dev_priv) && !IS_GEN9_LP(dev_priv))
> >>
> >> which in turn could just be written as
> >>
> >> if (!IS_GEN9(dev_priv))
> >>
> >> ...but since GLK has gen 10 display, so I'm wondering if the same issue
> >> will be present in gen 10 too, and whether this should just become
> >>
> >> if (INTEL_GEN(dev_priv) < 9)
> > +1. I opened here to exactly add same comment.
> I am checking with DINQ and without this patch for GLK it can enumerate
> HDA codec. Ofcourse after cdclk fix.
How about the other way around? i.e., does codec enumeration work this
patch but without the cdclk change?
> >
> >> BR,
> >> Jani.
> >>
> >>
> >>
> >>> return;
> >>>
> >>> i915_audio_component_get_power(kdev);
> >> --
> >> Jani Nikula, Intel Open Source Technology Center
> >> _______________________________________________
> >> Intel-gfx mailing list
> >> Intel-gfx@lists.freedesktop.org
> >> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] drm/i915/audio: Fix audio detection issue on GLK
2018-04-09 23:20 ` Dhinakaran Pandiyan
@ 2018-04-09 23:13 ` Kumar, Abhay
2018-04-10 11:02 ` Jani Nikula
0 siblings, 1 reply; 19+ messages in thread
From: Kumar, Abhay @ 2018-04-09 23:13 UTC (permalink / raw)
To: Pandiyan, Dhinakaran; +Cc: intel-gfx, Vivi, Rodrigo
On 4/9/2018 4:20 PM, Pandiyan, Dhinakaran wrote:
>
>
> On Mon, 2018-04-09 at 12:18 -0700, Kumar, Abhay wrote:
>> On 4/9/2018 12:10 PM, Rodrigo Vivi wrote:
>>> On Mon, Apr 09, 2018 at 05:07:31PM +0300, Jani Nikula wrote:
>>>> On Sun, 08 Apr 2018, Gaurav K Singh <gaurav.k.singh@intel.com> wrote:
>>>>> On Geminilake, sometimes audio card is not getting
>>>>> detected after reboot. This is a spurious issue happening on
>>>>> Geminilake. 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 GLK. Extending this fix to GLK as well.
>>>>>
>>>>> Tested by Du,Wenkai on GLK board.
>>>>>
>>>>> Signed-off-by: Gaurav K Singh <gaurav.k.singh@intel.com>
>>>>> ---
>>>>> drivers/gpu/drm/i915/intel_audio.c | 3 ++-
>>>>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c
>>>>> index 656f6c931341..73b1e0b96f88 100644
>>>>> --- a/drivers/gpu/drm/i915/intel_audio.c
>>>>> +++ b/drivers/gpu/drm/i915/intel_audio.c
>>>>> @@ -729,7 +729,8 @@ 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) && !IS_BROXTON(dev_priv))
>>>>> + if (!IS_GEN9_BC(dev_priv) && !IS_BROXTON(dev_priv) &&
>>>>> + !IS_GEMINILAKE(dev_priv))
>>>> That could be written as
>>>>
>>>> if (!IS_GEN9_BC(dev_priv) && !IS_GEN9_LP(dev_priv))
>>>>
>>>> which in turn could just be written as
>>>>
>>>> if (!IS_GEN9(dev_priv))
>>>>
>>>> ...but since GLK has gen 10 display, so I'm wondering if the same issue
>>>> will be present in gen 10 too, and whether this should just become
>>>>
>>>> if (INTEL_GEN(dev_priv) < 9)
>>> +1. I opened here to exactly add same comment.
>> I am checking with DINQ and without this patch for GLK it can enumerate
>> HDA codec. Ofcourse after cdclk fix.
> How about the other way around? i.e., does codec enumeration work this
> patch but without the cdclk change?
Nop. with DINQ we need to have cdclk change to make Codec detection
work. With or without this patch.
>
>
>
>>>> BR,
>>>> Jani.
>>>>
>>>>
>>>>
>>>>> return;
>>>>>
>>>>> i915_audio_component_get_power(kdev);
>>>> --
>>>> Jani Nikula, Intel Open Source Technology Center
>>>> _______________________________________________
>>>> Intel-gfx mailing list
>>>> Intel-gfx@lists.freedesktop.org
>>>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>>> _______________________________________________
>>> Intel-gfx mailing list
>>> Intel-gfx@lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] drm/i915/audio: Fix audio detection issue on GLK
2018-04-09 19:10 ` Rodrigo Vivi
@ 2018-04-09 19:18 ` Kumar, Abhay
2018-04-09 23:20 ` Dhinakaran Pandiyan
0 siblings, 1 reply; 19+ messages in thread
From: Kumar, Abhay @ 2018-04-09 19:18 UTC (permalink / raw)
To: Rodrigo Vivi, Jani Nikula; +Cc: intel-gfx
On 4/9/2018 12:10 PM, Rodrigo Vivi wrote:
> On Mon, Apr 09, 2018 at 05:07:31PM +0300, Jani Nikula wrote:
>> On Sun, 08 Apr 2018, Gaurav K Singh <gaurav.k.singh@intel.com> wrote:
>>> On Geminilake, sometimes audio card is not getting
>>> detected after reboot. This is a spurious issue happening on
>>> Geminilake. 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 GLK. Extending this fix to GLK as well.
>>>
>>> Tested by Du,Wenkai on GLK board.
>>>
>>> Signed-off-by: Gaurav K Singh <gaurav.k.singh@intel.com>
>>> ---
>>> drivers/gpu/drm/i915/intel_audio.c | 3 ++-
>>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c
>>> index 656f6c931341..73b1e0b96f88 100644
>>> --- a/drivers/gpu/drm/i915/intel_audio.c
>>> +++ b/drivers/gpu/drm/i915/intel_audio.c
>>> @@ -729,7 +729,8 @@ 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) && !IS_BROXTON(dev_priv))
>>> + if (!IS_GEN9_BC(dev_priv) && !IS_BROXTON(dev_priv) &&
>>> + !IS_GEMINILAKE(dev_priv))
>> That could be written as
>>
>> if (!IS_GEN9_BC(dev_priv) && !IS_GEN9_LP(dev_priv))
>>
>> which in turn could just be written as
>>
>> if (!IS_GEN9(dev_priv))
>>
>> ...but since GLK has gen 10 display, so I'm wondering if the same issue
>> will be present in gen 10 too, and whether this should just become
>>
>> if (INTEL_GEN(dev_priv) < 9)
> +1. I opened here to exactly add same comment.
I am checking with DINQ and without this patch for GLK it can enumerate
HDA codec. Ofcourse after cdclk fix.
>
>> BR,
>> Jani.
>>
>>
>>
>>> return;
>>>
>>> i915_audio_component_get_power(kdev);
>> --
>> Jani Nikula, Intel Open Source Technology Center
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] drm/i915/audio: Fix audio detection issue on GLK
2018-04-09 14:07 ` Jani Nikula
@ 2018-04-09 19:10 ` Rodrigo Vivi
2018-04-09 19:18 ` Kumar, Abhay
0 siblings, 1 reply; 19+ messages in thread
From: Rodrigo Vivi @ 2018-04-09 19:10 UTC (permalink / raw)
To: Jani Nikula; +Cc: intel-gfx
On Mon, Apr 09, 2018 at 05:07:31PM +0300, Jani Nikula wrote:
> On Sun, 08 Apr 2018, Gaurav K Singh <gaurav.k.singh@intel.com> wrote:
> > On Geminilake, sometimes audio card is not getting
> > detected after reboot. This is a spurious issue happening on
> > Geminilake. 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 GLK. Extending this fix to GLK as well.
> >
> > Tested by Du,Wenkai on GLK board.
> >
> > Signed-off-by: Gaurav K Singh <gaurav.k.singh@intel.com>
> > ---
> > drivers/gpu/drm/i915/intel_audio.c | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c
> > index 656f6c931341..73b1e0b96f88 100644
> > --- a/drivers/gpu/drm/i915/intel_audio.c
> > +++ b/drivers/gpu/drm/i915/intel_audio.c
> > @@ -729,7 +729,8 @@ 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) && !IS_BROXTON(dev_priv))
> > + if (!IS_GEN9_BC(dev_priv) && !IS_BROXTON(dev_priv) &&
> > + !IS_GEMINILAKE(dev_priv))
>
> That could be written as
>
> if (!IS_GEN9_BC(dev_priv) && !IS_GEN9_LP(dev_priv))
>
> which in turn could just be written as
>
> if (!IS_GEN9(dev_priv))
>
> ...but since GLK has gen 10 display, so I'm wondering if the same issue
> will be present in gen 10 too, and whether this should just become
>
> if (INTEL_GEN(dev_priv) < 9)
+1. I opened here to exactly add same comment.
>
> BR,
> Jani.
>
>
>
> > return;
> >
> > i915_audio_component_get_power(kdev);
>
> --
> Jani Nikula, Intel Open Source Technology Center
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH] drm/i915/audio: Fix audio detection issue on GLK
2018-04-08 13:36 Gaurav K Singh
@ 2018-04-09 14:07 ` Jani Nikula
2018-04-09 19:10 ` Rodrigo Vivi
0 siblings, 1 reply; 19+ messages in thread
From: Jani Nikula @ 2018-04-09 14:07 UTC (permalink / raw)
To: Gaurav K Singh, intel-gfx
On Sun, 08 Apr 2018, Gaurav K Singh <gaurav.k.singh@intel.com> wrote:
> On Geminilake, sometimes audio card is not getting
> detected after reboot. This is a spurious issue happening on
> Geminilake. 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 GLK. Extending this fix to GLK as well.
>
> Tested by Du,Wenkai on GLK board.
>
> Signed-off-by: Gaurav K Singh <gaurav.k.singh@intel.com>
> ---
> drivers/gpu/drm/i915/intel_audio.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c
> index 656f6c931341..73b1e0b96f88 100644
> --- a/drivers/gpu/drm/i915/intel_audio.c
> +++ b/drivers/gpu/drm/i915/intel_audio.c
> @@ -729,7 +729,8 @@ 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) && !IS_BROXTON(dev_priv))
> + if (!IS_GEN9_BC(dev_priv) && !IS_BROXTON(dev_priv) &&
> + !IS_GEMINILAKE(dev_priv))
That could be written as
if (!IS_GEN9_BC(dev_priv) && !IS_GEN9_LP(dev_priv))
which in turn could just be written as
if (!IS_GEN9(dev_priv))
...but since GLK has gen 10 display, so I'm wondering if the same issue
will be present in gen 10 too, and whether this should just become
if (INTEL_GEN(dev_priv) < 9)
BR,
Jani.
> return;
>
> i915_audio_component_get_power(kdev);
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH] drm/i915/audio: Fix audio detection issue on GLK
@ 2018-04-08 13:36 Gaurav K Singh
2018-04-09 14:07 ` Jani Nikula
0 siblings, 1 reply; 19+ messages in thread
From: Gaurav K Singh @ 2018-04-08 13:36 UTC (permalink / raw)
To: intel-gfx
On Geminilake, sometimes audio card is not getting
detected after reboot. This is a spurious issue happening on
Geminilake. 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 GLK. Extending this fix to GLK as well.
Tested by Du,Wenkai on GLK board.
Signed-off-by: Gaurav K Singh <gaurav.k.singh@intel.com>
---
drivers/gpu/drm/i915/intel_audio.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c
index 656f6c931341..73b1e0b96f88 100644
--- a/drivers/gpu/drm/i915/intel_audio.c
+++ b/drivers/gpu/drm/i915/intel_audio.c
@@ -729,7 +729,8 @@ 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) && !IS_BROXTON(dev_priv))
+ if (!IS_GEN9_BC(dev_priv) && !IS_BROXTON(dev_priv) &&
+ !IS_GEMINILAKE(dev_priv))
return;
i915_audio_component_get_power(kdev);
--
1.9.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 19+ messages in thread
end of thread, other threads:[~2018-04-18 8:07 UTC | newest]
Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-17 7:18 [PATCH] drm/i915/audio: Fix audio detection issue on GLK Gaurav K Singh
2018-04-17 8:00 ` ✗ Fi.CI.BAT: failure for drm/i915/audio: Fix audio detection issue on GLK (rev2) Patchwork
2018-04-17 8:05 ` Martin Peres
2018-04-17 8:02 ` [PATCH] drm/i915/audio: Fix audio detection issue on GLK Jani Nikula
2018-04-17 9:18 ` ✓ Fi.CI.BAT: success for drm/i915/audio: Fix audio detection issue on GLK (rev2) Patchwork
2018-04-17 10:05 ` ✗ Fi.CI.IGT: failure " Patchwork
2018-04-17 18:00 ` [PATCH] drm/i915/audio: Fix audio detection issue on GLK Kumar, Abhay
-- strict thread matches above, loose matches on Subject: below --
2018-04-17 18:22 Gaurav K Singh
2018-04-17 18:28 ` Du,Wenkai
2018-04-18 6:04 ` Singh, Gaurav K
2018-04-18 8:07 ` Jani Nikula
2018-04-08 13:36 Gaurav K Singh
2018-04-09 14:07 ` Jani Nikula
2018-04-09 19:10 ` Rodrigo Vivi
2018-04-09 19:18 ` Kumar, Abhay
2018-04-09 23:20 ` Dhinakaran Pandiyan
2018-04-09 23:13 ` Kumar, Abhay
2018-04-10 11:02 ` Jani Nikula
2018-04-11 19:27 ` Singh, Gaurav K
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.