On Sunday, November 29, 2015 at 09:18:07 PM, Simon Glass wrote:
> Convert this driver to use the new driver model PCI API.
> 
> Signed-off-by: Simon Glass <s...@chromium.org>
> Reviewed-by: Bin Meng <bmeng...@gmail.com>

ad subject -- it's EHCI, not ECHI .

> ---
> 
> Changes in v2:
> - Use u32 instead of uint32_t
> - Use uintptr_t to cast the hcor address
> 
>  drivers/usb/host/ehci-pci.c | 51
> ++++++++++++++++++++++++++++++++++----------- 1 file changed, 39
> insertions(+), 12 deletions(-)
> 
> diff --git a/drivers/usb/host/ehci-pci.c b/drivers/usb/host/ehci-pci.c
> index d607088..cda1c6d 100644
> --- a/drivers/usb/host/ehci-pci.c
> +++ b/drivers/usb/host/ehci-pci.c
> @@ -18,32 +18,34 @@ struct ehci_pci_priv {
>       struct ehci_ctrl ehci;
>  };
> 
> -static void ehci_pci_common_init(pci_dev_t pdev, struct ehci_hccr
> **ret_hccr, -                          struct ehci_hcor **ret_hcor)
> +#ifdef CONFIG_DM_USB
> +
> +static void ehci_pci_init(struct udevice *dev, struct ehci_hccr
> **ret_hccr, +                   struct ehci_hcor **ret_hcor)
>  {
>       struct ehci_hccr *hccr;
>       struct ehci_hcor *hcor;
> -     uint32_t cmd;
> +     u32 cmd;
> 
> -     hccr = (struct ehci_hccr *)pci_map_bar(pdev,
> +     hccr = (struct ehci_hccr *)dm_pci_map_bar(dev,
>                       PCI_BASE_ADDRESS_0, PCI_REGION_MEM);
> -     hcor = (struct ehci_hcor *)((uint32_t) hccr +
> +     hcor = (struct ehci_hcor *)((uintptr_t) hccr +
>                       HC_LENGTH(ehci_readl(&hccr->cr_capbase)));
> 
>       debug("EHCI-PCI init hccr 0x%x and hcor 0x%x hc_length %d\n",
> -           (uint32_t)hccr, (uint32_t)hcor,
> -           (uint32_t)HC_LENGTH(ehci_readl(&hccr->cr_capbase)));
> +           (u32)hccr, (u32)hcor,
> +           (u32)HC_LENGTH(ehci_readl(&hccr->cr_capbase)));

Use uintptr here as well then ?

>       *ret_hccr = hccr;
>       *ret_hcor = hcor;
> 
>       /* enable busmaster */
> -     pci_read_config_dword(pdev, PCI_COMMAND, &cmd);
> +     dm_pci_read_config32(dev, PCI_COMMAND, &cmd);
>       cmd |= PCI_COMMAND_MASTER;
> -     pci_write_config_dword(pdev, PCI_COMMAND, cmd);
> +     dm_pci_write_config32(dev, PCI_COMMAND, cmd);
>  }

Looks good otherwise,

Acked-by: Marek Vasut <ma...@denx.de>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to