Module Name:    src
Committed By:   cegger
Date:           Sat Jul  4 21:01:10 UTC 2009

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

Log Message:
remove useless parenthesis


To generate a diff of this commit:
cvs rdiff -u -r1.122 -r1.123 src/sys/dev/pci/pci.c
cvs rdiff -u -r1.16 -r1.17 src/sys/dev/pci/pci_usrreq.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/pci.c
diff -u src/sys/dev/pci/pci.c:1.122 src/sys/dev/pci/pci.c:1.123
--- src/sys/dev/pci/pci.c:1.122	Thu Apr  2 00:09:33 2009
+++ src/sys/dev/pci/pci.c	Sat Jul  4 21:01:10 2009
@@ -1,4 +1,4 @@
-/*	$NetBSD: pci.c,v 1.122 2009/04/02 00:09:33 dyoung Exp $	*/
+/*	$NetBSD: pci.c,v 1.123 2009/07/04 21:01:10 cegger Exp $	*/
 
 /*
  * Copyright (c) 1995, 1996, 1997, 1998
@@ -36,7 +36,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: pci.c,v 1.122 2009/04/02 00:09:33 dyoung Exp $");
+__KERNEL_RCSID(0, "$NetBSD: pci.c,v 1.123 2009/07/04 21:01:10 cegger Exp $");
 
 #include "opt_pci.h"
 
@@ -118,17 +118,17 @@
 	/* Check the locators */
 	if (cf->cf_loc[PCIBUSCF_BUS] != PCIBUSCF_BUS_DEFAULT &&
 	    cf->cf_loc[PCIBUSCF_BUS] != pba->pba_bus)
-		return (0);
+		return 0;
 
 	/* sanity */
 	if (pba->pba_bus < 0 || pba->pba_bus > 255)
-		return (0);
+		return 0;
 
 	/*
 	 * XXX check other (hardware?) indicators
 	 */
 
-	return (1);
+	return 1;
 }
 
 void
@@ -264,7 +264,7 @@
 		}
 		printf(")");
 	}
-	return (UNCONF);
+	return UNCONF;
 }
 
 int
@@ -282,11 +282,11 @@
 
 	/* a driver already attached? */
 	if (sc->PCI_SC_DEVICESC(device, function).c_dev != NULL && !match)
-		return (0);
+		return 0;
 
 	bhlcr = pci_conf_read(pc, tag, PCI_BHLC_REG);
 	if (PCI_HDRTYPE_TYPE(bhlcr) > 2)
-		return (0);
+		return 0;
 
 	id = pci_conf_read(pc, tag, PCI_ID_REG);
 	csr = pci_conf_read(pc, tag, PCI_COMMAND_STATUS_REG);
@@ -294,10 +294,10 @@
 
 	/* Invalid vendor ID value? */
 	if (PCI_VENDOR(id) == PCI_VENDOR_INVALID)
-		return (0);
+		return 0;
 	/* XXX Not invalid, but we've done this ~forever. */
 	if (PCI_VENDOR(id) == 0)
-		return (0);
+		return 0;
 
 	pa.pa_iot = sc->sc_iot;
 	pa.pa_memt = sc->sc_memt;
@@ -376,7 +376,7 @@
 		ret = (subdev != NULL);
 	}
 
-	return (ret);
+	return ret;
 }
 
 void
@@ -414,7 +414,7 @@
 
 	reg = pci_conf_read(pc, tag, PCI_COMMAND_STATUS_REG);
 	if (!(reg & PCI_STATUS_CAPLIST_SUPPORT))
-		return (0);
+		return 0;
 
 	/* Determine the Capability List Pointer register to start with. */
 	reg = pci_conf_read(pc, tag, PCI_BHLC_REG);
@@ -427,7 +427,7 @@
 		ofs = PCI_CARDBUS_CAPLISTPTR_REG;
 		break;
 	default:
-		return (0);
+		return 0;
 	}
 
 	ofs = PCI_CAPLIST_PTR(pci_conf_read(pc, tag, ofs));
@@ -447,12 +447,12 @@
 				*offset = ofs;
 			if (value)
 				*value = reg;
-			return (1);
+			return 1;
 		}
 		ofs = PCI_CAPLIST_NEXT(reg);
 	}
 
-	return (0);
+	return 0;
 }
 
 int
@@ -472,9 +472,9 @@
 		if (pcidev != NULL &&
 		    pci_enumerate_bus(device_private(pcidev), wildcard,
 		    		      match, pa) != 0)
-			return (1);
+			return 1;
 	}
-	return (0);
+	return 0;
 }
 
 #ifndef PCI_MACHDEP_ENUMERATE_BUS
@@ -544,10 +544,10 @@
 			tag = pci_make_tag(pc, sc->sc_bus, device, function);
 			ret = pci_probe_device(sc, tag, match, pap);
 			if (match != NULL && ret != 0)
-				return (ret);
+				return ret;
 		}
 	}
-	return (0);
+	return 0;
 }
 #endif /* PCI_MACHDEP_ENUMERATE_BUS */
 
@@ -567,7 +567,7 @@
 	KASSERT((offset + count) < 0x7fff);
 
 	if (pci_get_capability(pc, tag, PCI_CAP_VPD, &ofs, &reg) == 0)
-		return (1);
+		return 1;
 
 	for (i = 0; i < count; offset += sizeof(*data), i++) {
 		reg &= 0x0000ffff;
@@ -582,14 +582,14 @@
 		j = 0;
 		do {
 			if (j++ == 20)
-				return (1);
+				return 1;
 			delay(4);
 			reg = pci_conf_read(pc, tag, ofs);
 		} while ((reg & PCI_VPD_OPFLAG) == 0);
 		data[i] = pci_conf_read(pc, tag, PCI_VPD_DATAREG(ofs));
 	}
 
-	return (0);
+	return 0;
 }
 
 int
@@ -603,7 +603,7 @@
 	KASSERT((offset + count) < 0x7fff);
 
 	if (pci_get_capability(pc, tag, PCI_CAP_VPD, &ofs, &reg) == 0)
-		return (1);
+		return 1;
 
 	for (i = 0; i < count; offset += sizeof(*data), i++) {
 		pci_conf_write(pc, tag, PCI_VPD_DATAREG(ofs), data[i]);
@@ -620,13 +620,13 @@
 		j = 0;
 		do {
 			if (j++ == 20)
-				return (1);
+				return 1;
 			delay(1);
 			reg = pci_conf_read(pc, tag, ofs);
 		} while (reg & PCI_VPD_OPFLAG);
 	}
 
-	return (0);
+	return 0;
 }
 
 int

Index: src/sys/dev/pci/pci_usrreq.c
diff -u src/sys/dev/pci/pci_usrreq.c:1.16 src/sys/dev/pci/pci_usrreq.c:1.17
--- src/sys/dev/pci/pci_usrreq.c:1.16	Wed Jun 11 19:27:03 2008
+++ src/sys/dev/pci/pci_usrreq.c	Sat Jul  4 21:01:10 2009
@@ -1,4 +1,4 @@
-/*	$NetBSD: pci_usrreq.c,v 1.16 2008/06/11 19:27:03 cegger Exp $	*/
+/*	$NetBSD: pci_usrreq.c,v 1.17 2009/07/04 21:01:10 cegger Exp $	*/
 
 /*
  * Copyright 2001 Wasabi Systems, Inc.
@@ -40,7 +40,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: pci_usrreq.c,v 1.16 2008/06/11 19:27:03 cegger Exp $");
+__KERNEL_RCSID(0, "$NetBSD: pci_usrreq.c,v 1.17 2009/07/04 21:01:10 cegger Exp $");
 
 #include <sys/param.h>
 #include <sys/conf.h>
@@ -62,9 +62,9 @@
 
 	dv = device_lookup(&pci_cd, minor(dev));
 	if (dv == NULL)
-		return (ENXIO);
+		return ENXIO;
 
-	return (0);
+	return 0;
 }
 
 static int
@@ -81,7 +81,7 @@
 	case PCI_IOC_BDF_CFGWRITE:
 		if (bdfr->bus > 255 || bdfr->device >= sc->sc_maxndevs ||
 		    bdfr->function > 7)
-			return (EINVAL);
+			return EINVAL;
 		tag = pci_make_tag(sc->sc_pc, bdfr->bus, bdfr->device,
 		    bdfr->function);
 		if (cmd == PCI_IOC_BDF_CFGREAD)
@@ -89,7 +89,7 @@
 			    bdfr->cfgreg.reg);
 		else {
 			if ((flag & FWRITE) == 0)
-				return (EBADF);
+				return EBADF;
 			pci_conf_write(sc->sc_pc, tag, bdfr->cfgreg.reg,
 			    bdfr->cfgreg.val);
 		}
@@ -101,10 +101,10 @@
 		break;
 
 	default:
-		return (ENOTTY);
+		return ENOTTY;
 	}
 
-	return (0);
+	return 0;
 }
 
 static paddr_t
@@ -120,10 +120,10 @@
 	 *
 	 * XXX Need a way to deal with linear/prefetchable/etc.
 	 */
-	return (bus_space_mmap(sc->sc_memt, offset, 0, prot, 0));
+	return bus_space_mmap(sc->sc_memt, offset, 0, prot, 0);
 #else
 	/* XXX Consider this further. */
-	return (-1);
+	return -1;
 #endif
 }
 
@@ -150,14 +150,14 @@
 			r->val = pci_conf_read(pc, tag, r->reg);
 		else {
 			if ((flag & FWRITE) == 0)
-				return (EBADF);
+				return EBADF;
 			pci_conf_write(pc, tag, r->reg, r->val);
 		}
 		break;
 
 	default:
-		return (EPASSTHROUGH);
+		return EPASSTHROUGH;
 	}
 
-	return (0);
+	return 0;
 }

Reply via email to