Re: [PATCH 13/18] ARM: dts: Use simple-pm-bus for genpd for am3 l4_wkup
From: Vaittinen, Matti <hidden>
Date: 2021-01-14 07:59:41
Also in:
linux-omap
Morning Tony & All, On Thu, 2021-01-14 at 09:20 +0200, Tony Lindgren wrote:
* Tony Lindgren [off-list ref] [210113 17:30]:quoted
* Tony Lindgren [off-list ref] [210113 16:48]:quoted
* Geert Uytterhoeven [off-list ref] [210113 16:42]:quoted
On Wed, Jan 13, 2021 at 5:23 PM Tony Lindgren <tony@atomide.comquoted
wrote: Strange, bbb has been working for me with NFSroot, that's with omap2plus_defconfig. Care to send your .config so I can give it a try tomorrow?Sent by private email.OK thanks.quoted
Just tried omap2plus_defconfig, and it also hangs during boot.Hmm and I just tried and bbb still boots NFSroot for me at commit 5a230524f879 ("ARM: dts: Use simple-pm-bus for genpd for am3 l4_wkup").quoted
Hmm OK. Will give it a try tomorrow.Not sure why your bbb would produce all the deferred probe while mine won't. However, I've noticed that we need the following patch on omap4/5 and dra7 when playing with updating them to use genpd. Maybe this also fixes your boot issue if you can give it a try?
I do also experience the boot problem described by Geert. Head being: b0625afe305253d0831af9289b37c906c18a781b ARM: OMAP2+: Drop legacy platform data for am3 mpuss => BBB boots Ok. Head being: 5a230524f87926f24d637fe62fd689f7f86f5036 ARM: dts: Use simple-pm-bus for genpd for am3 l4_wkup => BBB boot fails. (I don't have any debug parameters => last thing I see in UART is from uBoot: Bytes transferred = 89849 (15ef9 hex) Kernel image @ 0x81000000 [ 0x000000 - 0x928200 ] ## Flattened Device Tree blob at 82000000 Booting using the fdt blob at 0x82000000 Using Device Tree in place at 82000000, end 82018ef8 Starting kernel ... Head being 5a230524f87926f24d637fe62fd689f7f86f5036 + this patch ARM: dts: Use simple-pm-bus for genpd for am3 l4_wkup (+ patch) Boot still fails for me. Please note - my environment takes the intermediate build result file: arch/arm/boot/dts/.am335x-boneblack.dtb.dts.tmp and compiles this with '-@' to allow further overlays. Best Regards Matti Vaittinen