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 <bengardi...@nanometrics.ca> > > > > 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 <bengardi...@nanometrics.ca> > > Tested-by: Matt Porter <mpor...@ti.com> > > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
