Module Name:    src
Committed By:   msaitoh
Date:           Mon May 12 11:51:35 UTC 2014

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

Log Message:
Use macro.


To generate a diff of this commit:
cvs rdiff -u -r1.108 -r1.109 src/sys/dev/pci/pci_subr.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_subr.c
diff -u src/sys/dev/pci/pci_subr.c:1.108 src/sys/dev/pci/pci_subr.c:1.109
--- src/sys/dev/pci/pci_subr.c:1.108	Mon May 12 11:27:31 2014
+++ src/sys/dev/pci/pci_subr.c	Mon May 12 11:51:35 2014
@@ -1,4 +1,4 @@
-/*	$NetBSD: pci_subr.c,v 1.108 2014/05/12 11:27:31 msaitoh Exp $	*/
+/*	$NetBSD: pci_subr.c,v 1.109 2014/05/12 11:51:35 msaitoh Exp $	*/
 
 /*
  * Copyright (c) 1997 Zubin D. Dittia.  All rights reserved.
@@ -40,7 +40,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: pci_subr.c,v 1.108 2014/05/12 11:27:31 msaitoh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: pci_subr.c,v 1.109 2014/05/12 11:51:35 msaitoh Exp $");
 
 #ifdef _KERNEL_OPT
 #include "opt_pci.h"
@@ -1671,41 +1671,48 @@ pci_conf_print_type1(
 #endif
 	}
 
+	rval = regs[o2i(PCI_BRIDGE_BUS_REG)];
 	printf("    Primary bus number: 0x%02x\n",
-	    (regs[o2i(0x18)] >> 0) & 0xff);
+	    (rval >> 0) & 0xff);
 	printf("    Secondary bus number: 0x%02x\n",
-	    (regs[o2i(0x18)] >> 8) & 0xff);
+	    (rval >> 8) & 0xff);
 	printf("    Subordinate bus number: 0x%02x\n",
-	    (regs[o2i(0x18)] >> 16) & 0xff);
+	    (rval >> 16) & 0xff);
 	printf("    Secondary bus latency timer: 0x%02x\n",
-	    (regs[o2i(0x18)] >> 24) & 0xff);
+	    (rval >> 24) & 0xff);
 
-	pci_conf_print_ssr(__SHIFTOUT(regs[o2i(0x1c)], __BITS(31, 16)));
+	rval = regs[o2i(PCI_BRIDGE_STATIO_REG)];
+	pci_conf_print_ssr(__SHIFTOUT(rval, __BITS(31, 16)));
 
 	/* XXX Print more prettily */
 	printf("    I/O region:\n");
-	printf("      base register:  0x%02x\n", (regs[o2i(0x1c)] >> 0) & 0xff);
-	printf("      limit register: 0x%02x\n", (regs[o2i(0x1c)] >> 8) & 0xff);
+	printf("      base register:  0x%02x\n", (rval >> 0) & 0xff);
+	printf("      limit register: 0x%02x\n", (rval >> 8) & 0xff);
+	rval = regs[o2i(PCI_BRIDGE_IOHIGH_REG)];
 	printf("      base upper 16 bits register:  0x%04x\n",
-	    (regs[o2i(0x30)] >> 0) & 0xffff);
+	    (rval >> 0) & 0xffff);
 	printf("      limit upper 16 bits register: 0x%04x\n",
-	    (regs[o2i(0x30)] >> 16) & 0xffff);
+	    (rval >> 16) & 0xffff);
 
 	/* XXX Print more prettily */
+	rval = regs[o2i(PCI_BRIDGE_MEMORY_REG)];
 	printf("    Memory region:\n");
 	printf("      base register:  0x%04x\n",
-	    (regs[o2i(0x20)] >> 0) & 0xffff);
+	    (rval >> 0) & 0xffff);
 	printf("      limit register: 0x%04x\n",
-	    (regs[o2i(0x20)] >> 16) & 0xffff);
+	    (rval >> 16) & 0xffff);
 
 	/* XXX Print more prettily */
+	rval = regs[o2i(PCI_BRIDGE_PREFETCHMEM_REG)];
 	printf("    Prefetchable memory region:\n");
 	printf("      base register:  0x%04x\n",
-	    (regs[o2i(0x24)] >> 0) & 0xffff);
+	    (rval >> 0) & 0xffff);
 	printf("      limit register: 0x%04x\n",
-	    (regs[o2i(0x24)] >> 16) & 0xffff);
-	printf("      base upper 32 bits register:  0x%08x\n", regs[o2i(0x28)]);
-	printf("      limit upper 32 bits register: 0x%08x\n", regs[o2i(0x2c)]);
+	    (rval >> 16) & 0xffff);
+	printf("      base upper 32 bits register:  0x%08x\n",
+	    regs[o2i(PCI_BRIDGE_PREFETCHBASE32_REG)]);
+	printf("      limit upper 32 bits register: 0x%08x\n",
+	    regs[o2i(PCI_BRIDGE_PREFETCHLIMIT32_REG)]);
 
 	if (regs[o2i(PCI_COMMAND_STATUS_REG)] & PCI_STATUS_CAPLIST_SUPPORT)
 		printf("    Capability list pointer: 0x%02x\n",
@@ -1716,11 +1723,12 @@ pci_conf_print_type1(
 	/* XXX */
 	printf("    Expansion ROM Base Address: 0x%08x\n", regs[o2i(0x38)]);
 
+	rval = regs[o2i(PCI_INTERRUPT_REG)];
 	printf("    Interrupt line: 0x%02x\n",
-	    (regs[o2i(0x3c)] >> 0) & 0xff);
+	    (rval >> 0) & 0xff);
 	printf("    Interrupt pin: 0x%02x ",
-	    (regs[o2i(0x3c)] >> 8) & 0xff);
-	switch ((regs[o2i(0x3c)] >> 8) & 0xff) {
+	    (rval >> 8) & 0xff);
+	switch ((rval >> 8) & 0xff) {
 	case PCI_INTERRUPT_PIN_NONE:
 		printf("(none)");
 		break;
@@ -1741,7 +1749,8 @@ pci_conf_print_type1(
 		break;
 	}
 	printf("\n");
-	rval = (regs[o2i(0x3c)] >> 16) & 0xffff;
+	rval = (regs[o2i(PCI_BRIDGE_CONTROL_REG)] >> PCI_BRIDGE_CONTROL_SHIFT)
+	    & PCI_BRIDGE_CONTROL_MASK;
 	printf("    Bridge control register: 0x%04x\n", rval); /* XXX bits */
 	onoff("Parity error response", 0x0001);
 	onoff("Secondary SERR forwarding", 0x0002);
@@ -1781,22 +1790,25 @@ pci_conf_print_type2(
 	pci_conf_print_bar(regs, 0x10, "CardBus socket/ExCA registers");
 #endif
 
+	/* Capability list pointer and secondary status register */
+	rval = regs[o2i(PCI_CARDBUS_CAPLISTPTR_REG)];
 	if (regs[o2i(PCI_COMMAND_STATUS_REG)] & PCI_STATUS_CAPLIST_SUPPORT)
 		printf("    Capability list pointer: 0x%02x\n",
-		    PCI_CAPLIST_PTR(regs[o2i(PCI_CARDBUS_CAPLISTPTR_REG)]));
+		    PCI_CAPLIST_PTR(rval));
 	else
 		printf("    Reserved @ 0x14: 0x%04" PRIxMAX "\n",
-		       __SHIFTOUT(regs[o2i(0x14)], __BITS(15, 0)));
-	pci_conf_print_ssr(__SHIFTOUT(regs[o2i(0x14)], __BITS(31, 16)));
+		       __SHIFTOUT(rval, __BITS(15, 0)));
+	pci_conf_print_ssr(__SHIFTOUT(rval, __BITS(31, 16)));
 
+	rval = regs[o2i(PCI_BRIDGE_BUS_REG)];
 	printf("    PCI bus number: 0x%02x\n",
-	    (regs[o2i(0x18)] >> 0) & 0xff);
+	    (rval >> 0) & 0xff);
 	printf("    CardBus bus number: 0x%02x\n",
-	    (regs[o2i(0x18)] >> 8) & 0xff);
+	    (rval >> 8) & 0xff);
 	printf("    Subordinate bus number: 0x%02x\n",
-	    (regs[o2i(0x18)] >> 16) & 0xff);
+	    (rval >> 16) & 0xff);
 	printf("    CardBus latency timer: 0x%02x\n",
-	    (regs[o2i(0x18)] >> 24) & 0xff);
+	    (rval >> 24) & 0xff);
 
 	/* XXX Print more prettily */
 	printf("    CardBus memory region 0:\n");
@@ -1812,11 +1824,12 @@ pci_conf_print_type2(
 	printf("      base register:  0x%08x\n", regs[o2i(0x34)]);
 	printf("      limit register: 0x%08x\n", regs[o2i(0x38)]);
 
+	rval = regs[o2i(PCI_INTERRUPT_REG)];
 	printf("    Interrupt line: 0x%02x\n",
-	    (regs[o2i(0x3c)] >> 0) & 0xff);
+	    (rval >> 0) & 0xff);
 	printf("    Interrupt pin: 0x%02x ",
-	    (regs[o2i(0x3c)] >> 8) & 0xff);
-	switch ((regs[o2i(0x3c)] >> 8) & 0xff) {
+	    (rval >> 8) & 0xff);
+	switch ((rval >> 8) & 0xff) {
 	case PCI_INTERRUPT_PIN_NONE:
 		printf("(none)");
 		break;

Reply via email to