Re: [PATCH 4/5] ARM: novena: Read Ethernet MAC address from EEPROM

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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-> 21e8000.serial@xxxxxxxxxx
>         probe-> 20e0000.pinctrl@xxxxxxxxxx
>     probe-> 21b0000.memory-controller@xxxxxxxxxx
>     probe-> ramfs0
>     probe-> devfs0
>     probe-> mem0
>     probe-> mem1
> netconsole: port not set
> netconsole: registered as netconsole-1
>     probe-> machine
>         probe-> 21a8000.i2c@xxxxxxxxxx
>             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-> 110000.dma-apbh@xxxxxxxxx
>     probe-> 120000.hdmi@xxxxxxxxx
> imx-hdmi 120000.hdmi@xxxxxxxxx: Detected HDMI controller 0x13:0xa:0xa0:0xc1
>     probe-> 2010000.spi@xxxxxxxxxx
>     probe-> 2080000.pwm@xxxxxxxxxx
>     probe-> 209c000.gpio@xxxxxxxxxx
>     probe-> 20a0000.gpio@xxxxxxxxxx
>     probe-> 20a4000.gpio@xxxxxxxxxx
>     probe-> 20a8000.gpio@xxxxxxxxxx
>     probe-> 20ac000.gpio@xxxxxxxxxx
>     probe-> 20b0000.gpio@xxxxxxxxxx
>     probe-> 20b4000.gpio@xxxxxxxxxx
>     probe-> 20bc000.watchdog@xxxxxxxxxx
>     probe-> 20c9000.usbphy@xxxxxxxxxx
>     probe-> 20ca000.usbphy@xxxxxxxxxx
>     probe-> 20d8000.reset-controller@xxxxxxxxxx
>     probe-> 2184000.usb@xxxxxxxxxx
>         probe-> 2184800.usbmisc@xxxxxxxxxx
>         probe-> regulator-usb-otg-vbus.of
>     probe-> 2184200.usb@xxxxxxxxxx
>         probe-> 21a4000.i2c@xxxxxxxxxx
>             probe-> pfuze1000
> imx-usb 2184200.usb@xxxxxxxxxx: Cannot get vbus regulator: No such device (ignoring)
> imx-usb 2184200.usb@xxxxxxxxxx: USB EHCI 1.00
>     probe-> 2188000.ethernet@xxxxxxxxxx
> mdio_bus: miibus0: probed
>     probe-> 2194000.mmc@xxxxxxxxxx
> imx-esdhc 2194000.mmc@xxxxxxxxxx: registered as mmc1
>     probe-> 2198000.mmc@xxxxxxxxxx
> imx-esdhc 2198000.mmc@xxxxxxxxxx: registered as mmc2
>     probe-> 21a0000.i2c@xxxxxxxxxx
>     probe-> 21bc000.efuse@xxxxxxxxxx
>     probe-> 21ec000.serial@xxxxxxxxxx
>     probe-> 21f0000.serial@xxxxxxxxxx
>     probe-> 2400000.ipu@xxxxxxxxxx
> imx-ipuv3 2400000.ipu@xxxxxxxxxx: IPUv3H probed
>     probe-> 2200000.sata@xxxxxxxxxx
>     probe-> 2800000.ipu@xxxxxxxxxx
> imx-ipuv3 2800000.ipu@xxxxxxxxxx: 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 |





[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux