Re: [PATCH v2] staging: rtl8712: remove unused variable from rtl871x_mlme.c

[Edmundo Carmona Antoranz]

On Thu, Apr 8, 2021 at 6:10 AM Sergei Krainov
<sergei.krainov.lkd@xxxxxxxxx> wrote:
> No side effects can be seen locally or in r8712_find_network()

I am sorry to jump in. Sergei, what Greg is asking is basically why
you want to delete the r8712_find_network call in the first place.
Deleting an unused variable is fine, but the problem here is that you
are _also_ deleting a call to a function that _probably_ does some
things that, even if you want to get rid of the variable, you would
probably like to keep on doing (and so the call would remain). Is that
call really not doing anything relevant? That's what you will have to
explain in the patch in order for it to make sense.

As a side note on top of the question about the call, it's not like
the variable is not being used. It's used right after the call to
r8712_find_network to check the result of the call... so is the real
purpose of the patch to remove the call in the first place and then
having the variable removed because there is no point in having it if
the call goes away?

I hope that helps.