On Tue, Nov 01, 2011 at 06:46:52PM +0000, Tull Alan-R80115 wrote:

This patch creates and populates a ops struct for each call to soc_pcm_new.

This isn't very idiomatic - a much more idiomatic solution would be to add new indirection functions which do the lookups. While it's probably not going to make much difference for most things there's operations like ioctl() which definitely don't look at all DMA specific in there.