Date:   Mon, 10 Jan 2022 13:08:43 +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
Subject: Re: [PATCH 3/7] staging: r8188eu: convert DBG_88E calls in
 core/rtw_iol.c
Message-ID: <20220110100843.GC1951@kadam>
References: <20220109215427.887-1-phil@philpotter.co.uk>
 <20220109215427.887-4-phil@philpotter.co.uk>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220109215427.887-4-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?SdFFIpyW1mJWedRJizE4jWTogyE+Z+NJE2zwVvZB3NFrK6aA5z2iSDgdjLhU?=
 =?us-ascii?Q?L6CcgyPQhyDnwnMFVZKtw05MMw6KCww2ki6aikbsZYfpnGtjieTVc9TRFzwj?=
 =?us-ascii?Q?2fsuFynKVsznYYr7idyASGbkB8/h6ysRLX9/4sUpHqI6blNH5avuYPnfHf3r?=
 =?us-ascii?Q?IWhDP4oU0f64gxuMbAlex3UwD/IGH8wSKWpNA4B+mKbnhvcMrQLSCZV2qYwR?=
 =?us-ascii?Q?/0IMaLjDLEYJYy62p+akp6rkN29pYDWdPSbDMOz5VS3roShkZ77mqRaSb8aB?=
 =?us-ascii?Q?XWeunrUrhe9vm02JN8v4+TsMFHr75R+lJ3ljQXoRTpfQaSmgoxUImxmKhLDh?=
 =?us-ascii?Q?V2YjRzRJABG232HkZFVv7KUcVP9v17/9+TEEXqsSZqRWQdcpViV6dbzXfsQZ?=
 =?us-ascii?Q?uJbnWBEBUMu1mQlDG1a3xiKRTpphReWwdPyh0yv/j4D5SJl2+g3NUmJ1Xtd0?=
 =?us-ascii?Q?PfkzGNlKAExLWEXhvycdHDLrgMkhODJ5ud/mQlhJSOJOLAw56sdL+Rg2ei4l?=
 =?us-ascii?Q?RkNC9JfIfLkXZ6yAxPCHI3P1VvTzF+RRzR7xV9bNl5z9i+wUwPlwWNZOQ4ZH?=
 =?us-ascii?Q?3eIhJGuuq0vN6XD8WSgeWK3/FsHvn/oF8yWlwwSiAzSBAfmiw9Gfu5XI5Qcp?=
 =?us-ascii?Q?q9HNzRdWU6ep1Ivo3Rx0jyB0pDxVtFDJA60U1daBfstPSEmnwaOUnyMNabm2?=
 =?us-ascii?Q?e2KbGws7VAoOVcH3JDPtU0moXH4xd8pV9v157/2m1tVrjW731G7djeyx3NTX?=
 =?us-ascii?Q?YXXU7tODG9z9zfO78Nx0q5excfeVNiBtBKOVbTd9IwP+fEArvWk4jUKIdD7z?=
 =?us-ascii?Q?orSO9VApLF863tf/vWSZ+HidRU+icAQuSlL4mXRhVw9aEegQ4yq9gruF8+kd?=
 =?us-ascii?Q?ICywyVnXYA3UkTqtuNjxlxQuhuqEbNcgIZefR60V5nCkJOPwrEz4iW6E/ZL0?=
 =?us-ascii?Q?rJvIiLK+wr1l+smfLdvUU8TjRBoEJVhxO3adQuyxif9Ulz5syw9SBosMYl/7?=
 =?us-ascii?Q?KP002LgNP99uxLVP0ssIu9Nox1HCAkbi7BnxA5Hh59YyTtN/3eJsPys5uLba?=
 =?us-ascii?Q?iF46aa0mLHwuwi7ITh+gzCCYKGTM21LIHhXQDzlXqNtv9sT9eRzVKGe9Dv4r?=
 =?us-ascii?Q?HXomJNRM3q1qMzXOie3lvJ8OCNg06MBDQPoe8h4ogifdm3Vk3TjFdhPttwAd?=
 =?us-ascii?Q?/R7ef+yrRSjtAEbHvN0QiAAuH9aN2wEMtZMbhpfgppvamoCn3+B7t17dqug2?=
 =?us-ascii?Q?HEeAW9Sok0UYrN5DkipZ/geLymWsNWHy8BFOYJoCfz4Sl8VnWL9g+DBeZ3SU?=
 =?us-ascii?Q?tWYfn7PhB89wekvzXfr5tER1SPYX16M1OrcLed1ur3IkU5vfFPbmd+io/qIF?=
 =?us-ascii?Q?KAG0C/4th3LYbVVh1f5jDT9lHqQOkRmTarnDMYg0Dd6x01otcG90i7A33//d?=
 =?us-ascii?Q?8J55F5EL0UgCSVo7lnMPamqu5ykTzhPEpR8ClI1AkT8ynCM+hjf6wEQetZeh?=
 =?us-ascii?Q?hLt3xmlhpBrcuqg5oC7wuzQAaJvnnS603UGWUUMR8UJ+08GlkLzf6AP7D4cS?=
 =?us-ascii?Q?eP7AlTzZ7fPCpXdqrHlPgpMagi4EaZlP1D4GGKMtEDhA1Jatu5dqCaPP0lJG?=
 =?us-ascii?Q?4gDR5+S48TACVt7QK/xooAjvkqXPSt8yOPdlS3NLjBLirH4hoPq+0lzMr9+x?=
 =?us-ascii?Q?m0IC6LDELQGSBqZrHLpg2E7QRl0=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b4e6303c-19f0-4c23-4fbb-08d9d4213c1d
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2365.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Jan 2022 10:09:06.2198
 (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: VWkgnDLNuqpaTg+xX94KaDlWZ2FrON4czBSHZIsjae5VEv/tdHVxKx/wFj3aV+OzczAfTj3wTMq3w3wLoUkCZVtygbDPTW4XWKjkjauBjik=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR10MB1615
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10222 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxscore=0 suspectscore=0 spamscore=0
 mlxlogscore=999 adultscore=0 phishscore=0 malwarescore=0 bulkscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2110150000
 definitions=main-2201100071
X-Proofpoint-ORIG-GUID: dGAG1AFESUWshRIIGmeVhVKxF0bIqQkR
X-Proofpoint-GUID: dGAG1AFESUWshRIIGmeVhVKxF0bIqQkR
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Sun, Jan 09, 2022 at 09:54:23PM +0000, Phillip Potter wrote:
> Convert the DBG_88E macro calls in core/rtw_iol.c to use pr_debug
> or netdev_dbg appropriately, as their information may be useful to
> observers, and this gets the driver closer to the point of being
> able to remove DBG_88E itself.
> 
> Some calls are at points in the call chain where use of dev_dbg or
> netdev_dbg isn't possible due to lack of device pointer, so plain
> pr_debug is appropriate here.
> 
> Signed-off-by: Phillip Potter <phil@philpotter.co.uk>
> ---
>  drivers/staging/r8188eu/core/rtw_iol.c | 10 ++++++----
>  1 file changed, 6 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/staging/r8188eu/core/rtw_iol.c b/drivers/staging/r8188eu/core/rtw_iol.c
> index 7e78b47c1284..923da2a9f6ae 100644
> --- a/drivers/staging/r8188eu/core/rtw_iol.c
> +++ b/drivers/staging/r8188eu/core/rtw_iol.c
> @@ -12,13 +12,15 @@ struct xmit_frame	*rtw_IOL_accquire_xmit_frame(struct adapter  *adapter)
>  
>  	xmit_frame = rtw_alloc_xmitframe(pxmitpriv);
>  	if (!xmit_frame) {
> -		DBG_88E("%s rtw_alloc_xmitframe return null\n", __func__);
> +		netdev_dbg(adapter->pnetdev,
> +			   "rtw_alloc_xmitframe return null\n");

You're going to have to send this anyway because of the compile issue.

I feel like you are not being aggressive enough in the debug messages
that you delete.  For example, this one should definitely be deleted.
Don't print an error message for alloc failures.

It would be easier to Ack a mass delete of these messages.

regards,
dan carpenter