From mboxrd@z Thu Jan 1 00:00:00 1970 From: broonie@opensource.wolfsonmicro.com (Mark Brown) Date: Mon, 27 Aug 2012 09:33:38 -0700 Subject: FW: [PATCH 07/22] ASoC: Ux500: Initialise PCM from MSP probe rather than as a device In-Reply-To: <50377C0D.2060905@stericsson.com> References: <014401cd8165$99960c40$ccc224c0$@se> <50377C0D.2060905@stericsson.com> Message-ID: <20120827163337.GC4339@opensource.wolfsonmicro.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, Aug 24, 2012 at 03:05:17PM +0200, Ola Lilja wrote: > I haven't followed everything in this discussion, but what I meant in an earlier > mail was that using the device for platform-DAI (MSP_I2S) also > as platform int the DAI-link struct is not anything I can see fit (as in the > patch Lee first submitted). I'm not sure I fully understand the above - I didn't notice anything with the dai links but I have to say that I could've looked more closely as I was waiting for ARM side reviews. > I can see a few acceptable solutions: > 1) Use it as it is! (A virtual device for PCM is created in arch/arm and the > probe is invoked from the machine-driver). > 2) Since the actual platform is the DMA-block, maybe there is some way of making > the device "non-virtual" by having some sub-device in the DMA-driver that is > instead triggered in the same way as in 1). The option we're using is essentially your option 2 but doing it from the DAI driver rather than from the DMA driver (keeping things in the audio subsystem since it's an audio specific issue).