mbox series

[0/2] Add the download mode support for IPQ5018

Message ID 20250512-ipq5018-syscon-v1-0-eb1ad2414c3c@outlook.com
Headers show
Series Add the download mode support for IPQ5018 | expand

Message

George Moussalem via B4 Relay May 12, 2025, 2:36 p.m. UTC
Enable support for download mode to collect the RAM dumps in case of
system crashes, to perform post mortem analysis.

Signed-off-by: George Moussalem <george.moussalem@outlook.com>
---
George Moussalem (2):
      dt-bindings: mfd: qcom,tcsr: Add compatible for ipq5018
      arm64: dts: qcom: ipq5018: enable the download mode support

 Documentation/devicetree/bindings/mfd/qcom,tcsr.yaml | 1 +
 arch/arm64/boot/dts/qcom/ipq5018.dtsi                | 6 ++++++
 2 files changed, 7 insertions(+)
---
base-commit: edef457004774e598fc4c1b7d1d4f0bcd9d0bb30
change-id: 20250512-ipq5018-syscon-1e349e0b224f

Best regards,

Comments

Konrad Dybcio May 12, 2025, 2:43 p.m. UTC | #1
On 5/12/25 4:36 PM, George Moussalem via B4 Relay wrote:
> From: George Moussalem <george.moussalem@outlook.com>
> 
> Enable support for download mode to collect RAM dumps in case of a
> system crash, allowing post mortem analysis.
> 
> Signed-off-by: George Moussalem <george.moussalem@outlook.com>
> ---

Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>

Konrad
Bjorn Andersson May 14, 2025, 9:03 p.m. UTC | #2
On Mon, 12 May 2025 18:36:45 +0400, George Moussalem wrote:
> Enable support for download mode to collect the RAM dumps in case of
> system crashes, to perform post mortem analysis.
> 
> 

Applied, thanks!

[2/2] arm64: dts: qcom: ipq5018: enable the download mode support
      commit: 43fefd6c71291b5793e7c4052b6e3e54d1d87715

Best regards,
Rob Herring May 14, 2025, 9:54 p.m. UTC | #3
On Wed, May 14, 2025 at 10:03:45PM +0100, Bjorn Andersson wrote:
> 
> On Mon, 12 May 2025 18:36:45 +0400, George Moussalem wrote:
> > Enable support for download mode to collect the RAM dumps in case of
> > system crashes, to perform post mortem analysis.
> > 
> > 
> 
> Applied, thanks!
> 
> [2/2] arm64: dts: qcom: ipq5018: enable the download mode support
>       commit: 43fefd6c71291b5793e7c4052b6e3e54d1d87715

You should really either wait a bit for the binding to be applied or 
just take the binding patch when it is trivial. Then we're not getting 
temporary warnings added which seems to happen a lot on QCom platforms.

Rob
Lee Jones May 15, 2025, 7 a.m. UTC | #4
On Wed, 14 May 2025, Rob Herring wrote:

> On Wed, May 14, 2025 at 10:03:45PM +0100, Bjorn Andersson wrote:
> > 
> > On Mon, 12 May 2025 18:36:45 +0400, George Moussalem wrote:
> > > Enable support for download mode to collect the RAM dumps in case of
> > > system crashes, to perform post mortem analysis.
> > > 
> > > 
> > 
> > Applied, thanks!
> > 
> > [2/2] arm64: dts: qcom: ipq5018: enable the download mode support
> >       commit: 43fefd6c71291b5793e7c4052b6e3e54d1d87715
> 
> You should really either wait a bit for the binding to be applied or 
> just take the binding patch when it is trivial. Then we're not getting 
> temporary warnings added which seems to happen a lot on QCom platforms.

First option is fine, but please don't apply the bindings, since it'll
cause annoying conflicts.