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
