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

Create a porting case study for the BCM2711/Raspberry Pi 4B #15354

Merged
merged 1 commit into from
Dec 28, 2024

Conversation

linguini1
Copy link
Contributor

Summary

This PR introduces a porting guide for the BCM2711 and Raspberry Pi 4B based off of the journal entries I made while creating the initial port in #15188.

Impact

A second porting case study is now available for people looking to port a new board or SoC to NuttX. I also created a new directory for storing the porting case studies so that they are listed under a "case studies" heading in the porting guide, making them easy to browse through.

Testing

Built the documentation locally and rendered in the browser to make sure the build passed and the webpages looked correct. Verified that links all work.

@github-actions github-actions bot added Area: Documentation Improvements or additions to documentation Size: M The size of the change in this PR is medium labels Dec 27, 2024
@nuttxpr
Copy link

nuttxpr commented Dec 27, 2024

[Experimental Bot, please feedback here]

Fill In The Commit Message: This PR contains a Commit with an Empty Commit Message. Please fill in the Commit Message with the PR Summary.

Yes, this PR appears to meet the NuttX requirements. The summary clearly explains the "why", "what", and "how" of the change. The impact section addresses all the relevant points, highlighting the addition of a new feature (the porting guide) and its positive impact on documentation. The testing section, while not showing specific logs, describes the verification process undertaken, which is appropriate for a documentation change. The absence of related issue numbers suggests this might be a new contribution without a pre-existing tracking issue.

Copy link
Contributor

@acassis acassis left a comment

Choose a reason for hiding this comment

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

Really good Matteo! Congratulations!

@acassis acassis merged commit b15cd35 into apache:master Dec 28, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Documentation Improvements or additions to documentation Size: M The size of the change in this PR is medium
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants