Skip to content

Update parameters for cn0577 and axi_ltc2387 #1870

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

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

StancaPop
Copy link
Contributor

@StancaPop StancaPop commented Aug 6, 2025

PR Description

Update the cn0577 reference design by adding a parametrizable value for the clk_gate width.
Update axi_ltc2387 and add IP GUI parameters.

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

IuliaCMoldovan and others added 7 commits July 18, 2025 11:34
- projects/cn0577: Added make parameters ADC_RES (the ADC input
  resolution) and TWOLANES (two-lane mode or one-lane mode). Also fixed
  the sysid parameters logging
- library/axi_ltc2387/axi_ltc2387_if.v: Changed the parameter name RESOLUTION
  to ADC_RES, as it was redundant

Signed-off-by: Iulia Moldovan <[email protected]>
Signed-off-by: Stanca Pop <[email protected]>
@StancaPop StancaPop marked this pull request as draft August 6, 2025 12:54
@StancaPop
Copy link
Contributor Author

To be merged after #1850

@StancaPop StancaPop requested a review from AndreiGrozav August 6, 2025 12:56
@StancaPop StancaPop changed the title Update parameters for cn0577 and axi_ltc2385 Update parameters for cn0577 and axi_ltc2387 Aug 6, 2025
StancaPop and others added 3 commits August 12, 2025 14:18
@StancaPop StancaPop force-pushed the merge_cn0577_updates branch from 31cc611 to 2774652 Compare August 12, 2025 11:18
@IuliaCMoldovan
Copy link
Contributor

Could you squash the doc commits into one?
I cherry-picked the code commits into #1850.

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.

2 participants