stm32: pm: Enable to driver JTAG port pns to analog when not required #63495
Conversation
This file contains 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
zephyrbot
added
platform: STM32
zephyrbot
requested review from
ABOSTM,
Desvauxm-st,
FRASTM,
gautierg-st and
GeorgeCGV
gmarull
reviewed
Oct 4, 2023
| #include <zephyr/drivers/gpio.h> | ||
| #include <zephyr/init.h> | ||
|
|
||
| #if !defined(CONFIG_DEBUG) && defined(CONFIG_PM) |
There was a problem hiding this comment.
is CONFIG_PM a strict condition here?
There was a problem hiding this comment.
If CONFIG_PM is not a condition, it would mean that you can't use gdb unless CONFIG_DEBUG is set, which is not a condition today.
I already foresee that I will get tens of questions around this..
Using CONFIG_PM, I don't break anything as gdb can't be used today with CONFIG_PM unless CONFIG_DEBUG is set.
erwango
force-pushed
the
dev/stm32_pm_jtag_analog
branch
from
5911c1d
to
45c54db
Compare
zephyrbot
added
manifest
manifest-hal_stm32
DNM
|
The following west manifest projects have been modified in this Pull Request:
Note: This message is automatically posted and updated by the Manifest GitHub Action. |
gmarull
reviewed
Oct 16, 2023
erwango
commented
Dec 5, 2023
erwango
force-pushed
the
dev/stm32_pm_jtag_analog
branch
from
45c54db
to
b74881e
Compare
erwango
force-pushed
the
dev/stm32_pm_jtag_analog
branch
from
b74881e
to
3c69c6a
Compare
OpenOCD can now be used to flash and debug nucleo_wba52cg. However it required use of STMicroelectronics OpenOCD fork. Add instructions on how to use it. Signed-off-by: Erwan Gouriou <[email protected]>
FRASTM
previously approved these changes
Jan 17, 2024
gautierg-st
requested changes
Jan 18, 2024
Comment on lines
+468
to
+471
| &debug_jtdi_pa15 &debug_jtdo_swo_pb3 | ||
| &debug_jtrst_pb4>; | ||
| pinctrl-1 = <&analog_pa13 &analog_pa14 &analog_pa15 | ||
| &analog_pb3 &analog_pb4>; |
There was a problem hiding this comment.
I'm not sure the dtsi is the best place to put that since the pins can be used for something else than debug.
For example, in the nucleo_wba55cg.dts, PA15, PB3 and PB4 are used for SPI1. Depending on the order of initialization, I think it could result in drivers not functioning.
There was a problem hiding this comment.
Right, so we should make sure this is done before configuring any other device and change priority
SW/JTAG signals description are useful when we don't need them. Signed-off-by: Erwan Gouriou <[email protected]>
At chip startup, jtag pins are configured by default to enable debug. This configuration adds consumption and when using PM profile, we can save ~40uA by resetting this configuration and setting pins to analog mode. Signed-off-by: Erwan Gouriou <[email protected]>
Provide jtag port pins description, so they can be used to be set in analog mode when not required to save power (around 40uA saved in total). Signed-off-by: Erwan Gouriou <[email protected]>
In some cases, PINCTRL_STATE_SLEEP may be required even when CONFIG_PM_DEVICE is not defined. One example is the possibility/need to put JTAG pins to analog when CONIG_PM=y and CONFIG_DEBUG=n. Signed-off-by: Erwan Gouriou <[email protected]>
erwango
force-pushed
the
dev/stm32_pm_jtag_analog
branch
from
3c69c6a
to
8fbf9bd
Compare
|
@gautierg-st PTAL |
gautierg-st
approved these changes
Jan 25, 2024
FRASTM
approved these changes
Jan 26, 2024
FRASTM
added a commit
to FRASTM/zephyr
that referenced
this pull request
Mar 28, 2024
During the migration to Hw model V2 the PR zephyrproject-rtos#63495 was not fully reported. This change is adding the support Serial Wire / JTAG port pins Signed-off-by: Francois Ramu <[email protected]>
fabiobaltieri
pushed a commit
that referenced
this pull request
Apr 1, 2024
During the migration to Hw model V2 the PR #63495 was not fully reported. This change is adding the support Serial Wire / JTAG port pins Signed-off-by: Francois Ramu <[email protected]>
Lahyllas
pushed a commit
to tsnlab/zephyr
that referenced
this pull request
May 16, 2024
During the migration to Hw model V2 the PR zephyrproject-rtos#63495 was not fully reported. This change is adding the support Serial Wire / JTAG port pins Signed-off-by: Francois Ramu <[email protected]>
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.
Configuring JTAG port pins in analog when not required (build time) can save few uA.
Requires #64109
Note:
This operation can also be done using pinctrl API, which would allow dynamic handling of the pins (JTAG function are alternate functions which can't be configured using GPIO API)
Main reason to push this PR using GPIO API is that this is my first shot at this. If everyone agrees this is good enough, this is fine with me