Re: [PATCH] MIPS: Fixups of ALSA memory maps

From: Takashi Iwai
Date: Thu Nov 26 2009 - 09:51:18 EST


At Mon, 23 Nov 2009 12:36:15 +0000,
Ralf Baechle wrote:
>
> On Mon, Nov 23, 2009 at 09:56:39AM +0100, Takashi Iwai wrote:
>
> > > On Wed, 2009-11-18 at 18:47 +0100, Takashi Iwai wrote:
> > > [...]
> > > >
> > > > Well, we haven't reached the consensus. The discussion faded away
> > > > somehow mainly because I had too little time to update and ping people
> > > > again.
> > > >
> > > > In Tokyo, I talked with some guys regarding this. Ben agreed to take
> > > > this approach for ppc, and David said that he doesn't mind for sparc
> > > > part. Fujita-san mentioned it's no big problem to add one op from
> > > > the generic dma_ops.
> > > >
> > > > So, maybe somehow need to convince James in the end (and ask Paul to
> > > > check SH part, too), then it'll be all up... theoretically :)
> > > >
> > > > Anyway, I'm going to raise the discussion again on linux-arch.
> > > > I'm afraid it's a bit too late game for 2.6.33, but starting now is
> > > > better than too late again.
> > >
> > > Hi, Takashi Iwai
> > >
> > > Before the API stuff going into the mainline(2.6.33), can we apply this
> > > "[PATCH] MIPS: Fixups of ALSA memory maps"(This is the minimal
> > > necessares) as a current fixup. and then we will not get a broken sound
> > > support for MIPS, and also the support to the latest Loongson2F family
> > > machines will benefit from it.
> > >
> > > and Ralf, what about your suggestion?
> >
> > The question is whether this hack can be safely added for all MIPS
> > platforms just by checking kconfig. I had an impression that rather
> > many things have to be checked in the runtime.
> >
> > As I have really little clue about MIPS architecture, I'd like let
> > MIPS guys decide about it...
>
> Okay, I'll cook up a nice kludge. It can't cover all cases but it will
> be an improvment for most platforms.

OK, but I found that the original patch doesn't work correctly for
some cases, as I mentioned in another thread. I'm going to send a patch
set soon as a new thread.


thanks,

Takashi
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/