[alsa-devel] i.MX audio support

Sascha Hauer s.hauer at pengutronix.de
Fri Nov 20 06:10:33 EST 2009


On Fri, Nov 20, 2009 at 11:32:10AM +0100, javier Martin wrote:
> > I would agree if the current code worked and would be good shape.
> > Unfortunately it's not. It hardcodes the fact that there are 2 SSI units
> > (i.MX51 has three of them). Instead of writel/readl it uses direct pointer
> > derefs. The driver skips audio while it reinitialises the DMA engine. In
> > mx1_mx2_pcm_open() there are several memory leaks. The driver does not
> > call clk_get itself but depends on the board doing it. The only
> > available board support depends on a board not present in the kernel, so
> > we are not able to compile it (and it won't because the audmux
> > definitions are missing)
> 
> We have still some pending HW changes in our board so we are not ready
> to submit it now. Anyway it would have been nice to have such this
> comments when I submitted my patches to avoid unnecessary rework.

Sorry, I missed them. I'm subscribed to alsa-devel for one day now and
AFAIK the patches haven't been posted to any other list.

Sascha


-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



More information about the linux-arm-kernel mailing list