[PATCH 1/3] staging: rtl8723bs: remove redundant ternary operators from return

[PATCH 1/3] staging: rtl8723bs: remove redundant ternary operators from return

[Ashmit Kumar]

The Linux kernel coding style specifies that boolean expressions
do not need a redundant `? true : false` ternary operator.
Additionally, return is not a function and therefore parentheses
are not required.

Signed-off-by: Ashmit Kumar <work.ashmitkumar@gmail.com>
---
 drivers/staging/rtl8723bs/os_dep/osdep_service.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/osdep_service.c b/drivers/staging/rtl8723bs/os_dep/osdep_service.c
index 4cfdf7c6234..214c1fd9b3e 100644
--- a/drivers/staging/rtl8723bs/os_dep/osdep_service.c
+++ b/drivers/staging/rtl8723bs/os_dep/osdep_service.c
@@ -130,7 +130,7 @@ void rtw_buf_update(u8 **buf, u32 *buf_len, u8 *src, u32 src_len)
  */
 inline bool rtw_cbuf_full(struct rtw_cbuf *cbuf)
 {
-	return (cbuf->write == cbuf->read - 1) ? true : false;
+	return cbuf->write == cbuf->read - 1;
 }
 
 /**
@@ -141,7 +141,7 @@ inline bool rtw_cbuf_full(struct rtw_cbuf *cbuf)
  */
 inline bool rtw_cbuf_empty(struct rtw_cbuf *cbuf)
 {
-	return (cbuf->write == cbuf->read) ? true : false;
+	return cbuf->write == cbuf->read;
 }
 
 /**
-- 
2.43.0

Re: [PATCH 1/3] staging: rtl8723bs: remove redundant ternary operators from return

[Greg KH]

On Mon, Aug 03, 2026 at 07:39:22PM +0000, Ashmit Kumar wrote:
> The Linux kernel coding style specifies that boolean expressions
> do not need a redundant `? true : false` ternary operator.
> Additionally, return is not a function and therefore parentheses
> are not required.

Why is this mixed with the media patches?  And they didn't seem to be
threaded, how was this sent?

And it's a resend?

thanks,

greg k-h