mbox series

[v3,0/4] qcom_scm: Add support for MC boot address API

Message ID 20211201130505.257379-1-stephan@gerhold.net
Headers show
Series qcom_scm: Add support for MC boot address API | expand

Message

Stephan Gerhold Dec. 1, 2021, 1:05 p.m. UTC 
The "firmware: qcom: scm: Add support for MC boot address API" commit
was reverted again in 5.16 [1]. This is a new attempt to add it back
with much less potential build problems.

For that I first simplify the existing qcom_scm_set_cold/warm_boot_addr()
implementations. The idea is that cpu_logical_map(), MPIDR_AFFINITY_LEVEL()
etc are not needed if we just set the entry address for all CPUs.
Nothing in the mainline tree actually requires setting a different entry
address for one particular CPU and I cannot really think of a use case for this.

[1]: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=7db2bc925e4642df6255612e5521af0423ada18a

Stephan Gerhold (4):
  cpuidle: qcom-spm: Check if any CPU is managed by SPM
  firmware: qcom: scm: Simplify set_cold/warm_boot_addr()
  firmware: qcom: scm: Drop cpumask parameter from set_boot_addr()
  firmware: qcom: scm: Add support for MC boot address API

 arch/arm/mach-qcom/platsmp.c       |   3 +-
 drivers/cpuidle/cpuidle-qcom-spm.c |  28 +++++-
 drivers/firmware/qcom_scm.c        | 132 ++++++++++++-----------------
 drivers/firmware/qcom_scm.h        |   5 ++
 include/linux/qcom_scm.h           |   4 +-
 5 files changed, 84 insertions(+), 88 deletions(-)

Comments

Bjorn Andersson Feb. 4, 2022, 6:35 p.m. UTC | #1 
On Wed, 1 Dec 2021 14:05:01 +0100, Stephan Gerhold wrote:
> The "firmware: qcom: scm: Add support for MC boot address API" commit
> was reverted again in 5.16 [1]. This is a new attempt to add it back
> with much less potential build problems.
> 
> For that I first simplify the existing qcom_scm_set_cold/warm_boot_addr()
> implementations. The idea is that cpu_logical_map(), MPIDR_AFFINITY_LEVEL()
> etc are not needed if we just set the entry address for all CPUs.
> Nothing in the mainline tree actually requires setting a different entry
> address for one particular CPU and I cannot really think of a use case for this.
> 
> [...]

Applied, thanks!

[1/4] cpuidle: qcom-spm: Check if any CPU is managed by SPM
      commit: 0ee30ace67e425ab83a1673bf51f50b577328cf9
[2/4] firmware: qcom: scm: Simplify set_cold/warm_boot_addr()
      commit: 7734c4b507cefbcf2f7a2a806e79c43e52528c5f
[3/4] firmware: qcom: scm: Drop cpumask parameter from set_boot_addr()
      commit: 52beb1fc237d67cdc64277dc90047767a6fc52d7
[4/4] firmware: qcom: scm: Add support for MC boot address API
      commit: f60a317bcbea5c5b8923d6de6c7288850fdd83fb

Best regards,