Thread (8 messages) 8 messages, 3 authors, 2012-05-29

Re: [PATCH] r6040: disable pci device if the subsequent calls (after pci_enable_device) fails

From: David Miller <davem@davemloft.net>
Date: 2012-05-29 21:28:51
Also in: lkml

From: David Miller <davem@davemloft.net>
Date: Tue, 29 May 2012 17:22:29 -0400 (EDT)
From: Florian Fainelli <florian@openwrt.org>
Date: Tue, 29 May 2012 11:20:50 +0200
quoted
On Monday 28 May 2012 17:27:03 Devendra Naga wrote:
quoted
the calls after the pci_enable_device may fail, and will error out with out
disabling it. disable the device at error paths.
Looks good, thanks Devendra!
quoted
Signed-off-by: Devendra Naga <redacted>
Acked-by: Florian Fainelli <florian@openwrt.org>
Applied.
Actually, reverted, you didn't test this patch at all.

You didn't even look at the warnings emitted by the compiler
with your changes installed.

You're passing a network device pointer into the PCI device
disable routine, which takes a PCI device pointer.

I can't express to you how extremely irritating it is when
people submit patches like this.

The bug in question is 1,000 times less harmful than your fix.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help