Re: [RFC] Port TI DSP BRIDGE for a new dedicated branch in linux-omap

[Tony Lindgren <tony@atomide.com>]

Hi,

* Felipe Contreras <felipe.contreras@gmail.com> [080816 12:58]:
> Hi Vijay,
> 
> On Sat, Aug 16, 2008 at 12:59 AM, Pasam, Vijay <vpasam@ti.com> 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