pci: Use a separate variable for the bus number
authorSimon Glass <sjg@chromium.org>
Fri, 27 Nov 2015 02:51:19 +0000 (19:51 -0700)
committerSimon Glass <sjg@chromium.org>
Tue, 1 Dec 2015 13:26:37 +0000 (06:26 -0700)
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>
common/cmd_pci.c

index f3148a3da04479cd216c70f203066dbfe3456b7b..bed880cbcc6eae4d60606b96e53ad893af397c5f 100644 (file)
@@ -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;
        }