Module Name:    src
Committed By:   msaitoh
Date:           Tue Sep 26 08:25:56 UTC 2017

Modified Files:
        src/sys/dev/pci: if_wm.c

Log Message:
- Correct NVM word size check in wm_nvm_version() to prevent
  "EERD polling failed" error on attach. if_wm.c rev. 1.534 was not perfect.
  Reported by ozaki-r@.
- Print NVM offset and word count when EERD polling failed.

XXX pullup-8.


To generate a diff of this commit:
cvs rdiff -u -r1.538 -r1.539 src/sys/dev/pci/if_wm.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_wm.c
diff -u src/sys/dev/pci/if_wm.c:1.538 src/sys/dev/pci/if_wm.c:1.539
--- src/sys/dev/pci/if_wm.c:1.538	Tue Sep 26 07:42:06 2017
+++ src/sys/dev/pci/if_wm.c	Tue Sep 26 08:25:56 2017
@@ -1,4 +1,4 @@
-/*	$NetBSD: if_wm.c,v 1.538 2017/09/26 07:42:06 knakahara Exp $	*/
+/*	$NetBSD: if_wm.c,v 1.539 2017/09/26 08:25:56 msaitoh Exp $	*/
 
 /*
  * Copyright (c) 2001, 2002, 2003, 2004 Wasabi Systems, Inc.
@@ -83,7 +83,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_wm.c,v 1.538 2017/09/26 07:42:06 knakahara Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_wm.c,v 1.539 2017/09/26 08:25:56 msaitoh Exp $");
 
 #ifdef _KERNEL_OPT
 #include "opt_net_mpsafe.h"
@@ -11768,7 +11768,8 @@ wm_nvm_read_eerd(struct wm_softc *sc, in
 		CSR_WRITE(sc, WMREG_EERD, eerd);
 		rv = wm_poll_eerd_eewr_done(sc, WMREG_EERD);
 		if (rv != 0) {
-			aprint_error_dev(sc->sc_dev, "EERD polling failed\n");
+			aprint_error_dev(sc->sc_dev, "EERD polling failed: "
+			    "offset=%d. wordcnt=%d\n", offset, wordcnt);
 			break;
 		}
 		data[i] = (CSR_READ(sc, WMREG_EERD) >> EERD_DATA_SHIFT);
@@ -12549,7 +12550,7 @@ printver:
 	}
 
 	/* Assume the Option ROM area is at avove NVM_SIZE */
-	if ((sc->sc_nvm_wordsize >= NVM_SIZE) && check_optionrom
+	if ((sc->sc_nvm_wordsize > NVM_SIZE) && check_optionrom
 	    && (wm_nvm_read(sc, NVM_OFF_COMB_VER_PTR, 1, &off) == 0)) {
 		/* Option ROM Version */
 		if ((off != 0x0000) && (off != 0xffff)) {

Reply via email to