Skip to content

CNN design added#18

Open
sfmth wants to merge 6 commits intoVLSIDA:mainfrom
sfmth:main
Open

CNN design added#18
sfmth wants to merge 6 commits intoVLSIDA:mainfrom
sfmth:main

Conversation

@sfmth
Copy link

@sfmth sfmth commented Jan 27, 2026

No description provided.

@bgoldbug
Copy link
Collaborator

The following changes are needed:

  • Add LICENSE file to src/cnn
  • Create a verilog.mk in src/cnn to allow for both make and make dev options
  • Add a pre-generated cnn verilog file to src/cnn
  • Remove the benchmark porting instructions pdf in the designs/ dir

@sfmth
Copy link
Author

sfmth commented Feb 4, 2026

Requested changes made

Copy link
Collaborator

@bgoldbug bgoldbug left a comment

Choose a reason for hiding this comment

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

Requested changes to config.mk and verilog.mk

Copy link
Collaborator

Choose a reason for hiding this comment

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

All DESIGN_HOME vars need to be replaced with BENCH_DESIGN_HOME

Copy link
Collaborator

Choose a reason for hiding this comment

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

By default, we want the cnn.v file to exist outside of the dev directory, such that it's available for users without needing to run make dev initially.

  • When I try to run make as the first command, it says: cnn.v is missing; using dev RTL. Run 'make dev' to regenerate and promote.
  • It should instead use an existing cnn.v (existing without runningmake dev)

@sfmth
Copy link
Author

sfmth commented Feb 12, 2026

This should fix those problems

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