cpc / openasip

Open Application-Specific Instruction Set processor tools (OpenASIP)
http://openasip.org
Other
140 stars 42 forks source link

Mismatches between xilinx_basic_alu.vhd and generic_sru.vhd #200

Closed ISDIES closed 1 year ago

ISDIES commented 1 year ago

Hi to all! I'm pretty new at Github, so I hope this is the correct way to raise my problem. I'm trying to create a new TTA processor which features an alu implemented using xilinx_basic_alu.vhd and generic_sru.vhd (is already present in xilinx_series7.hdb). The problem is that, when it comes the time to compile the HDL (using for example Vivado), the compiler tells me that some mismatch are present between xilinx_basic_alu.vhd and generic_sru.vhd. Fixing them manually, the system is working. To solve this issue, I tried to change the source code which I have under tce-level/tce/hdb/xilinx_vhdl and generate again the processor, but mismatches still occurs. Is there a way to fix the problem? Am I missing something? Thanks

pjaaskel commented 1 year ago

@TopiLeppanen do you know what's this about?

TopiLeppanen commented 1 year ago

It seems that the xilinx_basic_alu.vhd was out-dated. Some changes made to the generic_sru.vhd component weren't propagated to xilinx_basic_alu.vhd correctly. Fixing the VHDL of xilinx_basic_alu.vhd seems to fix the problem for me, and now I can generate processors that compile at least. I haven't used the xilinx_basic_alu.vhd component myself, so I can only hope that it works. (Nobody else had probably either, since it's been broken like this years).

If I had to guess why you changing the source codes didn't work, maybe you didn't run 'make install?. You can run it directly in the hdb-folder to only install the updated hdbs and sources. The generateprocessor will then pick up the hdbs and VHDL source files from the install directory.

ISDIES commented 1 year ago

Thanks @TopiLeppanen ! I was not running the 'make install' command to update my fixes on the VHDL.