Skip to content

projects/ad4630_fmc: Update READMEs #1710

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

projects/ad4630_fmc: Update READMEs #1710

wants to merge 2 commits into from
+122 −35

Conversation

cristianmihaipopa
Copy link
Collaborator

@cristianmihaipopa cristianmihaipopa commented May 9, 2025 •
edited
Loading

PR Description

Updated AD4630 READMEs & fixed a link from documentation.

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

StancaPop
StancaPop requested changes May 9, 2025
View reviewed changes
IuliaCMoldovan
IuliaCMoldovan requested changes May 19, 2025
View reviewed changes
@IuliaCMoldovan
Copy link
Contributor

Should split the commit in 2, one for Adding READMEs and one for updating the link in the documentation

@cristianmihaipopa
projects/ad4630_fmc: Updated README files
0e2794c 
Signed-off-by: Cristian Mihai Popa <[email protected]>
@cristianmihaipopa
docs/projects/ad4630_fmc: Fixed Linux driver link
8bdf026 
- Fixed a link in AD4630 doc. that wasn't
redirecting to the Linux driver

Signed-off-by: Cristian Mihai Popa <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ladace ladace Awaiting requested review from ladace ladace is a code owner

@sarpadi sarpadi Awaiting requested review from sarpadi sarpadi is a code owner

@gastmaier gastmaier Awaiting requested review from gastmaier gastmaier is a code owner

@LBFFilho LBFFilho Awaiting requested review from LBFFilho LBFFilho is a code owner

@StancaPop StancaPop Awaiting requested review from StancaPop StancaPop is a code owner

@IuliaCMoldovan IuliaCMoldovan Awaiting requested review from IuliaCMoldovan IuliaCMoldovan is a code owner

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
doc
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cristianmihaipopa @IuliaCMoldovan @StancaPop