I don't think these changes will break anything with the closed-source driver. However, it might be worth considering including driver information in the fdi file, so that different quirks can be applied depending on the driver used for a given set of hardware.
This command will suspend imbrium (Acer 5100) correctly with Intrepid 64-bit and open-source ATI video driver. It works fine on Jaunty as well:
pm-suspend --quirk-radeon-off --quirk- vbemode- restore --quirk- vbestate- restore --quirk-vbe-post
The corresponding entries in /usr/share/ hal/fdi/ information/ 10freedesktop/ 20-video- quirk-pm- acer.fdi are:
<match key="system. hardware. product" contains_ outof=" 1520;1650; 5100;5110; 5570;5920" > management. quirk.radeon_ off" type="bool" >true</ merge> management. quirk.vbe_ post" type="bool" >true</ merge> management. quirk.vbemode_ restore" type="bool" >true</ merge> management. quirk.vbestate_ restore" type="bool" >true</ merge>
<merge key="power_
<merge key="power_
<merge key="power_
<merge key="power_
</match>
I don't think these changes will break anything with the closed-source driver. However, it might be worth considering including driver information in the fdi file, so that different quirks can be applied depending on the driver used for a given set of hardware.