mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH v3 1/2] dt-bindings: embedded-controller: qcom,hamoa-crd-ec: add Lenovo Yoga Slim 7x
@ 2026-05-26 11:24 Daniel J Blueman
  2026-05-26 11:24 ` [PATCH v3 2/2] arm64: dts: qcom: x1e80100-lenovo-yoga-slim7x: Add Embedded Controller node Daniel J Blueman
                   ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Daniel J Blueman @ 2026-05-26 11:24 UTC (permalink / raw)
  To: Konrad Dybcio
  Cc: linux-arm-msm, devicetree, Sibi Sankar, Randy Dunlap,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, Hans de Goede,
	Bryan O'Donoghue, Bjorn Andersson, linux-kernel,
	Anvesh Jain P, Maya Matuszczyk, Krzysztof Kozlowski,
	Dmitry Baryshkov, Akhil P Oommen, Abel Vesa, Gaurav Kohli,
	Daniel J Blueman

The Lenovo Yoga Slim 7x uses the same Embedded Controller as the Qualcomm
Hamoa X1 CRD. Add a board-specific compatible with qcom,hamoa-crd-ec as
the fallback.

Signed-off-by: Daniel J Blueman <daniel@quora.org>
---
Changes in v3:
- new patch (binding was missing in v1/v2)

Dependencies:
https://lore.kernel.org/all/20260427-add-driver-for-ec-v8-0-702f74e495f7@oss.qualcomm.com/

 .../bindings/embedded-controller/qcom,hamoa-crd-ec.yaml          | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/embedded-controller/qcom,hamoa-crd-ec.yaml b/Documentation/devicetree/bindings/embedded-controller/qcom,hamoa-crd-ec.yaml
index ac5a08f8f76d..813d41769c0b 100644
--- a/Documentation/devicetree/bindings/embedded-controller/qcom,hamoa-crd-ec.yaml
+++ b/Documentation/devicetree/bindings/embedded-controller/qcom,hamoa-crd-ec.yaml
@@ -20,6 +20,7 @@ properties:
     oneOf:
       - items:
           - enum:
+              - lenovo,yoga-slim7x-ec
               - qcom,glymur-crd-ec
               - qcom,hamoa-iot-evk-ec
           - const: qcom,hamoa-crd-ec
-- 
2.53.0


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

* [PATCH v3 2/2] arm64: dts: qcom: x1e80100-lenovo-yoga-slim7x: Add Embedded Controller node
  2026-05-26 11:24 [PATCH v3 1/2] dt-bindings: embedded-controller: qcom,hamoa-crd-ec: add Lenovo Yoga Slim 7x Daniel J Blueman
@ 2026-05-26 11:24 ` Daniel J Blueman
  2026-05-26 12:25   ` Bryan O'Donoghue
                     ` (2 more replies)
  2026-05-26 12:24 ` [PATCH v3 1/2] dt-bindings: embedded-controller: qcom,hamoa-crd-ec: add Lenovo Yoga Slim 7x Bryan O'Donoghue
  2026-05-27  9:41 ` Krzysztof Kozlowski
  2 siblings, 3 replies; 8+ messages in thread
From: Daniel J Blueman @ 2026-05-26 11:24 UTC (permalink / raw)
  To: Konrad Dybcio
  Cc: linux-arm-msm, devicetree, Sibi Sankar, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Hans de Goede, Randy Dunlap,
	Bryan O'Donoghue, Bjorn Andersson, linux-kernel,
	Anvesh Jain P, Maya Matuszczyk, Krzysztof Kozlowski,
	Dmitry Baryshkov, Akhil P Oommen, Abel Vesa, Gaurav Kohli,
	Daniel J Blueman

The Lenovo Slim7x uses the same Embedded Controller as the Qualcomm Hamoa
X1 Customer Reference Device. Use the lenovo,yoga-slim7x-ec compatible
introduced by patch 1 for fan control, thermal sensor and suspend
behaviour.

Signed-off-by: Daniel J Blueman <daniel@quora.org>
---
Changes in v3:
- use lenovo,yoga-slim7x-ec compatible (introduced by patch 1)
- v2 link: https://lore.kernel.org/lkml/20260502063518.15153-1-daniel@quora.org/
Changes in v2:
- corrected DT compatible node
- v1 link: https://lore.kernel.org/all/20260429103301.17449-1-daniel@quora.org/

 .../dts/qcom/x1e80100-lenovo-yoga-slim7x.dts  | 22 +++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts b/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts
index beb1475d7fa0..1ee2a2296129 100644
--- a/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts
+++ b/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts
@@ -951,6 +951,22 @@ retimer_ss0_con_sbu_out: endpoint {
 	};
 };
 
+&i2c5 {
+	clock-frequency = <400000>;
+
+	status = "okay";
+
+	embedded-controller@76 {
+		compatible = "lenovo,yoga-slim7x-ec", "qcom,hamoa-crd-ec";
+		reg = <0x76>;
+
+		interrupts-extended = <&tlmm 66 IRQ_TYPE_EDGE_FALLING>;
+
+		pinctrl-0 = <&ec_int_n_default>;
+		pinctrl-names = "default";
+	};
+};
+
 &i2c7 {
 	clock-frequency = <400000>;
 
@@ -1352,6 +1368,12 @@ &tlmm {
 			       <44 4>, /* SPI (TPM) */
 			       <238 1>; /* UFS Reset */
 
+	ec_int_n_default: ec-int-n-state {
+		pins = "gpio66";
+		function = "gpio";
+		bias-disable;
+	};
+
 	edp_reg_en: edp-reg-en-state {
 		pins = "gpio70";
 		function = "gpio";
-- 
2.53.0


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

* Re: [PATCH v3 1/2] dt-bindings: embedded-controller: qcom,hamoa-crd-ec: add Lenovo Yoga Slim 7x
  2026-05-26 11:24 [PATCH v3 1/2] dt-bindings: embedded-controller: qcom,hamoa-crd-ec: add Lenovo Yoga Slim 7x Daniel J Blueman
  2026-05-26 11:24 ` [PATCH v3 2/2] arm64: dts: qcom: x1e80100-lenovo-yoga-slim7x: Add Embedded Controller node Daniel J Blueman
@ 2026-05-26 12:24 ` Bryan O'Donoghue
  2026-05-27  9:41 ` Krzysztof Kozlowski
  2 siblings, 0 replies; 8+ messages in thread
From: Bryan O'Donoghue @ 2026-05-26 12:24 UTC (permalink / raw)
  To: Daniel J Blueman, Konrad Dybcio
  Cc: linux-arm-msm, devicetree, Sibi Sankar, Randy Dunlap,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, Hans de Goede,
	Bjorn Andersson, linux-kernel, Anvesh Jain P, Maya Matuszczyk,
	Krzysztof Kozlowski, Dmitry Baryshkov, Akhil P Oommen, Abel Vesa,
	Gaurav Kohli

On 26/05/2026 12:24, Daniel J Blueman wrote:
> The Lenovo Yoga Slim 7x uses the same Embedded Controller as the Qualcomm
> Hamoa X1 CRD. Add a board-specific compatible with qcom,hamoa-crd-ec as
> the fallback.
> 
> Signed-off-by: Daniel J Blueman <daniel@quora.org>
> ---
> Changes in v3:
> - new patch (binding was missing in v1/v2)
> 
> Dependencies:
> https://lore.kernel.org/all/20260427-add-driver-for-ec-v8-0-702f74e495f7@oss.qualcomm.com/
> 
>   .../bindings/embedded-controller/qcom,hamoa-crd-ec.yaml          | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/embedded-controller/qcom,hamoa-crd-ec.yaml b/Documentation/devicetree/bindings/embedded-controller/qcom,hamoa-crd-ec.yaml
> index ac5a08f8f76d..813d41769c0b 100644
> --- a/Documentation/devicetree/bindings/embedded-controller/qcom,hamoa-crd-ec.yaml
> +++ b/Documentation/devicetree/bindings/embedded-controller/qcom,hamoa-crd-ec.yaml
> @@ -20,6 +20,7 @@ properties:
>       oneOf:
>         - items:
>             - enum:
> +              - lenovo,yoga-slim7x-ec
>                 - qcom,glymur-crd-ec
>                 - qcom,hamoa-iot-evk-ec
>             - const: qcom,hamoa-crd-ec

Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>

---
bod

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

* Re: [PATCH v3 2/2] arm64: dts: qcom: x1e80100-lenovo-yoga-slim7x: Add Embedded Controller node
  2026-05-26 11:24 ` [PATCH v3 2/2] arm64: dts: qcom: x1e80100-lenovo-yoga-slim7x: Add Embedded Controller node Daniel J Blueman
@ 2026-05-26 12:25   ` Bryan O'Donoghue
  2026-05-29 16:12   ` Sibi Sankar
  2026-06-03 12:58   ` Konrad Dybcio
  2 siblings, 0 replies; 8+ messages in thread
From: Bryan O'Donoghue @ 2026-05-26 12:25 UTC (permalink / raw)
  To: Daniel J Blueman, Konrad Dybcio
  Cc: linux-arm-msm, devicetree, Sibi Sankar, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Hans de Goede, Randy Dunlap,
	Bjorn Andersson, linux-kernel, Anvesh Jain P, Maya Matuszczyk,
	Krzysztof Kozlowski, Dmitry Baryshkov, Akhil P Oommen, Abel Vesa,
	Gaurav Kohli

On 26/05/2026 12:24, Daniel J Blueman wrote:
> The Lenovo Slim7x uses the same Embedded Controller as the Qualcomm Hamoa
> X1 Customer Reference Device. Use the lenovo,yoga-slim7x-ec compatible
> introduced by patch 1 for fan control, thermal sensor and suspend
> behaviour.
> 
> Signed-off-by: Daniel J Blueman <daniel@quora.org>
> ---
> Changes in v3:
> - use lenovo,yoga-slim7x-ec compatible (introduced by patch 1)
> - v2 link: https://lore.kernel.org/lkml/20260502063518.15153-1-daniel@quora.org/
> Changes in v2:
> - corrected DT compatible node
> - v1 link: https://lore.kernel.org/all/20260429103301.17449-1-daniel@quora.org/
> 
>   .../dts/qcom/x1e80100-lenovo-yoga-slim7x.dts  | 22 +++++++++++++++++++
>   1 file changed, 22 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts b/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts
> index beb1475d7fa0..1ee2a2296129 100644
> --- a/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts
> +++ b/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts
> @@ -951,6 +951,22 @@ retimer_ss0_con_sbu_out: endpoint {
>   	};
>   };
>   
> +&i2c5 {
> +	clock-frequency = <400000>;
> +
> +	status = "okay";
> +
> +	embedded-controller@76 {
> +		compatible = "lenovo,yoga-slim7x-ec", "qcom,hamoa-crd-ec";
> +		reg = <0x76>;
> +
> +		interrupts-extended = <&tlmm 66 IRQ_TYPE_EDGE_FALLING>;
> +
> +		pinctrl-0 = <&ec_int_n_default>;
> +		pinctrl-names = "default";
> +	};
> +};
> +
>   &i2c7 {
>   	clock-frequency = <400000>;
>   
> @@ -1352,6 +1368,12 @@ &tlmm {
>   			       <44 4>, /* SPI (TPM) */
>   			       <238 1>; /* UFS Reset */
>   
> +	ec_int_n_default: ec-int-n-state {
> +		pins = "gpio66";
> +		function = "gpio";
> +		bias-disable;
> +	};
> +
>   	edp_reg_en: edp-reg-en-state {
>   		pins = "gpio70";
>   		function = "gpio";

Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>

---
bod

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

* Re: [PATCH v3 1/2] dt-bindings: embedded-controller: qcom,hamoa-crd-ec: add Lenovo Yoga Slim 7x
  2026-05-26 11:24 [PATCH v3 1/2] dt-bindings: embedded-controller: qcom,hamoa-crd-ec: add Lenovo Yoga Slim 7x Daniel J Blueman
  2026-05-26 11:24 ` [PATCH v3 2/2] arm64: dts: qcom: x1e80100-lenovo-yoga-slim7x: Add Embedded Controller node Daniel J Blueman
  2026-05-26 12:24 ` [PATCH v3 1/2] dt-bindings: embedded-controller: qcom,hamoa-crd-ec: add Lenovo Yoga Slim 7x Bryan O'Donoghue
@ 2026-05-27  9:41 ` Krzysztof Kozlowski
  2026-05-27 12:10   ` Daniel J Blueman
  2 siblings, 1 reply; 8+ messages in thread
From: Krzysztof Kozlowski @ 2026-05-27  9:41 UTC (permalink / raw)
  To: Daniel J Blueman
  Cc: Konrad Dybcio, linux-arm-msm, devicetree, Sibi Sankar,
	Randy Dunlap, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Hans de Goede, Bryan O'Donoghue, Bjorn Andersson,
	linux-kernel, Anvesh Jain P, Maya Matuszczyk,
	Krzysztof Kozlowski, Dmitry Baryshkov, Akhil P Oommen, Abel Vesa,
	Gaurav Kohli

On Tue, May 26, 2026 at 07:24:04PM +0800, Daniel J Blueman wrote:
> The Lenovo Yoga Slim 7x uses the same Embedded Controller as the Qualcomm

Are you sure it is the same?

Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>

Best regards,
Krzysztof


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

* Re: [PATCH v3 1/2] dt-bindings: embedded-controller: qcom,hamoa-crd-ec: add Lenovo Yoga Slim 7x
  2026-05-27  9:41 ` Krzysztof Kozlowski
@ 2026-05-27 12:10   ` Daniel J Blueman
  0 siblings, 0 replies; 8+ messages in thread
From: Daniel J Blueman @ 2026-05-27 12:10 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Konrad Dybcio, linux-arm-msm, devicetree, Sibi Sankar,
	Randy Dunlap, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Hans de Goede, Bryan O'Donoghue, Bjorn Andersson,
	linux-kernel, Anvesh Jain P, Maya Matuszczyk,
	Krzysztof Kozlowski, Dmitry Baryshkov, Akhil P Oommen, Abel Vesa,
	Gaurav Kohli

On Wed, 27 May 2026 at 17:41, Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
> On Tue, May 26, 2026 at 07:24:04PM +0800, Daniel J Blueman wrote:
> > The Lenovo Yoga Slim 7x uses the same Embedded Controller as the Qualcomm
>
> Are you sure it is the same?

Yes, this is validated on my Slim7x. Both devices use the IT8987 EC
and Maya's amazing work found similar firmware strings.

Thanks,
  Dan
-- 
Daniel J Blueman

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

* Re: [PATCH v3 2/2] arm64: dts: qcom: x1e80100-lenovo-yoga-slim7x: Add Embedded Controller node
  2026-05-26 11:24 ` [PATCH v3 2/2] arm64: dts: qcom: x1e80100-lenovo-yoga-slim7x: Add Embedded Controller node Daniel J Blueman
  2026-05-26 12:25   ` Bryan O'Donoghue
@ 2026-05-29 16:12   ` Sibi Sankar
  2026-06-03 12:58   ` Konrad Dybcio
  2 siblings, 0 replies; 8+ messages in thread
From: Sibi Sankar @ 2026-05-29 16:12 UTC (permalink / raw)
  To: Daniel J Blueman, Konrad Dybcio
  Cc: linux-arm-msm, devicetree, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Hans de Goede, Randy Dunlap, Bryan O'Donoghue,
	Bjorn Andersson, linux-kernel, Anvesh Jain P, Maya Matuszczyk,
	Krzysztof Kozlowski, Dmitry Baryshkov, Akhil P Oommen, Abel Vesa,
	Gaurav Kohli


On 5/26/2026 4:54 PM, Daniel J Blueman wrote:
> The Lenovo Slim7x uses the same Embedded Controller as the Qualcomm Hamoa
> X1 Customer Reference Device. Use the lenovo,yoga-slim7x-ec compatible
> introduced by patch 1 for fan control, thermal sensor and suspend
> behaviour.
>
> Signed-off-by: Daniel J Blueman <daniel@quora.org>
> ---
> Changes in v3:
> - use lenovo,yoga-slim7x-ec compatible (introduced by patch 1)
> - v2 link: https://lore.kernel.org/lkml/20260502063518.15153-1-daniel@quora.org/
> Changes in v2:
> - corrected DT compatible node
> - v1 link: https://lore.kernel.org/all/20260429103301.17449-1-daniel@quora.org/


Reviewed-by: Sibi Sankar <sibi.sankar@oss.qualcomm.com>

>
>   .../dts/qcom/x1e80100-lenovo-yoga-slim7x.dts  | 22 +++++++++++++++++++
>   1 file changed, 22 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts b/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts
> index beb1475d7fa0..1ee2a2296129 100644
> --- a/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts
> +++ b/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts
> @@ -951,6 +951,22 @@ retimer_ss0_con_sbu_out: endpoint {
>   	};
>   };
>   
> +&i2c5 {
> +	clock-frequency = <400000>;
> +
> +	status = "okay";
> +
> +	embedded-controller@76 {
> +		compatible = "lenovo,yoga-slim7x-ec", "qcom,hamoa-crd-ec";
> +		reg = <0x76>;
> +
> +		interrupts-extended = <&tlmm 66 IRQ_TYPE_EDGE_FALLING>;
> +
> +		pinctrl-0 = <&ec_int_n_default>;
> +		pinctrl-names = "default";
> +	};
> +};
> +
>   &i2c7 {
>   	clock-frequency = <400000>;
>   
> @@ -1352,6 +1368,12 @@ &tlmm {
>   			       <44 4>, /* SPI (TPM) */
>   			       <238 1>; /* UFS Reset */
>   
> +	ec_int_n_default: ec-int-n-state {
> +		pins = "gpio66";
> +		function = "gpio";
> +		bias-disable;
> +	};
> +
>   	edp_reg_en: edp-reg-en-state {
>   		pins = "gpio70";
>   		function = "gpio";

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

* Re: [PATCH v3 2/2] arm64: dts: qcom: x1e80100-lenovo-yoga-slim7x: Add Embedded Controller node
  2026-05-26 11:24 ` [PATCH v3 2/2] arm64: dts: qcom: x1e80100-lenovo-yoga-slim7x: Add Embedded Controller node Daniel J Blueman
  2026-05-26 12:25   ` Bryan O'Donoghue
  2026-05-29 16:12   ` Sibi Sankar
@ 2026-06-03 12:58   ` Konrad Dybcio
  2 siblings, 0 replies; 8+ messages in thread
From: Konrad Dybcio @ 2026-06-03 12:58 UTC (permalink / raw)
  To: Daniel J Blueman
  Cc: linux-arm-msm, devicetree, Sibi Sankar, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Hans de Goede, Randy Dunlap,
	Bryan O'Donoghue, Bjorn Andersson, linux-kernel,
	Anvesh Jain P, Maya Matuszczyk, Krzysztof Kozlowski,
	Dmitry Baryshkov, Akhil P Oommen, Abel Vesa, Gaurav Kohli



On 26-May-26 13:24, Daniel J Blueman wrote:
> The Lenovo Slim7x uses the same Embedded Controller as the Qualcomm Hamoa
> X1 Customer Reference Device. Use the lenovo,yoga-slim7x-ec compatible
> introduced by patch 1 for fan control, thermal sensor and suspend
> behaviour.
> 
> Signed-off-by: Daniel J Blueman <daniel@quora.org>
> ---

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

Konrad

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

end of thread, other threads:[~2026-06-03 12:58 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-05-26 11:24 [PATCH v3 1/2] dt-bindings: embedded-controller: qcom,hamoa-crd-ec: add Lenovo Yoga Slim 7x Daniel J Blueman
2026-05-26 11:24 ` [PATCH v3 2/2] arm64: dts: qcom: x1e80100-lenovo-yoga-slim7x: Add Embedded Controller node Daniel J Blueman
2026-05-26 12:25   ` Bryan O'Donoghue
2026-05-29 16:12   ` Sibi Sankar
2026-06-03 12:58   ` Konrad Dybcio
2026-05-26 12:24 ` [PATCH v3 1/2] dt-bindings: embedded-controller: qcom,hamoa-crd-ec: add Lenovo Yoga Slim 7x Bryan O'Donoghue
2026-05-27  9:41 ` Krzysztof Kozlowski
2026-05-27 12:10   ` Daniel J Blueman

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

all inboxes | Powered by JetHome®