Messages in this thread | | | Date | Tue, 15 Nov 2022 11:16:53 +0000 | From | Mark Brown <> | Subject | Re: [PATCH v4 1/3] regulator: Add of_regulator_bulk_get_all |
| |
On Tue, Nov 15, 2022 at 10:42:50AM +0000, Russell King (Oracle) wrote: > On Tue, Nov 15, 2022 at 10:34:41AM +0000, Mark Brown wrote:
> > Well, it's not making this maintainer happy :/ If we know what > > PHY is there why not just look up the set of supplies based on > > the compatible of the PHY?
> It looks to me like this series fetches the regulators before the PHY > is bound to the driver, so what you're proposing would mean that the > core PHY code would need a table of all compatibles (which is pretty > hard to do, they encode the vendor/device ID, not some descriptive > name) and then a list of the regulator names. IMHO that doesn't scale.
Oh, PHYs have interesting enough drivers to dynamically load here? The last time I was looking at MDIO stuff it was all running from generic class devices but that was quite a while ago. [unhandled content-type:application/pgp-signature] | |