Re: next/master bisection: baseline.bootrr.intel-igb-probed on kontron-pitx-imx8m

From: Mark Brown <broonie@kernel.org>
To: Saravana Kannan <saravanak@google.com>
Cc: Michael Walle <michael@walle.cc>,
	Geert Uytterhoeven <geert+renesas@glider.be>,
	Ulf Hansson <ulf.hansson@linaro.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	kernelci-results@groups.io, bot@kernelci.org,
	gtucker@collabora.com, linux-arm-kernel@lists.infradead.org,
	linux-pci@vger.kernel.org
Subject: Re: next/master bisection: baseline.bootrr.intel-igb-probed on kontron-pitx-imx8m
Date: Tue, 16 Aug 2022 18:26:02 +0100	[thread overview]
Message-ID: <YvvTKkR4sOIsFxA4@sirena.org.uk> (raw)
In-Reply-To: <CAGETcx_JSViBcySNp+Rany2osBiKL7ON+tooPKW8TOTP6+t_HA@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 882 bytes --]

On Fri, Aug 12, 2022 at 04:54:25PM -0700, Saravana Kannan wrote:

> While you are here, I'm working towards patches on top of [1] where
> fw_devlink will tie the sync_state() callback to each regulator. Also,
> i realized that if you can convert the regulator_class to a
> regulator_bus, we could remove a lot of the "find the supply for this
> regulator when it's registered" code and let device links handle it.
> Let me know if that's something you'd be okay with. It would change
> the sysfs path for /sys/class/regulator and moves it to
> /sys/bus/regulator, but not sure if that's considered an ABI breakage
> (sysfs paths change all the time).

That *does* sound like it'd be an ABI issue TBH.  I thought there was
support for keeping class around even when converting to a bus (though
TBH given how entirely virtual this stuff us it seems odd that we'd be
going for a bus).

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]