Skip to content

solutions: reference-designs: Add eval-adin1320fmcz - #554

Draft
CiprianRegus wants to merge 1 commit into
mainfrom
dev/adin1320_doc
Draft

solutions: reference-designs: Add eval-adin1320fmcz#554
CiprianRegus wants to merge 1 commit into
mainfrom
dev/adin1320_doc

Conversation

@CiprianRegus

Copy link
Copy Markdown

Add a user-guide for setting up the EVAL-ADIN1320FMCZ with Zedboard as a carrier. Also provide a few steps on how to use standard Linux networking utils (such as ip link, ethtool and phytool) to configure the ADIN1320 PHYs.

This is intended to be a replacement for https://developer.analog.com/docs/linux/latest/drivers/net-phy/adin1320.html. Currently that Linux driver user-guide contains some out off place information, because it provides setup steps for testing the EVAL-ADIN1320 on Zedboard. Thus, I'll move the setup steps in here and later change the Linux user guide to only include details on how to test the PHY from userspace.

The current guide targets a previous version of the EVAL-ADIN1320, which is now deprecated. Update the content to match EVAL-ADIN1320FMCZ Rev C.

Type

  • Documentation
  • Bug fix
  • New feature
  • Continuous integration

Checklist

Add a user-guide for setting up the EVAL-ADIN1320FMCZ with Zedboard as a
carrier. Also provide a few steps on how to use standard Linux
networking utils (such as ip link, ethtool and phytool) to configure the
ADIN1320 PHYs.

Signed-off-by: Ciprian Regus <ciprian.regus@analog.com>
@CiprianRegus
CiprianRegus marked this pull request as draft July 28, 2026 10:57
@gastmaier gastmaier added the llm review Request a review from a LLM Reviewer label Aug 3, 2026
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

LLM review

This series adds the EVAL-ADIN1320FMCZ reference design (landing page +
Linux user guide for the Zedboard carrier).

run: 30812919384

e517102 - Add eval-adin1320fmcz

Contradiction bug: index.rst states the mainline adin PHY driver
already supports the ADIN1320 ("No ADIN1320-specific driver changes are
required"), while user-guide/linux.rst correctly states mainline does
not yet support it and instructs users to build from the
release/adin1320 branch. Verified against drivers/net/phy/adin.c:
main has no PHY_ID_ADIN1320, only release/adin1320 does. The
:git-linux: link in index.rst also points at main, so it doesn't even
show ADIN1320 support. Fixed by updating the note and link to match
linux.rst and point at release/adin1320.

Stale reference: linux.rst links to the old, pre-migration
en/resources/evaluation-hardware-and-software/software/kuiper-linux.html
page via :adi:. The rest of the documentation already uses
:external+kuiper:doc:Kuiper Linux `` for this (confirmed against the
kuiper intersphinx inventory). Fixed.

Minor: the Digilent ZedBoard link carried a ?srsltid=... tracking
parameter, inconsistent with the clean URL used elsewhere in the repo. Fixed.

CI warnings

The checkpatch/case/dangling-file warnings in the CI log are all
pre-existing and unrelated to the files added by this commit (verified with
.github/scripts/case.py and .github/scripts/dangling.py). The vale
failures for phytool (real tool name), autonegotiation, and VDD casing
are consistent with existing usage elsewhere in the repo and are treated as
noise, not fixed.

Verification data

  • Fetched analogdevicesinc/linux main and release/adin1320 branches
    (API + raw content) to confirm adin.c PHY ID support, the
    zynq-zed-adin1320fmcz-rgmii.dts MDIO topology (Port 0 = GEM1/addr 0x0,
    Port 1 = GEM0/addr 0x8, matching the guide), and the
    zynq_xcomm_adv7511_defconfig build target.
  • No ADIN1320 datasheet is published yet (checked the docling markdown
    mirror and the analog.com PDF sitemap), so the "Hardware Configuration
    Pins" tables could not be cross-checked against a datasheet; the ADIN1300
    datasheet was used as a plausibility reference for the pin naming
    convention (MACIF_SELx, PHY_CFGx, MDIX_MODE, LSFLD_CFG).
  • Built the added pages in isolation with adoc serve --once -s solutions/reference-designs/eval-adin1320fmcz: builds cleanly.
  • HTTP-checked the Kuiper v2.0.0 release zip and the
    swdownloads.analog.com/cse/kuiper/adin1320/artifacts.zip download links:
    both resolve.

Suggested patches

Apply the suggested patches with:

cd path/to/repository
export GITHUB_TOKEN=ghp_***
apply-patches --repo=analogdevicesinc/documentation 30812919384
Install instructions

The following one-liner installs the script if not present already:

curl -fSsL "https://raw.githubusercontent.com/analogdevicesinc/doctools/refs/heads/main/ci/scripts/apply-patches.sh"      -o ~/.local/bin/apply-patches.sh &&   grep -q "/apply-patches.sh" ~/.bashrc || echo "source ~/.local/bin/apply-patches.sh" >> $_ ; . $_

More information at AI Usage.

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

Labels

llm review Request a review from a LLM Reviewer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants