Thread (27 messages) flat view 27 messages, 3 authors, 2012-02-26

[PATCH v3 17/21] ARM: at91: merge SRAM Memory banks thanks to mirroring

From: Ryan Mallon <hidden>
Date: 2012-02-26 22:45:55
Also in: lkml

On 25/02/12 03:54, Jean-Christophe PLAGNIOL-VILLARD wrote:
On 08:47 Fri 24 Feb     , Ryan Mallon wrote:
quoted
On 24/02/12 01:26, Nicolas Ferre wrote:
quoted
From: Jean-Christophe PLAGNIOL-VILLARD <redacted>

On at91sam9260 and at91sam9g20 the SRAM banks are mirrored. We can
merge them together to be able to have bigger and continuous
internal RAM.
I think this patch needs some comments in the code. As mentioned in the
last round, this functionality is not documented, so it is not obvious
how/why this works. This is especially true for any user who has
problems with this code and then tries to look up those addresses in
their at91 manual only to find the regions are reserved.

Nicolas, are you able to give an official Atmel stance on this?
you have the ack from Nico and he send the e-mail
Sure, but it would be nice to get some official documentation on this.
Why was the feature, which seems quite useful, never documented? Is it
supported on all revisions of the chips? Are there any restrictions or
caveats to using the mirrored layout versus the separate layout?

At the very least, the code needs to be documented, since anyone using
this code is not able to look up the details in the at91 technical manuals.

~Ryan
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help