On Tuesday, April 29, 2014 12:25 PM, Fabio Estevam wrote:
>
> From: Fabio Estevam <fabio.estevam@xxxxxxxxxxxxx>
>
> Fix the following build warning that happens when building multi_v7_defconfig
> with CONFIG_ARM_LPAE=y:
>
> drivers/pci/host/pci-mvebu.c:334:5: warning: format '%x' expects argument of type 'unsigned int', but
> argument 3 has type 'phys_addr_t' [-Wformat=]
>
> Fix the warning by using '%pa' to printing 'phys_addr_t' type. While at it,
> also use the more standard notation [mem 0x - 0x] for memory region.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@xxxxxxxxxxxxx>
Reviewed-by: Jingoo Han <jg1.han@xxxxxxxxxxx>
Best regards,
Jingoo Han
> ---
> Changes since v1:
> - use the [mem 0x - 0x] notation
>
> drivers/pci/host/pci-mvebu.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/pci/host/pci-mvebu.c b/drivers/pci/host/pci-mvebu.c
> index e384e25..df57a6a 100644
> --- a/drivers/pci/host/pci-mvebu.c
> +++ b/drivers/pci/host/pci-mvebu.c
> @@ -330,8 +330,8 @@ static void mvebu_pcie_add_windows(struct mvebu_pcie_port *port,
> sz, remap);
> if (ret) {
> dev_err(&port->pcie->pdev->dev,
> - "Could not create MBus window at 0x%x, size 0x%x: %d\n",
> - base, sz, ret);
> + "Could not create MBus window at [mem %pa - %pa] :%d\n",
> + &base, &base + sz, ret);
> mvebu_pcie_del_windows(port, base - size_mapped,
> size_mapped);
> return;
> --
> 1.8.3.2
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
