diff options
author | Sebastian Andrzej Siewior <bigeasy@linutronix.de> | 2020-09-29 22:25:43 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2020-09-29 14:02:55 -0700 |
commit | a3b7b227f1250a90c8a9542ea6398a61e1708174 (patch) | |
tree | f3409bfc2e9595cc1bd92fd265801b269286fe5e /drivers/net/wireless/realtek/rtlwifi/debug.c | |
parent | 021b58ef51bb0e59d456e2101689052ff03a82e2 (diff) |
net: rtlwifi: Remove void* casts related to delayed work
INIT_DELAYED_WORK() takes two arguments: A pointer to the delayed work and
a function reference for the callback.
The rtl code casts all function references to (void *) because the
callbacks in use are not matching the required function signature. That's
error prone and bad pratice.
Some of the callback functions are also global, but only used in a single
file.
Clean the mess up by:
- Adding the proper arguments to the callback functions and using them in
the container_of() constructs correctly which removes the hideous
container_of_dwork_rtl() macro as well.
- Removing the type cast at the initializers
- Making the unnecessary global functions static
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Acked-by: Kalle Valo <kvalo@codeaurora.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/wireless/realtek/rtlwifi/debug.c')
0 files changed, 0 insertions, 0 deletions