-
Notifications
You must be signed in to change notification settings - Fork 8.2k
boards: mps2_an521: clean up memory map #52249
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
boards: mps2_an521: clean up memory map #52249
Conversation
I think you're looking for @microbuilder sorry (terrible name from a company I ran lifetimes ago). I'm sure someone is very confused about who you are and what you're tagging them for. 😉 |
4573d58 to
d679480
Compare
|
@microbuilder my mistake on the username :). Sorry for confusion; I was on autopilot and used your discord handle. Suggestions applied, thanks. I expect I'll have a system DT demo for this board by end of day. |
d679480 to
ad4310e
Compare
|
This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time. |
ad4310e to
2399084
Compare
|
Rebased with no changes to see if twister passes this time |
Fix comments in board DTS files referring to AN521 tables defining memory areas, and choose node label names that more accurately reflect the entries of interest in those tables. Adjust the one in-tree user of the affected node labels. Signed-off-by: Martí Bolívar <[email protected]>
2399084 to
06d8aab
Compare
Fix comments in board DTS files referring to AN521 tables defining memory areas, and choose node label names that more accurately reflect the entries of interest in those tables.
@microbuilder this reflects the first couple of things we talked about. I thought about how to plumb the cpu1 start address from DT all the way through, but I think that's going to require system devicetree. It's an interesting use case that I hadn't considered, though, and will definitely affect the design.