Skip to content

docs(cc_raspberry_pi): Better phrasing of config descriptions#6488

Merged
blackboxsw merged 1 commit into
canonical:mainfrom
paulober:cc-raspberrypi-update-serial-config
Oct 8, 2025
Merged

docs(cc_raspberry_pi): Better phrasing of config descriptions#6488
blackboxsw merged 1 commit into
canonical:mainfrom
paulober:cc-raspberrypi-update-serial-config

Conversation

@paulober
Copy link
Copy Markdown
Contributor

Proposed Commit Message

docs(cc_raspberry_pi): Better phrasing of config descriptions

Merge type

  • Squash merge using "Proposed Commit Message"
  • Rebase and merge unique commits. Requires commit messages per-commit each referencing the pull request number (#<PR_NUM>)

@tdewey-rpi @lurch

Signed-off-by: paulober <paul.oberosler@raspberrypi.com>
@github-actions github-actions Bot added the documentation This Pull Request changes documentation label Sep 25, 2025
Comment on lines +18 to +19
This example will enable the serial console on Raspberry Pi 5
while disabling the UART hardware (only Pi 5 allows this combination).
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Perhaps it's worth mentioning that (IIRC) this is because Pi 5 runs the serial console on a separate UART? (the one between the microHDMI ports)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hrm the example3 provided has the hardware: false and console: true yet the runtime code automatically resets enable_hw = True. Am I misreading the docs or the runtime behavior?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@blackboxsw You're right but it should only automatically enable hw if the platform is not a Pi 5 as the comment and the docs state:

if not is_pifive() and enable_console:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Oops ok I inverted the logic and missed the not. Looks good thank you.

@paulober paulober requested a review from blackboxsw October 7, 2025 12:05
Copy link
Copy Markdown
Collaborator

@blackboxsw blackboxsw left a comment

Choose a reason for hiding this comment

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

Thanks for the clarification @paulober LGTM

Comment on lines +18 to +19
This example will enable the serial console on Raspberry Pi 5
while disabling the UART hardware (only Pi 5 allows this combination).
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Oops ok I inverted the logic and missed the not. Looks good thank you.

@blackboxsw blackboxsw merged commit c6a1d1f into canonical:main Oct 8, 2025
21 checks passed
blackboxsw pushed a commit to blackboxsw/cloud-init that referenced this pull request Dec 12, 2025
…cal#6488)

Signed-off-by: paulober <paul.oberosler@raspberrypi.com>
holmanb pushed a commit that referenced this pull request Dec 18, 2025
Signed-off-by: paulober <paul.oberosler@raspberrypi.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation This Pull Request changes documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants