All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sumit Garg <sumit.garg@kernel.org>
To: Balaji Selvanathan <balaji.selvanathan@oss.qualcomm.com>
Cc: Sumit Garg <sumit.garg@kernel.org>,
	u-boot@lists.u-boot-project.org, u-boot-qcom@groups.io,
	Lukasz Majewski <lukma@denx.de>, Tom Rini <trini@konsulko.com>,
	Casey Connolly <casey.connolly@linaro.org>,
	Neil Armstrong <neil.armstrong@linaro.org>,
	Johan Jonker <jbx6244@gmail.com>,
	Luca Weiss <luca.weiss@fairphone.com>,
	Biswapriyo Nath <nathbappai@gmail.com>,
	Ilias Apalodimas <ilias.apalodimas@linaro.org>
Subject: Re: [PATCH v2 5/7] arm64: dts: monaco-evk: Add reboot-mode support via PON/SDAM
Date: Mon, 19 Oct 2026 17:28:29 +0530	[thread overview]
Message-ID: <atYF5T2LiJBOU3rT@sumit-xelite> (raw)
In-Reply-To: <20260812-b4-monaco-v2-5-865d288c0be6@oss.qualcomm.com>

On Wed, 12 Aug 2026 at 06:42:03 +0530, Balaji Selvanathan wrote:
>monaco-pmics.dtsi has no PON/SDAM reboot-reason storage, so
>U-Boot can't read the reboot-reason to tell if it should stop in
>fastboot instead of booting the kernel.
>
>Signed-off-by: Balaji Selvanathan <balaji.selvanathan@oss.qualcomm.com>
>---
> arch/arm/dts/monaco-evk-u-boot.dtsi | 36 ++++++++++++++++++++++++++++++++++++
> 1 file changed, 36 insertions(+)

Avoid adding DTS changes for targets that aren't yet accepted upstream.
Try to keep patch-sets independent such that they are easier to land.

-Sumit

>
>diff --git a/arch/arm/dts/monaco-evk-u-boot.dtsi b/arch/arm/dts/monaco-evk-u-boot.dtsi
>new file mode 100644
>index 00000000000..5520d111864
>--- /dev/null
>+++ b/arch/arm/dts/monaco-evk-u-boot.dtsi
>@@ -0,0 +1,36 @@
>+// SPDX-License-Identifier: BSD-3-Clause
>+/*
>+ * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
>+ */
>+
>+/ {
>+	reboot-mode {
>+		compatible = "nvmem-reboot-mode";
>+		nvmem-cells = <&reboot_reason>;
>+		nvmem-cell-names = "reboot-mode";
>+
>+		mode-bootloader = <0x02>;
>+		mode-recovery   = <0x01>;
>+	};
>+};
>+
>+&pmm8620au_0 {
>+	pmm8620au_0_pon: pon@1200 {
>+		compatible = "qcom,pmk8350-pon";
>+		reg = <0x1200>, <0x800>;
>+		reg-names = "hlos", "pbs";
>+	};
>+
>+	pmm8620au_0_sdam_0: nvram@7100 {
>+		compatible = "qcom,spmi-sdam";
>+		reg = <0x7100>;
>+		#address-cells = <1>;
>+		#size-cells = <1>;
>+		ranges = <0x0 0x7100 0x100>;
>+
>+		reboot_reason: reboot-reason@48 {
>+			reg = <0x48 0x1>;
>+			bits = <1 7>;
>+		};
>+	};
>+};
>
>-- 
>2.34.1
>

  reply	other threads:[~2026-10-19 11:58 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-12  1:11 [PATCH v2 0/7] qcom: qcs8300/monaco-evk: pinctrl, interconnect, RPMH clock and reboot-mode support Balaji Selvanathan via U-Boot
2026-08-12  1:11 ` [PATCH v2 1/7] clk: qcom: Add shared RPMH clock driver for SM8550/SM8650/SA8775P Balaji Selvanathan via U-Boot
2026-08-12 12:26   ` Neil Armstrong
2026-08-12  1:12 ` [PATCH v2 2/7] clk: qcom: qcs8300: Extend GCC clock driver to full gate/RCG coverage Balaji Selvanathan via U-Boot
2026-10-19 11:54   ` Sumit Garg
2026-08-12  1:12 ` [PATCH v2 3/7] pinctrl: qcom: Add QCS8300 pinctrl driver Balaji Selvanathan via U-Boot
2026-10-19 11:56   ` Sumit Garg
2026-08-12  1:12 ` [PATCH v2 4/7] interconnect: qcom: Add QCS8300 interconnect driver Balaji Selvanathan via U-Boot
2026-08-12  1:12 ` [PATCH v2 5/7] arm64: dts: monaco-evk: Add reboot-mode support via PON/SDAM Balaji Selvanathan via U-Boot
2026-10-19 11:58   ` Sumit Garg [this message]
2026-08-12  1:12 ` [PATCH v2 6/7] arm64: dts: monaco-evk: Force vreg_l8a always-on to protect UFS VCC Balaji Selvanathan via U-Boot
2026-10-19 12:01   ` Sumit Garg
2026-08-12  1:12 ` [PATCH v2 7/7] configs: qcom_qcs8300: Switch to monaco-evk DT and enable new drivers Balaji Selvanathan via U-Boot
2026-10-19 12:03   ` Sumit Garg
2026-10-08 12:44 ` [PATCH v2 0/7] qcom: qcs8300/monaco-evk: pinctrl, interconnect, RPMH clock and reboot-mode support Neil Armstrong

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=atYF5T2LiJBOU3rT@sumit-xelite \
    --to=sumit.garg@kernel.org \
    --cc=balaji.selvanathan@oss.qualcomm.com \
    --cc=casey.connolly@linaro.org \
    --cc=ilias.apalodimas@linaro.org \
    --cc=jbx6244@gmail.com \
    --cc=luca.weiss@fairphone.com \
    --cc=lukma@denx.de \
    --cc=nathbappai@gmail.com \
    --cc=neil.armstrong@linaro.org \
    --cc=trini@konsulko.com \
    --cc=u-boot-qcom@groups.io \
    --cc=u-boot@lists.u-boot-project.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.