[PATCH] drm/i915/audio: Add 801Mhz clock entries to dp_aud_n_m table

[PATCH] drm/i915/audio: Add 801Mhz clock entries to dp_aud_n_m table

[Radhakrishna Sripada]

From: "Sripada, Radhakrishna" <radhakrishna.sripada@intel.com>

Expand the Maud/Naud table according to DP 1.4 spec to include entries for
810 MHz clock. This is required for audio to work with HBR3.

Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Cc: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Radhakrishna Sripada <radhakrishna.sripada@intel.com>
---
 drivers/gpu/drm/i915/intel_audio.c | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c
index 3ea566f99450..6d1c33066987 100644
--- a/drivers/gpu/drm/i915/intel_audio.c
+++ b/drivers/gpu/drm/i915/intel_audio.c
@@ -59,6 +59,7 @@
  */
 
 /* DP N/M table */
+#define LC_810M	810000
 #define LC_540M	540000
 #define LC_270M	270000
 #define LC_162M	162000
@@ -99,6 +100,15 @@ static const struct dp_aud_n_m dp_aud_n_m[] = {
 	{ 128000, LC_540M, 4096, 33750 },
 	{ 176400, LC_540M, 3136, 18750 },
 	{ 192000, LC_540M, 2048, 11250 },
+	{ 32000, LC_810M, 1024, 50625 },
+	{ 44100, LC_810M, 784, 28125 },
+	{ 48000, LC_810M, 512, 16875 },
+	{ 64000, LC_810M, 2048, 50625 },
+	{ 88200, LC_810M, 1568, 28125 },
+	{ 96000, LC_810M, 1024, 16875 },
+	{ 128000, LC_810M, 4096, 50625 },
+	{ 176400, LC_810M, 3136, 28125 },
+	{ 192000, LC_810M, 2048, 16875 },
 };
 
 static const struct dp_aud_n_m *
-- 
2.9.3

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

✓ Fi.CI.BAT: success for drm/i915/audio: Add 801Mhz clock entries to dp_aud_n_m table

[Patchwork]

== Series Details ==

Series: drm/i915/audio: Add 801Mhz clock entries to dp_aud_n_m table
URL   : https://patchwork.freedesktop.org/series/44435/
State : success

== Summary ==

= CI Bug Log - changes from CI_DRM_4290 -> Patchwork_9235 =

== Summary - SUCCESS ==

  No regressions found.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/44435/revisions/1/mbox/

== Known issues ==

  Here are the changes found in Patchwork_9235 that come from known issues:

  === IGT changes ===

    ==== Issues hit ====

    igt@debugfs_test@read_all_entries:
      fi-snb-2520m:       PASS -> INCOMPLETE (fdo#103713)

    igt@gem_exec_flush@basic-wb-rw-before-default:
      fi-glk-j4005:       PASS -> DMESG-WARN (fdo#105719)

    igt@kms_busy@basic-flip-c:
      fi-glk-j4005:       PASS -> DMESG-WARN (fdo#106097)

    igt@kms_flip@basic-flip-vs-dpms:
      fi-glk-j4005:       PASS -> DMESG-WARN (fdo#106000)

    igt@kms_pipe_crc_basic@read-crc-pipe-b-frame-sequence:
      fi-glk-j4005:       PASS -> FAIL (fdo#103481)

    
    ==== Possible fixes ====

    igt@gem_exec_suspend@basic-s4-devices:
      fi-kbl-7500u:       DMESG-WARN (fdo#105128) -> PASS

    igt@kms_flip@basic-flip-vs-modeset:
      fi-glk-j4005:       DMESG-WARN (fdo#106000) -> PASS

    igt@kms_pipe_crc_basic@read-crc-pipe-c:
      fi-glk-j4005:       DMESG-WARN (fdo#106097) -> PASS

    igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
      fi-cnl-psr:         DMESG-WARN (fdo#104951) -> PASS

    
  fdo#103481 https://bugs.freedesktop.org/show_bug.cgi?id=103481
  fdo#103713 https://bugs.freedesktop.org/show_bug.cgi?id=103713
  fdo#104951 https://bugs.freedesktop.org/show_bug.cgi?id=104951
  fdo#105128 https://bugs.freedesktop.org/show_bug.cgi?id=105128
  fdo#105719 https://bugs.freedesktop.org/show_bug.cgi?id=105719
  fdo#106000 https://bugs.freedesktop.org/show_bug.cgi?id=106000
  fdo#106097 https://bugs.freedesktop.org/show_bug.cgi?id=106097


== Participating hosts (38 -> 35) ==

  Additional (1): fi-bdw-5557u 
  Missing    (4): fi-byt-squawks fi-ilk-m540 fi-cnl-y3 fi-skl-6700hq 


== Build changes ==

    * Linux: CI_DRM_4290 -> Patchwork_9235

  CI_DRM_4290: fb7bd7a9480b64dd46964ef5a00aeef33410376d @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4510: d1a93aa7e1507de76c6c71be15931cc4b90111bb @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_9235: 7347fd680334d350d80b041fb73c2f4bff24552e @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

7347fd680334 drm/i915/audio: Add 801Mhz clock entries to dp_aud_n_m table

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_9235/issues.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

✓ Fi.CI.IGT: success for drm/i915/audio: Add 801Mhz clock entries to dp_aud_n_m table

[Patchwork]

== Series Details ==

Series: drm/i915/audio: Add 801Mhz clock entries to dp_aud_n_m table
URL   : https://patchwork.freedesktop.org/series/44435/
State : success

== Summary ==

= CI Bug Log - changes from CI_DRM_4290_full -> Patchwork_9235_full =

== Summary - WARNING ==

  Minor unknown changes coming with Patchwork_9235_full need to be verified
  manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_9235_full, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  

== Possible new issues ==

  Here are the unknown changes that may have been introduced in Patchwork_9235_full:

  === IGT changes ===

    ==== Warnings ====

    igt@gem_exec_schedule@deep-bsd2:
      shard-kbl:          PASS -> SKIP

    
== Known issues ==

  Here are the changes found in Patchwork_9235_full that come from known issues:

  === IGT changes ===

    ==== Issues hit ====

    igt@gem_wait@write-busy-bsd2:
      shard-snb:          SKIP -> INCOMPLETE (fdo#105411)

    igt@kms_cursor_legacy@2x-long-flip-vs-cursor-legacy:
      shard-hsw:          PASS -> FAIL (fdo#104873)

    igt@kms_flip@plain-flip-ts-check-interruptible:
      shard-glk:          PASS -> FAIL (fdo#100368)

    igt@kms_flip_tiling@flip-to-y-tiled:
      shard-glk:          PASS -> FAIL (fdo#104724, fdo#103822)

    
    ==== Possible fixes ====

    igt@kms_flip@2x-flip-vs-expired-vblank-interruptible:
      shard-glk:          FAIL (fdo#105363) -> PASS

    igt@kms_flip@plain-flip-fb-recreate:
      shard-glk:          FAIL (fdo#100368) -> PASS

    igt@kms_setmode@basic:
      shard-hsw:          FAIL (fdo#99912) -> PASS

    
    ==== Warnings ====

    igt@drv_selftest@live_gtt:
      shard-glk:          FAIL (fdo#105347) -> INCOMPLETE (fdo#103359, k.org#198133)

    igt@kms_sysfs_edid_timing:
      shard-hsw:          FAIL (fdo#100047) -> WARN (fdo#100047)

    
  fdo#100047 https://bugs.freedesktop.org/show_bug.cgi?id=100047
  fdo#100368 https://bugs.freedesktop.org/show_bug.cgi?id=100368
  fdo#103359 https://bugs.freedesktop.org/show_bug.cgi?id=103359
  fdo#103822 https://bugs.freedesktop.org/show_bug.cgi?id=103822
  fdo#104724 https://bugs.freedesktop.org/show_bug.cgi?id=104724
  fdo#104873 https://bugs.freedesktop.org/show_bug.cgi?id=104873
  fdo#105347 https://bugs.freedesktop.org/show_bug.cgi?id=105347
  fdo#105363 https://bugs.freedesktop.org/show_bug.cgi?id=105363
  fdo#105411 https://bugs.freedesktop.org/show_bug.cgi?id=105411
  fdo#99912 https://bugs.freedesktop.org/show_bug.cgi?id=99912
  k.org#198133 https://bugzilla.kernel.org/show_bug.cgi?id=198133


== Participating hosts (5 -> 5) ==

  No changes in participating hosts


== Build changes ==

    * Linux: CI_DRM_4290 -> Patchwork_9235

  CI_DRM_4290: fb7bd7a9480b64dd46964ef5a00aeef33410376d @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4510: d1a93aa7e1507de76c6c71be15931cc4b90111bb @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_9235: 7347fd680334d350d80b041fb73c2f4bff24552e @ git://anongit.freedesktop.org/gfx-ci/linux
  piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_9235/shards.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH] drm/i915/audio: Add 801Mhz clock entries to dp_aud_n_m table

[Jani Nikula]

On Thu, 07 Jun 2018, Radhakrishna Sripada <radhakrishna.sripada@intel.com> wrote:
> From: "Sripada, Radhakrishna" <radhakrishna.sripada@intel.com>
>
> Expand the Maud/Naud table according to DP 1.4 spec to include entries for
> 810 MHz clock. This is required for audio to work with HBR3.
>
> Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
> Cc: Jani Nikula <jani.nikula@intel.com>
> Signed-off-by: Radhakrishna Sripada <radhakrishna.sripada@intel.com>

Reviewed-by: Jani Nikula <jani.nikula@intel.com>


> ---
>  drivers/gpu/drm/i915/intel_audio.c | 10 ++++++++++
>  1 file changed, 10 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c
> index 3ea566f99450..6d1c33066987 100644
> --- a/drivers/gpu/drm/i915/intel_audio.c
> +++ b/drivers/gpu/drm/i915/intel_audio.c
> @@ -59,6 +59,7 @@
>   */
>  
>  /* DP N/M table */
> +#define LC_810M	810000
>  #define LC_540M	540000
>  #define LC_270M	270000
>  #define LC_162M	162000
> @@ -99,6 +100,15 @@ static const struct dp_aud_n_m dp_aud_n_m[] = {
>  	{ 128000, LC_540M, 4096, 33750 },
>  	{ 176400, LC_540M, 3136, 18750 },
>  	{ 192000, LC_540M, 2048, 11250 },
> +	{ 32000, LC_810M, 1024, 50625 },
> +	{ 44100, LC_810M, 784, 28125 },
> +	{ 48000, LC_810M, 512, 16875 },
> +	{ 64000, LC_810M, 2048, 50625 },
> +	{ 88200, LC_810M, 1568, 28125 },
> +	{ 96000, LC_810M, 1024, 16875 },
> +	{ 128000, LC_810M, 4096, 50625 },
> +	{ 176400, LC_810M, 3136, 28125 },
> +	{ 192000, LC_810M, 2048, 16875 },
>  };
>  
>  static const struct dp_aud_n_m *

-- 
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH] drm/i915/audio: Add 801Mhz clock entries to dp_aud_n_m table

[Jani Nikula]

On Fri, 08 Jun 2018, Jani Nikula <jani.nikula@intel.com> wrote:
> On Thu, 07 Jun 2018, Radhakrishna Sripada <radhakrishna.sripada@intel.com> wrote:
>> From: "Sripada, Radhakrishna" <radhakrishna.sripada@intel.com>
>>
>> Expand the Maud/Naud table according to DP 1.4 spec to include entries for
>> 810 MHz clock. This is required for audio to work with HBR3.
>>
>> Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
>> Cc: Jani Nikula <jani.nikula@intel.com>
>> Signed-off-by: Radhakrishna Sripada <radhakrishna.sripada@intel.com>
>
> Reviewed-by: Jani Nikula <jani.nikula@intel.com>

And pushed, thanks for the patch.

>
>
>> ---
>>  drivers/gpu/drm/i915/intel_audio.c | 10 ++++++++++
>>  1 file changed, 10 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c
>> index 3ea566f99450..6d1c33066987 100644
>> --- a/drivers/gpu/drm/i915/intel_audio.c
>> +++ b/drivers/gpu/drm/i915/intel_audio.c
>> @@ -59,6 +59,7 @@
>>   */
>>  
>>  /* DP N/M table */
>> +#define LC_810M	810000
>>  #define LC_540M	540000
>>  #define LC_270M	270000
>>  #define LC_162M	162000
>> @@ -99,6 +100,15 @@ static const struct dp_aud_n_m dp_aud_n_m[] = {
>>  	{ 128000, LC_540M, 4096, 33750 },
>>  	{ 176400, LC_540M, 3136, 18750 },
>>  	{ 192000, LC_540M, 2048, 11250 },
>> +	{ 32000, LC_810M, 1024, 50625 },
>> +	{ 44100, LC_810M, 784, 28125 },
>> +	{ 48000, LC_810M, 512, 16875 },
>> +	{ 64000, LC_810M, 2048, 50625 },
>> +	{ 88200, LC_810M, 1568, 28125 },
>> +	{ 96000, LC_810M, 1024, 16875 },
>> +	{ 128000, LC_810M, 4096, 50625 },
>> +	{ 176400, LC_810M, 3136, 28125 },
>> +	{ 192000, LC_810M, 2048, 16875 },
>>  };
>>  
>>  static const struct dp_aud_n_m *

-- 
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx