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

reset of the error #771

Merged
merged 11 commits into from
Feb 18, 2024
Merged

reset of the error #771

merged 11 commits into from
Feb 18, 2024

Conversation

Vasco-Luz
Copy link
Contributor

made an bad error, this is a pr to revert the error

@jjts
Copy link
Contributor

jjts commented Feb 16, 2024

This PR isn't necessary , just the one with the iob_reg instead of always

@Vasco-Luz
Copy link
Contributor Author

Vasco-Luz commented Feb 16, 2024

an error is happening when .rst_i (boot_0) the system finds cpu trap condition, on sram_w_valid_reg

@Vasco-Luz
Copy link
Contributor Author

solved, syncronous reset cannot be dependent of boot_o because its because of boot_o that the output of sram_valid it becames equal to rom_valid. so it a 0 is putted in rst_i because there is no need for synchronous reset.

@Vasco-Luz
Copy link
Contributor Author

{3'b111, {(BOOTROM_ADDR_W - 2) {1'b0}}}
this line must substitute
{1'b1, {(BOOTROM_ADDR_W - 2) {1'b0}}}
in order for the sram pointer to point to the correct memory location.

@Vasco-Luz
Copy link
Contributor Author

changes, iob_reg_r to iob_reg, ((2SRAM_ADDR_W-2BOOTROM_ADDR_W)>>2) to get the righ pointer start(bits that point to the ram memory), changed so the reset of the wstrb is actually a 0.

@Vasco-Luz
Copy link
Contributor Author

(1'b1<<(SRAM_ADDR_W-2))-(1'b1<<(BOOTROM_ADDR_W-2))
changed the way it does the calcuation. to solve syntethesis errors.

.clk_i (clk_i),
.arst_i(arst_i),
.cke_i (cke_i),
.rst_i (1'b0),
.cke_i (1'b1),
Copy link
Contributor

Choose a reason for hiding this comment

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

.cke_i(cke_i)

cke_i must come all the way from the top

@Vasco-Luz
Copy link
Contributor Author

removed merge conflitcts with main

@jjts jjts merged commit e496f28 into IObundle:main Feb 18, 2024
8 checks passed
@Vasco-Luz Vasco-Luz deleted the remove_regs branch February 23, 2024 14:46
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