On Tue, Oct 28, 2025 at 01:18:44PM +0100, Stefan Sperling wrote:
> Fix PCI product ID of BCM57417.  Found while trying to use all
> bnxt product IDs in a switch statement. Which does not build due
> to a duplicate case value 0x16d8. FreeBSD uses 0x16d9 for BCM57417.
> 
> ok?

ok jan@

> M  sys/dev/pci/pcidevs  |  1+  1-
> 
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> commit - 86328f1ca48ae28269aeb8068dc2e34d6484a245
> commit + ff8da6dc667414de85cfcbe352eb32e974720d70
> blob - 7264bcaa6cbcdfd76b26a62cc9d1af56a7b89daf
> blob + e3488b177eb329e7703092fecbf8b36c52dacd1f
> --- sys/dev/pci/pcidevs
> +++ sys/dev/pci/pcidevs
> @@ -2679,7 +2679,7 @@ product BROADCOM BCM57407	0x16d5	BCM57407
>  product BROADCOM BCM57412	0x16d6	BCM57412
>  product BROADCOM BCM57414	0x16d7	BCM57414
>  product BROADCOM BCM57416	0x16d8	BCM57416
> -product BROADCOM BCM57417	0x16d8	BCM57417
> +product BROADCOM BCM57417	0x16d9	BCM57417
>  product BROADCOM BCM5781	0x16dd	BCM5781
>  product BROADCOM BCM57314	0x16df	BCM57314
>  product BROADCOM BCM57417_SFP	0x16e2	BCM57417 SFP
>