This series cleans up some defines related to usb transfers. It's based on
the series I sent yesterday.
Martin Kaiser (4):
staging: r8188eu: usb_read_port_complete needs no regs parameter
staging: r8188eu: remove defines that strip regs parameter
staging: r8188eu: remove unused defines
staging: r8188eu: remove usb_ops_linux.h
drivers/staging/r8188eu/hal/usb_ops_linux.c | 6 ++++-
drivers/staging/r8188eu/include/usb_ops.h | 2 --
.../staging/r8188eu/include/usb_ops_linux.h | 27 -------------------
.../staging/r8188eu/os_dep/usb_ops_linux.c | 1 -
4 files changed, 5 insertions(+), 31 deletions(-)
delete mode 100644 drivers/staging/r8188eu/include/usb_ops_linux.h
--
2.30.2
There's only two defines left in usb_ops_linux.h. Move those defines to
usb_ops_linux.c, where they are used.
Remove the now empty file usb_ops_linux.h.
Signed-off-by: Martin Kaiser <[email protected]>
---
drivers/staging/r8188eu/hal/usb_ops_linux.c | 4 ++++
drivers/staging/r8188eu/include/usb_ops.h | 2 --
drivers/staging/r8188eu/include/usb_ops_linux.h | 11 -----------
drivers/staging/r8188eu/os_dep/usb_ops_linux.c | 1 -
4 files changed, 4 insertions(+), 14 deletions(-)
delete mode 100644 drivers/staging/r8188eu/include/usb_ops_linux.h
diff --git a/drivers/staging/r8188eu/hal/usb_ops_linux.c b/drivers/staging/r8188eu/hal/usb_ops_linux.c
index 0f789d5250db..9c940ab8fd41 100644
--- a/drivers/staging/r8188eu/hal/usb_ops_linux.c
+++ b/drivers/staging/r8188eu/hal/usb_ops_linux.c
@@ -7,6 +7,10 @@
#include "../include/usb_ops.h"
#include "../include/rtl8188e_hal.h"
+#define VENDOR_CMD_MAX_DATA_LEN 254
+
+#define RTW_USB_CONTROL_MSG_TIMEOUT 500/* ms */
+
static int usb_read(struct adapter *adapt, u16 value, void *data, u8 size)
{
struct dvobj_priv *dvobjpriv = adapter_to_dvobj(adapt);
diff --git a/drivers/staging/r8188eu/include/usb_ops.h b/drivers/staging/r8188eu/include/usb_ops.h
index ddc46cb44358..5bd8ce37aebf 100644
--- a/drivers/staging/r8188eu/include/usb_ops.h
+++ b/drivers/staging/r8188eu/include/usb_ops.h
@@ -17,8 +17,6 @@
#define MAX_VENDOR_REQ_CMD_SIZE 254 /* 8188cu SIE Support */
#define MAX_USB_IO_CTL_SIZE (MAX_VENDOR_REQ_CMD_SIZE + ALIGNMENT_UNIT)
-#include "usb_ops_linux.h"
-
/*
* Increase and check if the continual_urb_error of this @param dvobjprivei
* is larger than MAX_CONTINUAL_URB_ERR
diff --git a/drivers/staging/r8188eu/include/usb_ops_linux.h b/drivers/staging/r8188eu/include/usb_ops_linux.h
deleted file mode 100644
index b149df7957dd..000000000000
--- a/drivers/staging/r8188eu/include/usb_ops_linux.h
+++ /dev/null
@@ -1,11 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause */
-/* Copyright(c) 2007 - 2011 Realtek Corporation. */
-
-#ifndef __USB_OPS_LINUX_H__
-#define __USB_OPS_LINUX_H__
-
-#define VENDOR_CMD_MAX_DATA_LEN 254
-
-#define RTW_USB_CONTROL_MSG_TIMEOUT 500/* ms */
-
-#endif
diff --git a/drivers/staging/r8188eu/os_dep/usb_ops_linux.c b/drivers/staging/r8188eu/os_dep/usb_ops_linux.c
index 583533631133..c9ee1173a1d6 100644
--- a/drivers/staging/r8188eu/os_dep/usb_ops_linux.c
+++ b/drivers/staging/r8188eu/os_dep/usb_ops_linux.c
@@ -4,7 +4,6 @@
#define _USB_OPS_LINUX_C_
#include "../include/drv_types.h"
-#include "../include/usb_ops_linux.h"
#include "../include/rtl8188e_recv.h"
static unsigned int ffaddr2pipehdl(struct dvobj_priv *pdvobj, u32 addr)
--
2.30.2
usb_ops_linux.h has a number of defines that strip the regs parameter from
function that should be used as urb callbacks.
Remove the defines for functions that don't exist in the r8188eu driver.
Signed-off-by: Martin Kaiser <[email protected]>
---
drivers/staging/r8188eu/include/usb_ops_linux.h | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/drivers/staging/r8188eu/include/usb_ops_linux.h b/drivers/staging/r8188eu/include/usb_ops_linux.h
index 0a8fefbd4c2e..c7a0dc482761 100644
--- a/drivers/staging/r8188eu/include/usb_ops_linux.h
+++ b/drivers/staging/r8188eu/include/usb_ops_linux.h
@@ -13,13 +13,4 @@
#define RTW_USB_BULKOUT_TIME 5000/* ms */
-#define _usbctrl_vendorreq_async_callback(urb, regs) \
- _usbctrl_vendorreq_async_callback(urb)
-#define usb_bulkout_zero_complete(purb, regs) \
- usb_bulkout_zero_complete(purb)
-#define usb_write_mem_complete(purb, regs) \
- usb_write_mem_complete(purb)
-#define usb_read_interrupt_complete(purb, regs) \
- usb_read_interrupt_complete(purb)
-
#endif
--
2.30.2
Remove some unused defines from usb_ops_linux.h.
Signed-off-by: Martin Kaiser <[email protected]>
---
drivers/staging/r8188eu/include/usb_ops_linux.h | 5 -----
1 file changed, 5 deletions(-)
diff --git a/drivers/staging/r8188eu/include/usb_ops_linux.h b/drivers/staging/r8188eu/include/usb_ops_linux.h
index c7a0dc482761..b149df7957dd 100644
--- a/drivers/staging/r8188eu/include/usb_ops_linux.h
+++ b/drivers/staging/r8188eu/include/usb_ops_linux.h
@@ -6,11 +6,6 @@
#define VENDOR_CMD_MAX_DATA_LEN 254
-#define RTW_USB_CONTROL_MSG_TIMEOUT_TEST 10/* ms */
#define RTW_USB_CONTROL_MSG_TIMEOUT 500/* ms */
-#define MAX_USBCTRL_VENDORREQ_TIMES 10
-
-#define RTW_USB_BULKOUT_TIME 5000/* ms */
-
#endif
--
2.30.2
On 1/24/23 22:01, Martin Kaiser wrote:
> This series cleans up some defines related to usb transfers. It's based on
> the series I sent yesterday.
>
> Martin Kaiser (4):
> staging: r8188eu: usb_read_port_complete needs no regs parameter
> staging: r8188eu: remove defines that strip regs parameter
> staging: r8188eu: remove unused defines
> staging: r8188eu: remove usb_ops_linux.h
>
> drivers/staging/r8188eu/hal/usb_ops_linux.c | 6 ++++-
> drivers/staging/r8188eu/include/usb_ops.h | 2 --
> .../staging/r8188eu/include/usb_ops_linux.h | 27 -------------------
> .../staging/r8188eu/os_dep/usb_ops_linux.c | 1 -
> 4 files changed, 5 insertions(+), 31 deletions(-)
> delete mode 100644 drivers/staging/r8188eu/include/usb_ops_linux.h
>
Tested-by: Philipp Hortmann <[email protected]> # Edimax N150