Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 5c29e21d authored by Cindy H Kao's avatar Cindy H Kao Committed by Inaky Perez-Gonzalez
Browse files

wimax/iwmc3200: don't disable the SDIO function if enable fails 



In the iwmc3200, disabling the WiMAX SDIO function when enable fails
would possibly result in a device reset triggered by the iwmc3200's
top controller since it monitors the bus reset activities from each
SDIO function. In any case, the disable makes no sense; if the enable
fails, it should not be disabled.

Thus we remove the unecessary sdio_disable_func() in
i2400ms_enable_function().

Signed-off-by: default avatarCindy H Kao <cindy.h.kao@intel.com>
Signed-off-by: default avatarInaky Perez-Gonzalez <inaky@linux.intel.com>
parent f2696fbd

drivers/net/wimax/i2400m/sdio.c

+0 −1
Original line number Diff line number Diff line
@@ -130,7 +130,6 @@ int i2400ms_enable_function(struct sdio_func *func) 
			goto function_enabled;

		}

		d_printf(2, dev, "SDIO function failed to enable: %d\n", err);

		sdio_disable_func(func);

		sdio_release_host(func);

		msleep(I2400MS_INIT_SLEEP_INTERVAL);

	}