From: Krzysztof Kozlowski <krzk@kernel.org>
To: Maya Matuszczyk <maccraft123mc@gmail.com>
Cc: Stephan Gerhold <stephan.gerhold@linaro.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
linux-arm-msm@vger.kernel.org,
devicetree <devicetree@vger.kernel.org>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 1/3] dt-bindings: platform: Add bindings for Qcom's EC on IT8987
Date: Mon, 23 Dec 2024 15:25:23 +0100 [thread overview]
Message-ID: <26ba3710-9f7f-40ae-a65d-e1d6e384a142@kernel.org> (raw)
In-Reply-To: <CAO_Mup+YFAT5oFRGYucW5h_eGcfp4C5FzF20hM6xF=qMR1DdQw@mail.gmail.com>
On 22/12/2024 16:07, Maya Matuszczyk wrote:
>
> With similar ones at the end:
> Devkit:
> EC VER:00.29.00$
> LsFv:00.29.00$
> Qualcomm$
> WoS 8c GenX$
> ODM$
> MB:A0$
> BUILD DATE:
> 02/0//2/24$
> TIME:
> 14:33:35$
>
> Slim7x:
> EC VER:00.60.00$
> LsFv:00.20.00$
> Qualcomm$
> WoS 8c GenX$
> ODM$
> MB:A0$
> BUILD DATE:
> 2024/07/25$
> TIME:
> 09:58:00$
>
>
>
>>
>> If the FW across devices is the same, then fallbacks for these are fine
>> with me.
>
> As the devkit has EC firmware that is handled the same way in DSDT
> tables of most of other x1e laptops with the same EC, and is a subset
> of what's done on Lenovo Yoga Slim 7x and Honor Magicbook Art 14 I
> think the devkit's compatible + -ec would be a good pick.
>
> This conversation is getting long and I feel like I've said everything
> I wanted to say, I'll just do what you tell me to do about the
> fallback and binding filename.
Go with a device specific compatible (I think I proposed in other
email). It is up to you if you want to add fallbacks. If you add
fallbacks, then please include some summary of above in commit msg or
binding description, so there will be a trace of that explanation/reason
for fallbacks.
Best regards,
Krzysztof
next prev parent reply other threads:[~2024-12-23 14:25 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-12-19 20:08 [PATCH v2 1/3] dt-bindings: platform: Add bindings for Qcom's EC on IT8987 Maya Matuszczyk
2024-12-19 20:08 ` [PATCH v2 3/3] arm64: dts: qcom: x1e80100-lenovo-yoga-slim7x: Add the EC Maya Matuszczyk
2024-12-20 11:52 ` Aiqun(Maria) Yu
2024-12-20 12:10 ` Konrad Dybcio
2024-12-19 23:40 ` [PATCH v2 1/3] dt-bindings: platform: Add bindings for Qcom's EC on IT8987 Rob Herring (Arm)
2024-12-20 18:05 ` Stephan Gerhold
[not found] ` <CAO_MupJ7JtXNgGyXcxGa+EGAvsu-yG0O6MgneGUBdCEgKNG+MA@mail.gmail.com>
2024-12-20 18:24 ` Stephan Gerhold
2024-12-22 14:40 ` Krzysztof Kozlowski
2024-12-22 15:07 ` Maya Matuszczyk
2024-12-23 14:25 ` Krzysztof Kozlowski [this message]
2024-12-30 2:45 ` Aiqun(Maria) Yu
2025-01-14 10:23 ` Aiqun(Maria) Yu
2024-12-22 6:33 ` Krzysztof Kozlowski
2024-12-22 6:40 ` Krzysztof Kozlowski
2024-12-22 7:55 ` Maya Matuszczyk
2024-12-22 9:40 ` Maya Matuszczyk
2024-12-22 14:34 ` Krzysztof Kozlowski
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=26ba3710-9f7f-40ae-a65d-e1d6e384a142@kernel.org \
--to=krzk@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=maccraft123mc@gmail.com \
--cc=robh@kernel.org \
--cc=stephan.gerhold@linaro.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).