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.

Sign up for GitHub

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

Jump to bottom

zephyr: decouple from platform pm_runtime.h interface #9495

Merged
merged 5 commits into from
Sep 24, 2024
Merged

zephyr: decouple from platform pm_runtime.h interface #9495

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
a471a85
platform: intel: cavs: remove runtime_pm core/mem init
kv2019i Sep 19, 2024
9dd8e54
platform: intel: cavs: remove calls to platform_pm_runtime_get
kv2019i Sep 19, 2024
5e72903
zephyr: decouple from platform pm_runtime.h interface
kv2019i Sep 19, 2024
8797d37
platform: intel: remove pm_runtime.h
kv2019i Sep 19, 2024
8114e4f
platform: imx: remove pm_runtime.h
LaurentiuM1234 Sep 20, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 0 additions & 15 deletions src/platform/intel/cavs/platform.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@
#include <sof/lib/memory.h>
#include <sof/lib/mm_heap.h>
#include <sof/lib/notifier.h>
#include <sof/lib/pm_runtime.h>
#include <rtos/wait.h>
#include <sof/platform.h>
#include <sof/schedule/edf_schedule.h>
Expand Down Expand Up @@ -96,11 +95,6 @@ int platform_boot_complete(uint32_t boot_message)
{
struct ipc_cmd_hdr header;

#if CONFIG_TIGERLAKE
/* TGL specific HW recommended flow */
pm_runtime_get(PM_RUNTIME_DSP, PWRD_BY_HPRO | (CONFIG_CORE_COUNT - 1));
#endif

mailbox_dspbox_write(0, &ready, sizeof(ready));

/* get any IPC specific boot message and optional data */
Expand Down Expand Up @@ -139,15 +133,6 @@ int platform_init(struct sof *sof)
/* Set CPU to max frequency for booting (single shim_write below) */
trace_point(TRACE_BOOT_PLATFORM_CPU_FREQ);

#if CONFIG_TIGERLAKE
/* prevent DSP Common power gating */
pm_runtime_get(PM_RUNTIME_DSP, PLATFORM_PRIMARY_CORE_ID);

#if CONFIG_DSP_RESIDENCY_COUNTERS
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think this configuration option is now unused and can be removed?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@lyakh It's still implemented in XTOS pm_runtime.c and would be removing a feature from XTOS builds. So keeping in at least for now.

init_dsp_r_state(r0_r_state);
#endif /* CONFIG_DSP_RESIDENCY_COUNTERS */
#endif /* CONFIG_TIGERLAKE */

/* init DMACs */
trace_point(TRACE_BOOT_PLATFORM_DMA);
ret = dmac_init(sof);
Expand Down