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.

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

Add the support for changing bootrom #28

Merged
merged 2 commits into from
Sep 3, 2024
Merged

Conversation

Konste11ation
Copy link

This PR adds the support for changing bootrom address by boot_mode_i at the chip level

  1. Modify the occamy_soc_ctrl.sv: Add the boot_addr signals in occamy_soc_ctrl.sv.

    // boot address
    logic [31:0] boot_addr_d, boot_addr_q;
    logic [31:0] boot_addr_init;
    logic [1:0] boot_mode;
    assign boot_mode = hw2reg_i.boot_mode.d;

always_comb begin
boot_addr_init = (boot_mode == 2'b00)? ${default_boot_addr}:${backup_boot_addr};
boot_addr_d = (boot_mode == 2'b00)? ${default_boot_addr}:${backup_boot_addr};
boot_addr_o = boot_addr_q;
end
The ${backup_boot_addr} is defined in occamy_cfg and the ${default_boot_addr} is set to the starting address of from.
2. Do some clean up on the hbm xbar
3. Do some clean up on the occamy gen flag during rtl generation
4. Do some clean up on the bender file

1. Add the boot_addr in the occamy_soc_ctrl.sv. Now the boot_addr can be configured by boot_mode_i at chip level
2. Add the backup_boot_addr at the occamy_cfg
3. Do some clean up on the hbm xbar
4. Do some clean up on the occamygen flag
Copy link

@IveanEx IveanEx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job! A small suggestion: As there is AddrWidth argument, can you also define all boot_addr_i at the same width except the one goes into quadrant? This will be beneficial for our later system!

@IveanEx IveanEx self-requested a review September 3, 2024 12:03
@Konste11ation Konste11ation merged commit f767a8d into main Sep 3, 2024
2 checks passed
@Konste11ation Konste11ation deleted the fanchen/change-bootrom branch September 3, 2024 12:33
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