James h65 uart docs #592
Merged
James h65 uart docs #592
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Changed from review comments, added some extra stuff about mini-uart:
|
PR updated. More comment welcome. |
added section on disabling linux console output.
|
@pelwell Does this read OK, and can you think of anything else that ought to go in there? |
Added link in read me to the UART config
|
That looks good. We should probably mention the raspi-config utility which makes disabling the serial port and the console more user-friendly. |
Change heading sizes to all be consistent
pelwell
reviewed
Mar 23, 2017
configuration/uart.md
Outdated
| @@ -8,7 +8,7 @@ In Linux device terms, by default, /dev/ttyS0 refers to to the mini-UART, and /d | |||
|
|
|||
| ## Mini-UART and CPU Core frequency | |||
|
|
|||
| The baud rate of the Mini-UART is linked to the core frequency of the VPU on the VC4 GPU. This means that as the VPU frequency governor varies the core frequency, the baud rate of the UART also changes. This makes the UART of limited use in the default state. Also, when the linux console uses the mini-UART (Raspberry Pi 3, ZeroW), by default this is disabled for this very reason. | |||
| The baud rate of the Mini-UART is linked to the core frequency of the VPU on the VC4 GPU. This means that as the VPU frequency governor varies the core frequency, the baud rate of the UART also changes. This makes the UART of limited use in the default state. Also, when the linux console uses the mini-UART (Raspberry Pi 3, ZeroW), this varying baud rate means the console, by default, is disabled. | |||
There was a problem hiding this comment.
This section goes a bit wrong at the end because the causality gets muddled - yes, the console is disabled by default on 3/0W, but only because the UART is disabled.
|
Have made the change as per @pelwell suggestion, now pushing to copy edit. |
Minor change made. Ready for merging.
|
Files checked. Changes made. Pushed back for technical checking. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Work in progress - not yet ready for merging, just here for review.