Module Name:    src
Committed By:   kiyohara
Date:           Sun Aug  1 06:57:06 UTC 2010

Modified Files:
        src/sys/dev/marvell: gtidmac.c gtmpsc.c gtpci.c gttwsi.c if_gfe.c
            mvpex.c mvsata_mv.c

Log Message:
Cleanup foo_match().
  + Remove check for model.
  + Remove unnecessary null line.


To generate a diff of this commit:
cvs rdiff -u -r1.5 -r1.6 src/sys/dev/marvell/gtidmac.c
cvs rdiff -u -r1.39 -r1.40 src/sys/dev/marvell/gtmpsc.c
cvs rdiff -u -r1.26 -r1.27 src/sys/dev/marvell/gtpci.c
cvs rdiff -u -r1.2 -r1.3 src/sys/dev/marvell/gttwsi.c
cvs rdiff -u -r1.37 -r1.38 src/sys/dev/marvell/if_gfe.c
cvs rdiff -u -r1.1 -r1.2 src/sys/dev/marvell/mvpex.c \
    src/sys/dev/marvell/mvsata_mv.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/marvell/gtidmac.c
diff -u src/sys/dev/marvell/gtidmac.c:1.5 src/sys/dev/marvell/gtidmac.c:1.6
--- src/sys/dev/marvell/gtidmac.c:1.5	Tue Jul 20 11:47:59 2010
+++ src/sys/dev/marvell/gtidmac.c	Sun Aug  1 06:57:06 2010
@@ -1,4 +1,4 @@
-/*	$NetBSD: gtidmac.c,v 1.5 2010/07/20 11:47:59 kiyohara Exp $	*/
+/*	$NetBSD: gtidmac.c,v 1.6 2010/08/01 06:57:06 kiyohara Exp $	*/
 /*
  * Copyright (c) 2008 KIYOHARA Takashi
  * All rights reserved.
@@ -26,7 +26,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: gtidmac.c,v 1.5 2010/07/20 11:47:59 kiyohara Exp $");
+__KERNEL_RCSID(0, "$NetBSD: gtidmac.c,v 1.6 2010/08/01 06:57:06 kiyohara Exp $");
 
 #include <sys/param.h>
 #include <sys/bus.h>
@@ -285,10 +285,6 @@
 
 	if (strcmp(mva->mva_name, match->cf_name) != 0)
 		return 0;
-
-	if (mva->mva_model == MARVELL_ORION_1_88F6082)
-		return 0;
-
 	if (mva->mva_offset == MVA_OFFSET_DEFAULT ||
 	    mva->mva_irq == MVA_IRQ_DEFAULT)
 		return 0;

Index: src/sys/dev/marvell/gtmpsc.c
diff -u src/sys/dev/marvell/gtmpsc.c:1.39 src/sys/dev/marvell/gtmpsc.c:1.40
--- src/sys/dev/marvell/gtmpsc.c:1.39	Sun Jul 11 08:43:36 2010
+++ src/sys/dev/marvell/gtmpsc.c	Sun Aug  1 06:57:06 2010
@@ -1,4 +1,4 @@
-/*	$NetBSD: gtmpsc.c,v 1.39 2010/07/11 08:43:36 kiyohara Exp $	*/
+/*	$NetBSD: gtmpsc.c,v 1.40 2010/08/01 06:57:06 kiyohara Exp $	*/
 /*
  * Copyright (c) 2009 KIYOHARA Takashi
  * All rights reserved.
@@ -29,7 +29,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: gtmpsc.c,v 1.39 2010/07/11 08:43:36 kiyohara Exp $");
+__KERNEL_RCSID(0, "$NetBSD: gtmpsc.c,v 1.40 2010/08/01 06:57:06 kiyohara Exp $");
 
 #include "opt_kgdb.h"
 
@@ -216,21 +216,7 @@
 
 	if (strcmp(mva->mva_name, match->cf_name) != 0)
 		return 0;
-
-	switch (mva->mva_model) {
-	case MARVELL_DISCOVERY:
-	case MARVELL_DISCOVERY_II:
-	case MARVELL_DISCOVERY_III:
-#if 0
-	case MARVELL_DISCOVERY_LT:
-#endif
-		break;
-
-	default:
-		return 0;
-	}
-	if (mva->mva_offset == MVA_OFFSET_DEFAULT ||
-	    mva->mva_irq == MVA_IRQ_DEFAULT)
+	if (mva->mva_offset == MVA_OFFSET_DEFAULT)
 		return 0;
 
 	mva->mva_size = GTMPSC_SIZE;

Index: src/sys/dev/marvell/gtpci.c
diff -u src/sys/dev/marvell/gtpci.c:1.26 src/sys/dev/marvell/gtpci.c:1.27
--- src/sys/dev/marvell/gtpci.c:1.26	Sun Jul 11 08:43:36 2010
+++ src/sys/dev/marvell/gtpci.c	Sun Aug  1 06:57:06 2010
@@ -1,4 +1,4 @@
-/*	$NetBSD: gtpci.c,v 1.26 2010/07/11 08:43:36 kiyohara Exp $	*/
+/*	$NetBSD: gtpci.c,v 1.27 2010/08/01 06:57:06 kiyohara Exp $	*/
 /*
  * Copyright (c) 2008, 2009 KIYOHARA Takashi
  * All rights reserved.
@@ -26,7 +26,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: gtpci.c,v 1.26 2010/07/11 08:43:36 kiyohara Exp $");
+__KERNEL_RCSID(0, "$NetBSD: gtpci.c,v 1.27 2010/08/01 06:57:06 kiyohara Exp $");
 
 #include "opt_pci.h"
 #include "pci.h"
@@ -90,6 +90,8 @@
 	if (strcmp(mva->mva_name, match->cf_name) != 0)
 		return 0;
 
+	if (mva->mva_unit == MVA_UNIT_DEFAULT)
+		return 0;
 	switch (mva->mva_model) {
 	case MARVELL_DISCOVERY:
 	case MARVELL_DISCOVERY_II:
@@ -99,23 +101,8 @@
 	case MARVELL_DISCOVERY_V:
 	case MARVELL_DISCOVERY_VI:
 #endif
-		if (mva->mva_unit == MVA_UNIT_DEFAULT ||
-		    mva->mva_offset != MVA_OFFSET_DEFAULT)
-			return 0;
-		break;
-
-	case MARVELL_ORION_1_88F5180N:
-	case MARVELL_ORION_1_88F5181:
-	case MARVELL_ORION_1_88F5182:
-	case MARVELL_ORION_2_88F5281:
-	case MARVELL_ORION_1_88W8660:
-		if (mva->mva_offset == MVA_OFFSET_DEFAULT)
+		if (mva->mva_offset != MVA_OFFSET_DEFAULT)
 			return 0;
-		mva->mva_unit = 0;	/* unit 0 only */
-		break;
-
-	default:
-		return 0;
 	}
 
 	mva->mva_size = GTPCI_SIZE;

Index: src/sys/dev/marvell/gttwsi.c
diff -u src/sys/dev/marvell/gttwsi.c:1.2 src/sys/dev/marvell/gttwsi.c:1.3
--- src/sys/dev/marvell/gttwsi.c:1.2	Sun Jul 11 08:43:36 2010
+++ src/sys/dev/marvell/gttwsi.c	Sun Aug  1 06:57:06 2010
@@ -1,4 +1,4 @@
-/*	$NetBSD: gttwsi.c,v 1.2 2010/07/11 08:43:36 kiyohara Exp $	*/
+/*	$NetBSD: gttwsi.c,v 1.3 2010/08/01 06:57:06 kiyohara Exp $	*/
 /*
  * Copyright (c) 2008 Eiji Kawauchi.
  * All rights reserved.
@@ -66,7 +66,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: gttwsi.c,v 1.2 2010/07/11 08:43:36 kiyohara Exp $");
+__KERNEL_RCSID(0, "$NetBSD: gttwsi.c,v 1.3 2010/08/01 06:57:06 kiyohara Exp $");
 #include "locators.h"
 
 #include <sys/param.h>
@@ -157,7 +157,6 @@
 
 	if (strcmp(mva->mva_name, match->cf_name) != 0)
 		return 0;
-
 	if (mva->mva_offset == MVA_OFFSET_DEFAULT ||
 	    mva->mva_irq == MVA_IRQ_DEFAULT)
 		return 0;

Index: src/sys/dev/marvell/if_gfe.c
diff -u src/sys/dev/marvell/if_gfe.c:1.37 src/sys/dev/marvell/if_gfe.c:1.38
--- src/sys/dev/marvell/if_gfe.c:1.37	Sun Jul 11 08:43:36 2010
+++ src/sys/dev/marvell/if_gfe.c	Sun Aug  1 06:57:06 2010
@@ -1,4 +1,4 @@
-/*	$NetBSD: if_gfe.c,v 1.37 2010/07/11 08:43:36 kiyohara Exp $	*/
+/*	$NetBSD: if_gfe.c,v 1.38 2010/08/01 06:57:06 kiyohara Exp $	*/
 
 /*
  * Copyright (c) 2002 Allegro Networks, Inc., Wasabi Systems, Inc.
@@ -42,7 +42,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_gfe.c,v 1.37 2010/07/11 08:43:36 kiyohara Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_gfe.c,v 1.38 2010/08/01 06:57:06 kiyohara Exp $");
 
 #include "opt_inet.h"
 #include "rnd.h"
@@ -218,14 +218,6 @@
 
 	if (strcmp(mva->mva_name, cf->cf_name) != 0)
 		return 0;
-
-	switch (mva->mva_model) {
-	case MARVELL_DISCOVERY:
-		break;
-
-	default:
-		return 0;
-	}
 	if (mva->mva_offset == MVA_OFFSET_DEFAULT)
 		return 0;
 

Index: src/sys/dev/marvell/mvpex.c
diff -u src/sys/dev/marvell/mvpex.c:1.1 src/sys/dev/marvell/mvpex.c:1.2
--- src/sys/dev/marvell/mvpex.c:1.1	Tue Jul 13 11:16:02 2010
+++ src/sys/dev/marvell/mvpex.c	Sun Aug  1 06:57:06 2010
@@ -1,4 +1,4 @@
-/*	$NetBSD: mvpex.c,v 1.1 2010/07/13 11:16:02 kiyohara Exp $	*/
+/*	$NetBSD: mvpex.c,v 1.2 2010/08/01 06:57:06 kiyohara Exp $	*/
 /*
  * Copyright (c) 2008 KIYOHARA Takashi
  * All rights reserved.
@@ -26,7 +26,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: mvpex.c,v 1.1 2010/07/13 11:16:02 kiyohara Exp $");
+__KERNEL_RCSID(0, "$NetBSD: mvpex.c,v 1.2 2010/08/01 06:57:06 kiyohara Exp $");
 
 #include "opt_pci.h"
 #include "pci.h"
@@ -87,25 +87,8 @@
 {
 	struct marvell_attach_args *mva = aux;
 
-	switch (mva->mva_model) {
-#if 0
-	case MARVELL_DISCOVERY_V:
-	case MARVELL_DISCOVERY_VI:
-#endif
-	case MARVELL_ORION_1_88F1181:
-	case MARVELL_ORION_1_88F5082:
-	case MARVELL_ORION_1_88F5180N:
-	case MARVELL_ORION_1_88F5181:
-	case MARVELL_ORION_1_88F5182:
-	case MARVELL_ORION_1_88F6082:
-	case MARVELL_ORION_1_88W8660:
-	case MARVELL_ORION_2_88F1281:
-	case MARVELL_ORION_2_88F5281:
-		break;
-
-	default:
+	if (strcmp(mva->mva_name, match->cf_name) != 0)
 		return 0;
-	}
 	if (mva->mva_offset == MVA_OFFSET_DEFAULT ||
 	    mva->mva_irq == MVA_IRQ_DEFAULT)
 		return 0;
Index: src/sys/dev/marvell/mvsata_mv.c
diff -u src/sys/dev/marvell/mvsata_mv.c:1.1 src/sys/dev/marvell/mvsata_mv.c:1.2
--- src/sys/dev/marvell/mvsata_mv.c:1.1	Tue Jul 13 13:05:05 2010
+++ src/sys/dev/marvell/mvsata_mv.c	Sun Aug  1 06:57:06 2010
@@ -1,4 +1,4 @@
-/*	$NetBSD: mvsata_mv.c,v 1.1 2010/07/13 13:05:05 kiyohara Exp $	*/
+/*	$NetBSD: mvsata_mv.c,v 1.2 2010/08/01 06:57:06 kiyohara Exp $	*/
 /*
  * Copyright (c) 2008 KIYOHARA Takashi
  * All rights reserved.
@@ -26,7 +26,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: mvsata_mv.c,v 1.1 2010/07/13 13:05:05 kiyohara Exp $");
+__KERNEL_RCSID(0, "$NetBSD: mvsata_mv.c,v 1.2 2010/08/01 06:57:06 kiyohara Exp $");
 
 #include <sys/param.h>
 #include <sys/bus.h>
@@ -90,19 +90,16 @@
 	{ PCI_VENDOR_MARVELL, PCI_PRODUCT_MARVELL_88F5182, 1, 2, gen2e, 0 },
 	{ PCI_VENDOR_MARVELL, PCI_PRODUCT_MARVELL_88F6082, 1, 1, gen2e, 0 },
 
-#if 0
 	/* Kirkwood */
-							/* 88F6190 has a port */
 	{ PCI_VENDOR_MARVELL, PCI_PRODUCT_MARVELL_88F6192, 1, 2, gen2e, 0 },
 	{ PCI_VENDOR_MARVELL, PCI_PRODUCT_MARVELL_88F6281, 1, 2, gen2e, 0 },
 
+#if 0
 	/* Discovery Inovation */
 	{ PCI_VENDOR_MARVELL, PCI_PRODUCT_MARVELL_MV78200, 1, 2, gen2e, 0 },
 							/* MV76100 has a port */
 	{ PCI_VENDOR_MARVELL, PCI_PRODUCT_MARVELL_MV78100, 1, 2, gen2e, 0 },
 #endif
-
-	{ -1, -1,					0, 0, gen_unknown, 0 }
 };
 
 
@@ -113,6 +110,8 @@
 	struct marvell_attach_args *mva = aux;
 	int i;
 
+	if (strcmp(mva->mva_name, match->cf_name) != 0)
+		return 0;
 	if (mva->mva_offset == MVA_OFFSET_DEFAULT ||
 	    mva->mva_irq == MVA_IRQ_DEFAULT)
 		    return 0;

Reply via email to