On Fri, Mar 21, 2008 at 01:11:14PM +0100, Sjoerd Simons wrote:
> On Wed, Mar 19, 2008 at 09:15:32PM -0400, The Anarcat wrote:
> > Package: hal
> > Version: 0.5.10-5
> > Severity: important
> >
> > Since I upgraded from sarge to lenny, the screen backlight doesn't turn
> > off when suspending to ram from Gnome. Here's my original post about
> > this:
> >
> > http://lists.debian.org/debian-laptop/2008/01/msg00081.html
> >
> > I have since then developped a small patch that fixes the issue for me.
> > I apply it on
>
> I've done this somewhat differently. Could you test the attached patch?
>
> Also please test if suspend/resume still works properly if you remove <merge
> key="power_management.quirk.radeon_off" type="bool">true</merge> line from the
> X31 part? It's good to only have the quirks really needed per entry
And now with the patch actually attached
Sjoerd
--
Mathematics is the only science where one never knows what
one is talking about nor whether what is said is true.
-- Russell
diff --git a/fdi/information/10freedesktop/20-video-quirk-pm-ibm.fdi b/fdi/information/10freedesktop/20-video-quirk-pm-ibm.fdi
index 4a987be..b967725 100644
--- a/fdi/information/10freedesktop/20-video-quirk-pm-ibm.fdi
+++ b/fdi/information/10freedesktop/20-video-quirk-pm-ibm.fdi
@@ -28,19 +28,17 @@
</match>
<!-- X22, X40; X32 -->
<match key="system.hardware.product" prefix_outof="2662;2672;2673">
- <merge key="power_management.quirk.radeon_off" type="bool">true</merge>
- <merge key="power_management.quirk.s3_bios" type="bool">true</merge>
- <merge key="power_management.quirk.s3_mode" type="bool">true</merge>
+ <match key="system.hardware.version" contains_not="X31">
+ <merge key="power_management.quirk.radeon_off" type="bool">true</merge>
+ <merge key="power_management.quirk.s3_bios" type="bool">true</merge>
+ <merge key="power_management.quirk.s3_mode" type="bool">true</merge>
+ </match>
</match>
<!-- X31 -->
<match key="system.hardware.product" prefix="2672">
<match key="system.hardware.version" contains="X31">
<merge key="power_management.quirk.dpms_suspend" type="bool">true</merge>
- <!-- already added above
- <merge key="power_management.quirk.s3_bios" type="bool">true</merge>
- <merge key="power_management.quirk.s3_mode" type="bool">true</merge>
- <merge key="power_management.quirk.radeon_off" type="bool">true</merge>
- -->
+ <merge key="power_management.quirk.radeon_off" type="bool">true</merge>
</match>
</match>
<!-- R50p, R52, X41t, X40, T30, T41p, T41p, T42, X40, Z60t, X41, Z60m, T32 (2647) -->
