Skip to content

Conversation

@kstribrnAmzn
Copy link
Member

@kstribrnAmzn kstribrnAmzn commented Nov 21, 2022

Description

Adds RISC-V 32-bit Embedded extension support. The Embedded or "E" extension proposes a reduced register set (down to 16 from 32) to prioritize interrupt and contact switching speed [1].

Test Steps

Flashed to SiFive HiFive1 Rev B with JTAG header as covered in this page. Note - I did need to have the HiFive board plugged in over USB micro as well.

Blinky demo executed without issue.

Related Issue

None

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

kstribrnAmzn and others added 3 commits November 17, 2022 11:03
Adds RV32E support to the IAR port. This is done by
reducing our register udsage to the first 16 registers
only.
@kstribrnAmzn kstribrnAmzn requested a review from a team as a code owner November 21, 2022 22:49
@codecov
Copy link

codecov bot commented Nov 21, 2022

Codecov Report

Base: 94.30% // Head: 94.30% // No change to project coverage 👍

Coverage data is based on head (63082d0) compared to base (b213ad8).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #588   +/-   ##
=======================================
  Coverage   94.30%   94.30%           
=======================================
  Files           6        6           
  Lines        2370     2370           
  Branches      579      579           
=======================================
  Hits         2235     2235           
  Misses         85       85           
  Partials       50       50           
Flag Coverage Δ
unittests 94.30% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

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.

3 participants