prepare_new_cl.sh generates broken scripts and templates.
jahagirdar opened this issue · 4 comments
The script prepare_new_cl.sh in aws-fpga/hdk/cl/developer_designs/ generates broken code and scripts.
Issues noticed so far:
- Code is generated as
design/cl_template.sv
While scripts inbuild/scripts
referencescl_hello_world
everywhere. - Script references files which are not generated e.g
design/cl_id_defines.vh
andcommon/design/cl_common_defines.vh
- The
design/cl_template.sv
throws 40+ errors during compilation. e.g.
ERROR: [Synth 8-2715] syntax error near . [/home/centos/src/project_data/aws-fpga/hdk/cl/developer_designs/vijay/build/src_post_encryption/cl_template.sv:183]
ERROR: [Synth 8-1031] cl_sh_pcis_awready is not declared [/home/centos/src/project_data/aws-fpga/hdk/cl/developer_designs/vijay/build/src_post_encryption/cl_template.sv:218]
ERROR: [Synth 8-1031] cl_sh_pcis_wready is not declared [/home/centos/src/project_data/aws-fpga/hdk/cl/developer_designs/vijay/build/src_post_encryption/cl_template.sv:220]
ERROR: [Synth 8-1751] cannot index into non-array tie_zero_id [/home/centos/src/project_data/aws-fpga/hdk/cl/developer_designs/vijay/build/src_post_encryption/cl_template.sv:291]
ERROR: [Synth 8-1751] cannot index into non-array tie_zero_id [/home/centos/src/project_data/aws-fpga/hdk/cl/developer_designs/vijay/build/src_post_encryption/cl_template.sv:292]
10 Infos, 1 Warnings, 1 Critical Warnings and 44 Errors encountered.
synth_design failed
Hello,
Thanks for reporting this issue, we're taking a look at it
Thanks!
Dear customer,
Thanks for reporting the issue. The prepare_new_cl.sh script is intended to only set up the directory structure thats similar to our cl_examples for customers to add their designs. The cl_template.sv itself is not a standalone design and is just intended as a place holder for customer design. We recommend to run one of our examples like cl_hello_world or cl_dram_dma for a quick start and add the design to cl_template when want to create custom_design and update the build scripts with corresponding source files. Please feel free to let us know if you have any follow up questions.
https://github.com/aws/aws-fpga/tree/master/hdk/cl/examples/cl_hello_world
https://github.com/aws/aws-fpga/tree/master/hdk/cl/examples/cl_dram_dma
Thanks
The minimum expectation from a template is that it should be clean start point for a new design. The current script is obliviously not a clean start-point. I ended up using a cleaned up template and folder layout from another site for my design.
In case Amazon AWS does not have the resources to fix this issue will they accept a PR?
Dear customer,
Thanks for using the F1 developer kit and providing your valuable feedback to AWS. We are continuously improving our developer kit and we currently have cl_examples that are recommended as a starting point to explore F1. In addition, we will be happy to review PR for the cl_template. Please feel free to provide any additional feedback that you have.
Thanks