[U-Boot-Users] [PATCH] Fixed pcnet io_base

Bus and phys address are not always the same
Signed-off-by: Vlad Lungu vlad@comsys.ro --- drivers/pcnet.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/pcnet.c b/drivers/pcnet.c index 2af0e8f..22edb48 100644 --- 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); dev->iobase &= ~0xf;
PCNET_DEBUG1("%s: devbusfn=0x%x iobase=0x%x: ",

Dear Ben,
this patch is still on my "open" list. Can you please review and apply if it makes sense to you? Thanks.
------- Forwarded Message
Date: Wed, 10 Oct 2007 23:02:09 +0300 From: Vlad Lungu vlad@comsys.ro To: u-boot-users@lists.sourceforge.net, wd@denx.de Subject: [PATCH] Fixed pcnet io_base
Bus and phys address are not always the same
Signed-off-by: Vlad Lungu vlad@comsys.ro --- drivers/pcnet.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/pcnet.c b/drivers/pcnet.c index 2af0e8f..22edb48 100644 --- 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 *)&de v->iobase); + dev->iobase=pci_io_to_phys(devbusfn,dev->iobase); dev->iobase &= ~0xf;
PCNET_DEBUG1("%s: devbusfn=0x%x iobase=0x%x: ",

Hi Wolfgang,
On Sun, Apr 20, 2008 at 2:36 AM, Wolfgang Denk wd@denx.de wrote:
Dear Ben,
this patch is still on my "open" list. Can you please review and apply if it makes sense to you? Thanks.
This looks OK to me. Assuming that pci_io_to_phys() is defined on all relevant architectures, I have no objections. Unfortunately I can't run MAKEALL right now, so if it's not too much of a bother please pull directly. We can always pull it out if there's a problem, I guess.
regards, Ben
------- Forwarded Message
Date: Wed, 10 Oct 2007 23:02:09 +0300 From: Vlad Lungu vlad@comsys.ro To: u-boot-users@lists.sourceforge.net, wd@denx.de Subject: [PATCH] Fixed pcnet io_base
Bus and phys address are not always the same
Signed-off-by: Vlad Lungu vlad@comsys.ro
drivers/pcnet.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/pcnet.c b/drivers/pcnet.c index 2af0e8f..22edb48 100644 --- 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 *)&de v->iobase);
dev->iobase=pci_io_to_phys(devbusfn,dev->iobase); dev->iobase &= ~0xf; PCNET_DEBUG1("%s: devbusfn=0x%x iobase=0x%x: ",
-- 1.5.2.2
------- End of Forwarded Message
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@denx.de Where a calculator on the ENIAC is equipped with 18,000 vacuum tubes and weighs 30 tons, computers in the future may have only 1,000 vacuum tubes and weigh only 1/2 tons. - Popular Mechanics, March 1949

In message f8328f7c0804200702x40841be6k286dd6e335440692@mail.gmail.com you wrote:
this patch is still on my "open" list. Can you please review and apply if it makes sense to you? Thanks.
This looks OK to me. Assuming that pci_io_to_phys() is defined on all relevant architectures, I have no objections. Unfortunately I can't run MAKEALL right now, so if it's not too much of a bother please pull directly. We can always pull it out if there's a problem, I guess.
Applied, thanks.
Best regards,
Wolfgang Denk

In message 470D2FC1.3070906@comsys.ro you wrote:
Bus and phys address are not always the same
Signed-off-by: Vlad Lungu vlad@comsys.ro
drivers/pcnet.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)
Applied, thanks.
Best regards,
Wolfgang Denk
participants (3)
-
Ben Warren
-
Vlad Lungu
-
Wolfgang Denk