tests/periph/uart_locate_pins: new test/utility app#20253
Merged
maribu merged 2 commits intoRIOT-OS:masterfrom Jan 18, 2024
Merged
tests/periph/uart_locate_pins: new test/utility app#20253maribu merged 2 commits intoRIOT-OS:masterfrom
maribu merged 2 commits intoRIOT-OS:masterfrom
Conversation
maribu
added
Type: new feature
maribu
requested review from
MichelRottleuthner,
aabadie and
leandrolanzieri
as code owners
github-actions
bot
added
Area: doc
maribu
force-pushed
the
tests/periph/uart_locate_pins
branch
2 times, most recently
from
ea5767c to
1e4932d
Compare
benpicco
reviewed
Jan 17, 2024
|
|
||
| ifneq ($(MCU),esp32) | ||
| # We only need 1 thread (+ the Idle thread on some platforms) and we really | ||
| # want this app working on all boards. |
Contributor
There was a problem hiding this comment.
I guess you could even do
DISBALE_MODULE += core_thread
Mind you that esp32 is not the only platform that uses core_idle_thread
Member
Author
There was a problem hiding this comment.
That's why it is 2 (because of the idle thread) or 3 on ESP (because of some ESP32 specific worker thread)
Contributor
There was a problem hiding this comment.
ah ok - is that also true for esp8266?
Member
Author
There was a problem hiding this comment.
I think not, but then again I'm not sure. But DISABLE_MODULE += core_thread works even better anyway.
benpicco
reviewed
Jan 17, 2024
tests/periph/uart_locate_pins/main.c
Outdated
| static char buf[32]; | ||
| size_t len = snprintf(buf, sizeof(buf), "P%u.%u / P%c%u\n", | ||
| pins[i].port_num, pins[i].pin_num, | ||
| (int)'A' + (int)pins[i].port_num, pins[i].pin_num); |
Contributor
There was a problem hiding this comment.
If you want to save ROM you could use fmt and set CONFIG_SKIP_BOOT_MSG=1
Member
Author
There was a problem hiding this comment.
I added a commit to just skip the boot message with stdio_null regardless of CONFIG_SKIP_BOOT_MSG. If the output is thrown away anyway, we can just as well safe the ROM/CPU cycles needed to prepare the output.
github-actions
bot
added
Platform: ARM
Member
Author
|
Tested with diff --git a/tests/periph/uart_locate_pins/main.c b/tests/periph/uart_locate_pins/main.c
index eeed7b9cd2..e2eb33605c 100644
--- a/tests/periph/uart_locate_pins/main.c
+++ b/tests/periph/uart_locate_pins/main.c
@@ -38,9 +38,41 @@ static const struct {
uint8_t port_num;
uint8_t pin_num;
} pins[] = {
+ {
+ .port_num = 1,
+ .pin_num = 8,
+ },
+ {
+ .port_num = 1,
+ .pin_num = 7,
+ },
+ {
+ .port_num = 1,
+ .pin_num = 6,
+ },
+ {
+ .port_num = 1,
+ .pin_num = 5,
+ },
+ {
+ .port_num = 1,
+ .pin_num = 4,
+ },
+ {
+ .port_num = 1,
+ .pin_num = 3,
+ },
+ {
+ .port_num = 0,
+ .pin_num = 27,
+ },
+ {
+ .port_num = 0,
+ .pin_num = 26,
+ },
{
.port_num = 0,
- .pin_num = 0,
+ .pin_num = 2,
},
};
and got: |
This application uses `soft_uart` to bit-bang the name of a number of configured GPIO pins via said pins at 9600 Bd. This way attaching an USB UART bridge to one pin at a time easily reveals which MCU GPIO pin a given pin on a board corresponds to. This is useful when no schematic and no silkscreen labeling is available, or when the information is misleading or outright incorrect (looking at the E180-ZG120B-TB).
With `stdio_null` no one is reading the boot message anyway, so let's safe some ROM.
maribu
force-pushed
the
tests/periph/uart_locate_pins
branch
from
35b1a6c to
b17b2b0
Compare
github-actions
bot
removed
Platform: ARM
Member
Author
|
Sorry, an unrelated commit sneaked in. I had to force push to get rid of it |
benpicco
approved these changes
Jan 18, 2024
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Contribution description
This application uses
soft_uartto bit-bang the name of a number of configured GPIO pins via said pins at 9600 Bd. This way attaching an USB UART bridge to one pin at a time easily reveals which MCU GPIO pin a given pin on a board corresponds to. This is useful when no schematic and no silkscreen labeling is available, or when the information is misleading or outright incorrect (looking at the E180-ZG120B-TB).Testing procedure
main.cto contain some pins to detect where they are routed to on the board.Issues/PRs references
None