[PATCH 1/2] powerpc/module_64: Improve restore_r2() return semantics
Petr Mladek
pmladek at suse.com
Thu Jan 26 00:24:50 AEDT 2023
On Tue 2023-01-24 19:38:04, Josh Poimboeuf wrote:
> restore_r2() returns 1 on success, which is surprising for a non-boolean
> function. Change it to return 0 on success and -errno on error to match
> kernel coding convention.
>
> Signed-off-by: Josh Poimboeuf <jpoimboe at kernel.org>
Looks good:
Reviewed-by: Petr Mladek <pmladek at suse.com>
It is in the right direction. Just note that there are more functions
with the boolean semantic passed via int return value. But there
are also other functions already using the 0/-E* return values so
this is rather positive change.
Best Regards,
Petr
More information about the Linuxppc-dev
mailing list