On Fri, May 27, 2005 at 03:00:00PM -0400, John W. Linville wrote:

On Fri, May 27, 2005 at 07:53:35PM +0100, Christoph Hellwig wrote:

quoted

On Fri, May 27, 2005 at 02:47:52PM -0400, John W. Linville wrote:

quoted

quoted

+	1600  NetXtreme BCM5752 Gigabit Ethernet PCI Express

I don't think you should mention "PCI Express" here.  That can trivially
befound it looking at the configuration header.

I'm just following what is at pciids.sourceforge.net.  Plus, it is
already like that for nine other IDs:

        1659  NetXtreme BCM5721 Gigabit Ethernet PCI Express
        1677  NetXtreme BCM5751 Gigabit Ethernet PCI Express
        167d  NetXtreme BCM5751M Gigabit Ethernet PCI Express
        167e  NetXtreme BCM5751F Fast Ethernet PCI Express
        169d  NetLink BCM5789 Gigabit Ethernet PCI Express
        16dd  NetLink BCM5781 Gigabit Ethernet PCI Express
        16f7  NetXtreme BCM5753 Gigabit Ethernet PCI Express
        16fd  NetXtreme BCM5753M Gigabit Ethernet PCI Express
        16fe  NetXtreme BCM5753F Fast Ethernet PCI Express

The Broadcom guys can speak-up, but I figure they know if "PCI Express"
is appropriate for their device... :-)

ok, it should be an all or nothing.  I still think it's more than silly..