Re: [PATCH v2] arm64: dts: qcom: slim7x: Add Embedded Controller node

[Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>]

On 5/2/26 8:35 AM, Daniel J Blueman wrote:
> The Lenovo Slim7x uses the same Embedded Controller as the Qualcomm
> Hamoa Customer Reference Device (X1). Wire it up for fan control,
> thermal sensor and suspend behaviour.
> 
> Signed-off-by: Daniel J Blueman <daniel@quora.org>
> ---
> Changes in v2:
> - corrected DT compatible node

You still need to introduce one specific to your actual hardware,
even if the driver receives no changes at the moment.

This is a measure to enable retroactively enabling quirks for users
that update their kernel but not their DTB, so that if a year from
now it turns out that the yoga EC requires some special handling (but
other users of qcom,hamoa-crd-ec don't), we can easily quirk it out

i.e., the desired outcome here is something like:

compatible = "lenovo,yoga-slim7x-ec",
	     "qcom,hamoa-crd-ec";

(the first compatible is ignored by the driver matching code today,
the other ("fallback") one is matched on)

This will need an accompanying change, adding the new Lenovo
compatible to the dt-bindings file

Konrad