Re: [PATCH] staging: rtl8723bs: remove static HalDetectPwrDownMode() function

Nikolay Kulikov <[email protected]>
Newsgroups gmane.linux.kernel
Message-ID <anYKU1f7kjGrZusA@archlinux>
On Fri, Aug 07, 2026 at 09:03:21AM +0200, Greg Kroah-Hartman wrote:
> On Thu, Aug 06, 2026 at 10:34:48PM +0300, Nikolay Kulikov wrote:
> > The return value of this function is ignored at the call site, and since
> > the function has no side effects (except for the assignment to
> > pHalData->pwrdown, which is not used anywhere else), it can be removed.
> > 
> > At the same time, remove the efuse shadow functions, as they are
> > becoming unused.
> 
> Why not remove pwrdown first, which shows that it is never actually
> used, and then remove the functions in a separate patch?  that would
> "prove" that this is a safe change, as-is we need to manually verify
> this patch is correct.

Yes, that really would be better. I will do that in v2.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.