hi, this changes if() BUG(); constructs to BUG_ON() which is cleaner and can better optimized away Signed-off-by: Eric Sesterhenn --- linux-2.6.16-rc1-git4/drivers/net/tulip/winbond-840.c.orig 2006-01-26 20:31:30.000000000 +0100 +++ linux-2.6.16-rc1-git4/drivers/net/tulip/winbond-840.c 2006-01-26 20:33:11.000000000 +0100 @@ -1645,7 +1645,7 @@ static int w840_suspend (struct pci_dev /* no more hardware accesses behind this line. */ - if (np->csr6) BUG(); + BUG_ON(np->csr6); if (ioread32(ioaddr + IntrEnable)) BUG(); /* pci_power_off(pdev, -1); */