lkml.org 
[lkml]   [2021]   [Mar]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 01/40] staging: rtl8723bs: replace RT_TRACE with public printk wrappers in core/rtw_xmit.c
I'm sorry but we can't accept this patch.

> @@ -481,12 +481,13 @@ static s32 update_attrib_sec_info(struct adapter *padapter, struct pkt_attrib *p
> pattrib->mac_id = psta->mac_id;
>
> if (psta->ieee8021x_blocked == true) {
> - RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("\n psta->ieee8021x_blocked == true\n"));
> + pr_err("%s psta->ieee8021x_blocked == true\n", DRIVER_PREFIX);

Here we have change debug code that's never printed into an error
message.

A bunch of the rest are technically "wrong" but harmless.

> if (!pxmitpriv->pallocated_frame_buf) {
> pxmitpriv->pxmit_frame_buf = NULL;
> - RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("alloc xmit_frame fail!\n"));
> + pr_err("%s alloc xmit_frame fail!\n", DRIVER_PREFIX);

We don't print warning messages for allocation failures. Checkpatch
is supposed to complain.

I always encourage people to think about everything deeply and look at
the context and read the error messages. But if you send a patch which
mindlessly deletes all these RT_TRACE() messages, then we will apply
that.

It's unfortunate that you have to re-write the first patch in a 40
patch series. :/

regards,
dan carpenter

\
 
 \ /
  Last update: 2021-03-31 12:18    [W:0.209 / U:0.648 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site