From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9BD6C30E0E9 for ; Tue, 7 Jul 2026 08:20:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.180.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783412413; cv=none; b=i98g3Xl6mtrQ/9PADmPqSJ6wWpeBtEwcMDjO2mipnZlhbiyU9iiRGLBY8H4OIpZsAdQj194u+JJpBG70ayocQqfeHi6TY5JtBlqJGcZ7fQhVES9FVc3HFPRTs/dconHZqkGKMwgpKVj7uuPhxhPak0OHJIRCCBhh9tnWdMxT/MI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783412413; c=relaxed/simple; bh=n5OeMK1bQ8yHLqI/ZoKerpO3/6fLbnwqt2JRjj+RJ4I=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=Lsae1uC/GVb2W+kXMTMG/VkbcRT1/wJGa6Kq5fEUSfVqm+GLv/gINiIiPuDpb1j5M/jmHkskrR/x/TeW7sTWZEbhedP4eBLcjwChT/YyGTgyJi+KNUISkiYVZF6TdtQ4YYNsmdWDiliFn8yPDuMbps6Mn5WSDUq2FqTuHxKSm5w= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=OwR7a2IU; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=DWupFSWi; arc=none smtp.client-ip=205.220.180.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="OwR7a2IU"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="DWupFSWi" Received: from pps.filterd (m0279873.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 6678Djnt3138631 for ; Tue, 7 Jul 2026 08:20:10 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= ajiWotAS1acnmFOssC6CEZcKuC/BL6UhgNsm6/8ZsA8=; b=OwR7a2IU4grZsFRg tJ10AyJ767+wHooZigQqyP7lwpzdi3QdLvIFSI4IvL3agnXL0xvBSDDMx7nbEbDe w046s+2s6XOJwzSDHgfaVnQDNoc/rmfbOQDjlje9RH1Ypz8ZJ5fSr4cEIzKCUtLP bEJiKt9Ysv42GJY9bXE/vKCxsfnAFfLK7UvWZ5Zjx20ZA1cmi6kdWRvE0MWwUQAo GlZaKUAFocOyJw1gCeyHUG9jpPzXc4tBApLe4hOpkB9ujldJD3nEAy30/fcub8k7 x9Mc2WAvq+QuwW0j+jldxp+LNsSrO2+wBXd1+2MEmrkf5boAOistFdWg93RmPguL b9R5ig== Received: from mail-qt1-f197.google.com (mail-qt1-f197.google.com [209.85.160.197]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4f8t158y6u-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Tue, 07 Jul 2026 08:20:10 +0000 (GMT) Received: by mail-qt1-f197.google.com with SMTP id d75a77b69052e-51c01ff996dso53651451cf.2 for ; Tue, 07 Jul 2026 01:20:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1783412410; x=1784017210; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:from:to:cc:subject:date:message-id:reply-to :content-type; bh=ajiWotAS1acnmFOssC6CEZcKuC/BL6UhgNsm6/8ZsA8=; b=DWupFSWi5kAwf/750V4ez4Qfn0FKacmv4GMhLlti/dNYGGJ4rQn3DqEKAnsTllxWej 6d7bHVcMnO/ID4KhVnPxVQj1ZX6GcJ0TazJ6LyckjjdR3dIB1xyRDplpWMjgJLtkpZeh aL0fpszGEu067/XUDR4Ab0s0TonsfoBwObG0zGTT/esKHYg56uN5Sgoh3/mzmpesaunU 8vOf7hoirmnmKxnDVNDM+JVCvsVauVegMcFkeeqxXAprzzH027zCnd/4ALvdsZqV/TYs kQgs6YP2yTrOu0ulwhRPE1F0xzo76gEz6CTgAiNTzsgD/rhoQu6cLkd7aoRrjC4fLDg0 A8Dw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783412410; x=1784017210; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=ajiWotAS1acnmFOssC6CEZcKuC/BL6UhgNsm6/8ZsA8=; b=lAF21HrG3YXY3445Ll6WVzp0Y9iFvbQbh4GIzn+/GLsAv7insdHwUqdPXTMs75lQHl EYTVzOnXW5c74pfbAyIvwAwgPo1In6iXy12/Zw0mlGjqT3XmDrzQeViVTjSNmTPV0Ae3 kBGTk3dB2diEltXu3sIR0nX9V2U4W5JAE4jqo4fj0CExEMKnJcAogZbDFzVTS1HV2DBi /4aLkj6eDuuE8Xe9sKdPraoUONNvvVoBSqRY6xBHTqNVcDM6wu5V8mp05laIRAra1aSl /recvRRz2MOaRk6x7xhH+Rr/fyC/XGcKdbIfjLG/hXfR/XfLX1k6WjJFuuOuBVLhtHm4 eMGQ== X-Forwarded-Encrypted: i=1; AHgh+RpcueXjenpB6IdgN1+wf1q9/EQmMOAe7zJqVzh7GWvysDs6q9D2bY8xdkkkQmvmfDHroMryIXXcIvx7BUw=@vger.kernel.org X-Gm-Message-State: AOJu0YzyrLmqiLolN/QPsSzCDEhiihdJuVZTDDvusYPkQk90HQroAmMT 3kXwFh0Z8pG1bNBq+eb0Y7qxTDqnwKAv3w7LoHnPDlkE3cz5WTtnyczQYbzOwRBGwIJJoNGXgv3 EVvYDbdBfy5UGh+RSCchCxfwFUPXU0qrSLAxWI1aVOP/8q8734rF5PC3yh0wDoJ6QhVg= X-Gm-Gg: AfdE7cmH8mgvx5AeNPJFzxM7OnZN985zRdYnfsKQDSW0jAuyuQrkUpPDsarJwNIxuZg PKo8L8cE7uYfr652PfRQpofTS2ZiWUSnHgPJlG32Zbpi7nZ8nanUuuA2am9co+Zax/jsye4nWR5 ujSTVgm0QDr1O9Y65hHBQPJb/ZwM81zyfPeZ/UvJftlxUAmi7r21KyrFZd0GbYtiYQSz/IKE4fI QLxkNH1agmi0FC4EOW3X0k2rnwRmXBP+SBytovUzr4M+pKMfd8o3++OaiDPZis2CT9lt5/4vDDb HTI/KGg2PKwwhyc2T9kzoylYE/l3ffsXiCrQK+3XSSX8WCn1KkLTc3AFOP7rMbZ7So7Fim41iWi EcFOfvOr7/aP/kKqshZuUjLg3J5wD40C2u+aLNGc0uEcN0snxaA8CUpjtSplvOIaNE2OyjaSOXQ == X-Received: by 2002:ac8:7d0c:0:b0:51c:12bf:d4a6 with SMTP id d75a77b69052e-51c7487df08mr45476091cf.48.1783412409848; Tue, 07 Jul 2026 01:20:09 -0700 (PDT) X-Received: by 2002:ac8:7d0c:0:b0:51c:12bf:d4a6 with SMTP id d75a77b69052e-51c7487df08mr45475921cf.48.1783412409444; Tue, 07 Jul 2026 01:20:09 -0700 (PDT) Received: from ?IPV6:2a05:6e02:1041:c10:7d20:e20f:a77f:4ef7? ([2a05:6e02:1041:c10:7d20:e20f:a77f:4ef7]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-493e0f32660sm37747195e9.2.2026.07.07.01.20.08 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 07 Jul 2026 01:20:08 -0700 (PDT) Message-ID: <38f5a2ec-ddc5-48d8-b76b-c2ed617392ca@oss.qualcomm.com> Date: Tue, 7 Jul 2026 10:20:07 +0200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v5 1/3] dt-bindings: embedded-controller: Add Lenovo ThinkPad T14s thermal sensor provider support To: Krzysztof Kozlowski Cc: sre@kernel.org, hansg@kernel.org, ilpo.jarvinen@linux.intel.com, linux@roeck-us.net, andersson@kernel.org, konradybcio@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, bryan.odonoghue@linaro.org, platform-driver-x86@vger.kernel.org, linux-kernel@vger.kernel.org, linux-hwmon@vger.kernel.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, neil.armstrong@linaro.org, gaurav.kohli@oss.qualcomm.com, manaf.pallikunhi@oss.qualcomm.com, priyansh.jain@oss.qualcomm.com References: <20260706184648.35613-1-daniel.lezcano@oss.qualcomm.com> <20260706184648.35613-2-daniel.lezcano@oss.qualcomm.com> <20260707-armored-azure-terrier-4dc3d3@quoll> <8d579153-efa5-4c8e-acae-87f3bdfbfc24@oss.qualcomm.com> <457242e3-6a40-49a2-8d86-214f9167a928@kernel.org> Content-Language: en-US From: Daniel Lezcano In-Reply-To: <457242e3-6a40-49a2-8d86-214f9167a928@kernel.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Proofpoint-GUID: hJQgCdhsRISTj0kLmte7pEIgcDijZOcD X-Authority-Analysis: v=2.4 cv=HstG3UTS c=1 sm=1 tr=0 ts=6a4cb6ba cx=c_pps a=EVbN6Ke/fEF3bsl7X48z0g==:117 a=xqWC_Br6kY4A:10 a=IkcTkHD0fZMA:10 a=RAioF0-LDSMA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=rJkE3RaqiGZ5pbrm-msn:22 a=EUspDBNiAAAA:8 a=qYa7bHtEPbO8xArO7mAA:9 a=QEXdDO2ut3YA:10 a=a_PwQJl-kcHnX1M80qC6:22 X-Proofpoint-ORIG-GUID: hJQgCdhsRISTj0kLmte7pEIgcDijZOcD X-Proofpoint-Spam-Info: AW1haW4tMjYwNzA3MDA3OCBTYWx0ZWRfX2KV9oe8wa1Bq 7voGEocP1nl97HACsdxNQbarzX3oOuAeymU3SWOaRIv+gS4+XRT72w4g9ZDdNAcSpR5+tOXtr7c Hs8/9Bzjp/NUNbcBRpNq3EgBI0fZ4f0= X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNzA3MDA3OCBTYWx0ZWRfX0mf+JjWenYJf su7Bt82GCp3avwZInMhlEq95RdZfjIudqtflukFHXrTcdp2BIMSEzg+rU0BxOOVYrppu9rnK+b8 Bs4DpLFIARjGKI+PhiTUVgCDQxdhfKyNqH6Ni3HYcmLVp4nMJXIWzOGigNVg61TJDtOrIe2+8Yd gyRnHRLOGsxWNmtyAwPGxsI0iB+zRVME/owqLiYRUr4fPZIYIyMjvkVlzGSs6FeqQFIdG5ajjor pv6dxY7VXWCZR+flZ3xxDIW53n6FrlGn856YyQUHwSeXpPizBzHLDS9VH4YvkgoHXCxYkM1Jdwg IAA19yoiimX54h1lvEbmTgvHKbs2SUcLDzTVXqIq2N91+Hm54yMbV6G0aCxFPVHcZgfO28LhwVn 4VggAngaYrvJvVYTlW+9U0heQDvG9OpkUF4+dp1bOcJQ2ZVXvBAkzgpLNK74Od2JH3Sj3e0Ce7G Xxuuefe3SUDzchqxF9g== X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.134,FMLib:17.12.100.49 definitions=2026-07-07_02,2026-07-06_02,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 phishscore=0 impostorscore=0 suspectscore=0 malwarescore=0 bulkscore=0 clxscore=1015 spamscore=0 adultscore=0 lowpriorityscore=0 priorityscore=1501 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2606150000 definitions=main-2607070078 On 7/7/26 10:09, Krzysztof Kozlowski wrote: > On 07/07/2026 10:08, Daniel Lezcano wrote: >> On 7/7/26 08:38, Krzysztof Kozlowski wrote: >>> On Mon, Jul 06, 2026 at 08:46:46PM +0200, Daniel Lezcano wrote: >>>> Document the Lenovo ThinkPad T14s Embedded Controller as a thermal >>>> sensor provider by adding the '#thermal-sensor-cells' property. >>>> >>>> This allows the EC temperature sensors to be referenced from thermal >>>> zones in the device tree. >>>> >>>> Signed-off-by: Daniel Lezcano >>>> --- >>>> .../embedded-controller/lenovo,thinkpad-t14s-ec.yaml | 5 +++++ >>>> 1 file changed, 5 insertions(+) >>>> >>>> diff --git a/Documentation/devicetree/bindings/embedded-controller/lenovo,thinkpad-t14s-ec.yaml b/Documentation/devicetree/bindings/embedded-controller/lenovo,thinkpad-t14s-ec.yaml >>>> index c87ccb5b3086..0d049979e35b 100644 >>>> --- a/Documentation/devicetree/bindings/embedded-controller/lenovo,thinkpad-t14s-ec.yaml >>>> +++ b/Documentation/devicetree/bindings/embedded-controller/lenovo,thinkpad-t14s-ec.yaml >>>> @@ -26,10 +26,14 @@ properties: >>>> >>>> wakeup-source: true >>>> >>>> + "#thermal-sensor-cells": >>>> + const: 1 >>>> + >>>> required: >>>> - compatible >>>> - reg >>>> - interrupts >>>> + - "#thermal-sensor-cells" >>> >>> This looks like ABI break without explanation. >> >> What would you suggest ? explanation or removed from required properties? > > I don't know what is the explanation. If this is actual ABI break as I > read Linux code, what is the rationale behind breaking real users? IIUC, if the DT has not #thermal-sensor-cells, but the code has the patch 2/3 then the hwmon fails to initialize as well as the EC controller, right ?