Re: [PATCH v3 1/1] ata: sata_rescan must scan for block devices

Hi
The addition of the line
ret = device_chld_unbind(dev, NULL);
Broke booting from sata with qemu q35 with coreboot + u-boot for me.
Kind regards

On 10.10.24 13:48, Arthur Heymans wrote:
Hi
The addition of the line
ret = device_chld_unbind(dev, NULL);
Broke booting from sata with qemu q35 with coreboot + u-boot for me.
Kind regards
Hello Arthur,
Sorry to hear this. I would need a reproducible example for further analysis.
Could you, please, share an image, the U-Boot .config file and HEAD commit hash, as well as the QEMU command.
Best regards
Heinrich

Hi
Sorry for the delay. The coreboot.rom.xz contains the coreboot.rom that this was tested with. To add u-boot as a coreboot payload do the following: Download coreboot https://coreboot.org/downloads.html
run to build cbfstool make -C util/cbfstool
Add u-boot as payload: cbfstool coreboot.rom add-flat-binary -f u-boot-x86-with-spl.bin -n fallback/payload -c lzma -l 0x1110000 -e 0x1110000
Then run with qemu: qemu-system-x86_64 -M q35 -bios coreboot.rom -m 4G -serial stdio -hdd some_linux.iso -enable-kvm
Thanks for looking into this
Arthur
On Thu, Oct 10, 2024 at 2:23 PM Heinrich Schuchardt heinrich.schuchardt@canonical.com wrote:
On 10.10.24 13:48, Arthur Heymans wrote:
Hi
The addition of the line
ret = device_chld_unbind(dev, NULL);
Broke booting from sata with qemu q35 with coreboot + u-boot for me.
Kind regards
Hello Arthur,
Sorry to hear this. I would need a reproducible example for further analysis.
Could you, please, share an image, the U-Boot .config file and HEAD commit hash, as well as the QEMU command.
Best regards
Heinrich
participants (2)
-
Arthur Heymans
-
Heinrich Schuchardt