2002-02-06 08:37:19

by Christoph Hellwig

[permalink] [raw]
Subject: Re: [Linux-ia64] Proper fix for sym53c8xx_2 driver and dma64_addr_t

On Wed, Feb 06, 2002 at 12:29:06AM -0800, David S. Miller wrote:
> What driver wants to get at this type and what are they using it
> for? dma_addr_t should be used by every driver I am aware of
> except the clustering PCI cards I've been told about and that
> driver isn't in the kernel at this time.
>
> So who needs it? :-)

The new sym53c8xx driver (sym2), and that one only if is actually is
configured for DAC-mode (SYM_CONF_DMA_ADDRESSING_MODE > 0).

Christoph

--
Of course it doesn't work. We've performed a software upgrade.


2002-02-06 08:47:22

by David Miller

[permalink] [raw]
Subject: Re: [Linux-ia64] Proper fix for sym53c8xx_2 driver and dma64_addr_t

From: Christoph Hellwig <[email protected]>
Date: Wed, 6 Feb 2002 09:35:58 +0100

On Wed, Feb 06, 2002 at 12:29:06AM -0800, David S. Miller wrote:
> So who needs it? :-)

The new sym53c8xx driver (sym2), and that one only if is actually is
configured for DAC-mode (SYM_CONF_DMA_ADDRESSING_MODE > 0).

It is not using the DMA apis correctly then, it should be using
dma_addr_t which may or may not be 64-bits on a given platform.

The sym2 driver needs to be fixed.

Franks a lot,
David S. Miller
[email protected]

2002-02-06 20:24:25

by Gérard Roudier

[permalink] [raw]
Subject: Re: [Linux-ia64] Proper fix for sym53c8xx_2 driver and dma64_addr_t



On Wed, 6 Feb 2002, David S. Miller wrote:

> From: Christoph Hellwig <[email protected]>
> Date: Wed, 6 Feb 2002 09:35:58 +0100
>
> On Wed, Feb 06, 2002 at 12:29:06AM -0800, David S. Miller wrote:
> > So who needs it? :-)
>
> The new sym53c8xx driver (sym2), and that one only if is actually is
> configured for DAC-mode (SYM_CONF_DMA_ADDRESSING_MODE > 0).
>
> It is not using the DMA apis correctly then, it should be using
> dma_addr_t which may or may not be 64-bits on a given platform.
>
> The sym2 driver needs to be fixed.

The code written at 't' time was using what was available at that time.
If you can let me know when Linux will be rewritten in Java, this will be
useful for me to be ready in time. :-)

--- sym_glue.c Sun Dec 30 21:33:10 2001
+++ a.c Tue Feb 5 22:17:30 2002
@@ -313,11 +313,7 @@
#ifndef SYM_LINUX_DYNAMIC_DMA_MAPPING
typedef u_long bus_addr_t;
#else
-#if SYM_CONF_DMA_ADDRESSING_MODE > 0
-typedef dma64_addr_t bus_addr_t;
-#else
typedef dma_addr_t bus_addr_t;
-#endif
#endif

You may get a couple of warnings, but since I haven't hardware to check
this, I am ready to apply patches.


G?rard.

PS:
The driver Java version is planned to be available on April the 1rst. ;)