[PATCH] riscv: qemu: Remove fdt_high default

[E Shattow]

Hi Tom,  Add CC: Heinrich

On 9/15/25 07:32, Tom Rini wrote:
> On Mon, Sep 15, 2025 at 07:45:49AM +0800, Vivian Wang wrote:
> 
>> Setting fdt_high to all ones is discouraged and does not appear to be
>> useful for RISC-V QEMU. Moreover, it causes a boot failure when the FDT
>> generated internally by QEMU is used while booting. Remove it to allow
>> U-Boot to pick a suitable address and relocate the FDT.
>>
>> Closes: https://lore.kernel.org/u-boot/8397369a-9b0b-4798-9c30-3a81165657d6@iscas.ac.cn
>> Signed-off-by: Vivian Wang <wangruikang at iscas.ac.cn>
> 

> I'm very glad to see this finally and would like to see the rest of the
> RISC-V platforms follow and do the same now.
> 
> Reviewed-by: Tom Rini <trini at konsulko.com>
> 

When you say you would like the rest of the RISC-V platforms (to) follow
and do the same;  What do you mean?  Heinrich and I were trying to
figure this out off of something you said on IRC awhile back. I was
taking a guess it was the qemu config (and this patch makes sense to me
so I think that's part of it), but what else? Is that a reference to
sifive-unleashed config fragment, are there others?

If you give us a detailed hint what we should be looking at it would go
a long way and we can test / figure out the rest. Is initrd_high
all-ones on your radar too?

Thanks,

-E Shattow