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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 5CD30C25B4E for ; Tue, 24 Jan 2023 17:47:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=cQEI0jfwMw6ZAuwjhv7UzXIHVUXSvCjsRFOUyqMBSUU=; b=JDdrPXTmfbRSJB KjscrLKR1Ti3q9mq/BXEkP0KugNzZaU7U0E+A4mneCZh4uI11rnQ+39ubiSZ1x4v7cqrZs9gNwiyE WVvBBkd7+cexFMu1+9vbCu7ObhS0r2zmOsuiuiLjRTHHMwdrsuMwAodQAxuj06GIh9tEzojke8jTr 3d2DeKAT3Bi6n+80hfrEa7DQRtw0g9ki2f0stp3eojq0tM9nKA4ROkGeOuO5MSfuPgvRRYxIGPUcT oWW8fZVozCMPSirD9o6mwL+ygTky1TP+AONtCj0lwhiQfP0eMHSM7/K5+DlICBK2sw/9T+UvnU1JO 3aPO5+lUZOo0stIB7Ejw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pKNNR-004ryj-Fx; Tue, 24 Jan 2023 17:46:33 +0000 Received: from mail-wm1-x334.google.com ([2a00:1450:4864:20::334]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pKNNO-004rxm-06 for linux-arm-kernel@lists.infradead.org; Tue, 24 Jan 2023 17:46:32 +0000 Received: by mail-wm1-x334.google.com with SMTP id bg13-20020a05600c3c8d00b003d9712b29d2so13403266wmb.2 for ; Tue, 24 Jan 2023 09:46:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20210112.gappssmtp.com; s=20210112; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=/v2GAT/gW1z+AQQeKDDFE3wRS4Dyi8jFMlO2BJzi6HU=; b=A0a6WgFqz7DrQif4g1HqvElyktI4Z/cY5HAWnoBk2ke9Z1JMP1cfOqs+TAujzCc3gh eOl4ID+MHKyx4Gv1wn67rTNUCNdD43nOL4C5DPt6wvA2eB7xKKO0nS68WlKKlvApVNDC RPvVXjVtJW8iPjdtKOmL8Y8PlUaCEx4zjxtZf7Yp/3QcbWnTVCQCUy/W+j/fvb1XpPpV TrT09I6N0fPik30/a4R44afvDeA3zLyf3VPkdMujyvMiLFf++XTotE42ymERA+feMT5z RtiJdRHsSTuM+xE5kUAH3bUo9yrBt5zkS5JN3xQwjmE/i5Fs/EPq+MrS4PW/D+JuZ4g3 W85w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=/v2GAT/gW1z+AQQeKDDFE3wRS4Dyi8jFMlO2BJzi6HU=; b=sOe78fV1WNs7UBIw52u7/b3ZyvExItscb5gk8HNKDOrgC8b2MXCZpV3h8v+h23/qPz xkTnuYTz6waLKrBBddBBRgJVsXM5twouQQUmWkbQL4YE1GBH4i2crIJtRRxhmKPwX99B rsPjcktnGhg4TQEb/fBXY8rewUQSkflpofLB3yXS0wcpJkpsqe8Ep49CHQz/nwc7pYHF ALbHirDTJm/wNw2xkKv44cIXbs0C/1m2pkR01ywl6vT/pkvm4GLqChFZCFgMDxmjUoEC Bbv8NM6Zs3vBtY4uoIjkDrHx9pFig2ZGip8aE1mDVI60HVgljI3x0WPGE4yiTM7aI+4j K8Qw== X-Gm-Message-State: AFqh2koMaiMTx5/wuv6Slu0XXU8TtBsPnyW8BYBN8ZXiH9+UbA17ClDC uPco79OUBQDQd/FboNTgv9o65w== X-Google-Smtp-Source: AMrXdXuFSiF40FFxyHPG8XgZAakZ5Xxa9nWm74fz85CKfHOnpPxRnrajry8SoKbxN4RuSUj3gZi6cw== X-Received: by 2002:a05:600c:1d10:b0:3d3:58d1:2588 with SMTP id l16-20020a05600c1d1000b003d358d12588mr25731882wms.41.1674582387832; Tue, 24 Jan 2023 09:46:27 -0800 (PST) Received: from [192.168.0.20] (62.213.132.195.rev.sfr.net. [195.132.213.62]) by smtp.gmail.com with ESMTPSA id m21-20020a05600c091500b003d1de805de5sm2450567wmp.16.2023.01.24.09.46.26 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 24 Jan 2023 09:46:27 -0800 (PST) Message-ID: <2950b824-ad2f-2579-a956-a03d157453eb@baylibre.com> Date: Tue, 24 Jan 2023 18:46:25 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.7.0 Subject: Re: [PATCH v7 4/4] thermal: mediatek: add another get_temp ops for thermal sensors To: Daniel Lezcano , "Rafael J. Wysocki" , Amit Kucheria , Rob Herring , Krzysztof Kozlowski , Zhang Rui Cc: AngeloGioacchino Del Regno , Fabien Parent , Matthias Brugger , Markus Schneider-Pargmann , linux-pm@vger.kernel.org, Rob Herring , Michael Kao , linux-kernel@vger.kernel.org, Hsin-Yi Wang , linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, devicetree@vger.kernel.org References: <20221018-up-i350-thermal-bringup-v7-0-ebf08ff2eddb@baylibre.com> <20221018-up-i350-thermal-bringup-v7-4-ebf08ff2eddb@baylibre.com> <4121bb6b-30db-7a23-f4c8-40afdda7a0b5@linaro.org> <0644aede-c281-3919-50e0-4466f6587d81@linaro.org> <187b0fe1-1f14-d8b8-c827-1e824da0b1d3@baylibre.com> <9f9547a2-69dd-f91d-c587-289156bf9d55@linaro.org> Content-Language: en-US From: Amjad Ouled-Ameur In-Reply-To: <9f9547a2-69dd-f91d-c587-289156bf9d55@linaro.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230124_094630_323320_CC6A69D2 X-CRM114-Status: GOOD ( 18.48 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 1/24/23 17:54, Daniel Lezcano wrote: > > Hi Amjad, > > On 24/01/2023 11:08, Amjad Ouled-Ameur wrote: > > [ ... ] > >>>> >>>> IIUC, there is a sensor per couple of cores. 1 x 2Bigs, 1 x 2Bigs, 1 x 4 Little, right ? >>> >>> MT8365 SoC has 4 x A53 CPUs. The SoC has 4 thermal zones per sensor. Thermal zone 0 corresponds >>> >>> to all 4 x A53 CPUs, the other thermal zones (1, 2 and 3) has nothing to do with CPUs. The cooling device type >>> >>> used for CPUs is passive. FYI, thermal zones 1, 2 and 3 are present in the SoC for debug-purpose only, they are not supposed >>> >>> to be used for production. >>> >> After reconsidering the fact that zones 1, 2 and 3 are only used for dev/debug, it might be best to avo > >> aggregation as you suggested, and keep only support for zone 0 in this driver. Thus I suggest I send a V8 >> >> where I keep only below fixes for this patch if that's okay with you: >> >> - Define "raw_to_mcelsius" function pointer for "struct thermal_bank_cfg". >> >> - Fix "mtk_thermal" variable in mtk_read_temp(). >> >> - Set "mt->raw_to_mcelsius" in probe(). >> >> >> For zones 1, 2 and 3 we can later add a different driver specific for dev/debug to probe them to >> >> avoid confusion. > > You can add them in the driver and in the device tree, but just add the cooling device for the thermal zone 0. Thermal zone 0 uses CPU{0..3} for passive cooling, in this case we should register cooling device with cpufreq_cooling_register() for each CPU right ? > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel