Thanks for your comments.

From: "ext Russell King - ARM Linux" <[EMAIL PROTECTED]>
Subject: Re: [PATCH 2/3][OMAP 3/4]table end for wcd command definitions
Date: Wed, 10 Sep 2008 20:38:30 +0100

> On Wed, Sep 10, 2008 at 08:44:14AM -0700, Tony Lindgren wrote:
> > * Hiroshi DOYU <[EMAIL PROTECTED]> [080909 21:08]:
> > > Hi,
> > > 
> > > Although this TI bridge patches are not integrated and a bit
> > > independent S/W component, it would be nice to get some comments just
> > > on this bridge header file location from kernel maintainer's
> > > perspective.
> > > 
> > > "arch/arm/plat-omap/include/mach/bridge"
> > >   or "arch/arm/plat-omap/include/bridge"?
> > 
> > How about put the bridge specific headers under include/linux/dspbridge?
> > Then you can have just the omap specific headers under
> > arch/arm/plat-omap/include/mach.
> 
> There's an advantage to having them under arch/arm/plat-omap/include/dspbridge
> - it's portable.  If ever they need to become more visible, they can be easily
> moved to arch/arm/include/dspbridge or include/dspbridge instead.

"arch/arm/plat-omap/include/dspbridge" sounds reasonable, at least,
for the meanwhile.

For this header file directory name, 'dsp' prefix is needed in
"include/'dsp'bridge" because "bridge" is too generic naming and,
instead, "drivers/'dsp'/bridge" has 'dsp' in its path already.

        Hiroshi DOYU


--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to