Skip to content

Conversation

@LBFFilho
Copy link
Contributor

@LBFFilho LBFFilho commented Oct 28, 2025

PR Description

Unifies SPI Engine projects for different vendors (Xilinx & Altera for now) into a single script. It uses intel implementations of the ad_* procs to keep vendor-specific lines to a minimum

This is implemented with a separate adi_board_intel.tcl, now sourced in the carrier-specific tcl, since that's where the intel commands were defined. Note that these methods are not spi engine specific, so that's a direction we could explore in the future. We could in the future move these intel procs to adi_board.tcl and have everything in a single place.

PR Type

  • Bug fix (change that fixes an issue)
  • New feature (change that adds new functionality)
  • Breaking change (has dependencies in other repos or will cause CI to fail)
  • Documentation

PR Checklist

  • I have followed the code style guidelines
  • I have performed a self-review of changes
  • I have compiled all hdl projects and libraries affected by this PR
  • I have tested in hardware affected projects, at least on relevant boards
  • I have commented my code, at least hard-to-understand parts
  • I have signed off all commits from this PR
  • I have updated the documentation (wiki pages, ReadMe files, Copyright etc)
  • I have not introduced new Warnings/Critical Warnings on compilation
  • I have added new hdl testbenches or updated existing ones

Signed-off-by: Laez Barbosa <laez.barbosa@analog.com>
Ad intel-compatible ad_* commands to the carrier common _qsys.tcl
Now the same spi engine script is used for xilinx and intel projects.
Other vendors can be added to the script in the future.

Signed-off-by: Laez Barbosa <laez.barbosa@analog.com>
Signed-off-by: Laez Barbosa <laez.barbosa@analog.com>
@LBFFilho LBFFilho force-pushed the spi_engine_abstraction branch from eeb0677 to abf9ede Compare November 3, 2025 18:15
@LBFFilho LBFFilho marked this pull request as ready for review November 4, 2025 16:35
@LBFFilho
Copy link
Contributor Author

LBFFilho commented Nov 4, 2025

RetriggerCI

1 similar comment
@bia1708
Copy link
Collaborator

bia1708 commented Nov 7, 2025

RetriggerCI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants