On 2 August 2011 10:04, Vasily Khoruzhick <anarsoul@xxxxxxxxx> wrote: >> + ret = if_sdio_power_on(card); > > Why do you power on card here? Maybe it's possible do it in lbs_start_card? The device must be powered while lbs_start_card() is executed. This is because lbs_start_card() runs a few commands, such as reading the MAC address. Moving the poweron/poweroff into lbs_start card() is possible, and I guess you are also finding for SPI that the only thing you need to power up the card for during probe is to execute lbs_start_card(). But personally I think it makes more sense at the interface driver level. Moving it into lbs_start_card() would be problematic if you ever needed the device powered for something else during the probe routine - you'd be forced to power-cycle the card again before or after running lbs_start_card(). >> + if_sdio_power_off(card); > > Same for power off. The card should be powered off at this point because the network interface is not up. It will then be powered on when userspace brings it up. Daniel -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
