No! That's *exactly* the problem: you see:
eepro100: wait_for_cmd_done timeout!
in your logs, now *which* of the 5 eepro100 cards was it?
wait_for_cmd_done(long cmd_ioaddr) should take a 'struct net_device *'
and use net_problem, then no details needed.
> > + pci_introduce(pdev);
>
> bloat, we don't need foo_introduce() functions for every subsystem, when
> every subsystem always has an attach-new-device function.
Yes, this should be moved up into the generic pci/networking code, but I
guess they wanted a self-contained example.
> > - printk (KERN_ERR "eepro100: cannot reserve MMIO region\n");
> > + pci_problem(LOG_ERR, pdev, "eepro100: cannot reserve MMIO regio
n");
>
> bloat, no advantage over printk
Now, which of those 5 cards was it again?
> > - printk(KERN_WARNING "%s: Invalid EEPROM checksum %#4.4x
, "
> > - "check settings before activating this devic
e!\n",
> > - dev->name, sum);
> > + net_pci_problem(LOG_WARNING, dev, pdev, "Invalid EEPROM
checksum, "
> > + "check settings before activating this devic
e!",
>
> > + detail(checksum, "%#4.4x", sum));
>
> bloat, checksum is purely informational, and can be obtained through
> other means
It's a direct translation from the printk ferchissakes!
> likewise
>
> etcetera...
Exactly.
Rusty.
-- Anyone who quotes me in their sig is an idiot. -- Rusty Russell. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/