Attachment 88775 Details for Bug 69675 – patch 1/2: correct ACR table

[patch] patch 1/2: correct ACR table

0001-drm-radeon-audio-correct-ACR-table.patch (text/plain), 2.24 KB, created by Pierre Ossman on 2013-11-06 19:12:59 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Pierre Ossman

Created: 2013-11-06 19:12:59 UTC

Size: 2.24 KB

patch

obsolete

>From 4b611f8a2a75d0208af0a13086108bba3bfaec4d Mon Sep 17 00:00:00 2001
>From: Pierre Ossman <pierre@ossman.eu>
>Date: Wed, 6 Nov 2013 20:00:32 +0100
>Subject: [PATCH 1/2] drm/radeon/audio: correct ACR table
>
>The values were taken from the CEC HDMI spec, but they assumed
>exact x/1.001 clocks. Since we round the clocks, we also need
>to calculate different N and CTS values.
>
>Note that the N for 25.2/1.001 MHz at 44.1 kHz audio is out of
>spec. Hopefully this mode is rarely used and/or HDMI sinks
>tolerate overly large values of N.
>
>Signed-off-by: Pierre Ossman <pierre@ossman.eu>
>---
> drivers/gpu/drm/radeon/r600_hdmi.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/drivers/gpu/drm/radeon/r600_hdmi.c b/drivers/gpu/drm/radeon/r600_hdmi.c
>index 06022e3..c8dd566 100644
>--- a/drivers/gpu/drm/radeon/r600_hdmi.c
>+++ b/drivers/gpu/drm/radeon/r600_hdmi.c
>@@ -57,15 +57,15 @@ enum r600_hdmi_iec_status_bits {
> static const struct radeon_hdmi_acr r600_hdmi_predefined_acr[] = {
>     /*	     32kHz	  44.1kHz	48kHz    */
>     /* Clock      N     CTS      N     CTS      N     CTS */
>-    {  25175,  4576,  28125,  7007,  31250,  6864,  28125 }, /*  25,20/1.001 MHz */
>+    {  25175,  4096,  25175, 28224, 125875,  6144,  25175 }, /*  25,20/1.001 MHz */
>     {  25200,  4096,  25200,  6272,  28000,  6144,  25200 }, /*  25.20       MHz */
>     {  27000,  4096,  27000,  6272,  30000,  6144,  27000 }, /*  27.00       MHz */
>     {  27027,  4096,  27027,  6272,  30030,  6144,  27027 }, /*  27.00*1.001 MHz */
>     {  54000,  4096,  54000,  6272,  60000,  6144,  54000 }, /*  54.00       MHz */
>     {  54054,  4096,  54054,  6272,  60060,  6144,  54054 }, /*  54.00*1.001 MHz */
>-    {  74176, 11648, 210937, 17836, 234375, 11648, 140625 }, /*  74.25/1.001 MHz */
>+    {  74176,  4096,  74176,  5733,  75335,  6144,  74176 }, /*  74.25/1.001 MHz */
>     {  74250,  4096,  74250,  6272,  82500,  6144,  74250 }, /*  74.25       MHz */
>-    { 148352, 11648, 421875,  8918, 234375,  5824, 140625 }, /* 148.50/1.001 MHz */
>+    { 148352,  4096, 148352,  5733, 150670,  6144, 148352 }, /* 148.50/1.001 MHz */
>     { 148500,  4096, 148500,  6272, 165000,  6144, 148500 }, /* 148.50       MHz */
>     {      0,  4096,      0,  6272,      0,  6144,      0 }  /* Other */
> };
>-- 
>1.8.3.1
>

Actions: View | Splinter Review

Attachments on bug 69675: 86598 | 86739 | 86740 | 86741 | 86856 | 88450 | 88476 | 88775 | 88776

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.