On Sat, Sep 22, 2012 at 21:03:14, Mark Brown wrote:
On Fri, Aug 31, 2012 at 06:20:58PM +0530, Hebbar, Gururaja wrote:
+config SND_DAVINCI_HAVE_SRAM
- bool
- default y if ARCH_DAVINCI=y
- default n if ARCH_OMAP=y
I've been sitting on this mostly since it seems like a step back from multi-platform kernels (which is where we're trying to get to) and I've been trying to decide what the best approach is. I'm thinking that we do want a generic API for allocating this stuff, it's a fairly generic feature (there's TCMs as well).
Adding ifdefs like this does just doesn't seem good.
I have no knowledge about multi-platform builds yet.
Currently this driver is shared between OMAP & DaVinci and both of them doesn't exist in single image build.
There was a effort done for this SRAM Consolidation [1] but it didn't progress. Hence I took this approach as a time-being/workaround. This was we can get affected platforms (like AM335x) get going/working.
[1]. http://patchwork.ozlabs.org/patch/104059/
Regards, Gururaja