Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/sys/dev/pci Cosmetics: long lines, needless variables, PCI p...



details:   https://anonhg.NetBSD.org/src/rev/4bb07c61fc71
branches:  trunk
changeset: 796308:4bb07c61fc71
user:      riastradh <riastradh%NetBSD.org@localhost>
date:      Wed May 28 15:34:32 2014 +0000

description:
Cosmetics: long lines, needless variables, PCI parameter verbiage.

diffstat:

 sys/dev/pci/agp_i810.c |  38 +++++++++++++++++++-------------------
 1 files changed, 19 insertions(+), 19 deletions(-)

diffs (98 lines):

diff -r 4a31ec88a026 -r 4bb07c61fc71 sys/dev/pci/agp_i810.c
--- a/sys/dev/pci/agp_i810.c    Wed May 28 14:55:16 2014 +0000
+++ b/sys/dev/pci/agp_i810.c    Wed May 28 15:34:32 2014 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: agp_i810.c,v 1.82 2014/05/28 03:17:42 riastradh Exp $  */
+/*     $NetBSD: agp_i810.c,v 1.83 2014/05/28 15:34:32 riastradh Exp $  */
 
 /*-
  * Copyright (c) 2000 Doug Rabson
@@ -30,7 +30,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: agp_i810.c,v 1.82 2014/05/28 03:17:42 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: agp_i810.c,v 1.83 2014/05/28 15:34:32 riastradh Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -302,7 +302,8 @@
                        return agp_intel_attach(parent, self, aux);
                }
 #endif
-               aprint_error(": can't find internal VGA device config space\n");
+               aprint_error(": can't find internal VGA"
+                   " config space\n");
                error = ENOENT;
                goto fail1;
        }
@@ -441,7 +442,8 @@
        error = bus_space_map(isc->bst, mmadr, isc->size, mmadr_flags,
            &isc->bsh);
        if (error) {
-               aprint_error_dev(self, "can't map MMIO registers: %d\n", error);
+               aprint_error_dev(self, "can't map MMIO registers: %d\n",
+                   error);
                error = ENXIO;
                goto fail1;
        }
@@ -481,8 +483,8 @@
        case CHIP_G4X:
                error = agp_i810_setup_chipset_flush_page(sc);
                if (error) {
-                       aprint_error_dev(self, "can't set up chipset flush page: %d\n",
-                           error);
+                       aprint_error_dev(self,
+                           "can't set up chipset flush page: %d\n", error);
                        goto fail3;
                }
                break;
@@ -550,7 +552,9 @@
 agp_i810_setup_chipset_flush_page(struct agp_softc *sc)
 {
        struct agp_i810_softc *const isc = sc->as_chipc;
-       pcireg_t reg, lo, hi;
+       const pci_chipset_tag_t pc = isc->as_pc;
+       const pcitag_t tag = isc->as_tag;
+       pcireg_t lo, hi;
        bus_addr_t addr, minaddr, maxaddr;
        int error;
 
@@ -562,13 +566,12 @@
 
        /* Read the PCI config register: 4-byte on gen3, 8-byte on gen>=4.  */
        if (isc->chiptype == CHIP_I915) {
-               reg = pci_conf_read(sc->as_pc, sc->as_tag, AGP_I915_IFPADDR);
-               addr = reg;
+               addr = pci_conf_read(pc, tag, AGP_I915_IFPADDR);
                minaddr = PAGE_SIZE;    /* XXX PCIBIOS_MIN_MEM?  */
                maxaddr = UINT32_MAX;
        } else {
-               hi = pci_conf_read(sc->as_pc, sc->as_tag, AGP_I965_IFPADDR+4);
-               lo = pci_conf_read(sc->as_pc, sc->as_tag, AGP_I965_IFPADDR);
+               hi = pci_conf_read(pc, tag, AGP_I965_IFPADDR+4);
+               lo = pci_conf_read(pc, tag, AGP_I965_IFPADDR);
                /*
                 * Convert to uint64_t, rather than bus_addr_t which
                 * may be 32-bit, to avoid undefined behaviour with a
@@ -608,15 +611,12 @@
                /* Write it into the PCI config register.  */
                addr = isc->flush_addr | 1;
                if (isc->chiptype == CHIP_I915) {
-                       pci_conf_write(sc->as_pc, sc->as_tag, AGP_I915_IFPADDR,
-                           addr);
+                       pci_conf_write(pc, tag, AGP_I915_IFPADDR, addr);
                } else {
-                       pci_conf_write(sc->as_pc, sc->as_tag,
-                           AGP_I965_IFPADDR + 4,
-                           __SHIFTOUT(addr, __BITS(63, 32)));
-                       pci_conf_write(sc->as_pc, sc->as_tag,
-                           AGP_I965_IFPADDR,
-                           __SHIFTOUT(addr, __BITS(31, 0)));
+                       hi = __SHIFTOUT(addr, __BITS(63, 32));
+                       lo = __SHIFTOUT(addr, __BITS(31, 0))
+                       pci_conf_write(pc, tag, AGP_I965_IFPADDR+4, hi);
+                       pci_conf_write(pc, tag, AGP_I965_IFPADDR, lo);
                }
        }
 



Home | Main Index | Thread Index | Old Index