On Thu, Oct 10, 2013 at 1:39 AM, Rafał Miłecki zajec5@gmail.com wrote:
2013/10/10 Alex Deucher alexdeucher@gmail.com:
drm/radeon: use hw generated CTS/N values for audioI'd like to see such patches earlier :| I'm OK with the change for DCE4+ (Evergreen) (I even suggested that change myself recently), but I didn't have time to check how this should be programmed on DCE2/3/4...
In your patch 0001-WIP-port-of-hdmi-dp-audio-code-to-newer-kernel.patch based on (AFAIU) internal specs you were setting HDMI_ACR_SOURCE.
According to the Christian R6xx may really have some bug for generating values by the hw, see: https://bugs.freedesktop.org/show_bug.cgi?id=69675#c12
So it doesn't look like a good patch for -fixes, especially without testing it on some hardware.
Yeah, I thought about that, but on the other hand, there is already a regression caused by the 1001 modes patches that this series fixes. I can add a patch to disable the hw generated values on older families for now if you'd prefer.
Alex