Re: [PATCH v2 0/5] Trivial code cleanup patches

Dan Carpenter
2023-06-07T15:10:15Z

On Wed, Jun 07, 2023 at 08:31:19PM +0530, Yogesh Hegde wrote:
> These patches remove variables in the struct rtllib_device that were
> set only once throughout the driver. Instead of using these variables,
> the patches modify the relevant code to directly call the corresponding
> functions, eliminating unnecessary indirection and removing CamelCase.
> The patches are required to be applied in sequence.
>
> Yogesh Hegde (5):
> staging: rtl8192e: Remove variable SetWirelessMode
> staging: rtl8192e: Remove variable SetBWModeHandler
> staging: rtl8192e: Remove variable LeisurePSLeave
> staging: rtl8192e: Remove variable InitialGainHandler
> staging: rtl8192e: Remove DRV_NAME definition in rtllib_debug.h
>

These all look good.

Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>

regards,
dan carpenter