Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • M mlib_devel
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Graph
    • Compare revisions
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • Deployments
    • Deployments
    • Releases
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • ALPACA
  • CASPERCASPER
  • mlib_devel
  • Merge requests
  • !3

Fix 100G memory map -- removing axi4lite use of `design_name` convention

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Mitch Burnett requested to merge rfsocs/devel-m2021a into m2021a Sep 02, 2023
  • Overview 0
  • Commits 3
  • Changes 5

We have been here before, see commit 165c2c9e...

The 100G registers were not accessible because the naming convention in the yellow block did not match how the axi4lite made the nets and wired the design up. However, adopting the need for a design_name in the 100G didn't make sense as there is no real convenient way to do so.

How the axi4lite generates seems to be the larger issue and making sure that both a yellow block adding registers as part of a mempry map and the axi4lite follow the same convention is not really transparent. It is also too easy to not be consistent. For example, because sys_block uses "sys" the ports need to be added using this same name. Knowing this needs to be done is not obvious. Making adding the ports that end up being used by the registers in the axi4lite interconnect usually a "trial and error" experience. It would help if a single entity were responsible for keeping track of adding the wires where they needed to be... not exactly trivial as it would take some redesign of the axi work.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: rfsocs/devel-m2021a