Module Name: src Committed By: msaitoh Date: Tue Oct 15 15:56:26 UTC 2019
Modified Files: src/sys/dev/pci: if_alc.c Log Message: Apply FreeBSD r218141: > alc_rev was used without initialization such that it failed to > apply AR8152 v1.0 specific initialization code. Fix this bug by > explicitly reading PCI device revision id via PCI accessor. > > Reported by: Gabriel Linder ( linder.gabriel <> gmail dot com ) To generate a diff of this commit: cvs rdiff -u -r1.38 -r1.39 src/sys/dev/pci/if_alc.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/sys/dev/pci/if_alc.c diff -u src/sys/dev/pci/if_alc.c:1.38 src/sys/dev/pci/if_alc.c:1.39 --- src/sys/dev/pci/if_alc.c:1.38 Tue Jul 9 08:46:58 2019 +++ src/sys/dev/pci/if_alc.c Tue Oct 15 15:56:26 2019 @@ -1,4 +1,4 @@ -/* $NetBSD: if_alc.c,v 1.38 2019/07/09 08:46:58 msaitoh Exp $ */ +/* $NetBSD: if_alc.c,v 1.39 2019/10/15 15:56:26 msaitoh Exp $ */ /* $OpenBSD: if_alc.c,v 1.1 2009/08/08 09:31:13 kevlo Exp $ */ /*- * Copyright (c) 2009, Pyun YongHyeon <yong...@freebsd.org> @@ -1190,6 +1190,7 @@ alc_attach(device_t parent, device_t sel char intrbuf[PCI_INTRSTR_LEN]; sc->alc_ident = alc_find_ident(pa); + sc->alc_rev = PCI_REVISION(pa->pa_class); aprint_naive("\n"); aprint_normal(": %s\n", sc->alc_ident->name); @@ -1393,7 +1394,6 @@ alc_attach(device_t parent, device_t sel * Don't use Tx CMB. It is known to have silicon bug. */ sc->alc_flags |= ALC_FLAG_CMB_BUG; - sc->alc_rev = PCI_REVISION(pa->pa_class); sc->alc_chip_rev = CSR_READ_4(sc, ALC_MASTER_CFG) >> MASTER_CHIP_REV_SHIFT; if (alcdebug) {