Bug 34532

Summary: Properly restore CLOCK_CNTL_INDEX register
Product: xorg Reporter: Mark Kettenis <kettenis>
Component: Driver/RadeonAssignee: xf86-video-ati maintainers <xorg-driver-ati>
Status: RESOLVED FIXED QA Contact: Xorg Project Team <xorg-team>
Severity: normal    
Priority: medium    
Version: git   
Hardware: All   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
Fix none

Description Mark Kettenis 2011-02-21 05:36:21 UTC
Created attachment 43600 [details]
Fix

RADEONRestore() restores the CLOCK_CNTL_INDEX register before calling
RADEONRestorePLLRegisters(), which modifies the CLOCK_CNTL_INDEX register
to access the PLL registers.  As a result we may end up with the wrong clock
being selected when exiting X.  This happens on platforms where the driver
doesn't save and restore the VGA state.
Comment 1 Alex Deucher 2011-02-21 07:56:56 UTC
Pushed thanks!
acd54a48a6680956ed903fc890204c0c910b6712

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.