Hi, "mmu" and "mailbox" still have some dependency on dspgateway. They also should be cleaned up as being independent on specific implementation, ideally to be shared by any bridge implementations. Hiroshi DOYU From: "ext Tony Lindgren" <tony@xxxxxxxxxxx> Subject: Re: [RFC] Port TI DSP BRIDGE for a new dedicated branch in linux-omap Date: Mon, 18 Aug 2008 10:56:24 +0300 > Hi, > > * Felipe Contreras <felipe.contreras@xxxxxxxxx> [080816 12:58]: > > Hi Vijay, > > > > On Sat, Aug 16, 2008 at 12:59 AM, Pasam, Vijay <vpasam@xxxxxx> wrote: > > >> I'm not familiar with checkpatch, but I guess the purpose is > > >> not to highlight functional issues. > > >> > > >> However, there are functional issues, but it makes sense to > > >> cleanup the code first: there's no point in analyzing code > > >> that is never used. > > > > > > There are about 9 errors with the latest set of patches. These are all > > > false positives - not really errors. Majority of warnings also fall > > > under this category. > > > > I'm not talking about issues returned by checkpatch, but issues > > visible to the human observer. > > And the two pieces of code that need to be fixed for Linux ASAP > for DSP are: > > - External device MMU hardware > - Mailbox handling > > This code needs to be shared for dsp[bios|bridge|gateway|link]. > And it can be also used for any other devices and coprocessors > needing MMU handling. > > So far the sanest solution seems to be in: > > arch/arm/*omap*/mmu.[ch] > arch/arm/*omap*/mailbox.[ch] > > Is there any reason why this code cannot be used for dspbridge? > > BTW, the MMU code needs to be fixed so that the code > shared with ioremap() is moved to ioremap(). Basically we > need to enhance ioremap() in a way where it can support external > MMU hardware. Otherwise the maintenance will be a nightmare. > > This was pointed out by Russell King a while back when we tried > to get the MMU code integrated to the mainline kernel. > > Regards, > > Tony > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
