Module Name:    src
Committed By:   msaitoh
Date:           Thu May 30 08:55:02 UTC 2024

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

Log Message:
Move RECOVERY_MODE feature test code for readability. No functional change.

 ixgbe.c rev. 1.169 inserted RECOVERY_MODE feature test in between
 MSI-X allocation and legacy allocation. To improve code readability,
move it to earlier location.


To generate a diff of this commit:
cvs rdiff -u -r1.350 -r1.351 src/sys/dev/pci/ixgbe/ixgbe.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/ixgbe/ixgbe.c
diff -u src/sys/dev/pci/ixgbe/ixgbe.c:1.350 src/sys/dev/pci/ixgbe/ixgbe.c:1.351
--- src/sys/dev/pci/ixgbe/ixgbe.c:1.350	Mon May 13 03:15:33 2024
+++ src/sys/dev/pci/ixgbe/ixgbe.c	Thu May 30 08:55:02 2024
@@ -1,4 +1,4 @@
-/* $NetBSD: ixgbe.c,v 1.350 2024/05/13 03:15:33 msaitoh Exp $ */
+/* $NetBSD: ixgbe.c,v 1.351 2024/05/30 08:55:02 msaitoh Exp $ */
 
 /******************************************************************************
 
@@ -64,7 +64,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ixgbe.c,v 1.350 2024/05/13 03:15:33 msaitoh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ixgbe.c,v 1.351 2024/05/30 08:55:02 msaitoh Exp $");
 
 #ifdef _KERNEL_OPT
 #include "opt_inet.h"
@@ -1136,6 +1136,21 @@ ixgbe_attach(device_t parent, device_t d
 	error = ixgbe_read_pba_string(hw, buf, IXGBE_PBANUM_LENGTH);
 	aprint_normal_dev(dev, "PBA number %s\n", error ? "unknown" : buf);
 
+	/* Recovery mode */
+	switch (sc->hw.mac.type) {
+	case ixgbe_mac_X550:
+	case ixgbe_mac_X550EM_x:
+	case ixgbe_mac_X550EM_a:
+		/* >= 2.00 */
+		if (hw->eeprom.nvm_image_ver_high >= 2) {
+			sc->feat_cap |= IXGBE_FEATURE_RECOVERY_MODE;
+			sc->feat_en |= IXGBE_FEATURE_RECOVERY_MODE;
+		}
+		break;
+	default:
+		break;
+	}
+
 	if (sc->feat_en & IXGBE_FEATURE_MSIX) {
 		error = ixgbe_allocate_msix(sc, pa);
 		if (error) {
@@ -1155,21 +1170,6 @@ ixgbe_attach(device_t parent, device_t d
 		}
 	}
 
-	/* Recovery mode */
-	switch (sc->hw.mac.type) {
-	case ixgbe_mac_X550:
-	case ixgbe_mac_X550EM_x:
-	case ixgbe_mac_X550EM_a:
-		/* >= 2.00 */
-		if (hw->eeprom.nvm_image_ver_high >= 2) {
-			sc->feat_cap |= IXGBE_FEATURE_RECOVERY_MODE;
-			sc->feat_en |= IXGBE_FEATURE_RECOVERY_MODE;
-		}
-		break;
-	default:
-		break;
-	}
-
 	if ((sc->feat_en & IXGBE_FEATURE_MSIX) == 0)
 		error = ixgbe_allocate_legacy(sc, pa);
 	if (error)

Reply via email to