2013/10/10 Alex Deucher alexdeucher@gmail.com:
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.
It would be nice. I'm working on getting some old GPUs, tracing fglrx and testing. So I hope to get HDMI support on DCE2/3 cleaned soon.