brcmfmac: usb: call brcmf_usb_up() during brcmf_bus_preinit()
authorArend Van Spriel <arend.vanspriel@broadcom.com>
Mon, 19 Feb 2018 23:14:21 +0000 (00:14 +0100)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 27 Feb 2018 16:21:26 +0000 (18:21 +0200)
By calling brcmf_usb_up() during brcmf_bus_preinit() it does not need
to be called in brcmf_usb_bus_setup().

Reviewed-by: Hante Meuleman <hante.meuleman@broadcom.com>
Reviewed-by: Pieter-Paul Giesberts <pieter-paul.giesberts@broadcom.com>
Reviewed-by: Franky Lin <franky.lin@broadcom.com>
Signed-off-by: Arend van Spriel <arend.vanspriel@broadcom.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c

index b27170c12482de0ca4115b4bc9484c214d1c4b86..d22cd1662da60ed74f3a0b6cd7950ed6ea5595a4 100644 (file)
@@ -1146,8 +1146,9 @@ static int brcmf_usb_get_fwname(struct device *dev, u32 chip, u32 chiprev,
 }
 
 static const struct brcmf_bus_ops brcmf_usb_bus_ops = {
-       .txdata = brcmf_usb_tx,
+       .preinit = brcmf_usb_up,
        .stop = brcmf_usb_down,
+       .txdata = brcmf_usb_tx,
        .txctl = brcmf_usb_tx_ctlpkt,
        .rxctl = brcmf_usb_rx_ctlpkt,
        .wowl_config = brcmf_usb_wowl_config,
@@ -1165,10 +1166,6 @@ static int brcmf_usb_bus_setup(struct brcmf_usbdev_info *devinfo)
                return ret;
        }
 
-       ret = brcmf_usb_up(devinfo->dev);
-       if (ret)
-               goto fail;
-
        ret = brcmf_bus_started(devinfo->dev);
        if (ret)
                goto fail;