[U-Boot] [RFC PATCH 04/29] drivers: pci-uclass: fix incorrect argument in map_sysmem

Simon Glass sjg at chromium.org
Wed Nov 20 03:00:06 UTC 2019


Hi Suneel,

On Tue, 29 Oct 2019 at 14:08, Suneel Garapati <suneelglinux at gmail.com> wrote:
>
> From: Suneel Garapati <sgarapati at marvell.com>
>
> Minor spelling correction.

Also mention your bug, and ideally add a Fixes: tag.

>
> Signed-off-by: Suneel Garapati <sgarapati at marvell.com>
> ---
>  drivers/pci/pci-uclass.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Reviewed-by: Simon Glass <sjg at chromium.org>

>
> diff --git a/drivers/pci/pci-uclass.c b/drivers/pci/pci-uclass.c
> index 896cb6b23a..848ac5a65e 100644
> --- a/drivers/pci/pci-uclass.c
> +++ b/drivers/pci/pci-uclass.c
> @@ -1373,7 +1373,7 @@ static void *dm_pci_map_ea_bar(struct udevice *dev, int bar, int flags,
>                 }
>
>                 /* size ignored for now */
> -               return map_physmem(addr, flags, 0);
> +               return map_physmem(addr, 0, flags);
>         }
>
>         return 0;
> @@ -1399,7 +1399,7 @@ void *dm_pci_map_bar(struct udevice *dev, int bar, int flags)
>
>         /*
>          * Pass "0" as the length argument to pci_bus_to_virt.  The arg
> -        * isn't actualy used on any platform because u-boot assumes a static
> +        * isn't actually used on any platform because u-boot assumes a static
>          * linear mapping.  In the future, this could read the BAR size
>          * and pass that as the size if needed.
>          */
> --
> 2.23.0
>

Regards,
Simon


More information about the U-Boot mailing list