[U-Boot] [PATCH 032/126] pci: Show a message if PCI autoconfig fails
Bin Meng
bmeng.cn at gmail.com
Sat Oct 5 13:12:25 UTC 2019
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass <sjg at chromium.org> wrote:
>
> At present this fails silently which can be confusing since some devices
> on the PCI bus may not work correctly. Show a message in this case
nits: missing . after case
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> drivers/pci/pci_auto.c | 12 +++++++++---
> 1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/pci/pci_auto.c b/drivers/pci/pci_auto.c
> index 1a3bf708347..7755ffb6fa2 100644
> --- a/drivers/pci/pci_auto.c
> +++ b/drivers/pci/pci_auto.c
> @@ -39,6 +39,8 @@ void dm_pciauto_setup_device(struct udevice *dev, int bars_num,
>
> for (bar = PCI_BASE_ADDRESS_0;
> bar < PCI_BASE_ADDRESS_0 + (bars_num * 4); bar += 4) {
> + int ret = 0;
> +
> /* Tickle the BAR and get the response */
> if (!enum_only)
> dm_pci_write_config32(dev, bar, 0xffffffff);
> @@ -97,9 +99,13 @@ void dm_pciauto_setup_device(struct udevice *dev, int bars_num,
> (unsigned long long)bar_size);
> }
>
> - if (!enum_only && pciauto_region_allocate(bar_res, bar_size,
> - &bar_value,
> - found_mem64) == 0) {
> + if (!enum_only) {
> + ret = pciauto_region_allocate(bar_res, bar_size,
> + &bar_value, found_mem64);
> + if (ret)
> + printf("PCI: Failed autoconfig bar %x", bar);
nits: should have a '\n'
> + }
> + if (!enum_only && !ret) {
> /* Write it out and update our limit */
> dm_pci_write_config32(dev, bar, (u32)bar_value);
>
> --
Other than above,
Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
More information about the U-Boot
mailing list