Please patch i810switch to work with i945
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| i810switch (Ubuntu) |
Undecided
|
Unassigned |
Bug Description
Binary package hint: i810switch
Please patch i810switch with the attached patch so it works with the i945 chip found in ThinkPad X60, as the Fn+F7 combination for switching the display doesn’t work in Feisty. The patch was written by Dieter Jurzitza.
chastell (chastell) wrote : | #1 |
chastell (chastell) wrote : | #2 |
Master Ar2ro (ar2ro) wrote : | #4 |
Even with this patch the program fails with Segmentation Fault, when I try to run it on my Feisty amd64 machine. I've got an Aristo Slim 430 laptop with i945 chip.
(gdb) bt
#0 0x00002b172fae1483 in ?? () from /lib/libc.so.6
#1 0x00002b172fae526c in free () from /lib/libc.so.6
#2 0x00002b172fad1164 in fclose () from /lib/libc.so.6
#3 0x0000000000401376 in main ()
Blaise Alleyne (balleyne) wrote : | #5 |
I can confirm that this patch worked on my Thinkpad X60 running Feisty - thanks!
Chris Wagner (chris-wagner) wrote : | #6 |
@Master Ar2ro - Is it i810switch that is failing (with the seg-fault)? Does it fail even when the patch is *not* applied?
lezem (jhcepas) wrote : | #7 |
I have the same seg-fault after applying this patch (not before).
I'm using a macbook core2duo.
Chris Wagner (chris-wagner) wrote : | #8 |
Is anyone else having problems with the patch? It seems to work okay for me. (Well, I have had some problems, but I don't believe they're due to the i810switch.)
@lezem - Which patch are you using? The more recently posted, shorter one? Could you get a detailed backtrace (see https:/
lezem (jhcepas) wrote : | #9 |
I'm using the last patch ( the short one ). And I attach the backtrace.
Luca Falavigna (dktrkranz) wrote : | #10 |
You should inform upstream developers of your efforts, they can help you and eventually include your patch in the next release.
Sam Peterson (peabodyenator) wrote : | #11 |
Patch worked for me, I'm running a Compaq Presario C571NR. Thanks very much.
unggnu (unggnu) wrote : | #12 |
I am closing this bug because i810switch has become obsolete with the new Intel driver which is enabled by default since Gutsy. With this driver and xrandr it should be easily possible to change monitor configuration. If it still doesn't work for you please open a new bug against package xserver-
Changed in i810switch: | |
status: | New → Invalid |
chastell (chastell) wrote : | #13 |
There are lots of problems with the Intel driver (which I’ll report as bugs against xserver-
I won’t reopen this bug as I understand your decision, although I find it trivial to apply this four-line patch to the i810switch sources (and it’d be nice if it was applied ‘upstream’, instead of having to rebuild the package myself).
Gaah, sorry, only the i810switch.c part is relevant for Ubuntu. Attaching a trimmed patch.