Skip to content

Conversation

jvelasc2
Copy link
Contributor

@jvelasc2 jvelasc2 commented Oct 2, 2025

projects: AD7616-SDZ: Added readme doc

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)

PR Checklist

  • I have followed the Coding style guidelines
  • I have complied with the Submission Checklist
  • I have performed a self-review of the changes
  • I have commented my code, at least hard-to-understand parts
  • I have build all projects affected by the changes in this PR
  • I have tested in hardware affected projects, at the relevant boards
  • I have signed off all commits from this PR
  • I have updated the documentation (wiki pages, ReadMe etc), if applies


======================= ================ =====================
EVAL‑AD7616SDZ (via J5) Function ZedBoard (FMC LAxx)
SCLK SPI Clock FMC LA0 (example)

Choose a reason for hiding this comment

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

not familiar with the project itself but why is this written with an "(example)"
does it mean FMC LA0 isn't the actual pin and can be changed? but it says we can use an SDP-I-FMC
so the pin placement should be final

DOUT0 / DOUT1 (MISO) Serial Out FMC LA2
CNVST Start Conversion FMC LA3
BUSY Conversion Busy FMC LA4
RESET Reset ADC FMC LA5
Copy link

@actorreno actorreno Oct 10, 2025

Choose a reason for hiding this comment

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

are these differential pins from ad7616sdz? if not might need to be more specific which FMC wires.
FMC LA2 is more like "LA02_P and LA02_N"

@jvelasc2 jvelasc2 force-pushed the ad7616sdz-proj-readme branch from d9c4c79 to 477823a Compare October 14, 2025 06:05
@jvelasc2 jvelasc2 force-pushed the ad7616sdz-proj-readme branch from 477823a to 4a628ac Compare October 14, 2025 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants