firmware: tegra: Add suspend hook and reset BPMP IPC early on resume
authorSumit Gupta <sumitg@nvidia.com>
Mon, 9 Oct 2023 10:05:56 +0000 (15:35 +0530)
committerThierry Reding <treding@nvidia.com>
Fri, 13 Oct 2023 12:20:27 +0000 (14:20 +0200)
commitea608a01d4ee66f8b51070e623f9adb8684c0dd4
tree2d96caccf3c3359fbb07230d80717962b5aa7343
parentbde3ce7257ca68120f0d8db96b3df5bc14f8245d
firmware: tegra: Add suspend hook and reset BPMP IPC early on resume

Add suspend hook and a 'suspended' field in the 'struct tegra_bpmp'
to mark if BPMP is suspended. Also, add a 'flags' field in the
'struct tegra_bpmp_message' whose 'TEGRA_BPMP_MESSAGE_RESET' bit can be
set from the Tegra MC driver to signal that the reset of BPMP IPC
channels is required before sending MRQ to the BPMP FW. Together both
the fields allow us to handle any requests that might be sent too soon
as they can cause hang during system resume.

One case where we see BPMP requests being sent before the BPMP driver
has resumed is the memory bandwidth requests which are triggered by
onlining the CPUs during system resume. The CPUs are onlined before the
BPMP has resumed and we need to reset the BPMP IPC channels to handle
these requests.

The additional check for 'flags' is done to avoid any un-intended BPMP
IPC reset if the tegra_bpmp_transfer*() API gets called during suspend
sequence after the BPMP driver is suspended.

Fixes: f41e1442ac5b ("cpufreq: tegra194: add OPP support and set bandwidth")
Co-developed-by: Thierry Reding <treding@nvidia.com>
Signed-off-by: Sumit Gupta <sumitg@nvidia.com>
Acked-by: Thierry Reding <treding@nvidia.com>
Signed-off-by: Thierry Reding <treding@nvidia.com>
drivers/firmware/tegra/bpmp.c
include/soc/tegra/bpmp.h