Re: [patch] PCI Cleanup

From: Alan Cox (alan@lxorguk.ukuu.org.uk)
Date: Tue Aug 13 2002 - 06:45:33 EST


On Tue, 2002-08-13 at 01:08, Matthew Dobson wrote:

> - if (!value)
> - return -EINVAL;
> -
> - result = pci_conf1_read(0, dev->bus->number, PCI_SLOT(dev->devfn),
> - PCI_FUNC(dev->devfn), where, 2, &data);
> -

This stil has the same problems as it did last time you posted it. The
pointless NULL check and the increased complexity over duplicating about
60 lines of code and having pci_conf1 ops cleanly as we do in 2.4.

The !value check is extremely bad because it turns a critical debuggable
software error into a silent unnoticed mistake.

Fixing the code instead of just resending it might improve the changes
of it being merged no end.

Alan

-
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/



This archive was generated by hypermail 2b29 : Thu Aug 15 2002 - 22:00:31 EST