2002-01-24 10:02:17

by Adam J. Richter

[permalink] [raw]
Subject: linux-2.5.3-pre4/drivers/acenic.c: pci_unmap_addr_set not defined for x86

linux-2.5.3-pre4/drivers/acenic.c uses pci_unmap_addr_set,
which is defined for most architectures in include/asm-*/pci.h, but
not for i386. For i386 this results in undefined references. I
imagine that this is the result of a missed file (include/asm-i386/pci.h?)
from an Acenic update patch.

Adam J. Richter __ ______________ 4880 Stevens Creek Blvd, Suite 104
[email protected] \ / San Jose, California 95129-1034
+1 408 261-6630 | g g d r a s i l United States of America
fax +1 408 261-6631 "Free Software For The Rest Of Us."


2002-01-24 13:38:50

by David Miller

[permalink] [raw]
Subject: Re: linux-2.5.3-pre4/drivers/acenic.c: pci_unmap_addr_set not defined for x86

From: "Adam J. Richter" <[email protected]>
Date: Thu, 24 Jan 2002 02:01:55 -0800

linux-2.5.3-pre4/drivers/acenic.c uses pci_unmap_addr_set,
which is defined for most architectures in include/asm-*/pci.h, but
not for i386. For i386 this results in undefined references. I
imagine that this is the result of a missed file (include/asm-i386/pci.h?)
from an Acenic update patch.

No, just a dumb typo:

--- include/asm-i386/pci.h.~1~ Tue Jan 15 10:59:36 2002
+++ include/asm-i386/pci.h Thu Jan 24 05:32:28 2002
@@ -118,7 +118,7 @@
#define DECLARE_PCI_UNMAP_ADDR(ADDR_NAME)
#define DECLARE_PCI_UNMAP_LEN(LEN_NAME)
#define pci_unmap_addr(PTR, ADDR_NAME) (0)
-#define pci_unmap_addr_SET(PTR, ADDR_NAME, VAL) do { } while (0)
+#define pci_unmap_addr_set(PTR, ADDR_NAME, VAL) do { } while (0)
#define pci_unmap_len(PTR, LEN_NAME) (0)
#define pci_unmap_len_set(PTR, LEN_NAME, VAL) do { } while (0)

2002-01-24 13:45:10

by Jes Sorensen

[permalink] [raw]
Subject: Re: linux-2.5.3-pre4/drivers/acenic.c: pci_unmap_addr_set not defined for x86

>>>>> "Adam" == Adam J Richter <[email protected]> writes:

Adam> linux-2.5.3-pre4/drivers/acenic.c uses pci_unmap_addr_set, which
Adam> is defined for most architectures in include/asm-*/pci.h, but not
Adam> for i386. For i386 this results in undefined references. I
Adam> imagine that this is the result of a missed file
Adam> (include/asm-i386/pci.h?) from an Acenic update patch.

Hi Adam

I haven't had a chance to look at it yet. The patch wasn't done by me
and whoever submitted it didn't seem to think it was worth the effort of
Cc'ing me a copy of it ;-(

I'll take a look.

Thanks,
Jes

2002-01-24 13:48:10

by David Miller

[permalink] [raw]
Subject: Re: linux-2.5.3-pre4/drivers/acenic.c: pci_unmap_addr_set not defined for x86

From: Jes Sorensen <[email protected]>
Date: Thu, 24 Jan 2002 08:44:44 -0500

I haven't had a chance to look at it yet. The patch wasn't done by me
and whoever submitted it didn't seem to think it was worth the effort of
Cc'ing me a copy of it ;-(

I was changing APIs, do I have to CC: every driver author on the
planet when I do this?

2002-01-24 14:16:04

by Jes Sorensen

[permalink] [raw]
Subject: Re: linux-2.5.3-pre4/drivers/acenic.c: pci_unmap_addr_set not defined for x86

>>>>> "David" == David S Miller <[email protected]> writes:

> From: Jes Sorensen <[email protected]> Date: Thu, 24 Jan
> 2002 08:44:44 -0500

> I haven't had a chance to look at it yet. The patch wasn't
> done by me and whoever submitted it didn't seem to think it was
> worth the effort of Cc'ing me a copy of it ;-(

David> I was changing APIs, do I have to CC: every driver author on the
David> planet when I do this?

Considering a) it's just a few keystrokes to add a CC: line, b) it's the
driver authors who are the first to get the bug reports, then yes it
seems like a very reasonable request.

Jes

2002-01-24 14:17:43

by David Miller

[permalink] [raw]
Subject: Re: linux-2.5.3-pre4/drivers/acenic.c: pci_unmap_addr_set not defined for x86

From: Jes Sorensen <[email protected]>
Date: Thu, 24 Jan 2002 09:15:42 -0500

Considering a) it's just a few keystrokes to add a CC: line, b) it's the
driver authors who are the first to get the bug reports, then yes it
seems like a very reasonable request.

This list actually is the first place the reports typically
go to.

I'm a responsible maintainer, when I add a problem, I always go back
and fix it or revert my changes. I never disappear after making
these kinds of changes.

2002-01-24 14:44:01

by Jes Sorensen

[permalink] [raw]
Subject: Re: linux-2.5.3-pre4/drivers/acenic.c: pci_unmap_addr_set not defined for x86

>>>>> "David" == David S Miller <[email protected]> writes:

David> From: Jes Sorensen <[email protected]> Date: Thu, 24 Jan
David> 2002 09:15:42 -0500

> Considering a) it's just a few keystrokes to add a CC: line,
> b) it's the driver authors who are the first to get the bug
> reports, then yes it seems like a very reasonable request.

David> This list actually is the first place the reports typically go
David> to.

Actually thats not the case in my experience, very often thing like that
go to the driver specific mailing lists or the driver maintainer in
private email. In fact this more common than a report going
linux-kernel.

David> I'm a responsible maintainer, when I add a problem, I always go
David> back and fix it or revert my changes. I never disappear after
David> making these kinds of changes.

I never claimed you don't, but some reports you never receive.

All I am suggesting is a CC, no fairytales or other stories to go with
it that actually takes time to write.

Jes

2002-01-24 15:44:22

by David Miller

[permalink] [raw]
Subject: Re: linux-2.5.3-pre4/drivers/acenic.c: pci_unmap_addr_set not defined for x86

From: "Narayan Desai" <[email protected]>
Date: Thu, 24 Jan 2002 09:41:26 -0600

I think that this change is necessary for 2.4.18pre6 as well.

Already sent to Marcelo.

2002-01-24 15:43:12

by Narayan Desai

[permalink] [raw]
Subject: Re: linux-2.5.3-pre4/drivers/acenic.c: pci_unmap_addr_set not defined for x86

I think that this change is necessary for 2.4.18pre6 as well.
-nld