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:   Tue, 25 Jan 2022 12:02:33 +0300
From:   Dan Carpenter <dan.carpenter@oracle.com>
To:     Phillip Potter <phil@philpotter.co.uk>
Cc:     gregkh@linuxfoundation.org, Larry.Finger@lwfinger.net,
        straube.linux@gmail.com, martin@kaiser.cx,
        linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org,
        paskripkin@gmail.com
Subject: Re: [PATCH 02/10] staging: r8188eu: remove smaller sets of DBG_88E
 calls from core dir
Message-ID: <20220125090233.GR1978@kadam>
References: <20220124224415.831-1-phil@philpotter.co.uk>
 <20220124224415.831-3-phil@philpotter.co.uk>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220124224415.831-3-phil@philpotter.co.uk>
User-Agent: Mutt/1.9.4 (2018-02-28)
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?cyzpxPEclOeAXUf/S8y16MubIJ01kPpOlXhAqgRUcWjR9Jo7elKw5COxXHzx?=
 =?us-ascii?Q?g8WxSi+of6FWK50JfVXe7Yh1eEYHYlMbgcuNsEEUF5Vm1fEqNyYSZleeZPfm?=
 =?us-ascii?Q?Scft3WWl7+JNjsqadUEq5678VzkwW9OaIzgNz+vrd0lHbdsy/zpdePyvtmJS?=
 =?us-ascii?Q?ldE+2Q2m6ujIBiNZKbA+wLvI2uQRoPg/Vjg4WokMJcogdDPlQoyEFWCekWOa?=
 =?us-ascii?Q?Ov/rpGUYd+WajBDRACWIOxB7hVAgRd3E01mQ/yDEek0edLQkoncUSNUkKgNA?=
 =?us-ascii?Q?qpw72jfC4JVTQLjjsMcaBw3xv7zdffg3iXd+BStbsyATYbWRFrauXwXEH8DM?=
 =?us-ascii?Q?xGYMIL+QzR+7K5D3Y1IA/oQbA+DG7wuesdCUEKbRTWKqourwF1oN/WIWrC2W?=
 =?us-ascii?Q?tWZogm5Ryq0giJKadVZd686uSp5IPBjwO5nPucT7XHiHrSZIO+WEdFKZudjg?=
 =?us-ascii?Q?siFheDpqyWpsA7/zCswbfwaqdmFpTcBB56+x2G3bgKWNfMmMaWmMGzcq/Sfi?=
 =?us-ascii?Q?zKqRsxzcRPjPedGU5lJyxmAWkliEzJqPWCsMgQOyfOMBaD4js9sdwkUX8/oj?=
 =?us-ascii?Q?g2kUvkXhMTX6Bo+YakFigeTmcWYsVH1GtkZ+2lU6PXRMOcW9UNjL6GaovFbW?=
 =?us-ascii?Q?rFVxbVgfFWAcIFPljnow6Di2L7zycgoj7JV8KGzHnFoBPrBKyq8R6qAXcH4Y?=
 =?us-ascii?Q?5kJ5Xq8i3xPASacuwExL592OIwpuulCh4g4Lumlz3UnqGJTB8btbbeCNcrSk?=
 =?us-ascii?Q?oi2hodoKveBR+fh2dIUWcJgA2ZOAfgke9Sit5UTyQ2a3hQ0Q6iK7UnpfWFlQ?=
 =?us-ascii?Q?fn/OFKRygIfDrkAyhVwLQ/E1RVkp/KiUZQELxn0mAYgD3whTtdRWkzgKOzSk?=
 =?us-ascii?Q?QdgIQ87pPU4FHl4GkgjXa1wnDOi+lI3zUUUi8TDqgp5BBYdTJ6FJ+EQN6mkm?=
 =?us-ascii?Q?7x6RQprj836HPQgbGGwbvWL+0bS2niZ2oGV5D2qKCwBICJ9uHSuJk/BBZMwz?=
 =?us-ascii?Q?BsT5spppzGKebxNPMnZcxeKlNyzayTRf/jHa/4ittKMZIfSAWokFfjH2il23?=
 =?us-ascii?Q?/OUC11hOM0MDytbNUPF3Wg2Cua6egYqi/Lmq6uT4H/IoyRE/eVcoJzcGS1u3?=
 =?us-ascii?Q?hc6TMGQ4OprvbG3aWPvR0NOCJLUUWDnxExkta/cLeOdA87SKSMLdD4Fob1bZ?=
 =?us-ascii?Q?3I/bOsdK/Zq9PDD+4KKS1Ioe+okCgS24wS6mFzz9K/JF5sKHFmqF+fMQLMJc?=
 =?us-ascii?Q?P/tXuQrHTSNZV7tCjyePkBiihyk4WHXtW8BWcETGNdLFzPOZlhNazgt4Bvqw?=
 =?us-ascii?Q?B3nphseTHKI8vitkMUkn9u9bLfF8CbAnHx7lm8HeWHD+13obxko4oC2wAJlN?=
 =?us-ascii?Q?HmxD7bdd72F/hm6P8lPeLWOMyzTQ3Cy+UCOnW5fgxkcAIukWQn97cJ/PA4bJ?=
 =?us-ascii?Q?1uoI9xZCtPDLPe3/tr/z4ywR1YXtTKXsaLlg8m4XcaPSDfQcgcAU7fQF1ldZ?=
 =?us-ascii?Q?/5hAS8MQslNVtecq5zqy0QLDfUAcCERH4EhZyPE+MoBdupwti82nnD1RFsG6?=
 =?us-ascii?Q?Ji+Hh4Y3r9aib+ZSR6Da4a2UdnVBj1LQvVaY2v8JuUpjDaZWTgmaxiNCrejN?=
 =?us-ascii?Q?YK1+cR+9oPvciTqCVm5vlDTiE+H9sre9FSEJmNrvTBnZrdCETZIgesi0jf0H?=
 =?us-ascii?Q?RsudQCU/2G+0bTK21m7PSGBN1c0=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 6e480dce-b3d6-4e9e-507e-08d9dfe17911
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2365.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Jan 2022 09:02:54.3167
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: Ncn5G/6IOGBzMbEHJXgn4XlhDDLSjuVhzpq8xPbjNaMEqFJVXJFXmO/MqZr3vAQJ8qQQi9EUamtMnLP6YDc7e8N6fbysR1UASg3/ygyGFiA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN8PR10MB3732
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10237 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 bulkscore=0 mlxscore=0 malwarescore=0
 spamscore=0 phishscore=0 suspectscore=0 adultscore=0 mlxlogscore=999
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2201110000
 definitions=main-2201250060
X-Proofpoint-GUID: HIQ0QvWVGbkwqPzdmnI5PiZtpL9mIcEH
X-Proofpoint-ORIG-GUID: HIQ0QvWVGbkwqPzdmnI5PiZtpL9mIcEH
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, Jan 24, 2022 at 10:44:07PM +0000, Phillip Potter wrote:
>  void rtw_sctx_done_err(struct submit_ctx **sctx, int status)
>  {
>  	if (*sctx) {
> -		if (rtw_sctx_chk_waring_status(status))
> -			DBG_88E("%s status:%d\n", __func__, status);
> +		rtw_sctx_chk_waring_status(status);

In a follow on patch, can you just remove the rtw_sctx_chk_waring_status()
function?

Looking through these warnings, a mass delete is the right thing.  The
debugging messages are garbage.  Use ftrace.  Some of the errors are
debug level so they will never be seen.  If you enable printing error
messages, it print a debug information but at error level so that's
useless.  Forget about it.  Just delete it.

You're going to need to redo these again because kbuild found some
build warnings...

Normally we would say if you're deleting a printk() and that makes the
curly braces unnecessary, then you have to delete the curly braces in
the same patch.  But in this case, you're changing a ton of code and
deleting the curly braces makes it hard to review.  I would prefer it
if you just deleted the printks and fixed up the curly braces in a
follow up patch.  And please don't fix long line warnings either.  Do
that in another even later patch if you want.

You do need to fix any build warnings about unused variables etc, but
don't worry about checkpatch until the end.

Basically as much as possible I'd prefer to only see deleted lines in
this patch.  No insertions.

 12 files changed, 66 insertions(+), 472 deletions(-)

There was one function where you removed the padatper argument because
it wasn't necessary.  That's great but do it in a follow on patch.
Keep a notepad app next to your vim and write down notes as you go:

0) Re-run checkpatch and delete unnecessary curly braces
1) Delete rtw_sctx_chk_waring_status()
2) padatper not needed for frob_whatever()
3) Fix long lines

regards,
dan carpenter