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 846ABC38A02 for ; Mon, 31 Oct 2022 13:26:56 +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-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=p6xfe3MECMp/FxOI8RlLhXuSeLBh1velRDAhqzvXCEk=; b=Vy1KHCMV0z5Rd3 kk+bAGig4jWUo0Hrib8/UFozGkKozBuXvix7r9V6aqP8sWZgHn0Gxw9cbrJq8x3LcFhgG08YjoNtV gLCT4PUVh3mquscG7Np/dueR4uKf/FVQxRspbCGIWD7F8476+K7Dt7+ruSHy/ILyP1Qe+ny06V+KM 372601B9sbhspYDPduQDpJISrYMqdwxLXrrGuw0tfD8xg9TEgTzpgipz6aZtd8eoE/+PvLo65D6rp Ydcun24Qe3UdnNeKA+JqyZh5kJ/5PCSDkmWsBg7YFuFRhCmQcE+0RNLc8gNg/G2GIy5IXnPx7AZJf 8akvqjhbfYGVZI0ixB0A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1opUnh-00C2bO-6S; Mon, 31 Oct 2022 13:26:01 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1opUnf-00C2Z1-3J for linux-arm-kernel@lists.infradead.org; Mon, 31 Oct 2022 13:26:00 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 1A9621FB; Mon, 31 Oct 2022 06:26:00 -0700 (PDT) Received: from bogus (unknown [10.57.6.137]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 297E63F703; Mon, 31 Oct 2022 06:25:51 -0700 (PDT) Date: Mon, 31 Oct 2022 13:25:23 +0000 From: Sudeep Holla To: Cristian Marussi Cc: Guenter Roeck , linux-kernel@vger.kernel.org, Sudeep Holla , linux-arm-kernel@lists.infradead.org, Daniel Lezcano , linux-hwmon@vger.kernel.org Subject: Re: [PATCH v2 7/8] hwmon: (scmi) Register explicitly with Thermal Framework Message-ID: <20221031132523.mfp7d5esk5hohldk@bogus> References: <7acc7a49-debb-abdb-f01c-f8adef4c1f0e@roeck-us.net> <20221031114018.59048-1-cristian.marussi@arm.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20221031114018.59048-1-cristian.marussi@arm.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221031_062559_204990_6065A639 X-CRM114-Status: GOOD ( 13.27 ) 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-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Oct 31, 2022 at 11:40:18AM +0000, Cristian Marussi wrote: > Available sensors are enumerated and reported by the SCMI platform server > using a 16bit identification number; not all such sensors are of a type > supported by hwmon subsystem and, among the supported ones, only a subset > could be temperature sensors that have to be registered with the Thermal > Framework. > Potential clashes between hwmon channels indexes and the underlying real > sensors IDs do not play well with the hwmon<-->thermal bridge automatic > registration routines and could need a sensible number of fake dummy > sensors to be made up in order to keep indexes and IDs in sync. > > Avoid to use the hwmon<-->thermal bridge dropping the HWMON_C_REGISTER_TZ > attribute and instead explicit register temperature sensors directly with > the Thermal Framework. > Hi Guenter, FWIW from scmi perspective, Acked-by: Sudeep Holla I was about to ask for your ack to pickup myself but I see there is no strict dependency for that. Not sure if you want to take this as fix for v6.1 as the thermal changes broke the existing support in SCMI hwmon or do you still prefer v6.2 ? -- Regards, Sudeep _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel