[U-Boot-Users] [PATCH] Fixed pcnet io_base , added Am79C970A chip id

[Wolfgang Denk <wd@denx.de>]

In message <470CA8A6.5060900@comsys.ro> you wrote:
> Bus and phys address are not always the same

OK, so this would be a bug fix.

> --- a/drivers/pcnet.c
> +++ b/drivers/pcnet.c
> @@ -196,6 +196,7 @@ int pcnet_initialize(bd_t *bis)
>          * Setup the PCI device.
>          */        pci_read_config_dword(devbusfn, PCI_BASE_ADDRESS_0, (unsigned int *)&dev->iobase);
> +       dev->iobase=pci_io_to_phys(devbusfn,dev->iobase);

Here it goes.

> @@ -260,6 +261,9 @@ static int pcnet_probe(struct eth_device* dev, bd_t *bis, int dev_nr)
>         return -1;
>      chip_version = (chip_version >> 12) & 0xffff;
>      switch (chip_version) {
> +    case 0x2621:
> +       chipname = "PCnet/PCI II 79C970A"; /* PCI */
> +       break;

But this is new code and completely unrelated to the bug fix.

Please split and submit as two separate patched - the first will go
into 1.3.0, the second will have to wait.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
If you want strict real-time behavior, run in the real  time  schedu-
ling class.  But there are no seatbelts or airbags;  main(){for(;;);}
can hard hang your system.                          -- Bart Smaalders