Re: [PATCH] staging: rtl8192u: Align descendant arguments

[Gilbert Adikankwu <gilbertadikankwu@gmail.com>]

On Sun, Oct 15, 2023 at 10:24:06AM +0200, Andi Shyti wrote:
> Hi Gilbert,
> 
> On Sun, Oct 15, 2023 at 08:44:18AM +0100, Gilbert Adikankwu wrote:
> > Adhere to Linux kernel coding style.
> > 
> > "...A very commonly used style is to align descendants to a function
> > open parenthesis" - (Excerpted from Linux kernel coding style (#2))
> > 
> > Reported by checkpatch:
> > 
> > CHECK: Alignment should match open parenthesis
> > 
> > Signed-off-by: Gilbert Adikankwu <gilbertadikankwu@gmail.com>
> > ---
> >  drivers/staging/rtl8192u/r8180_93cx6.c     |  4 ++--
> >  drivers/staging/rtl8192u/r8190_rtl8256.c   | 16 +++++++--------
> >  drivers/staging/rtl8192u/r8192U_core.c     | 20 +++++++++---------
> >  drivers/staging/rtl8192u/r8192U_dm.c       | 24 +++++++++++-----------
> >  drivers/staging/rtl8192u/r819xU_firmware.c |  4 ++--
> >  drivers/staging/rtl8192u/r819xU_phy.c      |  2 +-
> >  6 files changed, 35 insertions(+), 35 deletions(-)
> 
> Do you mind splitting one patch per file? It's not really
> necessary, but normally easier to review.
> 
> As you are here, if you then run checkpatch on your patch you
> will gather ideas for your next patches.
> 
> Andi

Hi Andi,

Thank you for your response. If it makes the review process easier, I
don't mind splitting one patch per file. I will do as you have
suggested. Should I make it a patchset?

Gilbert