From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Roese <sr@denx.de>
Date: Tue, 22 Dec 2015 09:06:15 +0100
Subject: [U-Boot] [PATCH v2 1/8] drivers/pci: Fix for debug builds
 without CONFIG_PCI_ENUM_ONLY
In-Reply-To: <20151221232449.653FC61223@mail.nwl.cc>
References: <1450740358-5014-1-git-send-email-phil@nwl.cc>
 <20151221232449.653FC61223@mail.nwl.cc>
Message-ID: <56790477.6040507@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

(adding Simon and Bin to Cc)

On 22.12.2015 00:25, Phil Sutter wrote:
> The debug printing references bar_res, which exists only if
> CONFIG_PCI_ENUM_ONLY is not defined. Therefore move it into the ifdef'd
> area.
>
> Signed-off-by: Phil Sutter <phil@nwl.cc>
> ---
>   drivers/pci/pci_auto_old.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/pci/pci_auto_old.c b/drivers/pci/pci_auto_old.c
> index 932eab8..8f17779 100644
> --- a/drivers/pci/pci_auto_old.c
> +++ b/drivers/pci/pci_auto_old.c
> @@ -98,11 +98,11 @@ void pciauto_setup_device(struct pci_controller *hose,
>   				bar_res = prefetch;
>   			else
>   				bar_res = mem;
> -#endif
>
>   			debug("PCI Autoconfig: BAR %d, %s, size=0x%llx, ",
>   			      bar_nr, bar_res == prefetch ? "Prf" : "Mem",
>   			      (unsigned long long)bar_size);
> +#endif
>   		}
>
>   #ifndef CONFIG_PCI_ENUM_ONLY
>

Reviewed-by: Stefan Roese <sr@denx.de>

Thanks,
Stefan