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

[Arthur Heymans]

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 at 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: .config
Type: application/octet-stream
Size: 50667 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20241022/b49a6d3c/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: coreboot.rom.xz
Type: application/x-xz
Size: 96824 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20241022/b49a6d3c/attachment.xz>