From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f54.google.com (mail-ej1-f54.google.com [209.85.218.54]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 822E33ED5C7 for ; Fri, 31 Jul 2026 11:20:54 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.54 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785496856; cv=none; b=f2Lhsrd7T9PkLpSnJF/sh2E4ymz2g/1lDRa+qhgCwM4Qcow4ixQ7hCYdp2uaX1mWpCMof1OqvBWvu7nro2d3NlUEvFXxbzZlK29zJpO3Ysp4NBcFm0a/jAUo+0l7lqlK2UoPgN4SbXJJ1V5vNSiwlZenyiOxrJtErF0XsPdif8I= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785496856; c=relaxed/simple; bh=bhq3MzeqOsauBzIWfBC+LweuxxY/XXQ4w63ZT64jvPM=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=Bs+G+xY3nOLN+Js+6cFgoZEpW3cxQBYL4Tt34ildIcuB5h9wzLQHZ6WhPoiqdsdh+/0l+BwN948Jzd0hJGmVLFI9m54DxDNK/REsObBCMWa14pEIIvYloSvI7CvmRx7RTzoChCJisc7hI7RkE/nrbJk+v9QHziAs7Vw/pvin1EM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=dmVncrkE; arc=none smtp.client-ip=209.85.218.54 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="dmVncrkE" Received: by mail-ej1-f54.google.com with SMTP id a640c23a62f3a-c1fbe461f59so125498666b.2 for ; Fri, 31 Jul 2026 04:20:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1785496853; x=1786101653; darn=vger.kernel.org; h=in-reply-to:content-transfer-encoding:content-disposition :content-type:mime-version:references:message-id:subject:cc:to:from :date:from:to:cc:subject:date:message-id:reply-to:content-type; bh=bzkNS7AH3AQpIYPHQeQ9hpg4r1cVjOJtrFQ4q2xajzU=; b=dmVncrkEQMeZ3LCEtrYL7CKtvI9z3zNHGLiZrU4uVjApYP+2LAbk6nesi/bE8VH9Ku Pdz+s/ZWQKVRlwGNRKxLlmHqvoId39fwxRp+ju/k9w0tkNvwwKw/7k2dsvnFByGknyb5 6ZSHtJkGMFJsfKI2lJLXqGC/ewN7t/2+vW1qB3/JLlnXq2MUQQZLbipQGZNVs1HmSTre s6GdXVTtawgh+wv3dZMEad0Q5zl5IYOmgRGOFFKo6XHOIeOApbMioQ8axdw3b+VDGGSG nxVqUhpg+LqSTbjjrNFu3XUItIdvf0jz5CNbQ/h6l+UoGwQTlAZFnr9umE3F/oaF+Yhs sseA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1785496853; x=1786101653; h=in-reply-to:content-transfer-encoding:content-disposition :content-type:mime-version:references:message-id:subject:cc:to:from :date:x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to:content-type; bh=bzkNS7AH3AQpIYPHQeQ9hpg4r1cVjOJtrFQ4q2xajzU=; b=dbTfnh7La0LHNg/J7A3DMAyLuy+0HRaoZAjwdl3ilsVUxs2mnnERJfhvjjrssx2g40 IpoUJZoga4djwCxKXoP1cb5w998GGrw9KeJgzuXZPY16cIDNUwWlgyrSCv0lcap/QdJy DITI8ghvGx3kdv+cfVjcT2UxYpELSdiBJuKVuaw9MOzmwkNhUSy/cTjwVwWoNAwLWwQp I7OLg327fx6PYEITFw4yXbtGS2R3CkY12F3T/MijcCdN9QAiLuJ3Zmz+kzh6on6fIg7m os0clSZ88MYCpaizIrt4bHBqSYmZLIiKc6N5g6DtocMAPEtnTADPbE/rKaWyNMkVTkJn J5og== X-Forwarded-Encrypted: i=1; AHgh+RrzQyWwf3D5GkEQmNUToh4LeMYqNWNbUtiMgQuz3AYN0d3xvY7h9wUeMP10M530sAg7RQ6mghsFCg==@vger.kernel.org X-Gm-Message-State: AOJu0YzWrZ/zvR2qrlGUxLgRyy5iG+HTr5qq0PVsGFjmsX9m+2iHfzFf JD8NKoiObmyFLab8fUNULkdogVunGRqpAnQDsPuZdabwYzCRpW9mXk8z0qr9XHZIJLA= X-Gm-Gg: AR+sD11/QpZm8wuzldKSym7hKAUGzRZjDfStL72bkfgo4Gk16MdqTz5LUKugMJCdxd6 l2k8RSB9CHQmNvUyyfLU7DHfenG25sjXf2WlWzR8i2ZzHE8wbC1kUjgxDv2GlkPOO5NlTaHIMsb h20yCiDNXaAzCJiHWcoc5C1bD9ChXBCH1fEq1VdttuYGczNG0mFhR/E/vW6kDNvLKbnNp4snXy+ qrM4WVisKDlJT5N+yXEGj+LmAL1Mxzl2D1d2jICMW3O9jNeMItEfzbR4vGfB2HXHuvozUGm18n/ 0XhhW12RSIi681bk7y9zPbGsOqEINpMrses8/aQ4NvSzUjiFjpN9p1GKx0lby4qovO//xABs0MZ tfI3Bp5r6w1hsEd+7OHo9J9VQT07arT7tMN/bCF5VkzZ13JvHsuvgBhR38YbnBEubRDfVuHY7kZ xiMMaovOIv+mjJmLRwgZDe2+WDkSZfgyNwJarBXV5sZAx4v2JakoZMgMprKLfWTsj3U+c1IlEy X-Received: by 2002:a17:907:1ca4:b0:c12:6dbf:4a80 with SMTP id a640c23a62f3a-c1fd295ec35mr86137366b.63.1785496852551; Fri, 31 Jul 2026 04:20:52 -0700 (PDT) Received: from linaro.org ([2a02:2454:ff24:7210:4eea:ae3f:f2e0:1952]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-c1fd3cc331fsm128489966b.28.2026.07.31.04.20.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 31 Jul 2026 04:20:52 -0700 (PDT) Date: Fri, 31 Jul 2026 13:20:48 +0200 From: Stephan Gerhold To: Gaurav Kohli Cc: Bjorn Andersson , Mathieu Poirier , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Daniel Lezcano , Amit Kucheria , Konrad Dybcio , "Rafael J. Wysocki" , Daniel Lezcano , Zhang Rui , Lukasz Luba , Manivannan Sadhasivam , Kees Cook , "Gustavo A. R. Silva" , cros-qcom-dts-watchers@chromium.org, linux-arm-msm@vger.kernel.org, linux-remoteproc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, linux-hardening@vger.kernel.org, manaf.pallikunhi@oss.qualcomm.com, Dipa Ramesh Mantre , Dmitry Baryshkov Subject: Re: [PATCH v7 9/9] arm64: dts: qcom: hamoa: Enable CDSP cooling Message-ID: References: <20260731-b4-qmi-tmd-v7-0-3befd44403d5@oss.qualcomm.com> <20260731-b4-qmi-tmd-v7-9-3befd44403d5@oss.qualcomm.com> Precedence: bulk X-Mailing-List: linux-pm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20260731-b4-qmi-tmd-v7-9-3befd44403d5@oss.qualcomm.com> On Fri, Jul 31, 2026 at 04:03:49PM +0530, Gaurav Kohli wrote: > From: Dipa Ramesh Mantre > > Unlike the CPU, the CDSP does not throttle its speed automatically > when it reaches high temperatures in hamoa. > > Set up CDSP cooling by throttling the cdsp, when it reaches 95°C. > > Add polling-delay-passive so the governor periodically evaluates > the zone during passive cooling and steps up cooling levels when > temperature stabilizes below the next trip. > Would be good to mention why you are treating hamoa-iot-evk differently. > Signed-off-by: Dipa Ramesh Mantre > Reviewed-by: Dmitry Baryshkov > Signed-off-by: Gaurav Kohli > --- > arch/arm64/boot/dts/qcom/hamoa-iot-evk.dts | 32 +++++++++++++++ > arch/arm64/boot/dts/qcom/hamoa.dtsi | 63 ++++++++++++++++++++++++++++++ > 2 files changed, 95 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/hamoa-iot-evk.dts b/arch/arm64/boot/dts/qcom/hamoa-iot-evk.dts > index 9fa86bb6438e..a146be1cb5fb 100644 > --- a/arch/arm64/boot/dts/qcom/hamoa-iot-evk.dts > +++ b/arch/arm64/boot/dts/qcom/hamoa-iot-evk.dts > @@ -1684,4 +1684,36 @@ trip-point0 { > }; > }; > }; > + > + nsp0-thermal { > + trips { > + trip-point1 { > + temperature = <105000>; > + }; > + }; > + }; > + > + nsp1-thermal { > + trips { > + trip-point1 { > + temperature = <105000>; > + }; > + }; > + }; > + > + nsp2-thermal { > + trips { > + trip-point1 { > + temperature = <105000>; > + }; > + }; > + }; > + > + nsp3-thermal { > + trips { > + trip-point1 { > + temperature = <105000>; > + }; > + }; > + }; > }; Please avoid overriding node values like this... One quick rename of one of the involved nodes and this will be silently broken without compile error. Nowadays dtbs_check will *probably* catch these, but if you override values using the label reference you would notice it immediately when compiling. All those nodes already have a label you can use: &nsp0_alert0 { temperature = <105000>; }; &nsp1_alert0 { temperature = <105000>; }; If the same pattern is already used for other thermal overrides in hamoa-iot-evk.dts, it would be good to prepend a patch to fix those too. Thanks, Stephan