From: Finn Thain <fthain@telegraphics.com.au>
To: Ralf Baechle <ralf@linux-mips.org>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>,
Jeff Garzik <jgarzik@pobox.com>,
Linux/m68k <linux-m68k@vger.kernel.org>,
Linux/m68k on Mac <linux-mac68k@mac.linux-m68k.org>,
Linux MIPS <linux-mips@vger.kernel.org>,
Linux kernel <linux-kernel@vger.kernel.org>,
Thomas Bogendoerfer <tsbogend@alpha.franken.de>
Subject: Re: [PATCH] Jazzsonic driver updates
Date: Sun, 12 Jun 2005 16:05:30 +1000 (EST) [thread overview]
Message-ID: <Pine.LNX.4.61.0506121454410.1470@loopy.telegraphics.com.au> (raw)
In-Reply-To: <20050323100139.GB8813@linux-mips.org>
On Wed, 23 Mar 2005, Ralf Baechle wrote:
> On Tue, Mar 22, 2005 at 06:13:17PM +0100, Geert Uytterhoeven wrote:
>
> > On Fri, 28 Jan 2005, Linux Kernel Mailing List wrote:
> > > ChangeSet 1.1986, 2005/01/28 00:12:28-05:00, ralf@linux-mips.org
> > >
> > > [PATCH] Jazzsonic driver updates
> > >
> > > o Resurrect the Jazz SONIC driver after years of it not having been tested
> > > o Convert from Space.c initialization to module_init / platform device.
> > >
> > > Signed-off-by: Jeff Garzik <jgarzik@pobox.com>
> >
> > > --- a/drivers/net/sonic.c 2005-03-06 18:10:39 -08:00
> > > +++ b/drivers/net/sonic.c 2005-03-06 18:10:39 -08:00
> > > @@ -116,7 +116,7 @@
> > > /*
> > > * Map the packet data into the logical DMA address space
> > > */
> > > - if ((laddr = vdma_alloc(PHYSADDR(skb->data), skb->len)) == ~0UL) {
> > > + if ((laddr = vdma_alloc(CPHYSADDR(skb->data), skb->len)) == ~0UL) {
> > ^^^^^^^^^
> > This part broke compilation for Mac/m68k.
> >
> > > printk("%s: no VDMA entry for transmit available.\n",
> > > dev->name);
> > > dev_kfree_skb(skb);
>
> Oh funny. vdma_alloc() was created 10 years ago as an internal API for
> the Jazz machines. Didn't realize m68k had cloned it :-) If anything
> it seems this should be converted to the modern DMA API.
I've just started merging my Mac sonic work into 2.6.12-rc6. m68k doesn't
yet implement the modern DMA API, but it is easy to fake it for a while
for macsonic.c. So I don't mind converting macsonic, jazzsonic and the
shared sonic driver core to the new API.
But, I knowing nothing about the Jazz DMA controller. I need some help
from the MIPS people:
Would I be right to say that vdma_{alloc,free}() can be changed to
dma_{,un}map_single? The other Jazz specific routine that sonic uses is
vdma_log2phys, and I don't know if that has a better alternative.
-f
> Ralf
> -
> To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
next prev parent reply other threads:[~2005-06-12 6:31 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <200503070210.j272ARii023023@hera.kernel.org>
2005-03-22 17:13 ` [PATCH] Jazzsonic driver updates Geert Uytterhoeven
2005-03-23 2:30 ` [PATCH] " Finn Thain
2005-03-23 10:01 ` Ralf Baechle
2005-03-23 10:47 ` Finn Thain
2005-06-12 6:05 ` Finn Thain [this message]
2005-06-15 11:41 ` Ralf Baechle
2005-06-15 14:02 ` Finn Thain
2005-06-15 14:27 ` Ralf Baechle
2005-06-15 16:53 ` Finn Thain
2005-06-15 17:45 ` Finn Thain
2005-06-16 9:22 ` Ralf Baechle
2005-06-16 12:14 ` Finn Thain
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=Pine.LNX.4.61.0506121454410.1470@loopy.telegraphics.com.au \
--to=fthain@telegraphics.com.au \
--cc=geert@linux-m68k.org \
--cc=jgarzik@pobox.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-m68k@vger.kernel.org \
--cc=linux-mac68k@mac.linux-m68k.org \
--cc=linux-mips@vger.kernel.org \
--cc=ralf@linux-mips.org \
--cc=tsbogend@alpha.franken.de \
/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;
as well as URLs for NNTP newsgroup(s).