* [PATCH] staging: rtl8723bs: fix checkpatch warnings in os_dep/osdep_service.c
@ 2026-01-28 18:28 Gus Stoermer
2026-01-29 7:20 ` Dan Carpenter
0 siblings, 1 reply; 2+ messages in thread
From: Gus Stoermer @ 2026-01-28 18:28 UTC (permalink / raw)
To: gregkh; +Cc: linux-staging, linux-kernel
From 64ab45283c7e006c3420242e0f7b00acc1c3fd38 Mon Sep 17 00:00:00 2001
From: xlumen1 <gstoermer1@gmail.com>
Date: Wed, 28 Jan 2026 18:08:46 +0000
Subject: [PATCH] staging: rtl8723bs: fix checkpatch warnings in
os_dep/osdep_service.c
Signed-off-by: xlumen1 <gstoermer1@gmail.com>
---
drivers/staging/rtl8723bs/os_dep/osdep_service.c | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/rtl8723bs/os_dep/osdep_service.c
b/drivers/staging/rtl8723bs/os_dep/osdep_service.c
index a00f9f0c8..95b1d4e8e 100644
--- a/drivers/staging/rtl8723bs/os_dep/osdep_service.c
+++ b/drivers/staging/rtl8723bs/os_dep/osdep_service.c
@@ -7,9 +7,9 @@
#include <drv_types.h>
/*
-* Translate the OS dependent @param error_code to OS independent
RTW_STATUS_CODE
-* @return: one of RTW_STATUS_CODE
-*/
+ * Translate the OS dependent @param error_code to OS independent
RTW_STATUS_CODE
+ * @return: one of RTW_STATUS_CODE
+ */
inline int RTW_STATUS_CODE(int error_code)
{
if (error_code >= 0)
@@ -152,7 +152,6 @@ void rtw_buf_update(u8 **buf, u32 *buf_len, u8
*src, u32 src_len)
kfree(ori);
}
-
/**
* rtw_cbuf_full - test if cbuf is full
* @cbuf: pointer of struct rtw_cbuf
@@ -204,6 +203,7 @@ bool rtw_cbuf_push(struct rtw_cbuf *cbuf, void *buf)
void *rtw_cbuf_pop(struct rtw_cbuf *cbuf)
{
void *buf;
+
if (rtw_cbuf_empty(cbuf))
return NULL;
@@ -226,7 +226,8 @@ struct rtw_cbuf *rtw_cbuf_alloc(u32 size)
cbuf = rtw_malloc(struct_size(cbuf, bufs, size));
if (cbuf) {
- cbuf->write = cbuf->read = 0;
+ cbuf->write = 0;
+ cbuf->read = 0;
cbuf->size = size;
}
--
2.34.1
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [PATCH] staging: rtl8723bs: fix checkpatch warnings in os_dep/osdep_service.c
2026-01-28 18:28 [PATCH] staging: rtl8723bs: fix checkpatch warnings in os_dep/osdep_service.c Gus Stoermer
@ 2026-01-29 7:20 ` Dan Carpenter
0 siblings, 0 replies; 2+ messages in thread
From: Dan Carpenter @ 2026-01-29 7:20 UTC (permalink / raw)
To: Gus Stoermer; +Cc: gregkh, linux-staging, linux-kernel
On Wed, Jan 28, 2026 at 06:28:59PM +0000, Gus Stoermer wrote:
> >From 64ab45283c7e006c3420242e0f7b00acc1c3fd38 Mon Sep 17 00:00:00 2001
> From: xlumen1 <gstoermer1@gmail.com>
> Date: Wed, 28 Jan 2026 18:08:46 +0000
> Subject: [PATCH] staging: rtl8723bs: fix checkpatch warnings in
> os_dep/osdep_service.c
>
> Signed-off-by: xlumen1 <gstoermer1@gmail.com>
> ---
Please use your real name. No commit message. The patch is corrupted.
You're doing too many things in one patch. Fix one type of warning at
a time.
regards,
dan carpenter
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-01-29 7:20 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-01-28 18:28 [PATCH] staging: rtl8723bs: fix checkpatch warnings in os_dep/osdep_service.c Gus Stoermer
2026-01-29 7:20 ` Dan Carpenter
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox