From: Simon Glass <sjg@chromium.org>
Date: Fri, 27 Nov 2015 02:51:19 +0000 (-0700)
Subject: pci: Use a separate variable for the bus number
X-Git-Tag: v2016.01-rc2~84
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=ca7de76d8c5a12e178593770e0d2272cca2596bb;p=u-boot

pci: Use a separate variable for the bus number

At present in do_pci(), bdf can either mean a bus number or a PCI bus number.
Use separate variables instead to reduce confusion.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
---

diff --git a/common/cmd_pci.c b/common/cmd_pci.c
index f3148a3da0..bed880cbcc 100644
--- a/common/cmd_pci.c
+++ b/common/cmd_pci.c
@@ -408,6 +408,7 @@ pci_cfg_modify (pci_dev_t bdf, ulong addr, ulong size, ulong value, int incrflag
 static int do_pci(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 {
 	ulong addr = 0, value = 0, size = 0;
+	int busnum = 0;
 	pci_dev_t bdf = 0;
 	char cmd = 's';
 	int ret = 0;
@@ -438,16 +439,15 @@ static int do_pci(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 #endif
 	default:		/* scan bus */
 		value = 1; /* short listing */
-		bdf = 0;   /* bus number  */
 		if (argc > 1) {
 			if (argv[argc-1][0] == 'l') {
 				value = 0;
 				argc--;
 			}
 			if (argc > 1)
-				bdf = simple_strtoul(argv[1], NULL, 16);
+				busnum = simple_strtoul(argv[1], NULL, 16);
 		}
-		pciinfo(bdf, value);
+		pciinfo(busnum, value);
 		return 0;
 	}