public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: "Gérard Roudier" <groudier@free.fr>
To: "David S. Miller" <davem@redhat.com>
Cc: <hch@caldera.de>, <davidm@hpl.hp.com>, <mmadore@turbolinux.com>,
	<linux-ia64@linuxia64.org>, <linux-kernel@vger.kernel.org>,
	<torvalds@transmeta.com>
Subject: Re: [Linux-ia64] Proper fix for sym53c8xx_2 driver and dma64_addr_t
Date: Tue, 5 Feb 2002 22:23:02 +0100 (CET)	[thread overview]
Message-ID: <20020205221314.F1707-100000@gerard> (raw)
In-Reply-To: <20020206.004503.118628125.davem@redhat.com>



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

>    From: Christoph Hellwig <hch@caldera.de>
>    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.  ;)



  reply	other threads:[~2002-02-06 20:24 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <3C6043E5.D1F40E5D@turbolinux.com>
     [not found] ` <20020205223804.A22012@caldera.de>
     [not found]   ` <15456.21030.840746.209377@napali.hpl.hp.com>
2002-02-06  8:21     ` [Linux-ia64] Proper fix for sym53c8xx_2 driver and dma64_addr_t Christoph Hellwig
2002-02-06  8:29       ` David S. Miller
2002-02-06  8:35         ` Christoph Hellwig
2002-02-06  8:45           ` David S. Miller
2002-02-05 21:23             ` Gérard Roudier [this message]
2002-02-06 17:10             ` Christoph Hellwig
2002-02-06 17:15               ` Christoph Hellwig
2002-02-06 17:15               ` David Mosberger
2002-02-05 21:34                 ` Gérard Roudier
2002-02-07  2:09               ` David S. Miller
2002-02-06 21:38                 ` Gérard Roudier
2002-02-06 16:54       ` David Mosberger

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20020205221314.F1707-100000@gerard \
    --to=groudier@free.fr \
    --cc=davem@redhat.com \
    --cc=davidm@hpl.hp.com \
    --cc=hch@caldera.de \
    --cc=linux-ia64@linuxia64.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mmadore@turbolinux.com \
    --cc=torvalds@transmeta.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox