-
Notifications
You must be signed in to change notification settings - Fork 5
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
Remove ecc scrub memory labels from linker scripts. #45
base: master
Are you sure you want to change the base?
Conversation
templates/base.lds
Outdated
@@ -104,7 +104,7 @@ SECTIONS | |||
{% else %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This {% else %}
clause was technical debt I added, and forgot to remove after you had weak
-ened __metal_eccscrub_bit
. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nick changes was also cherry picked into Kiwi.
Do we need to cherry pick this into Kiwi too?
templates/base.lds
Outdated
@@ -87,11 +87,11 @@ SECTIONS | |||
* certain core features */ | |||
PROVIDE(__metal_chicken_bit = {{ chicken_bit|default(0) }}); | |||
|
|||
{% if eccscrub_en %} | |||
{#{% if eccscrub_en %}#} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if this is no longer need and already verify that it I the case, better to simply delete this,
yes. |
update the above comment and I'll approve and merge this |
acc8031
to
d5b9f39
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@nategraff-sifive I will let you decide if you want this in 20.08 or not, and merge accordingly |
I'm waiting until after the release. |
Can you add this into Kiwi, so Kiwi can collapse into Koala next Monday |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Co-authored-by: Nick Knight <nick.knight@sifive.com>
150dafd
to
b93d782
Compare
'scrub.s' no longer uses these generated labels.
Generator code is commented out, for a possible re-use should the need arise.