[U-Boot] [PATCH 03/14] pci: Use a separate variable for the bus number

[Simon Glass]

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 <s...@chromium.org>
---

 common/cmd_pci.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/common/cmd_pci.c b/common/cmd_pci.c
index 5762769..6e28b70 100644
--- a/common/cmd_pci.c
+++ b/common/cmd_pci.c
@@ -407,6 +407,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;
@@ -437,14 +438,13 @@ 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);
                }
                cmd = 's';
                break;
@@ -476,7 +476,7 @@ static int do_pci(cmd_tbl_t *cmdtp, int flag, int argc, 
char * const argv[])
                ret = pci_cfg_modify(bdf, addr, size, value, 1);
                break;
        case 's':
-               pciinfo(bdf, value);
+               pciinfo(busnum, value);
                break;
        case 'w':               /* write */
                if (argc < 5)
-- 
2.6.0.rc2.230.g3dd15c0

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot