[Top] [All Lists]

Re: [PATCH] mips: Add dma_mmap_coherent()

To: "Joel Soete" <>
Subject: Re: [PATCH] mips: Add dma_mmap_coherent()
From: Takashi Iwai <>
Date: Fri, 22 Aug 2008 14:17:54 +0200
Cc: "James.Bottomley" <>, "linux-mips" <>, "ralf" <>, "linux-parisc" <>
In-reply-to: <K6047O$>
Original-recipient: rfc822;
References: <K6047O$>
User-agent: Wanderlust/2.12.0 (Your Wildest Dreams) SEMI/1.14.6 (Maruoka) FLIM/1.14.7 (Sanjō) APEL/10.6 Emacs/22.2 (x86_64-suse-linux-gnu) MULE/5.0 (SAKAKI)
At Fri, 22 Aug 2008 13:04:36 +0100,
Joel Soete wrote:
> Hello Takashi et al.,
> I build and boot successfully kernel 32bit including your patch on 2 systems
> (a b2k using sba and a d380 using ccio).

Thanks for testing!

> I just noticed that the above code is ~ the same; otoh there is also a
> iommu-helpers.h containing also common code to those 2 drivers. So may be for
> easiest maintenance, could you merge and move this code in this 'helper' as
> follow:
> --- ./drivers/parisc/iommu-helpers.h.Orig     2008-08-01 12:57:22.000000000 
> +0000
> +++ ./drivers/parisc/iommu-helpers.h  2008-08-22 08:07:26.000000000 +0000

That sounds like a good idea.

One concern is to define a non-inline function in *.h.  But,
iommu-helper.h is included only by these two drivers, so there is no
problem as now, although a comment would be more helpful.

> >
> > diff --git a/include/asm-parisc/dma-mapping.h 
> > b/include/asm-parisc/dma-mapping.h
> > index 53af696..5b357b3 100644
> > --- a/include/asm-parisc/dma-mapping.h
> > +++ b/include/asm-parisc/dma-mapping.h
> The small issue encountered: against latest Kyle git tree (dated 2008-07-29)
> this file was moved in arch/parisc/include/asm.

Yes.  My patches are still based on older version (2.6.27-rc2 or so).

git cares renaming well, so it shouldn't be a big problem.
I just tested it now and git-pull (oh now it's "git pull" :) renames
it automatically indeed.



<Prev in Thread] Current Thread [Next in Thread>