Re: [ 01/10] [Suspend2] kernel/power/modules.h

[Pekka J Enberg]

On Wednesday 01 February 2006 23:01, Pekka Enberg wrote:
> > > +
> > > +static inline void suspend_initialise_module_lists(void) {
> > > +       INIT_LIST_HEAD(&suspend_filters);
> > > +       INIT_LIST_HEAD(&suspend_writers);
> > > +       INIT_LIST_HEAD(&suspend_modules);
> > > +}
> >
> > I couldn't find a user for this. I would imagine there's only one,
> > though, and this should be inlined there?

On Thu, 2 Feb 2006, Nigel Cunningham wrote:
> I forgot to mention re this - yes, there's just one caller, in another set of 
> patches I'll send later (this was just the first set!). Having the function 
> to be inlined in this .h so that it's with other module specific code, and 
> then used in the caller once it has been #included, isn't that the right way 
> to do things?

Sorry, I can't parse the above :-). My point was that this is 
probably called in a .c file so move the function in that file and 
introduce it whenever you introduce the caller.

			Pekka
-
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/