On Thu, Oct 04, 2012 at 05:23:34PM +0530, Sekhar Nori wrote: > On 10/3/2012 8:25 PM, Matt Porter wrote: > > From: Ben Gardiner <[email protected]> > > > > The sram regions defined for da850-dm646x in their iotable entries are also > > defined in their davinci_soc_info's. > > > > Remove this duplicate information which is now uneccessary since sram > > init will ioremap the regions defined by their davinci_soc_info's. > > > > Since this removal completely removes all uses of SRAM_VIRT, also remove > > the SRAM_VIRT definition. > > > > Signed-off-by: Ben Gardiner <[email protected]> > > Tested-by: Matt Porter <[email protected]> > > What testing was done with this patch? Can you please add that > information to the commit text as well.
I have only tested on AM180x with uio_pruss and davinci-pcm (series for that posted a short while ago). In the uio_pruss case, I run the sample applications from the userspace PRU package that test communication with the shared SRAM, DDR, and internal PRU SRAM. For davinci-pcm, my series enables shared SRAM ping-pong buffering which is working comparably to pcm capture/playback without ping-pong buffering enabled on AM180x. I have not tested PM at all...yet. I'll add this detail to the commit. -Matt _______________________________________________ Davinci-linux-open-source mailing list [email protected] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
