On 25.01.23 15:04, John Watts wrote: > On Wed, Jan 25, 2023 at 02:48:20PM +0100, Ahmad Fatoum wrote: >> regulator-fixed only understands vin-supply. > > Right, changed. I was under the assumption that ANY node could specify an > X-supply.
vin-supply is handled by regulator core. Other X-supply needs to be explicitly requested by driver. >> I actually updated the eeprom node to use >>> reg_3p3v as vcc-supply and things were still probed in the wrong order. >> >> That's odd. You can enable CONFIG_DEBUG_PROBES to see in what order >> probes happened and at what depth (each probe within a probe is >> indented more than its parent). > > Should that even work? The vcc-supply property isn't a real one. It does in Linux :-) We could add the binding to the barebox AT24/AT25 driver. We'll need to make sure we don't fail the probe in the error path as not to break existing users. Fortunately, with deep probe, we will have no -EPROBE_DEFER that would complicate this. > > Currently the probe log is this: > > detected i.MX6 Quad revision 1.2 > i.MX6 Quad unique ID: c1d99d4d8191854 > probe-> [email protected] > probe-> [email protected] > probe-> [email protected] > probe-> ramfs0 > probe-> devfs0 > probe-> mem0 > probe-> mem1 > netconsole: port not set > netconsole: registered as netconsole-1 > probe-> machine > probe-> [email protected] > probe-> 24c5120 > Unable to read Novena EEPROM: Connection timed out > probe-> ldb.of > imx-ldb ldb.of: Neither display timings in ldb node nor remote panel found > imx-ldb ldb.of: dual-channel mode, ignoring second output > probe-> usbphynop1.of > probe-> usbphynop2.of > probe-> [email protected] > probe-> [email protected] > imx-hdmi [email protected]: Detected HDMI controller 0x13:0xa:0xa0:0xc1 > probe-> [email protected] > probe-> [email protected] > probe-> [email protected] > probe-> [email protected] > probe-> [email protected] > probe-> [email protected] > probe-> [email protected] > probe-> [email protected] > probe-> [email protected] > probe-> [email protected] > probe-> [email protected] > probe-> [email protected] > probe-> [email protected] > probe-> [email protected] > probe-> [email protected] > probe-> regulator-usb-otg-vbus.of > probe-> [email protected] > probe-> [email protected] > probe-> pfuze1000 > imx-usb [email protected]: Cannot get vbus regulator: No such device > (ignoring) > imx-usb [email protected]: USB EHCI 1.00 > probe-> [email protected] > mdio_bus: miibus0: probed > probe-> [email protected] > imx-esdhc [email protected]: registered as mmc1 > probe-> [email protected] > imx-esdhc [email protected]: registered as mmc2 > probe-> [email protected] > probe-> [email protected] > probe-> [email protected] > probe-> [email protected] > probe-> [email protected] > imx-ipuv3 [email protected]: IPUv3H probed > probe-> [email protected] > probe-> [email protected] > imx-ipuv3 [email protected]: IPUv3H probed > probe-> gpio-keys.of > probe-> leds.of > probe-> regulator-2p5v.of > probe-> regulator-3p3v.of > probe-> regulator-5p0v.of > probe-> regulator-audio-codec.of > probe-> regulator-display.of > probe-> regulator-lvds-lcd.of > probe-> regulator-pcie.of > probe-> regulator-sata.of > malloc space: 0xbfcf55c0 -> 0xffcf55bf (size 1 GiB) > probe-> imx-ipuv3-crtc0 > probe-> imx-ipuv3-crtc1 > probe-> imx-ipuv3-crtc2 > probe-> imx-ipuv3-crtc3 > >> >> Cheers, >> Ahmad > > John. > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
