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=-6.8 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS 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 936C6C7618B for ; Wed, 24 Jul 2019 05:21:15 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 4F10E218B0 for ; Wed, 24 Jul 2019 05:21:15 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=verdurent-com.20150623.gappssmtp.com header.i=@verdurent-com.20150623.gappssmtp.com header.b="R4yhWNRP" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1725883AbfGXFVK (ORCPT ); Wed, 24 Jul 2019 01:21:10 -0400 Received: from mail-vs1-f67.google.com ([209.85.217.67]:35973 "EHLO mail-vs1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725829AbfGXFVK (ORCPT ); Wed, 24 Jul 2019 01:21:10 -0400 Received: by mail-vs1-f67.google.com with SMTP id y16so30515694vsc.3 for ; Tue, 23 Jul 2019 22:21:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=verdurent-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=k/eNVz2Gv3iMOFCg2AdyVAu7xAsfyLCtEzF/NktEOgU=; b=R4yhWNRPmdXkwJRdn+s+5aJJB+VbHlkpuqSZC04pdadEiR3evwImFA+aaB+9aaKdWk biKBL45VJSOAtaB6bxzzUYkNhIAE/+QsGtTP5rWt5rXPd8nXQdVxGSoMHAH0yZemIGzq pjS7KTr2a2z0M8BaP0Ofng1oGdYoIQwULvKNJbnGD9jTAn3YptaMzdCimSxfGsursQRv CAoHgj0JEVn5IxYEBTMb64YSs4MUiy7VpzmiSXQ68bFgC25Y82gksvLAGdKaMznfnZMA RCVLQJuzp5uA5Ehn4flK087NEQtd1MqPsiLMLukyOeryOLk+Njalmz1vchjRKRFvCbbr Zp7A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=k/eNVz2Gv3iMOFCg2AdyVAu7xAsfyLCtEzF/NktEOgU=; b=eAuJLgRXJXFuzgTiK0oZ7p0iiBFx2XU0ca8OJvNyAkfkCCt2glx0qhAILoX/beB/GR EG1rRNOMUoGP4mgpqovGvYUCJ+86SDpmVT3qirNb2ta7UtqIWhqec0bZaI0hr3sS8D+n h1K6rZpedkmpcdFTwINM48brmb2C8/oARDWo6Wg8LvDCTVbOqyBityM6fO2+7/f2J/E1 kr45mK8KyAkCeneaqZTDvNOJMp7Qe2nf4vu1ABl6Gco5Zg6tDdIb7HZEum6utjlTPdEq y0PjSNNNF7+ANePumI3vTJdi+DLIZlPZExEwUJSx2ygnsvVXV3/aCHq12/O6FZ+ZnAQV Cq3w== X-Gm-Message-State: APjAAAVb8g76VKKecxgu2j7tSwSD1YBM+N2ikYGbjZLKHBIgRHkHWEhf 9CAsE5yGQozH2nTtLYWt80qiy4WLrKLAJ+aU2Ds= X-Google-Smtp-Source: APXvYqwxjkD6dGzt/PeAHHclFyFju/OEpRaPtlopsHdKsks/3sgcc5VLdmlEJszA9OpFozSYbnczrZA+pgiTdPijW2s= X-Received: by 2002:a67:f899:: with SMTP id h25mr3287593vso.159.1563945668489; Tue, 23 Jul 2019 22:21:08 -0700 (PDT) MIME-Version: 1.0 References: <20190724044906.12007-1-vkoul@kernel.org> <20190724044906.12007-4-vkoul@kernel.org> In-Reply-To: <20190724044906.12007-4-vkoul@kernel.org> From: Amit Kucheria Date: Wed, 24 Jul 2019 10:50:57 +0530 Message-ID: Subject: Re: [PATCH v2 3/5] arm64: dts: qcom: sdm845: remove unit name for thermal trip points To: Vinod Koul Cc: Andy Gross , linux-arm-msm , Bjorn Andersson , Rob Herring , Mark Rutland , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , LKML , Stephen Boyd , Marc Gonzalez Content-Type: text/plain; charset="UTF-8" Sender: linux-arm-msm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org On Wed, Jul 24, 2019 at 10:20 AM Vinod Koul wrote: > > The thermal trip points have unit name but no reg property, so we can > remove them > > arch/arm64/boot/dts/qcom/sdm845.dtsi:2824.31-2828.7: Warning (unit_address_vs_reg): /thermal-zones/cpu0-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:2830.31-2834.7: Warning (unit_address_vs_reg): /thermal-zones/cpu0-thermal/trips/trip-point@1: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:2868.31-2872.7: Warning (unit_address_vs_reg): /thermal-zones/cpu1-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:2874.31-2878.7: Warning (unit_address_vs_reg): /thermal-zones/cpu1-thermal/trips/trip-point@1: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:2912.31-2916.7: Warning (unit_address_vs_reg): /thermal-zones/cpu2-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:2918.31-2922.7: Warning (unit_address_vs_reg): /thermal-zones/cpu2-thermal/trips/trip-point@1: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:2956.31-2960.7: Warning (unit_address_vs_reg): /thermal-zones/cpu3-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:2962.31-2966.7: Warning (unit_address_vs_reg): /thermal-zones/cpu3-thermal/trips/trip-point@1: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3000.31-3004.7: Warning (unit_address_vs_reg): /thermal-zones/cpu4-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3006.31-3010.7: Warning (unit_address_vs_reg): /thermal-zones/cpu4-thermal/trips/trip-point@1: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3044.31-3048.7: Warning (unit_address_vs_reg): /thermal-zones/cpu5-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3050.31-3054.7: Warning (unit_address_vs_reg): /thermal-zones/cpu5-thermal/trips/trip-point@1: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3088.31-3092.7: Warning (unit_address_vs_reg): /thermal-zones/cpu6-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3094.31-3098.7: Warning (unit_address_vs_reg): /thermal-zones/cpu6-thermal/trips/trip-point@1: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3132.31-3136.7: Warning (unit_address_vs_reg): /thermal-zones/cpu7-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3138.31-3142.7: Warning (unit_address_vs_reg): /thermal-zones/cpu7-thermal/trips/trip-point@1: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3176.32-3180.7: Warning (unit_address_vs_reg): /thermal-zones/aoss0-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3191.35-3195.7: Warning (unit_address_vs_reg): /thermal-zones/cluster0-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3211.35-3215.7: Warning (unit_address_vs_reg): /thermal-zones/cluster1-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3231.31-3235.7: Warning (unit_address_vs_reg): /thermal-zones/gpu-thermal-top/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3246.31-3250.7: Warning (unit_address_vs_reg): /thermal-zones/gpu-thermal-bottom/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3261.32-3265.7: Warning (unit_address_vs_reg): /thermal-zones/aoss1-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3276.35-3280.7: Warning (unit_address_vs_reg): /thermal-zones/q6-modem-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3291.30-3295.7: Warning (unit_address_vs_reg): /thermal-zones/mem-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3306.31-3310.7: Warning (unit_address_vs_reg): /thermal-zones/wlan-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3321.33-3325.7: Warning (unit_address_vs_reg): /thermal-zones/q6-hvx-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3336.33-3340.7: Warning (unit_address_vs_reg): /thermal-zones/camera-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3351.32-3355.7: Warning (unit_address_vs_reg): /thermal-zones/video-thermal/trips/trip-point@0: node has a unit name, but no reg property > arch/arm64/boot/dts/qcom/sdm845.dtsi:3366.32-3370.7: Warning (unit_address_vs_reg): /thermal-zones/modem-thermal/trips/trip-point@0: node has a unit name, but no reg property > > Signed-off-by: Vinod Koul > Reviewed-by: Stephen Boyd Reviewed-by: Amit Kucheria > --- > arch/arm64/boot/dts/qcom/sdm845.dtsi | 58 ++++++++++++++-------------- > 1 file changed, 29 insertions(+), 29 deletions(-) > > diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi > index 2985df032179..48bd07646eb4 100644 > --- a/arch/arm64/boot/dts/qcom/sdm845.dtsi > +++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi > @@ -2815,13 +2815,13 @@ > thermal-sensors = <&tsens0 1>; > > trips { > - cpu0_alert0: trip-point@0 { > + cpu0_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "passive"; > }; > > - cpu0_alert1: trip-point@1 { > + cpu0_alert1: trip-point1 { > temperature = <95000>; > hysteresis = <2000>; > type = "passive"; > @@ -2859,13 +2859,13 @@ > thermal-sensors = <&tsens0 2>; > > trips { > - cpu1_alert0: trip-point@0 { > + cpu1_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "passive"; > }; > > - cpu1_alert1: trip-point@1 { > + cpu1_alert1: trip-point1 { > temperature = <95000>; > hysteresis = <2000>; > type = "passive"; > @@ -2903,13 +2903,13 @@ > thermal-sensors = <&tsens0 3>; > > trips { > - cpu2_alert0: trip-point@0 { > + cpu2_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "passive"; > }; > > - cpu2_alert1: trip-point@1 { > + cpu2_alert1: trip-point1 { > temperature = <95000>; > hysteresis = <2000>; > type = "passive"; > @@ -2947,13 +2947,13 @@ > thermal-sensors = <&tsens0 4>; > > trips { > - cpu3_alert0: trip-point@0 { > + cpu3_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "passive"; > }; > > - cpu3_alert1: trip-point@1 { > + cpu3_alert1: trip-point1 { > temperature = <95000>; > hysteresis = <2000>; > type = "passive"; > @@ -2991,13 +2991,13 @@ > thermal-sensors = <&tsens0 7>; > > trips { > - cpu4_alert0: trip-point@0 { > + cpu4_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "passive"; > }; > > - cpu4_alert1: trip-point@1 { > + cpu4_alert1: trip-point1 { > temperature = <95000>; > hysteresis = <2000>; > type = "passive"; > @@ -3035,13 +3035,13 @@ > thermal-sensors = <&tsens0 8>; > > trips { > - cpu5_alert0: trip-point@0 { > + cpu5_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "passive"; > }; > > - cpu5_alert1: trip-point@1 { > + cpu5_alert1: trip-point1 { > temperature = <95000>; > hysteresis = <2000>; > type = "passive"; > @@ -3079,13 +3079,13 @@ > thermal-sensors = <&tsens0 9>; > > trips { > - cpu6_alert0: trip-point@0 { > + cpu6_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "passive"; > }; > > - cpu6_alert1: trip-point@1 { > + cpu6_alert1: trip-point1 { > temperature = <95000>; > hysteresis = <2000>; > type = "passive"; > @@ -3123,13 +3123,13 @@ > thermal-sensors = <&tsens0 10>; > > trips { > - cpu7_alert0: trip-point@0 { > + cpu7_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "passive"; > }; > > - cpu7_alert1: trip-point@1 { > + cpu7_alert1: trip-point1 { > temperature = <95000>; > hysteresis = <2000>; > type = "passive"; > @@ -3167,7 +3167,7 @@ > thermal-sensors = <&tsens0 0>; > > trips { > - aoss0_alert0: trip-point@0 { > + aoss0_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "hot"; > @@ -3182,7 +3182,7 @@ > thermal-sensors = <&tsens0 5>; > > trips { > - cluster0_alert0: trip-point@0 { > + cluster0_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "hot"; > @@ -3202,7 +3202,7 @@ > thermal-sensors = <&tsens0 6>; > > trips { > - cluster1_alert0: trip-point@0 { > + cluster1_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "hot"; > @@ -3222,7 +3222,7 @@ > thermal-sensors = <&tsens0 11>; > > trips { > - gpu1_alert0: trip-point@0 { > + gpu1_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "hot"; > @@ -3237,7 +3237,7 @@ > thermal-sensors = <&tsens0 12>; > > trips { > - gpu2_alert0: trip-point@0 { > + gpu2_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "hot"; > @@ -3252,7 +3252,7 @@ > thermal-sensors = <&tsens1 0>; > > trips { > - aoss1_alert0: trip-point@0 { > + aoss1_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "hot"; > @@ -3267,7 +3267,7 @@ > thermal-sensors = <&tsens1 1>; > > trips { > - q6_modem_alert0: trip-point@0 { > + q6_modem_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "hot"; > @@ -3282,7 +3282,7 @@ > thermal-sensors = <&tsens1 2>; > > trips { > - mem_alert0: trip-point@0 { > + mem_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "hot"; > @@ -3297,7 +3297,7 @@ > thermal-sensors = <&tsens1 3>; > > trips { > - wlan_alert0: trip-point@0 { > + wlan_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "hot"; > @@ -3312,7 +3312,7 @@ > thermal-sensors = <&tsens1 4>; > > trips { > - q6_hvx_alert0: trip-point@0 { > + q6_hvx_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "hot"; > @@ -3327,7 +3327,7 @@ > thermal-sensors = <&tsens1 5>; > > trips { > - camera_alert0: trip-point@0 { > + camera_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "hot"; > @@ -3342,7 +3342,7 @@ > thermal-sensors = <&tsens1 6>; > > trips { > - video_alert0: trip-point@0 { > + video_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "hot"; > @@ -3357,7 +3357,7 @@ > thermal-sensors = <&tsens1 7>; > > trips { > - modem_alert0: trip-point@0 { > + modem_alert0: trip-point0 { > temperature = <90000>; > hysteresis = <2000>; > type = "hot"; > -- > 2.20.1 >