Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20211008081137.1948848-1-michal.vokac@ysoft.com>
 <20211018112820.qkebjt2gk2w53lp5@pengutronix.de> <37bc3702-bc98-dc54-e9c7-bf9bc92432f0@linaro.org>
In-Reply-To: <37bc3702-bc98-dc54-e9c7-bf9bc92432f0@linaro.org>
From:   Petr Benes <petrben@gmail.com>
Date:   Tue, 19 Oct 2021 09:24:44 +0200
Message-ID: <CAPwXO5YguJtSFSqnA_aGPch2NswmrP1EzOs0QH5O_iOdtn5W1A@mail.gmail.com>
Subject: Re: [PATCH] thermal: imx: Fix temperature measurements on i.MX6 after alarm
To:     Daniel Lezcano <daniel.lezcano@linaro.org>
Cc:     Oleksij Rempel <o.rempel@pengutronix.de>,
        =?UTF-8?B?TWljaGFsIFZva8OhxI0=?= <michal.vokac@ysoft.com>,
        Andrzej Pietrasiewicz <andrzej.p@collabora.com>,
        linux-pm@vger.kernel.org, Shawn Guo <shawnguo@kernel.org>,
        Amit Kucheria <amitk@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Pengutronix Kernel Team <kernel@pengutronix.de>,
        Fabio Estevam <festevam@gmail.com>,
        NXP Linux Team <linux-imx@nxp.com>,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        =?UTF-8?Q?Petr_Bene=C5=A1?= <petr.benes@ysoft.com>,
        stable@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Mon, 18 Oct 2021 at 13:38, Daniel Lezcano <daniel.lezcano@linaro.org> wr=
ote:
>
> On 18/10/2021 13:28, Oleksij Rempel wrote:
> > Hi Michal,
> >
> > I hope you have seen this patch:
> > https://lore.kernel.org/all/20210924115032.29684-1-o.rempel@pengutronix=
.de/
> >
> > Are there any reason why this was ignored?
>
> No reasons, I was waiting for some tags before merging it. But I forget
> about it when reviewing the current patch.

Tested Oleksij's patch. It works OK. A question arose. Does it require
CONFIG_PM=3Dy?
If this condition is mandatory and the requirement is valid, Kconfig
should be changed accordingly.
I'm not able to verify it works without PM, seems it doesn't.
>
>
>
> --
> <http://www.linaro.org/> Linaro.org =E2=94=82 Open source software for AR=
M SoCs
>
> Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
> <http://twitter.com/#!/linaroorg> Twitter |
> <http://www.linaro.org/linaro-blog/> Blog