Skip to content
Snippets Groups Projects

Resolve L2SDP-27

Merged Eric Kooistra requested to merge L2SDP-27 into master
1 unresolved thread

Closes L2SDP-27

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • 95 107 FOR I IN 0 TO g_nof_masters-1 LOOP
    96 108 IF master_mosi_arr(I).wr='1' OR master_mosi_arr(I).rd='1' THEN
    97 109 index <= I; -- index of active master
    98 110 EXIT;
    • The EXIT statement is supported by some logic synthesis tools, with certain restrictions. Have you checked if this code can be synthesized? If it cannot be synthesized, you can change the LOOP to "FOR I in g_nof_masters-1 DOWNTO 0 LOOP" to get the same behaviour.

    • Ik heb het EXIT statement in mm_master_mux en mm_bus_comb verduidelijkt, want eigenlijk zou het de EXIT zelfs weg mogen. In dp_latency_adapter gebruiken we de EXIT ook, het is dus met Quartus goed te synthetiseren, daarom is het OK denk ik om in een LOOP een EXIT te gebruiken.

    • changed this line in version 2 of the diff

    • Please register or sign in to reply
  • Eric Kooistra resolved all threads

    resolved all threads

  • Eric Kooistra added 4 commits

    added 4 commits

    • e5094cf3 - Replaced Windows LFCR by Linux LF to avoid ^R at end of line in vi (replace \r...
    • 5c2cc725 - Replaced Windows LFCR by Linux LF to avoid ^R at end of line in vi (replace \r...
    • f124d215 - Replaced Windows LFCR by Linux LF to avoid ^R at end of line in vi (replace \r...
    • 3ef797b6 - Clarified EXIT statement.

    Compare with previous version

  • mentioned in commit e6ba22e8

  • Please register or sign in to reply
    Loading