lkml.org 
[lkml]   [2023]   [Jun]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v3 0/4] Trivial code cleanup patches
From
On 6/18/23 15:03, Yogesh Hegde wrote:
> Rename variables in struct rtllib_device to avoid CamelCase which is not
> accepted by checkpatch.
>
> ---
> v3: The driver is split into 2 modules, calling the functions directly which
> the v2 of the patchset does breaks compile. So reverting back to v1 of
> the patch as suggested by Greg Kroah-Hartman <gregkh@linuxfoundation.org>.
>
> v2: Removed the variable and called the function directly instead of
> just renaming the variable as suggested by Greg Kroah-Hartman
> <gregkh@linuxfoundation.org>.
>
>
>
> Yogesh Hegde (4):
> staging: rtl8192e: Rename variable SetWirelessMode
> staging: rtl8192e: Rename variable SetBWModeHandler
> staging: rtl8192e: Rename variable LeisurePSLeave
> staging: rtl8192e: Rename variable InitialGainHandler
>
> drivers/staging/rtl8192e/rtl8192e/r8192E_phy.c | 4 ++--
> drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 8 ++++----
> drivers/staging/rtl8192e/rtl819x_HTProc.c | 4 ++--
> drivers/staging/rtl8192e/rtllib.h | 12 ++++++------
> drivers/staging/rtl8192e/rtllib_rx.c | 2 +-
> drivers/staging/rtl8192e/rtllib_softmac.c | 12 ++++++------
> drivers/staging/rtl8192e/rtllib_softmac_wx.c | 6 +++---
> 7 files changed, 24 insertions(+), 24 deletions(-)
>
This patch series alone is fine. But when my patch series ([PATCH 0/9]
staging: rtl8192e: Remove unsupported modes a and 5G) is applied before
it fails as we change same lines. Will see what happens.

Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>




\
 
 \ /
  Last update: 2023-06-18 22:27    [W:0.119 / U:1.236 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site