Devicetree
 help / color / mirror / Atom feed
* [PATCH 0/2] Initial support for Qualcomm Hamoa IOT COME board
@ 2026-06-30  7:10 Yuanjie Yang
  2026-06-30  7:10 ` [PATCH 1/2] dt-bindings: arm: qcom: Document HAMOA-IOT-COME board Yuanjie Yang
  2026-06-30  7:10 ` [PATCH 2/2] arm64: dts: qcom: Add base " Yuanjie Yang
  0 siblings, 2 replies; 10+ messages in thread
From: Yuanjie Yang @ 2026-06-30  7:10 UTC (permalink / raw)
  To: andersson, konradybcio, robh, krzk+dt, conor+dt
  Cc: linux-arm-msm, devicetree, linux-kernel, yuanjie.yang,
	tingwei.zhang

Introduce the device tree, DT bindings for HAMOA-IOT-COME board.

The system consists of a SoM mounted on a carrier board. The HAMOA-IOT-COME
SoM integrates the core system, including a SiP that contains the SoC and
related components.

The SiP on the HAMOA-IOT-COME SoM is equivalent to the HAMOA-IOT-EVK SoM.

┌───────────────────────────────┐
│          SoM (Module)         │
│   System on Module (small     │
│   pluggable board)            │
│                               │
│   ┌───────────────────────┐   │
│   │        SiP            │   │
│   │   System in Package   │   │
│   │        (chip)         │   │
│   │                       │   │
│   │   ┌───────────────┐   │   │
│   │   │     SoC       │   │   │
│   │   │               │   │   │
│   │   └───────────────┘   │   │
│   │                       │   │
│   │                       │   │
│   └───────────────────────┘   │
│                               │
└──────────────┬────────────────┘
               │ Plugs into
┌──────────────▼────────────────┐
│   Carrier Board (COME)        │
│                               │
└───────────────────────────────┘


Signed-off-by: Yuanjie Yang <yuanjie.yang@oss.qualcomm.com>
---

Yuanjie Yang (2):
  dt-bindings: arm: qcom: Document HAMOA-IOT-COME board
  arm64: dts: qcom: Add base HAMOA-IOT-COME board

 .../devicetree/bindings/arm/qcom.yaml         |   1 +
 arch/arm64/boot/dts/qcom/Makefile             |   1 +
 .../boot/dts/qcom/hamoa-iot-come-sip.dtsi     |   9 ++
 .../boot/dts/qcom/hamoa-iot-come-som.dtsi     |  38 ++++++
 arch/arm64/boot/dts/qcom/hamoa-iot-come.dts   | 108 ++++++++++++++++++
 5 files changed, 157 insertions(+)
 create mode 100644 arch/arm64/boot/dts/qcom/hamoa-iot-come-sip.dtsi
 create mode 100644 arch/arm64/boot/dts/qcom/hamoa-iot-come-som.dtsi
 create mode 100644 arch/arm64/boot/dts/qcom/hamoa-iot-come.dts


base-commit: 7de6ae9e12207ec146f2f3f1e58d1a99317e88bc
-- 
2.43.0


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH 1/2] dt-bindings: arm: qcom: Document HAMOA-IOT-COME board
  2026-06-30  7:10 [PATCH 0/2] Initial support for Qualcomm Hamoa IOT COME board Yuanjie Yang
@ 2026-06-30  7:10 ` Yuanjie Yang
  2026-07-01  7:28   ` Krzysztof Kozlowski
  2026-06-30  7:10 ` [PATCH 2/2] arm64: dts: qcom: Add base " Yuanjie Yang
  1 sibling, 1 reply; 10+ messages in thread
From: Yuanjie Yang @ 2026-06-30  7:10 UTC (permalink / raw)
  To: andersson, konradybcio, robh, krzk+dt, conor+dt
  Cc: linux-arm-msm, devicetree, linux-kernel, yuanjie.yang,
	tingwei.zhang

Document the device tree binding for the HAMOA-IOT-COME board, which uses
the Qualcomm X1E80100 SoC.

The system consists of a SoM mounted on a carrier board. The HAMOA-IOT-COME
SoM integrates the core system, including a SiP that contains the SoC and
related components.

Hierarchy:

    Carrier Board
        -> SoM
            -> SiP
                -> SoC

The SiP on the HAMOA-IOT-COME SoM is equivalent to the HAMOA-IOT-EVK SoM.

Signed-off-by: Yuanjie Yang <yuanjie.yang@oss.qualcomm.com>
---
 Documentation/devicetree/bindings/arm/qcom.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index 50cc18a6ec5e..a487b9e8dfb9 100644
--- a/Documentation/devicetree/bindings/arm/qcom.yaml
+++ b/Documentation/devicetree/bindings/arm/qcom.yaml
@@ -1172,6 +1172,7 @@ properties:
 
       - items:
           - enum:
+              - qcom,hamoa-iot-come
               - qcom,hamoa-iot-evk
           - const: qcom,hamoa-iot-som
           - const: qcom,x1e80100
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH 2/2] arm64: dts: qcom: Add base HAMOA-IOT-COME board
  2026-06-30  7:10 [PATCH 0/2] Initial support for Qualcomm Hamoa IOT COME board Yuanjie Yang
  2026-06-30  7:10 ` [PATCH 1/2] dt-bindings: arm: qcom: Document HAMOA-IOT-COME board Yuanjie Yang
@ 2026-06-30  7:10 ` Yuanjie Yang
  2026-06-30  7:26   ` sashiko-bot
  2026-07-01  7:29   ` Krzysztof Kozlowski
  1 sibling, 2 replies; 10+ messages in thread
From: Yuanjie Yang @ 2026-06-30  7:10 UTC (permalink / raw)
  To: andersson, konradybcio, robh, krzk+dt, conor+dt
  Cc: linux-arm-msm, devicetree, linux-kernel, yuanjie.yang,
	tingwei.zhang

Add the device tree for the HAMOA-IOT-COME platform.

The system consists of a SoM mounted on a carrier board. The HAMOA-IOT-COME
SoM integrates the core system, including a SiP that contains the SoC and
related components.

Hierarchy:

    Carrier Board
        -> SoM
            -> SiP
                -> SoC

The SiP on the HAMOA-IOT-COME SOM is equivalent to the HAMOA-IOT-EVK SoM.

The initial device tree includes support for:
- UART
- Regulators
- USB
- PCIe
- Pinctrl
- ADSP, CDSP
- UFS
- Graphic
- Video

Signed-off-by: Yuanjie Yang <yuanjie.yang@oss.qualcomm.com>
---
 arch/arm64/boot/dts/qcom/Makefile             |   1 +
 .../boot/dts/qcom/hamoa-iot-come-sip.dtsi     |   9 ++
 .../boot/dts/qcom/hamoa-iot-come-som.dtsi     |  38 ++++++
 arch/arm64/boot/dts/qcom/hamoa-iot-come.dts   | 108 ++++++++++++++++++
 4 files changed, 156 insertions(+)
 create mode 100644 arch/arm64/boot/dts/qcom/hamoa-iot-come-sip.dtsi
 create mode 100644 arch/arm64/boot/dts/qcom/hamoa-iot-come-som.dtsi
 create mode 100644 arch/arm64/boot/dts/qcom/hamoa-iot-come.dts

diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
index 6f33c4e2f09c..549287e3eeba 100644
--- a/arch/arm64/boot/dts/qcom/Makefile
+++ b/arch/arm64/boot/dts/qcom/Makefile
@@ -16,6 +16,7 @@ dtb-$(CONFIG_ARCH_QCOM)	+= apq8096sg-db820c.dtb
 dtb-$(CONFIG_ARCH_QCOM)	+= apq8096-ifc6640.dtb
 dtb-$(CONFIG_ARCH_QCOM)	+= eliza-mtp.dtb
 dtb-$(CONFIG_ARCH_QCOM)	+= glymur-crd.dtb
+dtb-$(CONFIG_ARCH_QCOM)	+= hamoa-iot-come.dtb
 dtb-$(CONFIG_ARCH_QCOM)	+= hamoa-iot-evk.dtb
 
 hamoa-iot-evk-el2-dtbs	:= hamoa-iot-evk.dtb x1-el2.dtbo
diff --git a/arch/arm64/boot/dts/qcom/hamoa-iot-come-sip.dtsi b/arch/arm64/boot/dts/qcom/hamoa-iot-come-sip.dtsi
new file mode 100644
index 000000000000..935af96c2b85
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/hamoa-iot-come-sip.dtsi
@@ -0,0 +1,9 @@
+// SPDX-License-Identifier: BSD-3-Clause
+/*
+ * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
+ */
+
+#include "hamoa-iot-som.dtsi"
+
+/ {
+};
diff --git a/arch/arm64/boot/dts/qcom/hamoa-iot-come-som.dtsi b/arch/arm64/boot/dts/qcom/hamoa-iot-come-som.dtsi
new file mode 100644
index 000000000000..5c6475b34cc6
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/hamoa-iot-come-som.dtsi
@@ -0,0 +1,38 @@
+// SPDX-License-Identifier: BSD-3-Clause
+/*
+ * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
+ */
+
+#include "hamoa-iot-come-sip.dtsi"
+
+/ {
+	vph_pwr: regulator-vph-pwr {
+		compatible = "regulator-fixed";
+
+		regulator-name = "vph_pwr";
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+
+		regulator-always-on;
+		regulator-boot-on;
+	};
+};
+
+&ufs_mem_phy {
+	vdda-phy-supply = <&vreg_l3i_0p8>;
+	vdda-pll-supply = <&vreg_l3e_1p2>;
+
+	status = "okay";
+};
+
+&ufs_mem_hc {
+	reset-gpios = <&tlmm 238 GPIO_ACTIVE_LOW>;
+
+	vcc-supply = <&vreg_l17b_2p5>;
+	vcc-max-microamp = <1300000>;
+	vccq-supply = <&vreg_l2i_1p2>;
+	vccq-max-microamp = <1200000>;
+
+	status = "okay";
+};
+
diff --git a/arch/arm64/boot/dts/qcom/hamoa-iot-come.dts b/arch/arm64/boot/dts/qcom/hamoa-iot-come.dts
new file mode 100644
index 000000000000..99c47cd953b8
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/hamoa-iot-come.dts
@@ -0,0 +1,108 @@
+// SPDX-License-Identifier: BSD-3-Clause
+/*
+ * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
+ */
+
+/dts-v1/;
+
+#include "hamoa-iot-come-som.dtsi"
+
+/ {
+	model = "Qualcomm Technologies, Inc. Hamoa IoT ComE";
+	compatible = "qcom,hamoa-iot-come", "qcom,hamoa-iot-som", "qcom,x1e80100";
+	chassis-type = "embedded";
+
+	aliases {
+		serial0 = &uart21;
+	};
+
+	chosen {
+		stdout-path = "serial0:115200n8";
+	};
+
+	vreg_pcie_12v: regulator-pcie-12v {
+		compatible = "regulator-fixed";
+
+		regulator-name = "VREG_PCIE_12V";
+		regulator-min-microvolt = <12000000>;
+		regulator-max-microvolt = <12000000>;
+
+		gpio = <&pm8550ve_8_gpios 8 GPIO_ACTIVE_HIGH>;
+		enable-active-high;
+
+		pinctrl-0 = <&pcie_x8_12v>;
+		pinctrl-names = "default";
+	};
+
+	vreg_pcie_3v3: regulator-pcie-3v3 {
+		compatible = "regulator-fixed";
+
+		regulator-name = "VREG_PCIE_3P3";
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+
+		gpio = <&pmc8380_3_gpios 6 GPIO_ACTIVE_HIGH>;
+		enable-active-high;
+
+		pinctrl-0 = <&pm_sde7_main_3p3_en>;
+		pinctrl-names = "default";
+	};
+
+	vreg_pcie_3v3_aux: regulator-pcie-3v3-aux {
+		compatible = "regulator-fixed";
+
+		regulator-name = "VREG_PCIE_3P3_AUX";
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+
+		gpio = <&pmc8380_3_gpios 8 GPIO_ACTIVE_HIGH>;
+		enable-active-high;
+
+		pinctrl-0 = <&pm_sde7_aux_3p3_en>;
+		pinctrl-names = "default";
+	};
+};
+
+&pcie3_port0 {
+	vpcie12v-supply = <&vreg_pcie_12v>;
+	vpcie3v3-supply = <&vreg_pcie_3v3>;
+	vpcie3v3aux-supply = <&vreg_pcie_3v3_aux>;
+
+	reset-gpios = <&tlmm 143 GPIO_ACTIVE_LOW>;
+	wake-gpios = <&tlmm 145 GPIO_ACTIVE_LOW>;
+};
+
+&pm8550ve_8_gpios {
+	pcie_x8_12v: pcie-12v-default-state {
+		pins = "gpio8";
+		function = "normal";
+		output-enable;
+		output-high;
+		bias-pull-down;
+		power-source = <0>;
+	};
+};
+
+&pmc8380_3_gpios {
+	pm_sde7_aux_3p3_en: pcie-aux-3p3-default-state {
+		pins = "gpio8";
+		function = "normal";
+		output-enable;
+		bias-pull-down;
+		power-source = <0>;
+	};
+
+	pm_sde7_main_3p3_en: pcie-main-3p3-default-state {
+		pins = "gpio6";
+		function = "normal";
+		output-enable;
+		bias-pull-down;
+		power-source = <0>;
+	};
+};
+
+&uart21 {
+	compatible = "qcom,geni-debug-uart";
+
+	status = "okay";
+};
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [PATCH 2/2] arm64: dts: qcom: Add base HAMOA-IOT-COME board
  2026-06-30  7:10 ` [PATCH 2/2] arm64: dts: qcom: Add base " Yuanjie Yang
@ 2026-06-30  7:26   ` sashiko-bot
  2026-06-30  9:37     ` yuanjiey
  2026-07-01  7:29   ` Krzysztof Kozlowski
  1 sibling, 1 reply; 10+ messages in thread
From: sashiko-bot @ 2026-06-30  7:26 UTC (permalink / raw)
  To: Yuanjie Yang; +Cc: conor+dt, robh, devicetree

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] Unused PCIe controllers (pcie4, pcie5, pcie6a) are left enabled in the ComE carrier board, causing boot delays and power waste.
--

commit 5654913fbfd858b8d2bf4684318b0d7145c8a5b6
Author: Yuanjie Yang <yuanjie.yang@oss.qualcomm.com>

arm64: dts: qcom: Add base HAMOA-IOT-COME board

This commit adds the device tree for the HAMOA-IOT-COME platform, which
consists of a SoM mounted on a carrier board. It includes initial support
for components like UART, Regulators, USB, PCIe, Pinctrl, and UFS.

> diff --git a/arch/arm64/boot/dts/qcom/hamoa-iot-come.dts b/arch/arm64/boot/dts/qcom/hamoa-iot-come.dts
> new file mode 100644
> index 0000000000000..99c47cd953b83
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/hamoa-iot-come.dts
[ ... ]
> +&pcie3_port0 {
> +	vpcie12v-supply = <&vreg_pcie_12v>;
> +	vpcie3v3-supply = <&vreg_pcie_3v3>;
> +	vpcie3v3aux-supply = <&vreg_pcie_3v3_aux>;
> +
> +	reset-gpios = <&tlmm 143 GPIO_ACTIVE_LOW>;
> +	wake-gpios = <&tlmm 145 GPIO_ACTIVE_LOW>;
> +};

[Severity: Medium]
The included hamoa-iot-som.dtsi enables pcie4, pcie5, and pcie6a with
status = "okay", but this carrier board does not disable them or provide
the required reset-gpios routing.

Will this result in the pcie-qcom driver powering on their PHYs and timing
out during link training?

It appears dw_pcie_wait_for_link() will loop and time out after 900ms per
controller without a connected device or toggled reset signal. This causes
boot delays of nearly 2.7 seconds and leaves unused PHYs powered on.

Should these unused controllers be explicitly disabled here with
status = "disabled"?

[ ... ]

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260630071022.3256-1-yuanjie.yang@oss.qualcomm.com?part=2

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 2/2] arm64: dts: qcom: Add base HAMOA-IOT-COME board
  2026-06-30  7:26   ` sashiko-bot
@ 2026-06-30  9:37     ` yuanjiey
  0 siblings, 0 replies; 10+ messages in thread
From: yuanjiey @ 2026-06-30  9:37 UTC (permalink / raw)
  To: sashiko-reviews; +Cc: conor+dt, robh, devicetree

On Tue, Jun 30, 2026 at 07:26:06AM +0000, sashiko-bot@kernel.org wrote:
> Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
> - [Medium] Unused PCIe controllers (pcie4, pcie5, pcie6a) are left enabled in the ComE carrier board, causing boot delays and power waste.
> --
> 
> commit 5654913fbfd858b8d2bf4684318b0d7145c8a5b6
> Author: Yuanjie Yang <yuanjie.yang@oss.qualcomm.com>
> 
> arm64: dts: qcom: Add base HAMOA-IOT-COME board
> 
> This commit adds the device tree for the HAMOA-IOT-COME platform, which
> consists of a SoM mounted on a carrier board. It includes initial support
> for components like UART, Regulators, USB, PCIe, Pinctrl, and UFS.
> 
> > diff --git a/arch/arm64/boot/dts/qcom/hamoa-iot-come.dts b/arch/arm64/boot/dts/qcom/hamoa-iot-come.dts
> > new file mode 100644
> > index 0000000000000..99c47cd953b83
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/qcom/hamoa-iot-come.dts
> [ ... ]
> > +&pcie3_port0 {
> > +	vpcie12v-supply = <&vreg_pcie_12v>;
> > +	vpcie3v3-supply = <&vreg_pcie_3v3>;
> > +	vpcie3v3aux-supply = <&vreg_pcie_3v3_aux>;
> > +
> > +	reset-gpios = <&tlmm 143 GPIO_ACTIVE_LOW>;
> > +	wake-gpios = <&tlmm 145 GPIO_ACTIVE_LOW>;
> > +};
> 
> [Severity: Medium]
> The included hamoa-iot-som.dtsi enables pcie4, pcie5, and pcie6a with
> status = "okay", but this carrier board does not disable them or provide
> the required reset-gpios routing.
> 
> Will this result in the pcie-qcom driver powering on their PHYs and timing
> out during link training?
> 
> It appears dw_pcie_wait_for_link() will loop and time out after 900ms per
> controller without a connected device or toggled reset signal. This causes
> boot delays of nearly 2.7 seconds and leaves unused PHYs powered on.
> 
> Should these unused controllers be explicitly disabled here with
> status = "disabled"?

Yes, Hamoa COMe does not include NVMe, Bluetooth, or Wi‑Fi,
So I think PCIe 4.0, 5.0, and 6.0 are not required. try to explicitly disabled in
DT is a way
 
Thanks,
Yuanjie


> [ ... ]
> 
> -- 
> Sashiko AI review · https://sashiko.dev/#/patchset/20260630071022.3256-1-yuanjie.yang@oss.qualcomm.com?part=2

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 1/2] dt-bindings: arm: qcom: Document HAMOA-IOT-COME board
  2026-06-30  7:10 ` [PATCH 1/2] dt-bindings: arm: qcom: Document HAMOA-IOT-COME board Yuanjie Yang
@ 2026-07-01  7:28   ` Krzysztof Kozlowski
  2026-07-01  8:47     ` yuanjiey
  0 siblings, 1 reply; 10+ messages in thread
From: Krzysztof Kozlowski @ 2026-07-01  7:28 UTC (permalink / raw)
  To: Yuanjie Yang
  Cc: andersson, konradybcio, robh, krzk+dt, conor+dt, linux-arm-msm,
	devicetree, linux-kernel, tingwei.zhang

On Tue, Jun 30, 2026 at 03:10:11PM +0800, Yuanjie Yang wrote:
> Document the device tree binding for the HAMOA-IOT-COME board, which uses
> the Qualcomm X1E80100 SoC.
> 
> The system consists of a SoM mounted on a carrier board. The HAMOA-IOT-COME
> SoM integrates the core system, including a SiP that contains the SoC and

This is contradictory to the code. Your binding said you are using IOT
SoM, not IoT COME SoM.

> related components.
> 
> Hierarchy:
> 
>     Carrier Board
>         -> SoM
>             -> SiP
>                 -> SoC

Drop. Compatibles define that.

> 
> The SiP on the HAMOA-IOT-COME SoM is equivalent to the HAMOA-IOT-EVK SoM.

I don't even know what does it mean. How two different hardware can be
equivalent? Are they the same? Are you creating duplicated compatibles
for same hardware?

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 2/2] arm64: dts: qcom: Add base HAMOA-IOT-COME board
  2026-06-30  7:10 ` [PATCH 2/2] arm64: dts: qcom: Add base " Yuanjie Yang
  2026-06-30  7:26   ` sashiko-bot
@ 2026-07-01  7:29   ` Krzysztof Kozlowski
  2026-07-01  8:51     ` yuanjiey
  1 sibling, 1 reply; 10+ messages in thread
From: Krzysztof Kozlowski @ 2026-07-01  7:29 UTC (permalink / raw)
  To: Yuanjie Yang
  Cc: andersson, konradybcio, robh, krzk+dt, conor+dt, linux-arm-msm,
	devicetree, linux-kernel, tingwei.zhang

On Tue, Jun 30, 2026 at 03:10:12PM +0800, Yuanjie Yang wrote:
> Signed-off-by: Yuanjie Yang <yuanjie.yang@oss.qualcomm.com>
> ---
>  arch/arm64/boot/dts/qcom/Makefile             |   1 +
>  .../boot/dts/qcom/hamoa-iot-come-sip.dtsi     |   9 ++
>  .../boot/dts/qcom/hamoa-iot-come-som.dtsi     |  38 ++++++
>  arch/arm64/boot/dts/qcom/hamoa-iot-come.dts   | 108 ++++++++++++++++++
>  4 files changed, 156 insertions(+)
>  create mode 100644 arch/arm64/boot/dts/qcom/hamoa-iot-come-sip.dtsi
>  create mode 100644 arch/arm64/boot/dts/qcom/hamoa-iot-come-som.dtsi
>  create mode 100644 arch/arm64/boot/dts/qcom/hamoa-iot-come.dts
> 
> diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
> index 6f33c4e2f09c..549287e3eeba 100644
> --- a/arch/arm64/boot/dts/qcom/Makefile
> +++ b/arch/arm64/boot/dts/qcom/Makefile
> @@ -16,6 +16,7 @@ dtb-$(CONFIG_ARCH_QCOM)	+= apq8096sg-db820c.dtb
>  dtb-$(CONFIG_ARCH_QCOM)	+= apq8096-ifc6640.dtb
>  dtb-$(CONFIG_ARCH_QCOM)	+= eliza-mtp.dtb
>  dtb-$(CONFIG_ARCH_QCOM)	+= glymur-crd.dtb
> +dtb-$(CONFIG_ARCH_QCOM)	+= hamoa-iot-come.dtb
>  dtb-$(CONFIG_ARCH_QCOM)	+= hamoa-iot-evk.dtb
>  
>  hamoa-iot-evk-el2-dtbs	:= hamoa-iot-evk.dtb x1-el2.dtbo
> diff --git a/arch/arm64/boot/dts/qcom/hamoa-iot-come-sip.dtsi b/arch/arm64/boot/dts/qcom/hamoa-iot-come-sip.dtsi
> new file mode 100644
> index 000000000000..935af96c2b85
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/hamoa-iot-come-sip.dtsi

What is the point of this file?

> @@ -0,0 +1,9 @@
> +// SPDX-License-Identifier: BSD-3-Clause
> +/*
> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> + */
> +
> +#include "hamoa-iot-som.dtsi"
> +
> +/ {
> +};

It is completely empty.

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 1/2] dt-bindings: arm: qcom: Document HAMOA-IOT-COME board
  2026-07-01  7:28   ` Krzysztof Kozlowski
@ 2026-07-01  8:47     ` yuanjiey
  0 siblings, 0 replies; 10+ messages in thread
From: yuanjiey @ 2026-07-01  8:47 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: andersson, konradybcio, robh, krzk+dt, conor+dt, linux-arm-msm,
	devicetree, linux-kernel, tingwei.zhang

On Wed, Jul 01, 2026 at 09:28:41AM +0200, Krzysztof Kozlowski wrote:
> On Tue, Jun 30, 2026 at 03:10:11PM +0800, Yuanjie Yang wrote:
> > Document the device tree binding for the HAMOA-IOT-COME board, which uses
> > the Qualcomm X1E80100 SoC.
> > 
> > The system consists of a SoM mounted on a carrier board. The HAMOA-IOT-COME
> > SoM integrates the core system, including a SiP that contains the SoC and
> 
> This is contradictory to the code. Your binding said you are using IOT
> SoM, not IoT COME SoM.

Hierarchy:

Hamoa ComE
 Carrier Board
     -> SoM
         -> SiP(This SiP hardware is effectively the same as the SoM on the Hamoa EVK)
             -> SoC

Hamoa evk
 Carrier Board
     -> SoM
         -> SoC


So I think I write a wrong dt binding about Hamoa ComE:
correct here:
I think the binding should be: 

1.
      - items:
          - enum:
              - qcom,hamoa-iot-come
          - const: qcom,hamoa-iot-come-som
          - const: qcom,hamoa-iot-come-sip
          - const: qcom,x1e80100

or
2.
      - items:
          - enum:
              - qcom,hamoa-iot-come
          - const: qcom,hamoa-iot-come-som
          - const: qcom,hamoa-iot-som
          - const: qcom,x1e80100


> > related components.
> > 
> > Hierarchy:
> > 
> >     Carrier Board
> >         -> SoM
> >             -> SiP
> >                 -> SoC
> 
> Drop. Compatibles define that.

Sure , will do.

> > 
> > The SiP on the HAMOA-IOT-COME SoM is equivalent to the HAMOA-IOT-EVK SoM.
> 
> I don't even know what does it mean. How two different hardware can be
> equivalent? Are they the same? Are you creating duplicated compatibles
> for same hardware?

This ComE SiP hardware is effectively the same as the SoM on the Hamoa EVK, 
It's just called differently on a different board. I should update my
commit msg show this.

> Best regards,
> Krzysztof
> 
Thanks,
Yuanjie



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 2/2] arm64: dts: qcom: Add base HAMOA-IOT-COME board
  2026-07-01  7:29   ` Krzysztof Kozlowski
@ 2026-07-01  8:51     ` yuanjiey
  2026-07-01 10:19       ` Krzysztof Kozlowski
  0 siblings, 1 reply; 10+ messages in thread
From: yuanjiey @ 2026-07-01  8:51 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: andersson, konradybcio, robh, krzk+dt, conor+dt, linux-arm-msm,
	devicetree, linux-kernel, tingwei.zhang

On Wed, Jul 01, 2026 at 09:29:34AM +0200, Krzysztof Kozlowski wrote:
> On Tue, Jun 30, 2026 at 03:10:12PM +0800, Yuanjie Yang wrote:
> > Signed-off-by: Yuanjie Yang <yuanjie.yang@oss.qualcomm.com>
> > ---
> >  arch/arm64/boot/dts/qcom/Makefile             |   1 +
> >  .../boot/dts/qcom/hamoa-iot-come-sip.dtsi     |   9 ++
> >  .../boot/dts/qcom/hamoa-iot-come-som.dtsi     |  38 ++++++
> >  arch/arm64/boot/dts/qcom/hamoa-iot-come.dts   | 108 ++++++++++++++++++
> >  4 files changed, 156 insertions(+)
> >  create mode 100644 arch/arm64/boot/dts/qcom/hamoa-iot-come-sip.dtsi
> >  create mode 100644 arch/arm64/boot/dts/qcom/hamoa-iot-come-som.dtsi
> >  create mode 100644 arch/arm64/boot/dts/qcom/hamoa-iot-come.dts
> > 
> > diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
> > index 6f33c4e2f09c..549287e3eeba 100644
> > --- a/arch/arm64/boot/dts/qcom/Makefile
> > +++ b/arch/arm64/boot/dts/qcom/Makefile
> > @@ -16,6 +16,7 @@ dtb-$(CONFIG_ARCH_QCOM)	+= apq8096sg-db820c.dtb
> >  dtb-$(CONFIG_ARCH_QCOM)	+= apq8096-ifc6640.dtb
> >  dtb-$(CONFIG_ARCH_QCOM)	+= eliza-mtp.dtb
> >  dtb-$(CONFIG_ARCH_QCOM)	+= glymur-crd.dtb
> > +dtb-$(CONFIG_ARCH_QCOM)	+= hamoa-iot-come.dtb
> >  dtb-$(CONFIG_ARCH_QCOM)	+= hamoa-iot-evk.dtb
> >  
> >  hamoa-iot-evk-el2-dtbs	:= hamoa-iot-evk.dtb x1-el2.dtbo
> > diff --git a/arch/arm64/boot/dts/qcom/hamoa-iot-come-sip.dtsi b/arch/arm64/boot/dts/qcom/hamoa-iot-come-sip.dtsi
> > new file mode 100644
> > index 000000000000..935af96c2b85
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/qcom/hamoa-iot-come-sip.dtsi
> 
> What is the point of this file?

This SiP hardware is effectively the same as the SoM on the Hamoa EVK.
It's just called differently on a different board. I want to use this
file to include hamoa-iot-som.dtsi header to show this idea.

Thanks,
Yuanjie

> > @@ -0,0 +1,9 @@
> > +// SPDX-License-Identifier: BSD-3-Clause
> > +/*
> > + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> > + */
> > +
> > +#include "hamoa-iot-som.dtsi"
> > +
> > +/ {
> > +};
> 
> It is completely empty.
> 
> Best regards,
> Krzysztof
> 

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 2/2] arm64: dts: qcom: Add base HAMOA-IOT-COME board
  2026-07-01  8:51     ` yuanjiey
@ 2026-07-01 10:19       ` Krzysztof Kozlowski
  0 siblings, 0 replies; 10+ messages in thread
From: Krzysztof Kozlowski @ 2026-07-01 10:19 UTC (permalink / raw)
  To: yuanjiey
  Cc: andersson, konradybcio, robh, krzk+dt, conor+dt, linux-arm-msm,
	devicetree, linux-kernel, tingwei.zhang

On 01/07/2026 10:51, yuanjiey wrote:
> On Wed, Jul 01, 2026 at 09:29:34AM +0200, Krzysztof Kozlowski wrote:
>> On Tue, Jun 30, 2026 at 03:10:12PM +0800, Yuanjie Yang wrote:
>>> Signed-off-by: Yuanjie Yang <yuanjie.yang@oss.qualcomm.com>
>>> ---
>>>  arch/arm64/boot/dts/qcom/Makefile             |   1 +
>>>  .../boot/dts/qcom/hamoa-iot-come-sip.dtsi     |   9 ++
>>>  .../boot/dts/qcom/hamoa-iot-come-som.dtsi     |  38 ++++++
>>>  arch/arm64/boot/dts/qcom/hamoa-iot-come.dts   | 108 ++++++++++++++++++
>>>  4 files changed, 156 insertions(+)
>>>  create mode 100644 arch/arm64/boot/dts/qcom/hamoa-iot-come-sip.dtsi
>>>  create mode 100644 arch/arm64/boot/dts/qcom/hamoa-iot-come-som.dtsi
>>>  create mode 100644 arch/arm64/boot/dts/qcom/hamoa-iot-come.dts
>>>
>>> diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
>>> index 6f33c4e2f09c..549287e3eeba 100644
>>> --- a/arch/arm64/boot/dts/qcom/Makefile
>>> +++ b/arch/arm64/boot/dts/qcom/Makefile
>>> @@ -16,6 +16,7 @@ dtb-$(CONFIG_ARCH_QCOM)	+= apq8096sg-db820c.dtb
>>>  dtb-$(CONFIG_ARCH_QCOM)	+= apq8096-ifc6640.dtb
>>>  dtb-$(CONFIG_ARCH_QCOM)	+= eliza-mtp.dtb
>>>  dtb-$(CONFIG_ARCH_QCOM)	+= glymur-crd.dtb
>>> +dtb-$(CONFIG_ARCH_QCOM)	+= hamoa-iot-come.dtb
>>>  dtb-$(CONFIG_ARCH_QCOM)	+= hamoa-iot-evk.dtb
>>>  
>>>  hamoa-iot-evk-el2-dtbs	:= hamoa-iot-evk.dtb x1-el2.dtbo
>>> diff --git a/arch/arm64/boot/dts/qcom/hamoa-iot-come-sip.dtsi b/arch/arm64/boot/dts/qcom/hamoa-iot-come-sip.dtsi
>>> new file mode 100644
>>> index 000000000000..935af96c2b85
>>> --- /dev/null
>>> +++ b/arch/arm64/boot/dts/qcom/hamoa-iot-come-sip.dtsi
>>
>> What is the point of this file?
> 
> This SiP hardware is effectively the same as the SoM on the Hamoa EVK.
> It's just called differently on a different board. I want to use this
> file to include hamoa-iot-som.dtsi header to show this idea.

You do not need empty file for such purpose.

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2026-07-01 10:19 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-06-30  7:10 [PATCH 0/2] Initial support for Qualcomm Hamoa IOT COME board Yuanjie Yang
2026-06-30  7:10 ` [PATCH 1/2] dt-bindings: arm: qcom: Document HAMOA-IOT-COME board Yuanjie Yang
2026-07-01  7:28   ` Krzysztof Kozlowski
2026-07-01  8:47     ` yuanjiey
2026-06-30  7:10 ` [PATCH 2/2] arm64: dts: qcom: Add base " Yuanjie Yang
2026-06-30  7:26   ` sashiko-bot
2026-06-30  9:37     ` yuanjiey
2026-07-01  7:29   ` Krzysztof Kozlowski
2026-07-01  8:51     ` yuanjiey
2026-07-01 10:19       ` Krzysztof Kozlowski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox