Re: [PATCH 00/19] staging: rtl8723bs: remove DBG_871X trace macro

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Apr 07, 2021 at 05:14:28PM +0200, Greg KH wrote:
> On Wed, Apr 07, 2021 at 03:49:24PM +0200, Fabio Aiuto wrote:
> > This patchset removes all DBG_871X usages and definitions.
> > 
> > The whole private tracing system is not tied to a configuration
> > symbol and the default behaviour is _trace nothing_.
> > 
> > DBG_871X macros require the code to be modified by
> > hand in order to be turned on. This obviously has not happened
> > since the code was merged, so just remove them as they are unused.
> > 
> > First patch fix a DBG_871X call. It has three args over only two
> > placeholders in the first format string argument.
> > If I would not make this fix, the semantic patch
> > used to bulk remove all macro occurences would ignore the abnormal
> > macro (the one with three args) and all subsequent occurrences in
> > the same file (core/rtw_recv.c).
> > 
> > The second patch applies the cocci script.
> > 
> > This is the semantic patch:
> > 
> > @@
> > expression a, b, c, d, e, f, g, h, i, j, k;
> > constant B, C, D, E;
> > @@
> > 
> > (
> > -	DBG_871X(a);
> > |
> > -	DBG_871X(a, b);
> > |
> > -	DBG_871X(a, B);
> > |
> > -	DBG_871X(a, b, c);
> > |
> > -	DBG_871X(a, B, c);
> > |
> > -	DBG_871X(a, b, C);
> > |
> > -	DBG_871X(a, B, C);
> > |
> > -	DBG_871X(a, b, c, d);
> > |
> > -	DBG_871X(a, B, c, d);
> > |
> > -	DBG_871X(a, b, C, d);
> > |
> > -	DBG_871X(a, b, c, D);
> > |
> > -	DBG_871X(a, B, C, d);
> > |
> > -	DBG_871X(a, B, c, D);
> > |
> > -	DBG_871X(a, b, C, D);
> > |
> > -	DBG_871X(a, B, C, D);
> > |
> > -	DBG_871X(a, b, c, d, e);
> > |
> > -	DBG_871X(a, B, c, d, e);
> > |
> > -	DBG_871X(a, b, C, d, e);
> > |
> > -	DBG_871X(a, b, c, D, e);
> > |
> > -	DBG_871X(a, b, c, d, E);
> > |
> > -	DBG_871X(a, B, C, d, e);
> > |
> > -	DBG_871X(a, B, c, D, e);
> > |
> > -	DBG_871X(a, B, c, d, E);
> > |
> > -	DBG_871X(a, b, C, D, e);
> > |
> > -	DBG_871X(a, b, C, d, E);
> > |
> > -	DBG_871X(a, b, c, D, E);
> > |
> > -	DBG_871X(a, B, C, D, e);
> > |
> > -	DBG_871X(a, B, C, d, E);
> > |
> > -	DBG_871X(a, B, c, D, E);
> > |
> > -	DBG_871X(a, b, C, D, E);
> > |
> > -	DBG_871X(a, B, C, D, E);
> > |
> > -	DBG_871X(a, b, c, d, e, f);
> > |
> > -	DBG_871X(a, b, c, d, e, f, g);
> > |
> > -	DBG_871X(a, b, c, d, e, f, g, h);
> > |
> > -	DBG_871X(a, b, c, d, e, f, g, h, i);
> > |
> > -	DBG_871X(a, b, c, d, e, f, g, h, i, j);
> > |
> > -	DBG_871X(a, b, c, d, e, f, g, h, i, j, k);
> > )
> > 
> > The third one removes an unmatched macro call,
> > maybe due to the trailing \, because the occurrence is
> > inside a macro expansion.
> > 
> > The fourth one deletes all commented out call spread all over
> > the rtl8723bs code.
> > 
> > Fifth one removes definitions.
> > 
> > All remaining patches do some code cleaning on all
> > places where DBG_871X has been removed.
> 
> Nice work, thanks for doing this, all now queued up!
> 
> greg k-h

thank you,

fabio




[Index of Archives]     [Linux Driver Development]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux