Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Optimize function layout in hps_accel project #323

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Optimize function layout in hps_accel project #323

wants to merge 3 commits into from

Commits on Oct 12, 2021

  1. Use custom linker script in hps_accel project 

    To avoid the possibility of the LoadInput function conflicting with
ConvPerChannel4x4 (its caller) in the instruction cache, lay them out
sequentially in the final binary.

Signed-off-by: Jakub Piecuch <[email protected]>
    @j-piecuch
    j-piecuch committed Oct 12, 2021
    Configuration menu
    Copy the full SHA
    fba55a6 View commit details
    Browse the repository at this point in the history

  2. hps_accel: include proj.mk before checking $(TARGET) 

    If we don't supply an explicit TARGET to make, it should be set to the default
value in proj.mk, which is digilent_arty. However, this assignment of the
default value is currently done _after_ the ifeq in hps_accel/Makefile, so the
clock frequency isn't reduced to 75MHz even though it should.

Signed-off-by: Jakub Piecuch <[email protected]>
    @j-piecuch
    j-piecuch committed Oct 12, 2021
    Configuration menu
    Copy the full SHA
    a1300c6 View commit details
    Browse the repository at this point in the history

  3. Add an explanatory comment to hps_accel/linker.ld 

    Signed-off-by: Jakub Piecuch <[email protected]>
    @j-piecuch
    j-piecuch committed Oct 12, 2021
    Configuration menu
    Copy the full SHA
    45ec30f View commit details
    Browse the repository at this point in the history