Re: [PATCH] staging: rtl8192e: replace variable with direct return

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Michael Straube <straube.linux@xxxxxxxxx>
Subject: Re: [PATCH] staging: rtl8192e: replace variable with direct return
From: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
Date: Tue, 12 Mar 2024 12:06:00 +0300
Cc: gregkh@xxxxxxxxxxxxxxxxxxx, philipp.g.hortmann@xxxxxxxxx, linux-staging@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <20240312051816.18717-1-straube.linux@gmail.com>
References: <20240312051816.18717-1-straube.linux@gmail.com>

On Tue, Mar 12, 2024 at 06:18:16AM +0100, Michael Straube wrote:
> Remove the variable rt_status from rtl92e_send_cmd_pkt() and return
> true/false directly to improve readability.
> 
> Signed-off-by: Michael Straube <straube.linux@xxxxxxxxx>

Thanks!

Reviewed-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

regards,
dan carpenter

References:
- [PATCH] staging: rtl8192e: replace variable with direct return
  - From: Michael Straube

Prev by Date: Re: [PATCH] staging: p80211conv: Rename local foo to decrypt_check
Next by Date: Re: [PATCH] staging: p80211conv: Rename local foo to decrypt_check
Previous by thread: [PATCH] staging: rtl8192e: replace variable with direct return
Index(es):
- Date
- Thread

[Index of Archives] [Linux Driver Development] [Linux Driver Backports] [DMA Engine] [Linux GPIO] [Linux SPI] [Video for Linux] [Linux USB Devel] [Linux Coverity] [Linux Audio Users] [Linux Kernel] [Linux SCSI] [Yosemite Backpacking]

Powered by Linux