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

"eFPGA_top" parameters for larger fabric #142

Open
anudeepdharavathu opened this issue Dec 10, 2023 · 4 comments
Open

"eFPGA_top" parameters for larger fabric #142

anudeepdharavathu opened this issue Dec 10, 2023 · 4 comments

Comments

@anudeepdharavathu
Copy link

eFPGA_top #(
.include_eFPGA(1),
.NumberOfRows(ROW_WIDTH),
.NumberOfCols(COL_WIDTH),
.FrameBitsPerRow(32),
.MaxFramesPerCol(20),
.desync_flag(20),
.FrameSelectWidth(5),
.RowSelectWidth(5)
)

which of these parameters need to be changed for larger fabrics say 256 IO tiles?
the default parameters do not seem to work.
I changed ROW_WIDTH but what about FrameSelectWidth and RowSelectWidth?
I don't see any documentation on them.
@KelvinChung2000
Copy link
Collaborator

The parameterisation of the eFPGA is not from the Verilog code itself but from the CSV file.

@anudeepdharavathu
Copy link
Author

FrameSelectWidth and RowSelectWidth parameters are not available in the .csv file.

when I generate large fabric, it doesn't work. Reducing the size works fine so I am requesting the procedure to optimize parameters for larger fabric.
I have only increased the row size to accommodate more IO tiles.
Do I need to change anything else other than ROW_WIDTH, BITSTREAM length, I/O register width in the test bench, top_wrapper.v file ?

@KelvinChung2000
Copy link
Collaborator

I have looked into the generator. The FrameSelectWidth and RowSelectWidth are not parsed. This will require a fix in the CSV parser. For now, you will need to change the FrameSelectWidth and RowSelectWidth manually in the Verilog.

@anudeepdharavathu
Copy link
Author

can you please explain how would these values change with increase in fabric size?

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

No branches or pull requests

2 participants