From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.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 1F4EF29B78B for ; Tue, 10 Jun 2025 14:09:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.168.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749564565; cv=none; b=MT/Rk1M0ZBJNyF83iR9HjWf7zz6bs9E0ZNcTFT01jL4swyMRcCmoV9+u0uj7qUWAmU7l9XcgeiNugAKfBh4HswJdQL6zagHjVBfujEwvdxehonmfAyfDXaZ0did9XE3LDLJcVgBSQiVRB0JmlI11m1UvTu8E1j9ABzmwg5EttB8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749564565; c=relaxed/simple; bh=64oL1IuUL6LDShJrW7z+kKTXElQQl4FAWK8vmLE6BNg=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=ZbA8drADkOLrZDc16VCvFoRwS2a77UPOFdct800n1fE90ssBS3YwfJk6FlUzZJ91Xbftf8Uv86EHzjpeguanW0NLXVjfbZkZSlZt+BOU35HVAycL/kNjGp1PXRxt4OlylaAwZmDSFdIWJHkQ/uQ75kURolPH2L49vmrU200xR98= 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=XBIx6CtN; arc=none smtp.client-ip=205.220.168.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="XBIx6CtN" Received: from pps.filterd (m0279867.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 55A6PlhI003755 for ; Tue, 10 Jun 2025 14:09:23 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= gP4Jd0wZuukS80MSwtMl4EeLp4f+KtIALdhGXarYaPU=; b=XBIx6CtNaZzvB0WU a6Jyu2bBHjsjyam2qsljk1cGYDp644pMzaMwezxnE80cCmN2AduJtDXRYv8JlN3d sanK5mMNYey47Fqu6Fun4HV5HL39fM2tNcGMVw+/5xlKEfMPCQhxm74xutYPbjNk XaEpffomlc94SLryZ8Sgxw5dAuy0uEoiayVkyTL3DLwitAVhSp9g3FsK8BEQzr55 2WDTdwu/va8MPQfarXvqzRC6cJ/Pv55DOcA+yKixkpExrBoeubw/VjaYwPhI6HKM HFNSWPnH5yaVGejWAFrw+7/alL5b6Zvab9F5NB2G+/VG43n2h4cEOh+c7fsifZKv p/+c5A== Received: from mail-qt1-f200.google.com (mail-qt1-f200.google.com [209.85.160.200]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 475v2y4g7y-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NOT) for ; Tue, 10 Jun 2025 14:09:23 +0000 (GMT) Received: by mail-qt1-f200.google.com with SMTP id d75a77b69052e-4a5ae436badso13451481cf.2 for ; Tue, 10 Jun 2025 07:09:23 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1749564562; x=1750169362; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=gP4Jd0wZuukS80MSwtMl4EeLp4f+KtIALdhGXarYaPU=; b=ej0pBETpaqFxKYceTZ2Glzps5UCyZJWbXjlXibwIRJE7oSMRLlo9KHM52pvWslF+VI y2I5jmUOkbbh2pDD1hcsWgqkiLkLg0xut+hMMmtWPJ1nKw2M3UkFplgTO+8k31kDBAqK UL6dmDwMwebX0tReaMDJnVKMDnfjla1MhOjjWq1DHt4EMSfP56A1fWcB2KRzZ2uCKHFi xvwA32E1s4oSsZx+q/5ZNdGRucMTRMMWweiC/nhnCsk95O+GMJE2T83QW2uMQ2UWaLAf Imxi67wl7X38+Ug0Kdhbo/B/9vqC6njLQlah7KnufCiBqou6ljoCZTlefVwvFN/+KUKy C9Jg== X-Forwarded-Encrypted: i=1; AJvYcCV/DiLNgcgKtIURV1EOPP3DVAcQSmzaI4sBELPGmIN6B1amq0OOXWuMePh0WK7/EdXKoVmS13t+Bw==@vger.kernel.org X-Gm-Message-State: AOJu0YwRFWMLm2ikLxYFtskgA/iuGVUhskjf6K++fcvu5om/EArCV0w7 vH9al97kORUIbYePaniocgylbDd0wBeKd25BdGCvvDCQcV1acPo5Jdbb+mjJ6ecP8B0c0QkKbho 9SCuT8yT3MdSPU+IYuKnZbqSK9kufVnXuSsbNUUjv9HspvC3Sla2R9K99tJyxAA== X-Gm-Gg: ASbGncs9JfLCWuCmbPGjEBsYMW+hMon3hC0VO8OgZcCCGkTVoZZtnkMpheM8XJ/bd1e r+ZTm8YUMTrFTr4FhWIEQUluZbkUYvf7W+Fqvlh+IlFXl1ajgcjKwQMzkE+a6FqUzAC2ud4Stp3 sdn+83vAfIc+yVLhqGsruodJkxflAbEg/w++JcuUZ4sn59034t/CzF3tVwtOaDU5lzAdnEQLmXN AWM0Z3P3axXd8y1ZqmnYFj5RjhsUPQa4DIhgHHO9Xy25uNvj4ObKw5cuEY8vuU+KDcDvGYLGiow qubiSFHNDIDauVKqnFmHDeVybr6vOa3zv0m/0KQVhqSXFj4nRK74LVHf3BQ9BvBZATr6OAFWtFR J X-Received: by 2002:a05:620a:294a:b0:7d0:aafe:dd6 with SMTP id af79cd13be357-7d331c11a81mr969897385a.1.1749564561946; Tue, 10 Jun 2025 07:09:21 -0700 (PDT) X-Google-Smtp-Source: AGHT+IERgZgp2y5pYp48AInVGbjIfwav55zx9yDpitn4roraA79eZtpYgR9Fcxn3XP8O4dbac3evfg== X-Received: by 2002:a05:620a:294a:b0:7d0:aafe:dd6 with SMTP id af79cd13be357-7d331c11a81mr969893985a.1.1749564561524; Tue, 10 Jun 2025 07:09:21 -0700 (PDT) Received: from [192.168.65.90] (078088045245.garwolin.vectranet.pl. [78.88.45.245]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-ade1dc7e770sm729541666b.183.2025.06.10.07.09.19 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 10 Jun 2025 07:09:21 -0700 (PDT) Message-ID: <73e2881f-b899-45a4-a59e-47593283b63b@oss.qualcomm.com> Date: Tue, 10 Jun 2025 16:09:18 +0200 Precedence: bulk X-Mailing-List: linux-pm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v10] arm64: dts: qcom: ipq5018: Add tsens node To: George Moussalem , Amit Kucheria , Thara Gopinath , "Rafael J. Wysocki" , Daniel Lezcano , Zhang Rui , Lukasz Luba , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Bjorn Andersson , Konrad Dybcio Cc: linux-arm-msm@vger.kernel.org, linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Sricharan Ramabadhran , Dmitry Baryshkov References: <20250610-ipq5018-tsens-v10-1-3a10a5a2642c@outlook.com> Content-Language: en-US From: Konrad Dybcio In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUwNjEwMDExMiBTYWx0ZWRfX7+/k+wv2H4oo w7g786Hev/APVdBMdIvJivDoEy6kqEKRnlG60y3/k19gAi544+QJTcTSL871RESE3t3N8pqNTV8 p4nDihTwnV3M5icTYxi/R3vs2VyUdfRAsncqBk9glqbRMnS1bAPW069dxocZilTLFFr9kR7ozZZ YJqARd+q8caxfaL5J3rv/ua56NwdfvfVAdXxCOZL3YFhj+igAxczK9efmfotZdQwUaclLqtuboP fv4Coxyxx6yW1Qz53EjOY/U6Yua/HayBrIV/Ghg+acd//7BGYKQJ3ZPzANbggB5lxesAS8SEuRh LaGheXE9GFbTs4d4B4rSb76Jz4aQiISa8wyiZ9eiDk8p2vbQsTSp3H1jWqT2nG6sBKdwM2CMgS9 2KafoMbX7RHtMLrYqA32W12guENADMbpkfglheKAXZyo7m6qD8x5Xk+mbE4CZhjX8kZYJD0d X-Proofpoint-GUID: dWJzDuWemy7N_EjZUSwC6Go-ndaY6rUv X-Proofpoint-ORIG-GUID: dWJzDuWemy7N_EjZUSwC6Go-ndaY6rUv X-Authority-Analysis: v=2.4 cv=f+BIBPyM c=1 sm=1 tr=0 ts=68483c93 cx=c_pps a=JbAStetqSzwMeJznSMzCyw==:117 a=FpWmc02/iXfjRdCD7H54yg==:17 a=IkcTkHD0fZMA:10 a=6IFa9wvqVegA:10 a=COk6AnOGAAAA:8 a=KKAkSRfTAAAA:8 a=UqCG9HQmAAAA:8 a=E6Ly0PRcfj7wCC4qtTcA:9 a=3ZKOabzyN94A:10 a=QEXdDO2ut3YA:10 a=uxP6HrT_eTzRwkO_Te1X:22 a=TjNXssC_j7lpFel5tvFf:22 a=cvBusfyB2V15izCimMoJ:22 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1099,Hydra:6.0.736,FMLib:17.12.80.40 definitions=2025-06-10_05,2025-06-10_01,2025-03-28_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 suspectscore=0 spamscore=0 mlxlogscore=999 bulkscore=0 impostorscore=0 lowpriorityscore=0 priorityscore=1501 adultscore=0 phishscore=0 mlxscore=0 malwarescore=0 clxscore=1015 classifier=spam authscore=0 authtc=n/a authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.19.0-2505280000 definitions=main-2506100112 On 6/10/25 4:06 PM, George Moussalem wrote: > > > On 6/10/25 17:08, Konrad Dybcio wrote: >> On 6/10/25 3:03 PM, George Moussalem via B4 Relay wrote: >>> From: Sricharan Ramabadhran >>> >>> IPQ5018 has tsens V1.0 IP with 5 sensors, though 4 are in use. >>> There is no RPM, so tsens has to be manually enabled. Adding the tsens >>> and nvmem nodes and adding 4 thermal sensors (zones). With the >>> critical temperature being 120'C and action is to reboot. >>> >>> Reviewed-by: Dmitry Baryshkov >>> Signed-off-by: Sricharan Ramabadhran >>> Signed-off-by: George Moussalem >>> --- [...] >>> +        gephy-thermal { >>> +            polling-delay-passive = <0>; >>> +            polling-delay = <0>; >>> +            thermal-sensors = <&tsens 4>; >>> + >>> +            trips { >>> +                gephy-critical { >>> +                    temperature = <120000>; >> >> I'm not sure whether there's any firmware/hardware measure to shut >> down these beforehand. It's better to have a software trip at 120C >> than to not have any at all, but you may want to try and find a >> case_therm or so thermistor for your devices.. > > these are the temps as defined for the critical trips by qcom. IPQ8074 has the temp set at 110C, while IPQ6018 even higher at 125C. > They've also defined lower temp trips in the downstream DTS files. > Should I try and add those? I could check if the driver supports them.. you can register the CPUs as a cooling device and add some sort of a trip point (see e.g. ipq6018.dtsi) Konrad