Skip to content

Conversation

@machschmitt
Copy link
Contributor

Add device tree for using AD4000 on CoraZ7.

PR Description

Add a device tree for using AD4000 on CoraZ7. AD4000 eval boards interfaced the SPI host through an FMC connector. Though, there seem to be a new evaluation board for AD4000 that used a PMOD connector.

PR Type

  • Bug fix (a change that fixes an issue)
  • New feature (a change that adds new functionality)
  • Breaking change (a change that affects other repos or cause CIs to fail)

PR Checklist

  • I have conducted a self-review of my own code changes
  • I have compiled my changes, including the documentation
  • I have tested the changes on the relevant hardware
  • I have updated the documentation outside this repo accordingly
  • I have provided links for the relevant upstream lore

Copy link
Collaborator

@nunojsa nunojsa left a comment

Choose a reason for hiding this comment

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

Just one small nit... Also if you run git log --oneline --no-merges arch/arm/boot/dts (preferably on an upstream tree) you'll see slightly different style for git subjects in ARM (would be nice if we all start following the same style :)).

But, FWIW, PR accepted

};

&fpga_axi {

Copy link
Collaborator

Choose a reason for hiding this comment

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

nit. No need for the blank line

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hold on, I'll fix those.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed. Dropped blank line after &fpga_axi. Updated patch title to be similar to upstream dts title pattern.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks! Much appreciated 👍

Add device tree for using AD4000 on CoraZ7.

Signed-off-by: Marcelo Schmitt <[email protected]>
@nunojsa nunojsa merged commit 10d2a5d into analogdevicesinc:main Nov 5, 2025
27 checks passed
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