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.