Skip to content

Use TinyGo __app_ram_base#3

Closed
aykevl wants to merge 1 commit into
masterfrom
app-ram-base
Closed

Use TinyGo __app_ram_base#3
aykevl wants to merge 1 commit into
masterfrom
app-ram-base

Conversation

@aykevl

@aykevl aykevl commented May 24, 2020

Copy link
Copy Markdown
Member

This makes sure the SoftDevice start address is configured in only one
place, and thus avoid potential issues when it is changed. It also will
allow setting a different SoftDevice RAM size with a linker flag, in
LLVM 11 (when expressions are allowed in the MEMORY part of a linker
script).

@aykevl aykevl force-pushed the master branch 6 times, most recently from 3a84858 to c26709f Compare May 30, 2020 21:43
This makes sure the SoftDevice start address is configured in only one
place, and thus avoid potential issues when it is changed. It also will
allow setting a different SoftDevice RAM size with a linker flag, in
LLVM 11 (when expressions are allowed in the MEMORY part of a linker
script).
@deadprogram

Copy link
Copy Markdown
Member

Looks like this branch needs merge conflict resolution, please.

@aykevl

aykevl commented Jun 2, 2020

Copy link
Copy Markdown
Member Author

Rereading the code I saw a bug, will fix it eventually (there isn't any hurry, and it'll break the last stable TinyGo release).

@aykevl aykevl marked this pull request as draft June 2, 2020 00:02
@deadprogram

Copy link
Copy Markdown
Member

Is this PR still valid?

@aykevl

aykevl commented Apr 14, 2021

Copy link
Copy Markdown
Member Author

This change is included in #61, so closing.

@aykevl aykevl closed this Apr 14, 2021
@aykevl aykevl deleted the app-ram-base branch April 14, 2021 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants