]> git.neil.brown.name Git - history.git/commit
[PATCH] e100: early reset fix
authorAndrew Morton <akpm@osdl.org>
Wed, 24 Nov 2004 09:53:50 +0000 (04:53 -0500)
committerJeff Garzik <jgarzik@pobox.com>
Wed, 24 Nov 2004 09:53:50 +0000 (04:53 -0500)
commit139292f20a31e29e6d97eb266d6d253b56cb9181
treea64a38421828f5a2a9afc59bb3255d23cdc56771
parent0a57a616d65ff5c872ad9b010a89a09f82e27861
[PATCH] e100: early reset fix

Gents, what do we think of the below fix?

Thanks.

My IBM A21P laptop is getting an IRQ storm at boot.

IRQ #11 is shared between USB and e100, and USB is getting in there first.
Consequently, when e100 initialises the hardware, that interrupt line is ready
to go.  As soon as e100.c runs pci_set_master(), the interrupt hits the CPU.
But of course the e100 driver isn't ready to handle the interrupt yet, so the
system disables IRQ #11.

This only happens on warm boots (/sbin/reboot).  Things work OK from power-on.
 So I assume that the BIOS is failing to fully reset the NIC and that some
sort of interrupt is internally pending.

The patch rearranges e100_probe() so that we issue e100_hw_reset() prior to
running pci_set_master(), and fixes the problem.

Note that e100_probe() is now running e100_hw_reset() twice - I didn't remove
the later call for general paranoia reasons.

eepro100.c has the same lockup, and needs a similar fix.

Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Jeff Garzik <jgarzik@pobox.com>
drivers/net/e100.c