On 10/17/2015 11:49 AM, Simon Glass wrote:
This is not supported with driver model, so print a message instead of generating a build error. Rescanning PCI is not yet implemented.
diff --git a/common/cmd_pci.c b/common/cmd_pci.c
@@ -457,7 +457,11 @@ static int do_pci(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) return pci_cfg_display(bdf, addr, size, value); #ifdef CONFIG_CMD_PCI_ENUM case 'e': +# ifdef CONFIG_DM_PCI
printf("This command is not yet supported with driver model\n");+# else pci_init(); +# endif
That feature is enabled on most/all Tegra boards with PCI support. Hopefully nobody will miss it; I guess I haven't used it so I don't object to this change.
However, wouldn't it be better to remove CONFIG_CMD_PCI_ENUM from the config header rather than leaving the command enabled yet non-functional? Or are you planning on implementing this path in the near future?