Re: [PATCH 00/13] soc: qcom: mdt_loader: Support Qualcomm SM8450

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 28/01/2022 05:55, Bjorn Andersson wrote:
The Qualcomm SM8450 platform comes with both some smaller changes in the
firmware packaging and a new requirement to hold onto the metadata buffer until
PAS auth_and_reset has been completed.

Extend the PAS api and rework the mdt_loader to meet these new requirements,
then wire this up with the PAS remoteproc driver and finally add the SM8450
remoteproc instances.

Bjorn Andersson (13):
   firmware: qcom: scm: Introduce pas_metadata context
   soc: qcom: mdt_loader: Split out split-file-loader
   soc: qcom: mdt_loader: Allow hash segment to be split out
   soc: qcom: mdt_loader: Allow hash to reside in any segment
   soc: qcom: mdt_loader: Extend check for split firmware
   soc: qcom: mdt_loader: Reorder parts of __qcom_mdt_load()
   soc: qcom: mdt_loader: Always invoke PAS mem_setup
   soc: qcom: mdt_loader: Extract PAS operations
   remoteproc: qcom: pas: Carry PAS metadata context
   dt-bindings: remoteproc: qcom: pas: Add SM8450 PAS compatibles
   remoteproc: qcom: pas: Add SM8450 remoteproc support
   arm64: dts: qcom: sm8450: Add remoteproc enablers and instances
   arm64: dts: qcom: sm8450-qrd: Enable remoteproc instances

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>

Minor nitpicks:
- I'd reorder the series by moving patch 1 (pas_metadata) closer to patch 8&9 (pas metadata usage) - I would have added pas_metadata as an argument to qcom_mdt_load(). However I see, why you didn't want to add another argument to the list.


  .../bindings/remoteproc/qcom,adsp.yaml        |  16 +
  arch/arm64/boot/dts/qcom/sm8450-qrd.dts       |  20 ++
  arch/arm64/boot/dts/qcom/sm8450.dtsi          | 297 ++++++++++++++++++
  drivers/firmware/qcom_scm.c                   |  39 ++-
  drivers/remoteproc/qcom_q6v5_mss.c            |   7 +-
  drivers/remoteproc/qcom_q6v5_pas.c            |  36 ++-
  drivers/soc/qcom/mdt_loader.c                 | 232 +++++++++-----
  include/linux/qcom_scm.h                      |  10 +-
  include/linux/soc/qcom/mdt_loader.h           |  17 +-
  9 files changed, 579 insertions(+), 95 deletions(-)



--
With best wishes
Dmitry



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux