On Sun, 15 Apr 2007 02:36:12 +0200 Luca Tettamanti <[EMAIL PROTECTED]> wrote:
> > @@ -27,7 +27,16 @@ > > int s2ram_do(void) > > { > > int ret = 0; > > - FILE *f = fopen("/sys/power/state", "w"); > > + FILE *f; > > + > > + /* If this works we're done. Else we just continue as if nothing > > + * happened, future kernels will work with /s/p/s. */ > > + ret = s2ram_do_pmu(); > > + if (!ret) > > + return ret; > > + ret = 0; > > + > > + f = fopen("/sys/power/state", "w"); > > if (!f) { > > printf("/sys/power/state does not exist; what kind of ninja > > mutant machine is this?\n"); > > return ENODEV; > > This is what I was trying to avoid. PMU in generic code is not nice. > I've corrected the 'ARCH' bug above and added headers as per Pavel's > comment. > > Anyway, since PPC actually wants to override (as in "do something > different") s2ram_do it's also possibile to make it a weak symbol so > that s2ram-ppc.c can provides its own implementation, will generic code > contains a sensitive default. I'm attacching a patch with the weak > symbol version. I don't think the s2ram_do_pmu call is that problematic to have in the generic code. The trade of have to make is, do we want to duplicate the /sys/power/state stuff or do we want an empty function in x86 piece of the code. I think the latter is better choice. grts Tim
signature.asc
Description: PGP signature
------------------------------------------------------------------------- This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now. http://sourceforge.net/powerbar/db2/
_______________________________________________ Suspend-devel mailing list Suspend-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/suspend-devel