2021-03-25 08:05:19

by Jiabing Wan

[permalink] [raw]
Subject: [PATCH] [v3] drivers: staging: _adapter is declared twice

struct _adapter has been declared at 23rd line.
Remove the duplicate.

Signed-off-by: Wan Jiabing <[email protected]>
---
Changelog:
v3:
- Delete trailing whitespace.

v2:
- Delete the blank line.
---
drivers/staging/rtl8712/drv_types.h | 2 --
1 file changed, 2 deletions(-)

diff --git a/drivers/staging/rtl8712/drv_types.h b/drivers/staging/rtl8712/drv_types.h
index 0c4325073c63..a44d04effc8b 100644
--- a/drivers/staging/rtl8712/drv_types.h
+++ b/drivers/staging/rtl8712/drv_types.h
@@ -36,8 +36,6 @@ enum _NIC_VERSION {
RTL8716_NIC
};

-struct _adapter;
-
struct qos_priv {
/* bit mask option: u-apsd, s-apsd, ts, block ack... */
unsigned int qos_option;
--
2.25.1


2021-03-25 08:10:10

by Dan Carpenter

[permalink] [raw]
Subject: Re: [PATCH] [v3] drivers: staging: _adapter is declared twice

On Thu, Mar 25, 2021 at 04:00:46PM +0800, Wan Jiabing wrote:
> struct _adapter has been declared at 23rd line.
> Remove the duplicate.
>
> Signed-off-by: Wan Jiabing <[email protected]>
> ---
> Changelog:
> v3:
> - Delete trailing whitespace.
>
> v2:
> - Delete the blank line.

Bingo! :) Well done.

Reviewed-by: Dan Carpenter <[email protected]>

regards,
dan carpenter