From patchwork Mon Feb 12 18:25:03 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Rafael J. Wysocki" X-Patchwork-Id: 20266 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7300:bc8a:b0:106:860b:bbdd with SMTP id dn10csp103473dyb; Mon, 12 Feb 2024 10:47:08 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCUxKrBNO+I57STuTCKj2oQC+atfKxbho8IBhAJSE3c0kwcOBjZvp/84vWJuZ+J/NCXs8tV2sAHqrwIfRJeBLj2IsR5+wA== X-Google-Smtp-Source: AGHT+IEey5EUth/bz9s00719vPuFNP6Ga4yrnNfhZ06FMQw/QzobNNpMAwRSeah2X6KEA7DnWOK3 X-Received: by 2002:a17:903:1247:b0:1d9:7ba7:bba with SMTP id u7-20020a170903124700b001d97ba70bbamr8828876plh.20.1707763628303; Mon, 12 Feb 2024 10:47:08 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1707763628; cv=pass; d=google.com; s=arc-20160816; b=gv7N8Jcz9fQKR/OAVRHtSzE+9VaP+PSA7fbNo2AOY6fdNsctPMKsp42rJXuUFBMpJ5 MuOyM/yPLxWYufsBf1NK7KXfX1pi+8yY2OIJMQ67xkNRavx1CPq4G8JX/OfZnMM2I/tz pZLYIX08wy6x0i0NZr5HD7jsvDp2WUpTOrvxb+VlMdw7BwZ/73mko+s2s+nJeJQiuGQn oIGvlRplPkvWE1rRZgpitWR56zay5yKvRB7T5GZzKDIY7PUpHRbdguWgxFQ7j01eGPj0 uwf5SEzj8ZMY0jYAu1KXsnyslyXolr7LROmjmIVrzoToWoShMXoqJra+pP6Siw6JU0ki GQGg== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:message-id:date:subject:cc:to :from; bh=M5+ZP1ybQpJsRspLnPwVSQwkZkP25MDtcMAxat4KULc=; fh=/QTK0LaAZA866S/cQwS0MYUPhPMCnGYWA8WhQtFcptM=; b=YYf4biEp5FN/XqLmI5Zpe/zwf1353FAYB8k6787EGk1dSN60iKCdembkwc1eQ934X2 dvcx8OJxEKDPOyfgRWqa8dYUf7fWzOAobwMWGoJkggx56RZ+ofwsDh0+PHnfGAnOyaEc hN7g7G9bOHvwgQxAaL5IxhegEybqjWceL+RYseAwN3wFpWPAwPnbjTWYAZKyY7vA4unO ypY1hZYj5I92TeKWqwiMYwTGUQaAY82lNqbKNqkW5zXgEK84bzJp51p1xaDt9XLQ7AzY JUoW01oQOX66WFaWNQesG3y0CHKawtJ3XHudBKqBjDgC7zibzW3ZO9eEZvOhudXkm0Fg wU0A==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; arc=pass (i=1 spf=pass spfdomain=rjwysocki.net); spf=pass (google.com: domain of linux-kernel+bounces-62216-ouuuleilei=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-62216-ouuuleilei=gmail.com@vger.kernel.org" X-Forwarded-Encrypted: i=2; AJvYcCVs/Q81LCM+qWmBpozVwxYPnpZfUspHyOfKyyWFNZhTqPvj7e6frrpTMK5ZVh5kvU0RHn6geSj5MyeEC/xfLjlxRmNpxQ== Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [139.178.88.99]) by mx.google.com with ESMTPS id h8-20020a170902748800b001d9a61d7bf0si592954pll.186.2024.02.12.10.47.08 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 12 Feb 2024 10:47:08 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-62216-ouuuleilei=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99; Authentication-Results: mx.google.com; arc=pass (i=1 spf=pass spfdomain=rjwysocki.net); spf=pass (google.com: domain of linux-kernel+bounces-62216-ouuuleilei=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-62216-ouuuleilei=gmail.com@vger.kernel.org" Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 152622833C0 for ; Mon, 12 Feb 2024 18:47:08 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 399274F897; Mon, 12 Feb 2024 18:42:41 +0000 (UTC) Received: from cloudserver094114.home.pl (cloudserver094114.home.pl [79.96.170.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8CCC34C63D; Mon, 12 Feb 2024 18:42:36 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=79.96.170.134 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707763359; cv=none; b=Q5hehO6TQprwc6rU8k/4Qk88AfmDvTf4j4R+XE7mGO2gKwLaWxqf1fCMYDnaPsX4or2sr3PrhpDaFK3X2bfIfvwBAbsbTDpz5Fc32kVGKf5p3EU8MGgbLWTHU3xiGD1JE4FdoK87ZTYakHCEpuEPB7LXeAJ/QLigAqxuVyUZ9Uc= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707763359; c=relaxed/simple; bh=AZVaCsAriHLBfg1CD2bjELKhsNVl1DMTTJvKn6jlgmo=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version:Content-Type; b=ecPSuY2QrzaeCpWXtaD0ZprFUpWRQdd1LdpA8ClWmny4IIXKu2N/Agu8yx+kl337DgXp4iYQVkVnaHeu8mV1gMVHW4m7wVqC7H0dylGAk8k7dsHwfzSrBWNI5UUHFt2NAKWVGarpHI0hY4EDvv2D42WAU80zaLMciCJHb2CaOco= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=rjwysocki.net; spf=pass smtp.mailfrom=rjwysocki.net; arc=none smtp.client-ip=79.96.170.134 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=rjwysocki.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=rjwysocki.net Received: from localhost (127.0.0.1) (HELO v370.home.net.pl) by /usr/run/smtp (/usr/run/postfix/private/idea_relay_lmtp) via UNIX with SMTP (IdeaSmtpServer 5.4.0) id eb00a7d99c467735; Mon, 12 Feb 2024 19:42:34 +0100 Received: from kreacher.localnet (unknown [195.136.19.94]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by cloudserver094114.home.pl (Postfix) with ESMTPSA id EE0EA669CF2; Mon, 12 Feb 2024 19:42:33 +0100 (CET) From: "Rafael J. Wysocki" To: Linux PM Cc: Lukasz Luba , LKML , Daniel Lezcano , Stanislaw Gruszka , Srinivas Pandruvada , Zhang Rui , netdev@vger.kernel.org, Ido Schimmel , Petr Machata , Miri Korenblit , linux-wireless@vger.kernel.org, Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Manaf Meethalavalappu Pallikunhi Subject: [PATCH v2 0/9] thermal: Writable trip points handling rework Date: Mon, 12 Feb 2024 19:25:03 +0100 Message-ID: <6017196.lOV4Wx5bFT@kreacher> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-CLIENT-IP: 195.136.19.94 X-CLIENT-HOSTNAME: 195.136.19.94 X-VADE-SPAMSTATE: clean X-VADE-SPAMCAUSE: gggruggvucftvghtrhhoucdtuddrgedvledrudefgdduudegucetufdoteggodetrfdotffvucfrrhhofhhilhgvmecujffqoffgrffnpdggtffipffknecuuegrihhlohhuthemucduhedtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenucfjughrpefhvfevufffkfgggfgtsehtufertddttdejnecuhfhrohhmpedftfgrfhgrvghlucflrdcuhgihshhotghkihdfuceorhhjfiesrhhjfiihshhotghkihdrnhgvtheqnecuggftrfgrthhtvghrnhepgeffhfdujeelhfdtgeffkeetudfhtefhhfeiteethfekvefgvdfgfeeikeeigfehnecuffhomhgrihhnpehkvghrnhgvlhdrohhrghenucfkphepudelhedrudefiedrudelrdelgeenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepihhnvghtpeduleehrddufeeirdduledrleegpdhhvghlohepkhhrvggrtghhvghrrdhlohgtrghlnhgvthdpmhgrihhlfhhrohhmpedftfgrfhgrvghlucflrdcuhgihshhotghkihdfuceorhhjfiesrhhjfiihshhotghkihdrnhgvtheqpdhnsggprhgtphhtthhopeduiedprhgtphhtthhopehlihhnuhigqdhpmhesvhhgvghrrdhkvghrnhgvlhdrohhrghdprhgtphhtthhopehluhhkrghsiidrlhhusggrsegrrhhmrdgtohhmpdhrtghpthhtoheplhhinhhugidqkhgvrhhnvghlsehvghgvrhdrkhgvrhhnvghlrdhorhhgpdhrtghpthhtohepuggrnhhivghlrdhlvgiitggrnhhosehlihhn rghrohdrohhrghdprhgtphhtthhopehsthgrnhhishhlrgifrdhgrhhushiikhgrsehlihhnuhigrdhinhhtvghlrdgtohhmpdhrtghpthhtohepshhrihhnihhvrghsrdhprghnughruhhvrggurgeslhhinhhugidrihhnthgvlhdrtghomh X-DCC--Metrics: v370.home.net.pl 1024; Body=16 Fuz1=16 Fuz2=16 X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1790719954175107717 X-GMAIL-MSGID: 1790719954175107717 Hi Everyone, This is an update of https://lore.kernel.org/linux-pm/3232442.5fSG56mABF@kreacher/ fixing a few bugs and renaming the new trip point flags introduced by it. The original description of the patch series is still applicable: "The purpose of this patch series is to allow thermal zone creators to specify which properties (temperature or hysteresis) of which trip points can be set from user space via sysfs on a per-trip basis instead of passing writable trips masks to the thermal zone registration function which is both cumbersome and error prone and it doesn't even allow to request different treatment of different trip properties. The writable trip masks used today only affect trip temperatures (that is, if a trip point is in a writable trips mask, its temperature can be set via sysfs) and they only take effect if the CONFIG_THERMAL_WRITABLE_TRIPS kernel configuration option is set, which appears to be assumed by at least some of the drivers using writable trips masks. Some other drivers using them simply select CONFIG_THERMAL_WRITABLE_TRIPS which pretty much defeats its purpose (and imx even sets this option in its defconfig). For this reasons, patch [1/9] removes CONFIG_THERMAL_WRITABLE_TRIPS and makes the writable trips masks always work. Moreover, trip hysteresis, which is not affected either by the writable trips masks or by CONFIG_THERMAL_WRITABLE_TRIPS, can only be set via sysfs if the .set_trip_hyst() operation is provided by the given thermal zone, but currently this thermal zone operation is used by no one, so effectively trip hysteresis cannot be set via sysfs at all. This is not a problem for the majority of drivers that want trip temperatures to be set via sysfs, because they also don't want trip hysteresis to be changed for any trips (at least as far as I can say), but there are use cases in which it is desirable to be able to update trip hysteresis as well as trip temperature (for example see https://lore.kernel.org/linux-pm/20240106191502.29126-1-quic_manafm@quicinc.com/). Those use cases are not addressed here directly, but after this series addressing them should be relatively straightforward. Namely, patch [2/9] adds flags to struct thermal_trip and defines two of them to indicate whether or not setting the temperature or hysteresis of the given trip via sysfs is allowed. If a writable trips mask is passed to thermal_zone_device_register_with_trips(), is it is used to set the "writable temperature" flag for the trips covered by it and that flag is then consulted by the thermal sysfs code. The "writable hysteresis" trip flag is also taken into account by the thermal sysfs code, but it is not set automatically in any case. Patch [3/9] is based on the observation that the .set_trip_hyst() thermal zone operation is never used - it simply drops that callback from struct thermal_zone_device_ops and adjusts the code checking its presence. Patches [4-8/9] update drivers using writable trips masks to set the new "writable temperature" flag directly instead and some of them are simplified a bit as a result. After these patches, all of the callers of thermal_zone_device_register_with_trips() pass a zero writable trips mask to it, so patch [9/9] drops that mask from the functions argument list and adjusts all of its callers accordingly. After all of the changes in this series, allowing the hysteresis value to be set via sysfs for a given trip is a matter of setting its "writable hysteresis" flag (and analogously for trip temperature)." Thanks! Reviewed-by: Daniel Lezcano