On Thu, Oct 10, 2013 at 1:39 AM, Rafał Miłecki <zaj...@gmail.com> wrote: > 2013/10/10 Alex Deucher <alexdeuc...@gmail.com>: >> drm/radeon: use hw generated CTS/N values for audio > > I'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 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel