Re: [PATCH 06/12] imx-ssi sound driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Nov 21, 2009 at 08:12:34PM -0600, Timur Tabi wrote:
> On Thu, Nov 19, 2009 at 9:48 AM, Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> wrote:
> > Signed-off-by: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx>
> > ---
> >  arch/arm/plat-mxc/Makefile           |    1 +
> >  arch/arm/plat-mxc/include/mach/ssi.h |   17 +
> >  arch/arm/plat-mxc/ssi-fiq.S          |  134 ++++
> >  sound/soc/imx/devdma.c               |   80 ++
> >  sound/soc/imx/devdma.h               |    3 +
> >  sound/soc/imx/imx-ssi.c              | 1349 ++++++++++++++++++++++++++++++++++
> >  sound/soc/imx/imx-ssi.h              |   24 +
> 
> Why did you write a new SSI driver when there already is one in sound/soc/fsl?

Mainly because I wasn't aware of the fact that the fsl driver is the
same :(

I'm currently looking if and how the code can be merged. I don't know if
it's worth it though since the common code doesn't seem to be very big.

There will have to be some changes in the fsl-ssi driver. For example I
need the irq handler for my fiq handler which means that the fsl-ssi
driver is not allowed to request the irq for it's statistic. The other
thing is the hardware accesses as we do not have in_be*, out_be* and
setbits/clrbits. Of course all these things are doable and shouldn't be
a showstopper.

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 |
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel


[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux