Hi Rob or Tony, any insights or fixes? At the moment we have simply reverted cf081d009c44 in the Letux distro kernels. BR and thanks, Nikolaus Schaller > Am 03.02.2022 um 11:16 schrieb H. Nikolaus Schaller <hns@xxxxxxxxxxxxx>: > > Hi, > since v5.17-rc1 musb is broken on the gta04 device. > > Symptoms of this bug are > > a) main symptom > > [ 21.336517] using random host ethernet address > [ 21.341430] using host ethernet address: 32:70:05:18:ff:78 > [ 21.341461] using self ethernet address: 46:10:3a:b3:af:d9 > [ 21.358184] usb0: HOST MAC 32:70:05:18:ff:78 > [ 21.376678] usb0: MAC 46:10:3a:b3:af:d9 > [ 21.388305] using random self ethernet address > [ 21.393371] using random host ethernet address > [ 21.398162] g_ether gadget: Ethernet Gadget, version: Memorial Day 2008 > [ 21.421081] g_ether gadget: g_ether ready > [ 21.492156] musb-hdrc musb-hdrc.1.auto: Could not enable: -22 > [ 21.691345] musb-hdrc musb-hdrc.1.auto: Could not enable: -22 > [ 21.803192] musb-hdrc musb-hdrc.1.auto: Could not enable: -22 > [ 21.819427] musb-hdrc musb-hdrc.1.auto: Could not enable: -22 > [ 22.124450] musb-hdrc musb-hdrc.1.auto: Could not enable: -22 > [ 22.168518] musb-hdrc musb-hdrc.1.auto: Could not enable: -22 > [ 22.179382] musb-hdrc musb-hdrc.1.auto: Could not enable: -22 > [ 23.213592] musb-hdrc musb-hdrc.1.auto: pm runtime get failed in musb_gadget_queue > [ 23.221832] musb-hdrc musb-hdrc.1.auto: Could not enable: -22 > [ 23.227905] musb-hdrc musb-hdrc.1.auto: Could not enable: -22 > [ 23.239440] musb-hdrc musb-hdrc.1.auto: Could not enable: -22 > [ 23.401000] musb-hdrc musb-hdrc.1.auto: Could not enable: -22 > [ 23.407073] musb-hdrc musb-hdrc.1.auto: Could not enable: -22 > [ 23.426361] musb-hdrc musb-hdrc.1.auto: Could not enable: -22 > [ 23.734466] musb-hdrc musb-hdrc.1.auto: pm runtime get failed in musb_gadget_queue > [ 23.742462] musb-hdrc musb-hdrc.1.auto: pm runtime get failed in musb_gadget_queue > [ 23.750396] musb-hdrc musb-hdrc.1.auto: pm runtime get failed in musb_gadget_queue > ... (repeats with high frequency) > > This stops if the USB cable is unplugged and restarts if it is plugged in again. > > b) also found in the log > > [ 6.498107] ------------[ cut here ]------------ > [ 6.502960] WARNING: CPU: 0 PID: 868 at arch/arm/mach-omap2/omap_hwmod.c:1885 _enable+0x50/0x234 > [ 6.512207] omap_hwmod: usb_otg_hs: enabled state can only be entered from initialized, idle, or disabled state > [ 6.522766] Modules linked in: omap2430(+) bmp280_i2c bmp280 itg3200 at24 tsc2007 leds_tca6507 bma180 hmc5843_i2c hmc5843_core industrialio_triggered_buffer lis3lv02d_i2c kfifo_buf lis3lv02d phy_twl4030_usb snd_soc_omap_mcbsp snd_soc_ti_sdma musb_hdrc snd_soc_twl4030 gnss_sirf twl4030_vibra twl4030_madc twl4030_charger twl4030_pwrbutton gnss industrialio ehci_omap omapdrm drm_kms_helper syscopyarea sysfillrect sysimgblt fb_sys_fops drm drm_panel_orientation_quirks cec > [ 6.566436] CPU: 0 PID: 868 Comm: udevd Not tainted 5.16.0-rc5-letux+ #8251 > [ 6.573730] Hardware name: Generic OMAP36xx (Flattened Device Tree) > [ 6.580322] [<c010ed30>] (unwind_backtrace) from [<c010a1d0>] (show_stack+0x10/0x14) > [ 6.588470] [<c010a1d0>] (show_stack) from [<c0897c14>] (dump_stack_lvl+0x40/0x4c) > [ 6.596405] [<c0897c14>] (dump_stack_lvl) from [<c0130cc4>] (__warn+0xb4/0xdc) > [ 6.604003] [<c0130cc4>] (__warn) from [<c0130d5c>] (warn_slowpath_fmt+0x70/0x9c) > [ 6.611846] [<c0130d5c>] (warn_slowpath_fmt) from [<c011f4d4>] (_enable+0x50/0x234) > [ 6.619903] [<c011f4d4>] (_enable) from [<c012081c>] (omap_hwmod_enable+0x28/0x40) > [ 6.627838] [<c012081c>] (omap_hwmod_enable) from [<c0120ff4>] (omap_device_enable+0x4c/0x78) > [ 6.636779] [<c0120ff4>] (omap_device_enable) from [<c0121030>] (_od_runtime_resume+0x10/0x3c) > [ 6.645812] [<c0121030>] (_od_runtime_resume) from [<c05c688c>] (__rpm_callback+0x3c/0xf4) > [ 6.654510] [<c05c688c>] (__rpm_callback) from [<c05c6994>] (rpm_callback+0x50/0x54) > [ 6.662628] [<c05c6994>] (rpm_callback) from [<c05c66b0>] (rpm_resume+0x448/0x4e4) > [ 6.670593] [<c05c66b0>] (rpm_resume) from [<c05c6784>] (__pm_runtime_resume+0x38/0x50) > [ 6.678985] [<c05c6784>] (__pm_runtime_resume) from [<bf14ab20>] (musb_init_controller+0x350/0xa5c [musb_hdrc]) > [ 6.689727] [<bf14ab20>] (musb_init_controller [musb_hdrc]) from [<c05bccb8>] (platform_probe+0x58/0xa8) > [ 6.699737] [<c05bccb8>] (platform_probe) from [<c05badf0>] (really_probe+0x170/0x2fc) > [ 6.708068] [<c05badf0>] (really_probe) from [<c05bb040>] (__driver_probe_device+0xc4/0xd8) > [ 6.716827] [<c05bb040>] (__driver_probe_device) from [<c05bb084>] (driver_probe_device+0x30/0xac) > [ 6.726226] [<c05bb084>] (driver_probe_device) from [<c05bb3d0>] (__device_attach_driver+0x94/0xb4) > [ 6.735717] [<c05bb3d0>] (__device_attach_driver) from [<c05b93f8>] (bus_for_each_drv+0xa0/0xb4) > [ 6.744934] [<c05b93f8>] (bus_for_each_drv) from [<c05bb248>] (__device_attach+0xc0/0x134) > [ 6.753631] [<c05bb248>] (__device_attach) from [<c05b9fcc>] (bus_probe_device+0x28/0x80) > [ 6.762207] [<c05b9fcc>] (bus_probe_device) from [<c05b7e40>] (device_add+0x5fc/0x788) > [ 6.770507] [<c05b7e40>] (device_add) from [<c05bd240>] (platform_device_add+0x70/0x1bc) > [ 6.779022] [<c05bd240>] (platform_device_add) from [<bf177830>] (omap2430_probe+0x260/0x2d4 [omap2430]) > [ 6.789001] [<bf177830>] (omap2430_probe [omap2430]) from [<c05bccb8>] (platform_probe+0x58/0xa8) > [ 6.798309] [<c05bccb8>] (platform_probe) from [<c05badf0>] (really_probe+0x170/0x2fc) > [ 6.806610] [<c05badf0>] (really_probe) from [<c05bb040>] (__driver_probe_device+0xc4/0xd8) > [ 6.815399] [<c05bb040>] (__driver_probe_device) from [<c05bb084>] (driver_probe_device+0x30/0xac) > [ 6.824798] [<c05bb084>] (driver_probe_device) from [<c05bb4b4>] (__driver_attach+0xc4/0xd8) > [ 6.833648] [<c05bb4b4>] (__driver_attach) from [<c05b9308>] (bus_for_each_dev+0x64/0xa0) > [ 6.842224] [<c05b9308>] (bus_for_each_dev) from [<c05ba248>] (bus_add_driver+0x148/0x1a4) > [ 6.850891] [<c05ba248>] (bus_add_driver) from [<c05bbd1c>] (driver_register+0xb4/0xf8) > [ 6.859313] [<c05bbd1c>] (driver_register) from [<c0101f54>] (do_one_initcall+0x90/0x1c8) > [ 6.867889] [<c0101f54>] (do_one_initcall) from [<c0893968>] (do_init_module+0x4c/0x204) > [ 6.876373] [<c0893968>] (do_init_module) from [<c01b4c30>] (load_module+0x13f0/0x1928) > [ 6.884796] [<c01b4c30>] (load_module) from [<c01b53a0>] (sys_finit_module+0xa0/0xc0) > [ 6.893005] [<c01b53a0>] (sys_finit_module) from [<c0100080>] (ret_fast_syscall+0x0/0x54) > [ 6.901580] Exception stack(0xc2807fa8 to 0xc2807ff0) > [ 6.906890] 7fa0: b6e517d4 00052068 00000006 b6e509f8 00000000 b6e5131c > [ 6.915466] 7fc0: b6e517d4 00052068 cd718000 0000017b 00020000 00037f78 00050048 00063368 > [ 6.924011] 7fe0: bed8fef0 bed8fee0 b6e4ac4b b6f55a42 > [ 6.929321] ---[ end trace d715ff121b58763c ]--- > > c) git bisect result on testing for "musb-hdrc" in the console log: > > cf081d009c447647c6b36aced535ca427dbebe72 is the first bad commit > commit cf081d009c447647c6b36aced535ca427dbebe72 > Author: Rob Herring <robh@xxxxxxxxxx> > Date: Wed Dec 15 17:07:57 2021 -0600 > > usb: musb: Set the DT node on the child device > > The musb glue drivers just copy the glue resources to the musb child device. > Instead, set the musb child device's DT node pointer to the parent device's > node so that platform_get_irq_byname() can find the resources in the DT. > This removes the need for statically populating the IRQ resources from the > DT which has been deprecated for some time. > > Signed-off-by: Rob Herring <robh@xxxxxxxxxx> > Link: https://lore.kernel.org/r/20211215230756.2009115-3-robh@xxxxxxxxxx > Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > > drivers/usb/musb/am35x.c | 2 ++ > drivers/usb/musb/da8xx.c | 2 ++ > drivers/usb/musb/jz4740.c | 1 + > drivers/usb/musb/mediatek.c | 2 ++ > drivers/usb/musb/omap2430.c | 1 + > drivers/usb/musb/ux500.c | 1 + > 6 files changed, 9 insertions(+) > > Please advise a fix. > > BR and thanks, > Nikolaus Schaller >
