On Thu, Oct 10, 2013 at 1:39 AM, Rafał Miłecki <zajec5@xxxxxxxxx> wrote: > 2013/10/10 Alex Deucher <alexdeucher@xxxxxxxxx>: >> 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@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
