Re: wakeup code translated to .c

From: Sam Ravnborg
Date: Sun Feb 03 2008 - 17:27:26 EST


On Sun, Feb 03, 2008 at 11:24:07PM +0100, Pavel Machek wrote:
> On Sun 2008-02-03 19:49:31, Sam Ravnborg wrote:
> > On Sun, Feb 03, 2008 at 07:16:48PM +0100, Pavel Machek wrote:
> > > Hi!
> > >
> > > > > This version works on 32-bit, and builds on 64-bit (but I'm pretty
> > > > > sure it does not work. 32-bit code probably needs to go into rm/....)
> > > > >
> > > >
> > > > Do you have an updated version or is this the latest one?
> > >
> > > I'm glad you ask ;-). Here's reasonably-recent version (I have
> > > slightly cleaner one, but it got obscured by 2.6.24-git merge), I
> > > eventually got it to work on 64-bit, by reusing trampoline.S code.
> > >
> > > Pavel
> > > Signed-off-by: Pavel Machek <pavel@xxxxxxx>
> >
> > Hi PAvel.
> >
> > Before you submit this for real we need me to take a deper
> > look into your Makefiles.
>
> Ok, lets get this right.
>
> > We do not want this duplication between boot and acpi.
> > And the way you deal with code sharing from boot/ is not
> > OK either.
>
> Well, current version has one copy in .c, then two duplicate .S
> version, based on older code.... so few duplicate Makefiles do not
> seem like huge deal.
It is more the single files that just include another .c file.

Can you drop me an updated patch sometime Monday - I already lost
the inital mail and I guess you continue your updates.

Sam
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/