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 26592C4167B for ; Sat, 9 Dec 2023 10:57:46 +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:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=hwWheHW7U7ItbzsRbTfmULu+akYD+XsNoiW1e6EcWgc=; b=DGQZQoter9naUmm15k1rEii/f6 X9UdEfR1OU/zCrHL201vAm/WaxbtW4rVXcccOHa7xQyhvPbnJ49DRhgxxziyF+1NHZZmiF2MWmovi n4iojJTUj9iM5BY+j+HW5rS3xOa20qMNCv1q3kM11X3ZHecwa18LcSbemPoSyR7DIO8qnzHhbdMXT aXEsFnfgqDkIBt5ZGUJlOlJNUsClUOtvaff3EMxBJkIbQBUiO5OMiIFfeHY5FlZKFv37L6H5pDh9O rMXRtNT9Llg6622uJcGGF0dtjQKDUid6eNV5fXa4K4UNuqCk/45anNPGqqJwBqLqNw/ZmA5u2pFF0 nICnj7zw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rBv1J-00HT6Z-2W; Sat, 09 Dec 2023 10:57:17 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rBv1G-00HT57-1n for linux-arm-kernel@lists.infradead.org; Sat, 09 Dec 2023 10:57:16 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id D28CA603E0; Sat, 9 Dec 2023 10:57:06 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9D6E5C433C8; Sat, 9 Dec 2023 10:57:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1702119426; bh=LfIhP/af5BHnpvQo4uP7i1HD6/yJmre7AxHGR/IvG54=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=uUnmF7CDb4lBM7QWDc8vErl87dZFgiju4DLw8Ih0qLnT+w4aVy9wIgenKuUyH/v2X D5jSkv59gp0NSY/GJSlpq+e+eLxXAn7vEpz/TvH+2I10F0pu6DWDfibGZEgrpKQ2Wl Q3Uoj7/cYn3Pcq8FWY2nU7XR+nYRwn0m9wgZuMS7gWFzfcglm6ufaPsF7YdZAI5vEC cCgh34qJJnPXw97CYrqSuaBYUoZCWe8I8z2q+jp6Fc4SUJ1dGpfiIYaBhRTmn1c+mh hTC5LF7SvezZydDEUnCsv2gCGNwUJ1LiDDf23lCaHQvnUXUoFqVav5Uu/RnnOKEHki dnSOXEc026jLg== Date: Sat, 9 Dec 2023 10:57:01 +0000 From: Conor Dooley To: Dmitry Torokhov Subject: Re: [PATCH v1 1/2] dt-bindings: input: atmel,maxtouch: add poweroff-in-suspend property Message-ID: <20231209-sizzle-monthly-6e4f3c966b0f@spud> References: <20231207111300.80581-1-eichest@gmail.com> <20231207111300.80581-2-eichest@gmail.com> MIME-Version: 1.0 In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231209_025714_678539_640F152B X-CRM114-Status: GOOD ( 25.89 ) 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: , Cc: devicetree@vger.kernel.org, conor+dt@kernel.org, Stefan Eichenberger , nick@shmanahar.org, linux-kernel@vger.kernel.org, Linus Walleij , claudiu.beznea@tuxon.dev, Stefan Eichenberger , robh+dt@kernel.org, alexandre.belloni@bootlin.com, krzysztof.kozlowski+dt@linaro.org, linux-input@vger.kernel.org, linux-arm-kernel@lists.infradead.org Content-Type: multipart/mixed; boundary="===============6332257107082962706==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============6332257107082962706== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="C72rfSVWvZsQ7rnL" Content-Disposition: inline --C72rfSVWvZsQ7rnL Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Dec 08, 2023 at 11:37:47PM +0000, Dmitry Torokhov wrote: > Hi Linus, Krzysztof, >=20 > On Fri, Dec 08, 2023 at 01:54:21PM +0100, Linus Walleij wrote: > > On Thu, Dec 7, 2023 at 12:13=E2=80=AFPM Stefan Eichenberger wrote: > >=20 > > > From: Stefan Eichenberger > > > > > > Add a new property to indicate that the device should be powered off = in > > > suspend mode. > > > > > > Signed-off-by: Stefan Eichenberger > > (...) > > > + atmel,poweroff-in-suspend: > > > + description: | > > > + When this property is set, all supplies are turned off when th= e system is > > > + going to suspend. > > > + type: boolean > > wakeup-source: > > type: boolean > >=20 > > As Krzysztof says it seems you are describing an operating system featu= re. >=20 > It appears to be an OS feature, but I would argue that it is also a > property of a board. It is tempting to say that if DTS defines supplies > for the controller we should use them to power off the controller in > suspend, otherwise we should use the deep sleep functionality of the > controller. But a mere presence of regulators does not indicate if they > can actually be powered off in suspend (i.e. if controllers shares power > rails with another device that can be a wakeup source), so we need to > have additional hints on how OS should behave on a given device. >=20 > On top of that we have regulator framework supplying dummy regulators... Simply rephrasing the property might be sufficient? The current one sounds making policy decisions with the "should be". Reframing the commit message and property description etc in terms of what aspect of the hardware the ability to turn off all supplies in suspend comes from would make it more acceptable. Pretty much answering the question "why can't we try and turn off all supplies on all systems with this device" should get things rolling. Cheers, Conor. --C72rfSVWvZsQ7rnL Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZXRH/QAKCRB4tDGHoIJi 0gwwAP0f6ZUlt2hLfY6VvqK/jBmyh55L7p9l4zE7YNlHzIYoAgD/QhPbNUmbRBTQ ryYrjWFFE9seO+T3W6rJ5AXJgOQskw8= =9hx8 -----END PGP SIGNATURE----- --C72rfSVWvZsQ7rnL-- --===============6332257107082962706== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============6332257107082962706==--