[U-Boot] [PATCH v2 07/10] net: pcnet: use pci_virt_to_mem to obtain buffer addresses

[Daniel Schwierzeck]

2013/2/2 Gabor Juhos <juhosg at openwrt.org>:
> The pcnet driver uses the pci_phys_to_mem function
> to get the memory address of the DMA buffers. This
> This assumes an 1:1 mapping between the PCI and
> physical memory which is not true on all platforms.
>
> On MIPS platform U-Boot is running within a mapped
> memory region, and the pci_phys_to_mem macro can't
> be used to obtain the memory address of the buffers.
>
> Signed-off-by: Gabor Juhos <juhosg at openwrt.org>
> Cc: Daniel Schwierzeck <daniel.schwierzeck at googlemail.com>
> ---
> Changes since v1:
>  - rebased against mips/testing
>
> Changes since RFC: ---
>
> ---
> Note:
>
> This is only tested with the qemu-malta target. The change
> might break real platforms, however I have no suitable board
> to test it.

the consumers of pcnet and NE2000 are qemu-mips, R7780MP and SHMIN.
Nobuhiro, could you comment this change?

>
> -Gabor
> ---
>  drivers/net/pcnet.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/pcnet.c b/drivers/net/pcnet.c
> index c028a44..45a66fb 100644
> --- a/drivers/net/pcnet.c
> +++ b/drivers/net/pcnet.c
> @@ -146,7 +146,7 @@ static int pcnet_recv (struct eth_device *dev);
>  static void pcnet_halt (struct eth_device *dev);
>  static int pcnet_probe (struct eth_device *dev, bd_t * bis, int dev_num);
>
> -#define PCI_TO_MEM(d,a) pci_phys_to_mem((pci_dev_t)d->priv, (u_long)(a))
> +#define PCI_TO_MEM(d, a) pci_virt_to_mem((pci_dev_t)d->priv, (a))
>  #define PCI_TO_MEM_LE(d,a) (u32)(cpu_to_le32(PCI_TO_MEM(d,a)))
>
>  static struct pci_device_id supported[] = {
> --
> 1.7.10
>

-- 
Best regards,
Daniel