Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Sign in
P
Platform-independent core collection
  • Project
    • Project
    • Details
    • Activity
    • Cycle Analytics
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
  • Issues 10
    • Issues 10
    • List
    • Board
    • Labels
    • Milestones
  • Merge Requests 4
    • Merge Requests 4
  • Wiki
    • Wiki
  • image/svg+xml
    Discourse
    • Discourse
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Commits
  • Issue Boards
  • Projects
  • Platform-independent core collection
  • Merge Requests
  • !18

Closed
Opened Jun 27, 2022 by David Belohrad@mistrjoda
  • Report abuse
Report abuse

Making libs more versatile for Altera/Modelsim

This merge brings 3 changes:

  1. pcie_wb does not compile under mentor modelsim as default case is missing -> fixed
  2. spi master file is written twice in the manifest
  3. CONDITIONAL INSERTION OF XILINX ONLY MODULES INTO MANIFEST

As what concerns 3 there are few library components, which are bound to xilinx platform libraries. Under altera those do not exist and fail to resolve. Following patch conditionally inserts those library modules into the manifests only if target platform is xilinx. In all other cases it just throws info message that libs for the user-selected platform are not supported.

Check out, review, and merge locally

Step 1. Fetch and check out the branch for this merge request

git fetch origin
git checkout -b altera_works origin/altera_works

Step 2. Review the changes locally

Step 3. Merge the branch and fix any conflicts that come up

git fetch origin
git checkout origin/proposed_master
git merge --no-ff altera_works

Step 4. Push the result of the merge to GitLab

git push origin proposed_master

Note that pushing to GitLab requires write access to this repository.

Tip: You can also checkout merge requests locally by following these guidelines.

  • Discussion 1
  • Commits 16
  • Changes 8
Tristan Gingold
Assignee
Tristan Gingold @tgingold
Assign to
None
Milestone
None
Assign milestone
Time tracking
0
Labels
None
Assign labels
  • View project labels
Reference: project/general-cores!18