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

arm64: boot: xilinx: jupiter: re-arrange devicetrees #2704

Merged
merged 1 commit into from
Jan 28, 2025

Conversation

nunojsa
Copy link
Collaborator

@nunojsa nunojsa commented Jan 27, 2025

Having zynqmp-jupiter-sdr-rx2tx2.dts directly including zynqmp-jupiter-sdr.dts and then removing nodes was clumsy and a bit confusing. That also lead to having DT tags ("/dts-v1/;") included in both files which can be problematic for some build systems like petalinux. Hence, just do things properly and place the common pieces in a common .dtsi file.

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)

@nunojsa
Copy link
Collaborator Author

nunojsa commented Jan 27, 2025

v2:

  • Fix style complains.

Having zynqmp-jupiter-sdr-rx2tx2.dts directly including
zynqmp-jupiter-sdr.dts and then removing nodes was clumsy and a bit
confusing. That also lead to having DT tags ("/dts-v1/;") included in
both files which can be problematic for some build systems like
petalinux. Hence, just do things properly and place the common pieces in
a common .dtsi file.

Signed-off-by: Nuno Sá <[email protected]>
@nunojsa nunojsa force-pushed the staging/jupiter-dt-refactor branch from 166d329 to 4abf5e8 Compare January 27, 2025 13:28
@nunojsa nunojsa merged commit f1e0168 into main Jan 28, 2025
13 checks passed
@nunojsa nunojsa deleted the staging/jupiter-dt-refactor branch January 28, 2025 14:14
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