From patchwork Mon Nov 26 12:55:06 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simon Horman X-Patchwork-Id: 10698329 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 7A38117D5 for ; Mon, 26 Nov 2018 13:13:25 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 65FCF296BB for ; Mon, 26 Nov 2018 13:13:25 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 5717C297E5; Mon, 26 Nov 2018 13:13:25 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id F146B296BB for ; Mon, 26 Nov 2018 13:13:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:MIME-Version:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:References: In-Reply-To:Message-Id:Date:Subject:To:From:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Owner; bh=uXfwTogF/RUhBKbXin178HuoKOv5ZnRaS5I9A49VKbE=; b=oo0rkR9b4csZ/5lVLwCjIu20Rm MLBfNgrU3473ha7+rhT/ROsQ1RbtAwg6oY76Jg3wyZLnzY043xOJ25mzo6kELVzdwHN+1+duShySH OIct/ErLsaLik5XXLhU8bQ7PDVvQ2QR5n8JuXeU/IPk13XD2XB2irhQcNRH9IOf0X0ihaOwuXzGer lCHNnbyd/hZIb+9UeXjEhLgYmIs/yuNpKUyU+NJO2airjZTEn6Sv5tsaeoZb5OsZdiZVnrgYR18T7 ScdxgyeDCEHWq/KFnLBnjIZJjx8BeHbw3tMl58dC74Q4Ae75GqcqoqwSlUjGocRmP2QxIegn7PNcr 98qubIUQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1gRGhW-0006Zw-Fz; Mon, 26 Nov 2018 13:13:22 +0000 Received: from casper.infradead.org ([85.118.1.10]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1gRGgG-0005J5-A0 for linux-arm-kernel@bombadil.infradead.org; Mon, 26 Nov 2018 13:12:04 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; h=References:In-Reply-To:Message-Id:Date: Subject:Cc:To:From:Sender:Reply-To:MIME-Version:Content-Type: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=h4VMde8CeHf7gqGdXOsn82/XzTjHFSu3RsEQt8X0TmM=; b=XQOtT1y5H3ZwECef1tyvgL34r Hdjjdlt/wryLrPRR9/zuYLDy0jLzqbQqUfLXoLL9cMYrCLUVuVbLpUpC7CVAyeqxlsXaoE7dveSg3 HoaeShzhGp8QVXhWMC6biwN0eQCJTVdeKpZSV/KsJcpTlQ8vZ4lxBgeNak2JH5nOC9JKqsQWcQln1 2Anps89SO1NqNhWBwdcSpDJubkK/Z71UEmqpB5umn1PPzrY1nxDsiixtu44bvvv6UwJtDpc5BaT+m TxqGqSbyRCJFQpWFwAClJNTDdbFYc8s1xpipaftD3GV8Zj6/kSKRHgDGfGHrCDAeZtZU9beZ+ra94 7X8pyrlfQ==; Received: from kirsty.vergenet.net ([202.4.237.240]) by casper.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1gRGb6-0005w7-5L for linux-arm-kernel@lists.infradead.org; Mon, 26 Nov 2018 13:06:46 +0000 Received: from reginn.horms.nl (watermunt.horms.nl [80.127.179.77]) by kirsty.vergenet.net (Postfix) with ESMTPA id 7B6AE25BF51; Mon, 26 Nov 2018 23:55:12 +1100 (AEDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=verge.net.au; s=mail; t=1543236912; bh=yvExVxXbLxJprjTjWJS1DANHaTi/u+Js7LY3jpO13J8=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=MM5ydygph7TSbXw8pF/4Rc4MTPDyKbZ7Vq/y1uVlZNigd8hpP+nS8um922tV3Q5nk j7OhIU0x5W3hOJacLpIFfM5DIlVeh2o+MrHQaauvbTt8Bmkn5X3akAASynyKWduWog 52KGzHTNkJhFDlwIYqHAJRVBewgDplkccBHPp/Ss= Received: by reginn.horms.nl (Postfix, from userid 7100) id 5AA239404AF; Mon, 26 Nov 2018 13:55:10 +0100 (CET) From: Simon Horman To: linux-renesas-soc@vger.kernel.org Subject: [PATCH 40/40] arm64: dts: renesas: Add all CPUs in cooling maps Date: Mon, 26 Nov 2018 13:55:06 +0100 Message-Id: <275e4eb3f21a09b6b8bd4a353b9a01e500240385.1543229516.git.horms+renesas@verge.net.au> X-Mailer: git-send-email 2.11.0 In-Reply-To: References: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20181126_130644_806740_6737C3C1 X-CRM114-Status: GOOD ( 14.93 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Viresh Kumar , Magnus Damm , linux-arm-kernel@lists.infradead.org, Simon Horman MIME-Version: 1.0 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP From: Viresh Kumar Each CPU can (and does) participate in cooling down the system but the DT only captures a handful of them, normally CPU0, in the cooling maps. Things work by chance currently as under normal circumstances its the first CPU of each cluster which is used by the operating systems to probe the cooling devices. But as soon as this CPU ordering changes and any other CPU is used to bring up the cooling device, we will start seeing failures. Also the DT is rather incomplete when we list only one CPU in the cooling maps, as the hardware doesn't have any such limitations. Update cooling maps to include all devices affected by individual trip points. Signed-off-by: Viresh Kumar Signed-off-by: Simon Horman --- arch/arm64/boot/dts/renesas/r8a7795.dtsi | 15 ++++++++++++--- arch/arm64/boot/dts/renesas/r8a7796.dtsi | 6 +++--- 2 files changed, 15 insertions(+), 6 deletions(-) diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi b/arch/arm64/boot/dts/renesas/r8a7795.dtsi index e94a5f2dbd08..8643ecb1a3f3 100644 --- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi @@ -3115,7 +3115,10 @@ cooling-maps { map0 { trip = <&sensor1_passive>; - cooling-device = <&a57_0 4 4>; + cooling-device = <&a57_0 4 4>, + <&a57_1 4 4>, + <&a57_2 4 4>, + <&a57_3 4 4>; }; }; }; @@ -3141,7 +3144,10 @@ cooling-maps { map0 { trip = <&sensor2_passive>; - cooling-device = <&a57_0 4 4>; + cooling-device = <&a57_0 4 4>, + <&a57_1 4 4>, + <&a57_2 4 4>, + <&a57_3 4 4>; }; }; }; @@ -3167,7 +3173,10 @@ cooling-maps { map0 { trip = <&sensor3_passive>; - cooling-device = <&a57_0 4 4>; + cooling-device = <&a57_0 4 4>, + <&a57_1 4 4>, + <&a57_2 4 4>, + <&a57_3 4 4>; }; }; }; diff --git a/arch/arm64/boot/dts/renesas/r8a7796.dtsi b/arch/arm64/boot/dts/renesas/r8a7796.dtsi index e7614b781308..afedbf5728ec 100644 --- a/arch/arm64/boot/dts/renesas/r8a7796.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a7796.dtsi @@ -2839,7 +2839,7 @@ cooling-maps { map0 { trip = <&sensor1_passive>; - cooling-device = <&a57_0 5 5>; + cooling-device = <&a57_0 5 5>, <&a57_1 5 5>; }; }; }; @@ -2865,7 +2865,7 @@ cooling-maps { map0 { trip = <&sensor2_passive>; - cooling-device = <&a57_0 5 5>; + cooling-device = <&a57_0 5 5>, <&a57_1 5 5>; }; }; }; @@ -2891,7 +2891,7 @@ cooling-maps { map0 { trip = <&sensor3_passive>; - cooling-device = <&a57_0 5 5>; + cooling-device = <&a57_0 5 5>, <&a57_1 5 5>; }; }; };