On Wed, 2007-01-31 at 07:52 +0000, Matthew Garrett wrote: > Based on previous discussions, I've implemented a rough attempt at > providing some level of basic runtime power management on the ipw2100 > chipset. This patch does the following: > > 1) On load, it initialises the hardware and then quiesces it again > 2) On interface up, it powers the hardware back up > 3) On interface down, it powers the hardware down and puts the chip in > D3 > 4) It attempts to behave correctly over suspend/resume - ie, if the > interface was down beforehand, it will ensure that the chip is powered > down
>From my understanding, the intention of this patch is to defer the device self-initialization work (including firmware loading) from netdev initialization time to netdev open time (ifconfig up) and de-initialize the device when it is not being used (ifconfig down). This saves power during the time the driver is loaded but the interface is not open. You should remove ipw2100_up() from ipw2100_net_init() which is netdev->init() since it will be called in ->open() in your patch. I'd also suggest you to request_irq()/free_irq() in the netdev ->open() and ->close() in case the device shares IRQ with other devices, the interrupt handler should not be invoked anyway. Thanks, -yi - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
