On Sun, Jun 07, 2009 at 08:39:01PM +0200, Manuel Lauss wrote:
Replaces the sample Alchemy PSC AC97 machine code with a DB1200 machine driver with AC97 and I2S support.
Acked-by: Mark Brown broonie@opensource.wolfsonmicro.com
If this is going to go in during the merge window I'm happy for it to go in via the MIPS tree but otherwise I'd much rather it goes via ASoC in case API changes cause merge issues. I don't know what you prefer, Ralf?
We could also split the MIPS and ASoC parts into separate patches if that helps.
+/* it sucks that the ASoC headers are not under include/ */ +#include "../codecs/ac97.h" +#include "../codecs/wm8731.h"
This is because they're internal to ASoC - having them out of include should set off big red warning flags for something outside ASoC is looking at them. If there are things that should be referenced outside ASoC they should be in a separate header in include/sound like the WM9081 platform data.
+static int db1200_ac97_init(struct snd_soc_codec *codec) +{
- snd_soc_dapm_sync(codec);
- return 0;
+}
This could be removed but it does no harm.
+/*------------------------- COMMON PART ---------------------------*/
+static struct resource psc1_res[] = {
- [0] = {
.start = CPHYSADDR(PSC1_BASE_ADDR),.end = CPHYSADDR(PSC1_BASE_ADDR) + 0x000fffff,.flags = IORESOURCE_MEM,
If you conver the I2S driver to use the standard device probing model this could all me moved into the architecture code rather than placed in machine drivers.