From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1j6dls-0006xs-Dp for barebox@lists.infradead.org; Tue, 25 Feb 2020 17:13:25 +0000 References: <92a929cc-b9e6-db9b-352d-c7c271a29558@pengutronix.de> <43978130-6f7d-759c-ef63-a005cb6f0934@pengutronix.de> <64cbefba-5ce4-919e-1de2-48e2247ca391@pengutronix.de> From: Ahmad Fatoum Message-ID: <7ba631db-78a6-d0d6-8036-8516c0a9b948@pengutronix.de> Date: Tue, 25 Feb 2020 18:13:21 +0100 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: musb is broken in master To: Yegor Yefremov Cc: t.remmet@phytec.de, barebox Hello, On 2/24/20 3:08 PM, Yegor Yefremov wrote: > I've checked out barebox master and then reverted > 574eed3f6fcf056aa4c9e46c4b5224e3f7844d8d (dts: update to v5.5-rc1). > After examining the changes, I found out that am33xx.dtsi was changed. > One of this changes is the following: > > - usb: usb@47400000 { > - compatible = "ti,am33xx-usb"; > - reg = <0x47400000 0x1000>; > - ranges; > + usb: target-module@47400000 { > + compatible = "ti,sysc-omap4", "ti,sysc"; > + reg = <0x47400000 0x4>, > + <0x47400010 0x4>; > + reg-names = "rev", "sysc"; > + ti,sysc-mask = <(SYSC_OMAP4_FREEEMU | > + SYSC_OMAP2_SOFTRESET)>; > + ti,sysc-midle = , > + , > + ; > + ti,sysc-sidle = , > + , > + , > + ; > + clocks = <&l3s_clkctrl AM3_L3S_USB_OTG_HS_CLKCTRL 0>; > + clock-names = "fck"; > > If I understand it correctly, drivers/usb/musb/musb_am335x.c still has > the old .compatible string: > > static __maybe_unused struct of_device_id am335x_child_dt_ids[] = { > { > .compatible = "ti,am33xx-usb", > }, { > /* sentinel */ > }, > }; > > This node was removed: > > - usb_ctrl_mod: control@44e10620 { > - compatible = "ti,am335x-usb-ctrl-module"; > - reg = <0x44e10620 0x10 > - 0x44e10648 0x4>; > - reg-names = "phy_ctrl", "wakeup"; > - status = "disabled"; > - }; > > So there are many changes in device tree files that are not reflected > in the drivers. I haven't debugged it but just examined the introduced > changes. I see. I misunderstood your earlier answer to mean that reverting the _Linux_ patch in question fixed the issue for you instead of reverting the whole _barebox_ patch. I've bisected the issue in the kernel git and arrived at the same commit about ti,sysc that you suspected. While we have the necessary support of barebox to support this, the new device tree structuring changed the order in which we probed the different MUSB drivers. I have just sent out two patches to teach the MUSB drivers to accept the new world order. Your Tested-by: would be appreciated. Apparently, we already do some automatic testing for the BeagleBone, including barebox, with our internal DistroKit CI. I'll amend our tests, so it may catch new warnings after updates, like the one you encountered. Cheers Ahmad > > Yegor > -- 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 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox