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

caf: power_manager: Document Kconfig changes #21143

Merged
merged 1 commit into from
Mar 27, 2025

Conversation

MarekPieta
Copy link
Contributor

Change adds documentation for introduced Kconfig configuration changes.

Jira: NCSDK-32519

@MarekPieta MarekPieta requested review from a team as code owners March 25, 2025 14:19
@github-actions github-actions bot added the doc-required PR must not be merged without tech writer approval. label Mar 25, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 25, 2025

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 6

Inputs:

Sources:

sdk-nrf: PR head: 47a23a948eb0e81dab6a960eaf6cefeaae65e120

more details

sdk-nrf:

PR head: 47a23a948eb0e81dab6a960eaf6cefeaae65e120
merge base: 1b0df5c628155be0771a1df6b1af5753bc55ea2c
target head (main): 09ecf06019029fb3d2adec087f3bf701ef4c8fda
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (2)
doc
│  ├── nrf
│  │  ├── libraries
│  │  │  ├── caf
│  │  │  │  │ power_manager.rst
│  │  ├── releases_and_maturity
│  │  │  ├── releases
│  │  │  │  │ release-notes-changelog.rst

Outputs:

Toolchain

Version:
Build docker image:

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ Integration tests
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_positioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

@MarekPieta MarekPieta requested a review from a team March 25, 2025 14:27
Copy link

@MarekPieta MarekPieta requested a review from a team March 26, 2025 06:35
@MarekPieta MarekPieta force-pushed the caf_pm_device_runtime_doc branch 2 times, most recently from 4b73c63 to 0e2caa4 Compare March 26, 2025 06:45
@MarekPieta MarekPieta requested a review from divipillai March 26, 2025 06:47
Copy link
Contributor

@zycz zycz left a comment

Choose a reason for hiding this comment

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

I would add information about CONFIG_PM_DEVICE that it is also needed for PM_DEVICE_RUNTIME and that it should be off if you don't have any devices which can use it.

@MarekPieta MarekPieta force-pushed the caf_pm_device_runtime_doc branch from 0e2caa4 to fc61155 Compare March 26, 2025 10:13
Copy link
Contributor

@bjarki-andreasen bjarki-andreasen left a comment

Choose a reason for hiding this comment

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

Looks good :)

Change adds documentation for introduced Kconfig configuration changes.

Jira: NCSDK-32519

Signed-off-by: Marek Pieta <Marek.Pieta@nordicsemi.no>
Signed-off-by: Divya Pillai <divya.pillai@nordicsemi.no>
@MarekPieta MarekPieta force-pushed the caf_pm_device_runtime_doc branch from fc61155 to 47a23a9 Compare March 26, 2025 10:49
@MarekPieta MarekPieta requested review from zycz and divipillai March 26, 2025 10:49
@kapi-no kapi-no merged commit a73c390 into nrfconnect:main Mar 27, 2025
14 checks passed
@MarekPieta MarekPieta deleted the caf_pm_device_runtime_doc branch March 27, 2025 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-required PR must not be merged without tech writer approval.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants