lkml.org 
[lkml]   [2008]   [Oct]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [announce] new tree: "fix all build warnings, on all configs"
Date
On Tuesday, 21 of October 2008, Andi Kleen wrote:
> On Tue, Oct 21, 2008 at 01:17:16PM +0200, Ingo Molnar wrote:
> >
> > * Andi Kleen <andi@firstfloor.org> wrote:
> >
> > > > * acpi_pm_disable_gpes - Disable the GPEs.
> > > > */
> > > > -static int acpi_pm_disable_gpes(void)
> > > > +static inline int acpi_pm_disable_gpes(void)
> > >
> > > Just to satisfy my curiosity, what compiler warning does marking
> > > functions inline fix?
> >
> > the commit log below explains the situation. The warning exposed a maze
> > of #ifdefs in drivers/acpi/sleep/main.c. It's not the warning we need to
> > "fix" but that maze, obviously.
>
> Thanks. That makes sense,
>
> I also agree with you that the better alternative would be
> to just always force SUSPEND together with ACPI.
>
> I suspect the code delta wouldn't be very large compared to the rest
> of the ACPI code.

Is that _really_ necessary?

I mean, do the warnings appear in any case that's not theoretically impossible?

Rafael


\
 
 \ /
  Last update: 2008-10-21 21:37    [W:0.193 / U:0.008 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site