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

drivers/devicetree/fdt: add additional fdt parsing utilities #12405

Merged
merged 1 commit into from
Jun 3, 2024

Conversation

g2gps
Copy link
Contributor

@g2gps g2gps commented May 27, 2024

Summary

arch/risc-v/litex: add some litex specific FDT helpers.

Impact

These utilities remove some of the boilerplate needed for FDT parsing and device initialization.

Testing

Litex port with device tree support. Further PRs to follow with driver support.

@g2gps
Copy link
Contributor Author

g2gps commented May 27, 2024

There's potential to move these to the generic drivers/devicetree/fdt.c. However I'm not sure how much boilerplate to contribute there.

@xiaoxiang781216
Copy link
Contributor

xiaoxiang781216 commented May 27, 2024

There's potential to move these to the generic drivers/devicetree/fdt.c. However I'm not sure how much boilerplate to contribute there.

yes, all functions aren't specific to riscv, so why not move to the common placce?

@g2gps
Copy link
Contributor Author

g2gps commented May 28, 2024

There's potential to move these to the generic drivers/devicetree/fdt.c. However I'm not sure how much boilerplate to contribute there.

yes, all functions aren't specific to riscv, so why not move to the common placce?

Ok, I'll do an update and repush.

These utilities remove some of the boilerplate needed for FDT parsing and device initialization.
@g2gps g2gps changed the title arch/risc-v/litex: add some litex specific FDT helpers. drivers/devicetree/fdt: add additional fdt parsing utilities May 30, 2024
@xiaoxiang781216 xiaoxiang781216 merged commit 39e6e25 into apache:master Jun 3, 2024
26 checks passed
@g2gps g2gps deleted the litex_fdt branch June 6, 2024 01:25
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