[alsa-devel] SoC sound support for imx27

[Juergen Beisert]

On Thursday 24 April 2008 16:30, Liam Girdwood wrote:
> On Thu, 2008-04-24 at 16:11 +0200, Juergen Beisert wrote:
> > On Wednesday 23 April 2008 17:36, Liam Girdwood wrote:
> > > On Wed, 2008-04-23 at 16:54 +0200, Juergen Beisert wrote:
> > > > Hi,
> > > >
> > > > I'm going to adapt Freescale's i.MX27 CPU to the SoC sound support
> > > > via I2S. I found the imx21 already in the repository (may it will
> > > > help for my job). Is anybody else working on i.MX27 support (to avoid
> > > > to reinvention everything)?
> > >
> > > It's not fully supported atm. However I've had interest from a few
> > > folks who are interested in completing support.
> > >
> > > Currently working :-
> > > [...]
> >
> > Working in what tree? In the git repository I did not found a "struct
> > dma_channel_params" for imx. I'm confused.
>
> http://opensource.wolfsonmicro.com/cgi-bin/gitweb.cgi?p=linux-2.6-asoc.git;
>a=summary
>
> (dev branch)
>
> It's currently based on the 2.6.22 Freescale kernel (i.e. arch/arm/mxc)
> from their i.MX31ADS board's BSP.
>
> Fwiw, I will be updating stuff as more i.MX code goes upstream, it's
> just a matter of time.

Hmm, I did a deeper look into this code. Why - for example - does the file 
mx31ads-wm8753.c does the job of the BSP file? Why it registers devices? For 
my board I want to register platform_devices only in 
arc/arm/mach-mx2/pcm038.c. Nowhere else. If there is some glue required to 
bring components together, IMHO also arc/arm/mach-mx2/pcm038.c is the correct 
place to do so. Why you did it in sound/ ? If you do it in sound/ , you 
always need two locations to maintain. On in the arch/, one in sound/ .

Juergen

-- 
Dipl.-Ing. Juergen Beisert | http://www.pengutronix.de
 Pengutronix - Linux Solutions for Science and Industry
    Handelsregister: Amtsgericht Hildesheim, HRA 2686
         Vertretung Sued/Muenchen, Germany
   Phone: +49-8766-939 228 |  Fax: +49-5121-206917-9