lkml.org 
[lkml]   [2022]   [Jan]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v3 1/2] staging: r8188eu: remove unneeded ret variables
Hello Greg,

On Tue, Jan 25, 2022 at 04:10:57PM +0100, Greg KH wrote:
> Many, if not most, of these functions should either be having their
> return value checked, or be void functions as no one checks their return
> value and they can not fail. Please split this up and look at each
> function to determine which is is and how to fix it up properly. Just
> returning 0 all the time is not the correct thing to do all the time.
>
> One example would be rtw_p2p_get_status() It can not fail, so why does
> it return anything?

Thanks for reviewing the patches.

I had split the changes in a way that the first patch removes the
unneeded return variables and the second patch converts the functions
(changed by the first patch) whose return values are not used, to return
void.

But yes, I now think, it is better to just convert the functions
whose return values are not used, directly to return void instead of
first removing the unneeded return variable and then converting
to return void.

I'll resend this as a single patch.

Thanks,
Abdun Nihaal

\
 
 \ /
  Last update: 2022-01-27 16:26    [W:0.342 / U:0.052 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site