[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CA+E=qVf8_9gn0y=mcdKXvj2PFoHT2eF+JN=CmtTNdRGaSnpgKg@mail.gmail.com>
Date: Fri, 27 Dec 2024 14:34:10 -0800
From: Vasily Khoruzhick <anarsoul@...il.com>
To: Chris Morgan <macroalpha82@...il.com>
Cc: Andre Przywara <andre.przywara@....com>, Lee Jones <lee@...nel.org>,
Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>,
Chen-Yu Tsai <wens@...e.org>, Liam Girdwood <lgirdwood@...il.com>, Mark Brown <broonie@...nel.org>,
devicetree@...r.kernel.org, linux-sunxi@...ts.linux.dev,
linux-kernel@...r.kernel.org, Martin Botka <martin.botka@...ainline.org>,
Chris Morgan <macromorgan@...mail.com>
Subject: Re: [PATCH v2 3/5] mfd: axp20x: Allow multiple regulators
On Sat, Dec 7, 2024 at 9:45 AM Chris Morgan <macroalpha82@...il.com> wrote:
> Using git bisect, I found that this patch breaks the CONFIG_AXP20X_ADC
> option which is used by some of the battery and charger drivers for the
> axp20x PMIC series. My current assumption is that the
> devm_iio_channel_get() call made by these drivers worked correctly
> previously when the PLATFORM_DEVID_NONE, but now it's not working
> anymore. I'm still testing possible solutions for that problem.
I confirm that this patch breaks the battery driver on Pinebook (and
likely Pinephone). Reverting it fixes the issue for me.
> Thank you,
> Chris
>
Powered by blists - more mailing lists