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;