Thread (25 messages) flat view 25 messages, 3 authors, 2012-10-05

[PATCH v3 2/6] ARM: davinci: da850-dm646x: remove the SRAM_VIRT iotable entry

From: Matt Porter <hidden>
Date: 2012-10-04 12:54:29
Also in: lkml

On Thu, Oct 04, 2012 at 05:23:34PM +0530, Sekhar Nori wrote:
On 10/3/2012 8:25 PM, Matt Porter wrote:
quoted
From: Ben Gardiner <redacted>

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 <redacted>
Tested-by: Matt Porter <redacted>
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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help