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

Dan Carpenter <[email protected]>
Newsgroups org.kernel.vger.linux-kernel,dev.linux.lists.linux-staging
Message-ID <[email protected]>
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.
> 

The patch needs to get rid of pwrdown from include/hal_data.h.  There
are two reasons for this.

1) For correctness and cleanliness.
2) It makes the patch easier to review because as the patch is written
   now we need to check the kernel source to see if what you're saying
   is true.  But if we delete it and it is actually used somewhere then
   we can rely on the kbuild-bot to find the mistake.

regards,
dan carpenter
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.