MLK-20190-03 brcmfmac: check thre return value of .brcmf_fil_iovar_data_get()

Check the return value of .brcmf_fil_iovar_data_get() to fix
the coverity issue of "error handling issue".

Reviewed-by: Haibo Chen <haibo.chen@nxp.com>
Signed-off-by: Fugang Duan <fugang.duan@nxp.com>
Signed-off-by: Arulpandiyan Vadivel <arulpandiyan_vadivel@mentor.com>

diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c
index be1acfa..eaca7e5 100644 (file)
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c
@@ -2547,12 +2547,15 @@ brcmf_sdio_ulp_preinit(struct device *dev)
        struct brcmf_bus *bus_if = dev_get_drvdata(dev);
        struct brcmf_sdio_dev *sdiodev = bus_if->bus_priv.sdio;
        struct brcmf_if *ifp = bus_if->drvr->iflist[0];
+       s32 err = 0;
 
        brcmf_dbg(TRACE, "Enter\n");
 
        /* Query ulp_sdioctrl iovar to get the ULP related SHM offsets */
-       brcmf_fil_iovar_data_get(ifp, "ulp_sdioctrl", &sdiodev->shm_ulp,
-                                sizeof(sdiodev->shm_ulp));
+       err = brcmf_fil_iovar_data_get(ifp, "ulp_sdioctrl", &sdiodev->shm_ulp,
+                                      sizeof(sdiodev->shm_ulp));
+       if (err)
+               brcmf_err("ulp_sdioctrl iovar returned err = %d\n", err);
 
        sdiodev->ulp = false;