From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Wed, 25 Jan 2023 15:17:37 +0100 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1pKgap-001Zbc-6F for lore@lore.pengutronix.de; Wed, 25 Jan 2023 15:17:37 +0100 Received: from bombadil.infradead.org ([2607:7c80:54:3::133]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1pKgal-0001jI-VV for lore@pengutronix.de; Wed, 25 Jan 2023 15:17:36 +0100 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=V+AMEJwbP/Xdz2Tg+JbipHK99sBmXhi75h1qa6bXkwM=; b=wgWT82GsPn/q1Gg5dhssFO6HSV gfTKa1AHcTvxhQFy00J+r0/ncptAcNufTKRKfwZOLzcv1jNFtxQAYfC2nR2s/HJWiRM8UMCjauejb QgRaBRXh6HgMWSQVZT+xHC9VaR7B1paLWsfD8RD9ay52bHZ01CKKLdQ66YtCxHoyV1H2IwI46AEqD G88DqV3nree/pl0Mvk12lisxeuxMT4w8iF9Opv+ww7LyiIsoglZbjZ6Kik52lUJ9vOpQO3n5rt7Dp 0OIEbEY48YlSW7IRxSrdOFd4sPIGUHILYEeD7rA+pCKgjYAeV85ja3iMxvPu1q52PHhZk17u9nL1f i0dHEO0w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pKgZU-007U9w-6O; Wed, 25 Jan 2023 14:16:16 +0000 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pKgZP-007U8s-68 for barebox@lists.infradead.org; Wed, 25 Jan 2023 14:16:12 +0000 Received: from ptz.office.stw.pengutronix.de ([2a0a:edc0:0:900:1d::77] helo=[127.0.0.1]) by metis.ext.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1pKgZK-0001WI-CT; Wed, 25 Jan 2023 15:16:06 +0100 Message-ID: <77f25018-39ab-ec53-56a5-21bfdeb484d7@pengutronix.de> Date: Wed, 25 Jan 2023 15:16:05 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.5.0 Content-Language: en-US To: John Watts Cc: Sascha Hauer , barebox@lists.infradead.org References: <20230122175141.119834-1-contact@jookia.org> <20230122175141.119834-5-contact@jookia.org> <20230123093305.GS24755@pengutronix.de> <20230125080406.GW24755@pengutronix.de> <20230125081923.GY24755@pengutronix.de> <072ab8ce-d92f-8841-a44e-621a3ba345a4@pengutronix.de> From: Ahmad Fatoum In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230125_061611_257914_D34D068E X-CRM114-Status: GOOD ( 20.93 ) X-BeenThere: barebox@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "barebox" X-SA-Exim-Connect-IP: 2607:7c80:54:3::133 X-SA-Exim-Mail-From: barebox-bounces+lore=pengutronix.de@lists.infradead.org X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on metis.ext.pengutronix.de X-Spam-Level: X-Spam-Status: No, score=-5.3 required=4.0 tests=AWL,BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,NICE_REPLY_A,RCVD_IN_DNSWL_MED,SPF_HELO_NONE, SPF_NONE autolearn=unavailable autolearn_force=no version=3.4.2 Subject: Re: [PATCH 4/5] ARM: novena: Read Ethernet MAC address from EEPROM X-SA-Exim-Version: 4.2.1 (built Wed, 08 May 2019 21:11:16 +0000) X-SA-Exim-Scanned: Yes (on metis.ext.pengutronix.de) 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@21e8000.of > probe-> 20e0000.pinctrl@20e0000.of > probe-> 21b0000.memory-controller@21b0000.of > probe-> ramfs0 > probe-> devfs0 > probe-> mem0 > probe-> mem1 > netconsole: port not set > netconsole: registered as netconsole-1 > probe-> machine > probe-> 21a8000.i2c@21a8000.of > 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@110000.of > probe-> 120000.hdmi@120000.of > imx-hdmi 120000.hdmi@120000.of: Detected HDMI controller 0x13:0xa:0xa0:0xc1 > probe-> 2010000.spi@2010000.of > probe-> 2080000.pwm@2080000.of > probe-> 209c000.gpio@209c000.of > probe-> 20a0000.gpio@20a0000.of > probe-> 20a4000.gpio@20a4000.of > probe-> 20a8000.gpio@20a8000.of > probe-> 20ac000.gpio@20ac000.of > probe-> 20b0000.gpio@20b0000.of > probe-> 20b4000.gpio@20b4000.of > probe-> 20bc000.watchdog@20bc000.of > probe-> 20c9000.usbphy@20c9000.of > probe-> 20ca000.usbphy@20ca000.of > probe-> 20d8000.reset-controller@20d8000.of > probe-> 2184000.usb@2184000.of > probe-> 2184800.usbmisc@2184800.of > probe-> regulator-usb-otg-vbus.of > probe-> 2184200.usb@2184200.of > probe-> 21a4000.i2c@21a4000.of > probe-> pfuze1000 > imx-usb 2184200.usb@2184200.of: Cannot get vbus regulator: No such device (ignoring) > imx-usb 2184200.usb@2184200.of: USB EHCI 1.00 > probe-> 2188000.ethernet@2188000.of > mdio_bus: miibus0: probed > probe-> 2194000.mmc@2194000.of > imx-esdhc 2194000.mmc@2194000.of: registered as mmc1 > probe-> 2198000.mmc@2198000.of > imx-esdhc 2198000.mmc@2198000.of: registered as mmc2 > probe-> 21a0000.i2c@21a0000.of > probe-> 21bc000.efuse@21bc000.of > probe-> 21ec000.serial@21ec000.of > probe-> 21f0000.serial@21f0000.of > probe-> 2400000.ipu@2400000.of > imx-ipuv3 2400000.ipu@2400000.of: IPUv3H probed > probe-> 2200000.sata@2200000.of > probe-> 2800000.ipu@2800000.of > imx-ipuv3 2800000.ipu@2800000.of: 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 |