From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-16.4 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1BB13C43461 for ; Mon, 3 May 2021 17:56:14 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id E108F61278 for ; Mon, 3 May 2021 17:56:13 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231414AbhECR5G (ORCPT ); Mon, 3 May 2021 13:57:06 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39966 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231378AbhECR5F (ORCPT ); Mon, 3 May 2021 13:57:05 -0400 Received: from mail-pg1-x534.google.com (mail-pg1-x534.google.com [IPv6:2607:f8b0:4864:20::534]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F1161C061761 for ; Mon, 3 May 2021 10:56:10 -0700 (PDT) Received: by mail-pg1-x534.google.com with SMTP id m12so4174934pgr.9 for ; Mon, 03 May 2021 10:56:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=bPGNoffuv19YjjXYP1WFdfb1qoKP5Lg1nHaK0QpjVVw=; b=eyW8g2e77jYlYs5whvg43H5Ssak7Pr+zzh9QLq6e12BTjGDwXEpALVoYYYlIytZbaI EvBD9PTRKMXoNbS3g5vg+yttyvz5cvL4b8DJlGruJbr5Eg1I4G+Ak5plqIYK+0IIpdBR pRU6GW5kqk4DspHWi8juBxW6bABk7gODouTpw= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=bPGNoffuv19YjjXYP1WFdfb1qoKP5Lg1nHaK0QpjVVw=; b=YBTvcjTbM7Z5hkYV3TjJ5Qbw//r2VXwlQ8/PC9nkI8wELQzV2ZcK58nHJoCuXftDkx /DBeNU46cN6AGzylHKIhJ4GqZrsbQvUjx4pYdalBEdlQg3U2F0bdbsybekwmUcaQ399O ZO/othKbmUesM2VjZ0m1/neMXL2TD36qGUMt32NCVdeKjsQ4PuiG4JNpQgeB39gLklxf wQGUB6aBFXG8UxC+qNqvASlNxXSnRi2gdN8B2AbuKX2K1LCpj4tcmWNFvtHlbCp7W5ET /fs00qKp/fHe2ER3DrrXLNoG+iRTqTXp/ia25YTScrwrwaKAg9bUYcbj4Onj05FTtvmH 8vhg== X-Gm-Message-State: AOAM533o5eU6GJu+0+1qh67VN4SL/11YQJmJiQ28ABVcmUoKB2Qayf1O H8sMZMcrWl6Iwafx7PffAF9DyBPQnk9R0A== X-Google-Smtp-Source: ABdhPJzeuUMnOHQFQXipHo229NVknh0suN/NWgkQfOVNZtfwuJ4AH5TVUs80cJ6BaBLtaFRBf/hnkA== X-Received: by 2002:a63:1b5c:: with SMTP id b28mr19048190pgm.444.1620064570539; Mon, 03 May 2021 10:56:10 -0700 (PDT) Received: from localhost ([2620:15c:202:201:8584:3fd:2adf:a655]) by smtp.gmail.com with UTF8SMTPSA id 76sm9557600pfw.58.2021.05.03.10.56.09 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 03 May 2021 10:56:10 -0700 (PDT) Date: Mon, 3 May 2021 10:56:09 -0700 From: Matthias Kaehlcke To: Rajeshwari Cc: amitk@kernel.org, thara.gopinath@linaro.org, agross@kernel.org, bjorn.andersson@linaro.org, rui.zhang@intel.com, daniel.lezcano@linaro.org, robh+dt@kernel.org, linux-pm@vger.kernel.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, sanm@codeaurora.org, manafm@codeaurora.org Subject: Re: [PATCH V3 3/3] arm64: dts: qcom: SC7280: Add thermal zone support Message-ID: References: <1619778592-8112-1-git-send-email-rkambl@codeaurora.org> <1619778592-8112-4-git-send-email-rkambl@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1619778592-8112-4-git-send-email-rkambl@codeaurora.org> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Apr 30, 2021 at 03:59:52PM +0530, Rajeshwari wrote: > Adding thermal zone, cooling maps support and changing hysteresis value for critical trip point in SC7280. > Hysteresis is not needed for critical trip point, when it reaches critical threshold system > will go for shutdown. This patch doesn't change any hysteresis values, it adds them. As commented on v2, the sentence about the hysteresis doesn't add any value IMO, just remove it, or at least explain in a reply why you think it is valuable. > > Signed-off-by: Rajeshwari > --- patches with v > 1 should include a changelog > arch/arm64/boot/dts/qcom/sc7280.dtsi | 852 +++++++++++++++++++++++++++++++++++ > 1 file changed, 852 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi > index 275113c..d722081 100644 > --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi > +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi > @@ -12,6 +12,7 @@ > #include > #include > #include > +#include > > / { > > ... > > + thermal_zones: thermal-zones { > + cpu0-thermal { > + polling-delay-passive = <250>; > + polling-delay = <0>; > + > + thermal-sensors = <&tsens0 1>; > + > + trips { > + cpu0_alert0: trip-point0 { > + temperature = <90000>; > + hysteresis = <2000>; > + type = "passive"; > + }; > + > + cpu0_alert1: trip-point1 { > + temperature = <95000>; > + hysteresis = <2000>; > + type = "passive"; > + }; > + > + cpu0_crit: cpu_crit { The node name should use dashes as separator, not underscores, i.e. it should be 'cpu-crit'. Applicable to all critical trip points.