Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Sun, 23 Jul 2023 15:41:07 +0100
From:   Mark Brown <broonie@kernel.org>
To:     Daniel Lezcano <daniel.lezcano@linaro.org>
Cc:     Icenowy Zheng <uwu@icenowy.me>,
        Vasily Khoruzhick <anarsoul@gmail.com>,
        Yangtao Li <tiny.windzz@gmail.com>,
        "Rafael J. Wysocki" <rafael@kernel.org>,
        Amit Kucheria <amitk@kernel.org>,
        Zhang Rui <rui.zhang@intel.com>, Chen-Yu Tsai <wens@csie.org>,
        Jernej Skrabec <jernej.skrabec@gmail.com>,
        Samuel Holland <samuel@sholland.org>,
        Hugh Dickins <hughd@google.com>, linux-pm@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] thermal/drivers/sun8i: Don't fail probe due to zone
 registration failure
Message-ID: <ZL08A8J7fLJ3bJsl@finisterre.sirena.org.uk>
References: <20230718-thermal-sun8i-registration-v1-1-c95b1b070340@kernel.org>
 <5a9ccb708f004e70e2102417eb48b766b03777cd.camel@icenowy.me>
 <ZLwH9RFnJymdy7YD@finisterre.sirena.org.uk>
 <17dc1961-c629-e09f-8979-1e6cbc92534e@linaro.org>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="RmJB1P66QJ1Tbrkp"
Content-Disposition: inline
In-Reply-To: <17dc1961-c629-e09f-8979-1e6cbc92534e@linaro.org>
Precedence: bulk


--RmJB1P66QJ1Tbrkp
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Sun, Jul 23, 2023 at 11:40:34AM +0200, Daniel Lezcano wrote:
> On 22/07/2023 18:46, Mark Brown wrote:

> > It did seem to be a bit of an excessively strict requirement, I was
> > going to poke at that myself.  It does seem worthwhile doing the change
> > in the sun8i driver anyway, there might be some other issue that causes
> > registration to fail which would have the same issue.

> Why do you want a thermal zone if there is no trip point ?

It at least allows you to see what the hardware is monitoring.  It's not
terribly useful for thermal management (unless you're doing to try to do
something in userspace I guess) but it seems more helpful than
completely failing to register, especially given the knock on effects
when a single device registers multiple zones.

--RmJB1P66QJ1Tbrkp
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmS9PAIACgkQJNaLcl1U
h9CQpAf8CXrJb/uZ2uZiPQxWc3K4AUQnhODnS9900hcXgY503ZuYYzRc99HclUxV
oikMyqyYkxnws/U/08HeOrCZ57O9GJFKd0Zf6WwJn1T/e7ruktnMk1XmdHVw87Dq
RrN9gkary/8yU7iv1NDkbMS5TcD8pDOSkZVSWVXdPQKf/xxRftjUZiIdeF5rWnPb
vpRT50ukw26xbtGedwZcijgU1daHor1QX2yPCxNG5JTqdb9aGKuWgV/sgveyiztf
3SmtXPrg3jS37z72iyCEZbv92Rin9C6CW6B3TYPipr7txk0mveoiwAQza1hrOgR9
WH2ytODtyQOt7JpihXq/rzyYx+SO3A==
=hAw6
-----END PGP SIGNATURE-----

--RmJB1P66QJ1Tbrkp--