Re: [PATCH v2 2/3] firmware: qcom: scm: Expose download-mode control

[Rob Herring]

On Fri, May 26, 2017 at 11:33:07PM -0700, Bjorn Andersson wrote:
> In order to aid post-mortem debugging the Qualcomm platforms provides a
> "memory download mode", where the boot loader will provide an interface
> for custom tools to "download" the content of RAM to a host machine.
> 
> The mode is triggered by writing a magic value somehwere in RAM, that is
> read in the boot code path after a warm-restart. Two mechanism for
> setting this magic value are supported in modern platforms; a direct SCM
> call to enable the mode or through a secure io write of a magic value.
> 
> In order for a normal reboot not to trigger "download mode" the magic
> must be cleared during a clean reboot.

This must be happening somewhere before the kernel is entered? Or 
warm-restarts are not the norm?

> Download mode has to be enabled by including qcom_scm.download_mode=1 on
> the command line.

This looks similar to reboot reason functionality (i.e. boot into mode 
X). I'd expect this to use that at least for the kernel. Not sure about 
bindings though.

> Signed-off-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>
> ---
> 
> Changes since v1:
> - Specify DT propert being two-cell
> - Correct handling of scm-call return code
> 
>  .../devicetree/bindings/firmware/qcom,scm.txt      |  2 +
>  drivers/firmware/qcom_scm-32.c                     |  6 +++
>  drivers/firmware/qcom_scm-64.c                     | 13 +++++++
>  drivers/firmware/qcom_scm.c                        | 43 ++++++++++++++++++++++
>  drivers/firmware/qcom_scm.h                        |  2 +
>  5 files changed, 66 insertions(+)
>