[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <Zbl_y-KmQhCnN7Tu@sashalap>
Date: Tue, 30 Jan 2024 18:01:31 -0500
From: Sasha Levin <sashal@...nel.org>
To: Sebastian Reichel <sebastian.reichel@...labora.com>
Cc: linux-kernel@...r.kernel.org, stable@...r.kernel.org,
Konrad Dybcio <konrad.dybcio@...aro.org>,
Luca Weiss <luca.weiss@...rphone.com>, andersson@...nel.org,
linux-arm-msm@...r.kernel.org, linux-pm@...r.kernel.org,
Johan Hovold <johan+linaro@...nel.org>
Subject: Re: [PATCH AUTOSEL 6.7 15/39] power: supply: qcom_battmgr: Register
the power supplies after PDR is up
On Mon, Jan 29, 2024 at 02:03:14PM +0100, Sebastian Reichel wrote:
>Hi,
>
>On Sun, Jan 28, 2024 at 11:10:35AM -0500, Sasha Levin wrote:
>> From: Konrad Dybcio <konrad.dybcio@...aro.org>
>>
>> [ Upstream commit b43f7ddc2b7a5a90447d96cb4d3c6d142dd4a810 ]
>>
>> Currently, a not-yet-entirely-initialized battmgr (e.g. with pd-mapper
>> not having yet started or ADSP not being up etc.) results in a couple of
>> zombie power supply devices hanging around.
>>
>> This is particularly noticeable when trying to suspend the device (even
>> s2idle): the PSY-internal thermal zone is inaccessible and returns
>> -ENODEV, which causes log spam.
>>
>> Register the power supplies only after we received some notification
>> indicating battmgr is ready to take off.
>>
>> Signed-off-by: Konrad Dybcio <konrad.dybcio@...aro.org>
>> Tested-by: Luca Weiss <luca.weiss@...rphone.com>
>> Link: https://lore.kernel.org/r/20231218-topic-battmgr_fixture_attempt-v1-1-6145745f34fe@linaro.org
>> Signed-off-by: Sebastian Reichel <sebastian.reichel@...labora.com>
>> Signed-off-by: Sasha Levin <sashal@...nel.org>
>> ---
>
>Please drop it, I have a patch queued reverting this patch.
Dropped, thanks!
--
Thanks,
Sasha
Powered by blists - more mailing lists