[PATCH v3] staging: rtl8723bs: fix code style issues in rtl8723b_hal.h
From: Hüseyin Can Erdem
Date: Wed Mar 18 2026 - 11:09:44 EST
Replace __attribute__((__packed__)) with the preferred __packed macro.
Add parentheses around macro arguments to avoid potential operator
precedence issues.
Signed-off-by: Hüseyin Can Erdem <erdemhuseyincan09@xxxxxxxxx>
---
drivers/staging/rtl8723bs/include/rtl8723b_hal.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rtl8723bs/include/rtl8723b_hal.h b/drivers/staging/rtl8723bs/include/rtl8723b_hal.h
index 7ec84304a..cd3b49338 100644
--- a/drivers/staging/rtl8723bs/include/rtl8723b_hal.h
+++ b/drivers/staging/rtl8723bs/include/rtl8723b_hal.h
@@ -28,7 +28,7 @@
#define FW_8723B_END_ADDRESS 0x1FFF /* 0x5FFF */
#define IS_FW_HEADER_EXIST_8723B(fw_hdr) \
- ((le16_to_cpu(fw_hdr->signature) & 0xFFF0) == 0x5300)
+ ((le16_to_cpu((fw_hdr)->signature) & 0xFFF0) == 0x5300)
struct rt_firmware {
u32 fw_length;
@@ -133,7 +133,7 @@ struct rt_firmware_hdr {
#define EFUSE_IC_ID_OFFSET 506 /* For some inferiority IC purpose.
* Added by Roger, 2009.09.02.
*/
-#define AVAILABLE_EFUSE_ADDR(addr) (addr < EFUSE_REAL_CONTENT_LEN_8723B)
+#define AVAILABLE_EFUSE_ADDR(addr) ((addr) < EFUSE_REAL_CONTENT_LEN_8723B)
#define EFUSE_ACCESS_ON 0x69 /* For RTL8723 only. */
#define EFUSE_ACCESS_OFF 0x00 /* For RTL8723 only. */
@@ -171,7 +171,7 @@ struct c2h_evt_hdr_t {
u8 CmdID;
u8 CmdLen;
u8 CmdSeq;
-} __attribute__((__packed__));
+} __packed;
enum { /* tag_Package_Definition */
PACKAGE_DEFAULT,
--
2.47.3