mbox series

[v3,0/6] a series of checkpatch fixes on the file: rxtx.c

Message ID 20221022070612.13009-1-tanjubrunostar0@gmail.com
Headers
Series a series of checkpatch fixes on the file: rxtx.c |

Message

Tanjuate Brunostar Oct. 22, 2022, 7:06 a.m. UTC 
  The fixes are similar, mostly long lines splitting. I had to make
serveral patches to ease the work of inspectors

v2: fixed a compilation error found by the kernel test robot and
recompiled the code

v3: tends out the error persisted in the second version. this version is
a correction of that

Tanjuate Brunostar (6):
  staging: vt6655: fix lines ending in a '('
  staging: vt6655: refactor long lines of code in s_uGetRTSCTSDuration
  staging: vt6655: refactor long lines of code in s_uFillDataHead
  staging: vt6655: refactor long lines of code in s_vGenerateTxParameter
  staging: vt6655: refactor long lines of code in the rest of the file
  staging: vt6655: refactor code to avoid a line ending in '('

 drivers/staging/vt6655/rxtx.c | 532 ++++++++++++++++++++--------------
 1 file changed, 321 insertions(+), 211 deletions(-)

Comments

Greg KH Oct. 22, 2022, 7:35 a.m. UTC | #1 
On Sat, Oct 22, 2022 at 07:06:06AM +0000, Tanjuate Brunostar wrote:
> The fixes are similar, mostly long lines splitting. I had to make
> serveral patches to ease the work of inspectors
> 
> v2: fixed a compilation error found by the kernel test robot and
> recompiled the code
> 
> v3: tends out the error persisted in the second version. this version is
> a correction of that
> 
> Tanjuate Brunostar (6):
>   staging: vt6655: fix lines ending in a '('
>   staging: vt6655: refactor long lines of code in s_uGetRTSCTSDuration
>   staging: vt6655: refactor long lines of code in s_uFillDataHead
>   staging: vt6655: refactor long lines of code in s_vGenerateTxParameter
>   staging: vt6655: refactor long lines of code in the rest of the file
>   staging: vt6655: refactor code to avoid a line ending in '('
> 
>  drivers/staging/vt6655/rxtx.c | 532 ++++++++++++++++++++--------------
>  1 file changed, 321 insertions(+), 211 deletions(-)
> 
> -- 
> 2.34.1
> 
> 

Any reason you do not have "staging: vt6655:" as the prefix for this 0/X
email subject line?

thanks,

greg k-h
Tanjuate Brunostar Oct. 22, 2022, 8:22 a.m. UTC | #2 
On Sat, Oct 22, 2022 at 9:15 AM Greg KH <gregkh@linuxfoundation.org> wrote:
>
> On Sat, Oct 22, 2022 at 07:06:06AM +0000, Tanjuate Brunostar wrote:
> > The fixes are similar, mostly long lines splitting. I had to make
> > serveral patches to ease the work of inspectors
> >
> > v2: fixed a compilation error found by the kernel test robot and
> > recompiled the code
> >
> > v3: tends out the error persisted in the second version. this version is
> > a correction of that
> >
> > Tanjuate Brunostar (6):
> >   staging: vt6655: fix lines ending in a '('
> >   staging: vt6655: refactor long lines of code in s_uGetRTSCTSDuration
> >   staging: vt6655: refactor long lines of code in s_uFillDataHead
> >   staging: vt6655: refactor long lines of code in s_vGenerateTxParameter
> >   staging: vt6655: refactor long lines of code in the rest of the file
> >   staging: vt6655: refactor code to avoid a line ending in '('
> >
> >  drivers/staging/vt6655/rxtx.c | 532 ++++++++++++++++++++--------------
> >  1 file changed, 321 insertions(+), 211 deletions(-)
> >
> > --
> > 2.34.1
> >
> >
>
> Any reason you do not have "staging: vt6655:" as the prefix for this 0/X
> email subject line?
>
> thanks,
>
> greg k-h

oh I forgot to include it