Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Wed, 7 Apr 2021 17:14:28 +0200
From:   Greg KH <gregkh@linuxfoundation.org>
To:     Fabio Aiuto <fabioaiuto83@gmail.com>
Cc:     linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 00/19] staging: rtl8723bs: remove DBG_871X trace macro
Message-ID: <YG3MVGBomCeVicj+@kroah.com>
References: <cover.1617802415.git.fabioaiuto83@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <cover.1617802415.git.fabioaiuto83@gmail.com>
Precedence: bulk

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