353-v5.1-brcmfmac-remove-set-but-not-used-variable-old_state.patch 1.4 KB

1234567891011121314151617181920212223242526272829303132333435363738
  1. From e4d1b2716b8859199c28f2ac5f984bd05a146a6b Mon Sep 17 00:00:00 2001
  2. From: YueHaibing <[email protected]>
  3. Date: Mon, 18 Feb 2019 08:08:46 +0000
  4. Subject: [PATCH] brcmfmac: remove set but not used variable 'old_state'
  5. Fixes gcc '-Wunused-but-set-variable' warning:
  6. drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c: In function 'brcmf_usb_state_change':
  7. drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c:578:6: warning:
  8. variable 'old_state' set but not used [-Wunused-but-set-variable]
  9. It's never used and can be removed.
  10. Signed-off-by: YueHaibing <[email protected]>
  11. Acked-by: Arend van Spriel <[email protected]>
  12. Signed-off-by: Kalle Valo <[email protected]>
  13. ---
  14. drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c | 2 --
  15. 1 file changed, 2 deletions(-)
  16. --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c
  17. +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c
  18. @@ -575,7 +575,6 @@ static void
  19. brcmf_usb_state_change(struct brcmf_usbdev_info *devinfo, int state)
  20. {
  21. struct brcmf_bus *bcmf_bus = devinfo->bus_pub.bus;
  22. - int old_state;
  23. brcmf_dbg(USB, "Enter, current state=%d, new state=%d\n",
  24. devinfo->bus_pub.state, state);
  25. @@ -583,7 +582,6 @@ brcmf_usb_state_change(struct brcmf_usbd
  26. if (devinfo->bus_pub.state == state)
  27. return;
  28. - old_state = devinfo->bus_pub.state;
  29. devinfo->bus_pub.state = state;
  30. /* update state of upper layer */