From: Mugunthan V N <mugunthanvnm@xxxxxx> Date: Fri, 13 Dec 2013 14:35:27 +0530 > During checking the interrupts with "cat /proc/interrupts", it is showing > device name as (null), this change was done with commit id aa1a15e2d where > request_irq is changed to devm_request_irq also changing the irq name from > platform device name to net device name, but the net device is not > registered at this point with the network frame work, so devm_request_irq > is called with device name as NULL, by which it is showed as "(null)" in > "cat /proc/interrupts". So this patch moved the devm_request_irq after > the net device register so that the device name shows as eth*. This change is buggy. If the request irq fails, you have to unregister the network device, branching to clean_ale_ret is insufficient. And this shows the more fundamental problem with your change, you cannot register the network device before you request the IRQs if your ->open() method assumes that the IRQs are registered already. Which this driver does. The very moment you call request_irq(), the interface can be brought up and then ->open() method invoked. Therefore, register_netdevice() absolutely must be the very last action during the probe function. Why don't you just use the platform device name as the interrupt name? The other alternative is to only register the IRQs in the ->open() routine and free the IRQ in the ->close() method. I can't apply this, sorry. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
