wifi: brcmsmac: avoid function pointer casts
authorArnd Bergmann <arnd@arndb.de>
Tue, 13 Feb 2024 10:05:37 +0000 (11:05 +0100)
committerKalle Valo <kvalo@kernel.org>
Fri, 16 Feb 2024 15:48:59 +0000 (17:48 +0200)
commite1ea6db35fc3ba5ff063f097385e9f7a88c25356
treef49b970d7f9ecee37b3bd70b4b598502c3c8a1c8
parent42ffccd0a36e099dea3d3272c5d62a0454ded1f0
wifi: brcmsmac: avoid function pointer casts

An old cleanup went a little too far and causes a warning with clang-16
and higher as it breaks control flow integrity (KCFI) rules:

drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy_shim.c:64:34: error: cast from 'void (*)(struct brcms_phy *)' to 'void (*)(void *)' converts to incompatible function type [-Werror,-Wcast-function-type-strict]
   64 |                         brcms_init_timer(physhim->wl, (void (*)(void *))fn,
      |                                                       ^~~~~~~~~~~~~~~~~~~~

Change this one instance back to passing a void pointer so it can be
used with the timer callback interface.

Fixes: d89a4c80601d ("staging: brcm80211: removed void * from softmac phy")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com>
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://msgid.link/20240213100548.457854-1-arnd@kernel.org
drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_cmn.c
drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy_shim.c
drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy_shim.h