Skip to content
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

adrv902x: Add vck190 support #2654

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

adrv902x: Add vck190 support #2654

wants to merge 2 commits into from

Conversation

AndrDragomir
Copy link
Contributor

PR Description

Add device tree for adrv9025 with vck190
Include adrv9025 firmware in adi_versal_defconfig

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 tested the changes on the relevant hardware
  • I have updated the documentation outside this repo accordingly (if there is the case)

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.

Only comment is about the git messages.

Run git log --oneline DIR_OF_CHANGE and check the style used... For instance, for your two commits, it should be:

arm64: boot: dts: add vck190 support
arm64: configs: adi_versal_defconfig: add adrv9025 firmware

Then add a description (even if small and obvious) on the git message - at the very least to make checkpatch happy with it

New support for adrv902x with vck190

Signed-off-by: AndrDragomir <[email protected]>
Add adrv9025 firmware, needed for the new adrv9025_vck190 support

Signed-off-by: AndrDragomir <[email protected]>
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