From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f52.google.com (mail-wr1-f52.google.com [209.85.221.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D5D016D0D for ; Wed, 7 Apr 2021 16:49:07 +0000 (UTC) Received: by mail-wr1-f52.google.com with SMTP id f12so12712143wro.0 for ; Wed, 07 Apr 2021 09:49:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=s4obH9YIZgpmzsW+SCrIe4AakWZcXJUvY0gyNebmfoA=; b=VtgsnxbpikqPS8uUSFrDn8gWyKhnnnyLZ99c4HgIi+pY8j6cNTYKsrrkougkL7rv92 XRSOoTa+Qw9Qul+IPJOV6FEUu6kOPQ9KfYGn1ZkyKkJcelFe1SBSCEkPSmwvmrH+wsCa fbG4/QcHAFRHsxcl6gGeF21lFAOs0Sx4gI5d2LFBT332RVazDfwdfAFOxEVFokUv9TTz MixuGs9qmvKefyn2jfyOjti5tNlo1SdxWALGFKZT+cNiRfQ5T91A1MiFLIkeMhQOn8br kC0pquv8urJ+dgfiThUj1NXIusgBL537GxghTZLHqY2u6MemO/aC7L/nNghFymySH5Zi D/+Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=s4obH9YIZgpmzsW+SCrIe4AakWZcXJUvY0gyNebmfoA=; b=bvhkUoRyR5wNZ2Yl8uBDf4NssaS2AdjzaUPc+HiLwobOGyG7eLbghMJgWtMCDBh2rJ Q72ITzWBOf2oZwKN5ssuKoNYJPCt+UmblTh/6k7a/3k56qIO6oOG0S9pz5qnG0v6FbOl 3swHzyTHeHF6wSGMIgU9NrRJAzFFpSoKTt3kB+4khkbcfTGj+Ehc9aVFoLFxp3wXAmST jQhMnqvvOl7HCBugIChdtpEwilC7ilBSiOjvOdMXWk/NlPaws34gUMvEImkr14JqNZA2 1xAF3JiBAeJ+sRGwcbc7Xmqwz207Hzl73FaosbChCSvm2/pVuSdUMdVWvtmrvmTOWy6U dWtQ== X-Gm-Message-State: AOAM532gzWYM28EaMg2vsh0dzKIvK/JRJoDeVQ8kqV5jfgww4BR44+wT iCbnfOX3f/5cLtRAO408Z8I= X-Google-Smtp-Source: ABdhPJzHWR41KWvKZXmIW2NuUyMinGdiVpqZvO2/fzOf1XQYFyKq8/xBCI0iAuHAPBmYx2EcqyiLXQ== X-Received: by 2002:adf:b642:: with SMTP id i2mr5348617wre.8.1617814146380; Wed, 07 Apr 2021 09:49:06 -0700 (PDT) Received: from agape.jhs ([5.171.72.134]) by smtp.gmail.com with ESMTPSA id s8sm40255543wrn.97.2021.04.07.09.49.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 07 Apr 2021 09:49:05 -0700 (PDT) Date: Wed, 7 Apr 2021 18:49:03 +0200 From: Fabio Aiuto To: Greg KH 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: <20210407164901.GA1403@agape.jhs> References: X-Mailing-List: linux-staging@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) 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