[Top] [All Lists]

Re: [alsa-devel] [PATCH 4/7] Alchemy: DB1200 AC97+I2S audio support.

To: Mark Brown <>
Subject: Re: [alsa-devel] [PATCH 4/7] Alchemy: DB1200 AC97+I2S audio support.
From: Manuel Lauss <>
Date: Mon, 8 Jun 2009 14:21:18 +0200
Cc: Linux-MIPS <>, Ralf Baechle <>, Manuel Lauss <>,
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=gamma; h=domainkey-signature:mime-version:received:in-reply-to:references :date:message-id:subject:from:to:cc:content-type :content-transfer-encoding; bh=GSYwsSxcggRGcOh1FLgatIYCDc9l8K0sTeL63dWDrN4=; b=if75p0r2MOAbdIPCR3ljOSr/Ntzv6PGUDNVRahZifkUms8heNzlYe11UlPlDyoDOvQ TW30gOJE5LQJ9QSj5pSPW3J7V3Zylr9DcCnXecwaVZunQ+Mi6k1U7a2s05rMWkLaEmue Srp8rJllOfAvPXiSClZDB/4mzZN50aYwIqFIo=
Domainkey-signature: a=rsa-sha1; c=nofws;; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; b=gZVzDZ4U5jOeKkotdzJNenIz3C2o4THtdCnkudL9d83Ftge4UXYOvnBhVn7hlTl2Ez BMFYjr7Gfb7ulAUTVr+QnEIIymCvU05YOjU00Y2L+Tt0PM7O/azQkM9eig8EBQOoHbyW 5tMNPAgkDYLDAeA+lQfA3Nim1+zPJrphplLvE=
In-reply-to: <20090608115336.GA25827@rakim.wolfsonmicro.main>
Original-recipient: rfc822;
References: <> <> <> <> <> <> <> <20090608102018.GA6547@rakim.wolfsonmicro.main> <> <20090608115336.GA25827@rakim.wolfsonmicro.main>

> You should convert the DAI drivers to probe as normal platform devices
> and attach the resources used by the CPU to those devices rather than
> attaching the data to soc-audio.  pxa2xx-ac97 does this, as do the
> PowerPC drivers and the s3c64xx-i2s driver.  The DaVinci drivers
> currently on the davinci branch of my git for merge after the merge
> window do this too.

I see now what you mean, but this is ugly as sin:
I now need to register 2 platform devices in the board code: 1 for the DAI
(with resources mmio + irq) and 1 for the DMA engine (with ddma id resources),
or register the DMA engine device from within the AC97/I2S drivers.

This is in my opinion even worse than the current scheme, which at least allows
me to group all PSC resources into one struct resource which all audio-related
drivers can share without too much uglyness.

       Manuel Lauss

<Prev in Thread] Current Thread [Next in Thread>