Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Major updates towards v1.3.0 #2

Open
wants to merge 50 commits into
base: Vivado
Choose a base branch
from
Open

Conversation

stefanunrein
Copy link

@stefanunrein stefanunrein commented Mar 5, 2025

New Features

  • Support more boards and devices in config package.
  • Added new helper functions.
  • Added new vector and record types.
  • Added new feature to select the RAM type for ocram_sdp.
  • Added new feature for selecting set_false_path or set_max_delay for synchronizers.
  • Added new feature for selecting registered output for synchronizers.

Braking Changes

Changes

Bug Fixes

  • Many bugfixes.
  • Fixed bug in cache_tagunit_seq when FA_CHUNKS = 1.
  • Fixed bug in sync_Bit_Xilinx not obeying to SYNC_DEPTH.

Documentation

  • Documentation is now licensed under CC-BY-4.0.
    A documentation license counter part to Apache License 2.0 for the code.
  • New documentation pipeline running on GitHub Actions (Setup GitHub Action Pipeline using GHDL and Sphinx #7)
  • Reworked landing page
    • Updated shields.
  • Updated URLs to use github.com/VHDL/PoC
  • Fixed cross-references.
  • Updated link to OSVVM as well as license information of OSVVM (now Apache License 2.0 - like PoC).
  • Removed Xilinx ISE from documentation.
  • Added caution boxes to indicate potentially outdated documentation.
    This will be checked and updated in the upcoming v1.x releases.
  • Added PLC2 as a user of PoC.
  • Updated documentation to Sphinx v8.3
    • Removed sphinx_rtd_theme submodule.
    • Use latest sphinx_rtd_theme plus custom CSS enhancements.
    • Updated Sphinx extensions.
  • Added PoC logo and icon.

Tests

Related Issues and PullRequests

@stefanunrein stefanunrein changed the base branch from master to Vivado March 5, 2025 23:29
@Paebbels Paebbels assigned Paebbels and stefanunrein and unassigned Paebbels Mar 6, 2025
@Paebbels Paebbels self-requested a review March 6, 2025 00:00
@Paebbels Paebbels added the enhancement New feature or request label Mar 6, 2025
@Paebbels Paebbels added this to the v1.3 milestone Mar 6, 2025
@Paebbels Paebbels changed the title Plc2/release Major updates towards v1.3.0 Mar 6, 2025
}
if {[DirectoryExists AXI4]} {
include ./OSVVM-AXI4/AXI4.pro
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OSVVM-UART is missing, but included as a submodule.

regression.tcl Outdated
Comment on lines 24 to 27
build ../lib/osvvm/osvvm.pro
build ../lib/OSVVM-Common/Common.pro
build ../lib/OSVVM-AXI4/AXI4.pro
build ../lib/OSVVM-UART/UART.pro
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
build ../lib/osvvm/osvvm.pro
build ../lib/OSVVM-Common/Common.pro
build ../lib/OSVVM-AXI4/AXI4.pro
build ../lib/OSVVM-UART/UART.pro
build ../../lib/osvvm/OsvvmLibraries.pro

regression.tcl Outdated
Comment on lines 41 to 42
analyze ../tb/common/my_project.vhdl
analyze ../tb/common/my_config_GENERIC.vhdl
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
analyze ../tb/common/my_project.vhdl
analyze ../tb/common/my_config_GENERIC.vhdl
analyze ../../tb/common/my_project.vhdl
analyze ../../tb/common/my_config_GENERIC.vhdl

analyze ../tb/common/my_project.vhdl
analyze ../tb/common/my_config_GENERIC.vhdl

build ../src/PoC.pro
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
build ../src/PoC.pro
build ../../src/PoC.pro


SetSaveWaves true

build ../tb/RunAllTests.pro
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
build ../tb/RunAllTests.pro
build ../../tb/RunAllTests.pro

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants