Re: [linux-pm] [RFC][PATCH] PM: Introduce new top level suspendand hibernation callbacks (rev. 2)

From: Johannes Berg
Date: Thu Mar 20 2008 - 20:21:45 EST



> + * All of the above callbacks, except for @complete(), return error codes.
> + * However, the error codes returned by the resume operations, @resume(),
> + * @thaw(), and @restore(), are only printed in the system logs, since the PM
> + * core cannot do anything else about them.

Why bother and not just make them return void, the error printing can
most likely be done much much better in the callback since that possibly
has information on why it failed.

> + * All of the above callbacks, return error codes, but the error codes returned
> + * by the resume operations, @resume_noirq(), @thaw_noirq(), and
> + * @restore_noirq(), are only printed in the system logs, since the PM core
> + * cannot do anything else about them.

Same here, of course.

johannes

Attachment: signature.asc
Description: This is a digitally signed message part