On Wed, Jul 19, 2006 at 12:00:48PM +0200, Rafael J. Wysocki wrote: > Hi, > > On Wednesday 19 July 2006 11:17, Stefan Seyfried wrote: > > I dug this out again after updating the vbetool code. > > This hack does the equivalent of > > > > X=`vbetool vbemode get` > > <suspend> > > <maybe vbetool post> > > vbetool vbemode set $X > > Nice. [Perhaps I should try it on my box.] > VBE_SAVE notoriosly segfaults on x866_64 machines.
Well, i'm not sure if VBE_MODE is better :-) > > > a) "feels" better to me > > b) the HAL / pm-utils guys have a similar method in their repertoire, and > > i'd like to have "s2ram $SOME_ADVANCED_OPTIONS_DETERMINED_BY_HAL" as an > > drop-in-replacement to this stuff, so i need this. > > OK > > > I actually included the nx5000 whitelist change, to show a potential usage > > of this flag. > > Generally looks good (see below for one little comment). > > Rafael > > Index: vbetool/vbetool.c > > =================================================================== > > RCS file: /cvsroot/suspend/suspend/vbetool/vbetool.c,v > > retrieving revision 1.6 > > diff -u -p -r1.6 vbetool.c > > --- vbetool/vbetool.c 19 Jul 2006 08:51:04 -0000 1.6 > > +++ vbetool/vbetool.c 19 Jul 2006 09:13:23 -0000 > > @@ -379,6 +379,11 @@ int do_set_mode (int mode, int vga) { > > return 0; > > } > > > > +void set_vbe_mode(int mode) > > +{ > > + (void)do_set_mode(mode, 0); > > +} > > I'd remove the (void) here. I think i did this due to a compiler warning (do_set_mode is int). > > @@ -441,12 +446,18 @@ int do_set_panel_brightness(int brightne > > return 0; > > } > > > > -int do_get_mode() { > > +int __get_mode() > > +{ > > reg_frame regs; > > int error; > > > > memset(®s, 0, sizeof(regs)); > > error = do_vbe_service(0x4f03, 0, ®s); > > + return error; > > +} And i know the error variable is totally superfluous here, but it keeps the diff to upstream vbetool smaller ;-) -- Stefan Seyfried | "Please, just tell people QA / R&D Team Mobile Devices | to use KDE." SUSE LINUX Products GmbH, Nürnberg | -- Linus Torvalds ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys -- and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Suspend-devel mailing list Suspend-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/suspend-devel