Re: [PATCH for-8.2 6/7] target/riscv: add 'max' CPU type

[Daniel Henrique Barboza]

On 7/12/23 18:35, Conor Dooley wrote:
> On Wed, Jul 12, 2023 at 06:09:10PM -0300, Daniel Henrique Barboza wrote:
>
> > It is intentional. Those default marchid/mimpid vals were derived from the 
> > current
> > QEMU version ID/build and didn't mean much.
> >
> > It is still possible to set them via "-cpu rv64,marchid=N,mimpid=N" if needed 
> > when
> > using the generic (rv64,rv32) CPUs. Vendor CPUs can't have their machine IDs 
> > changed
> > via command line.
>
> Sounds good, thanks. I did just now go and check icicle to see what it
> would report & it does not boot. I'll go bisect...

BTW how are you booting the icicle board nowadays? I remember you mentioning 
about
some changes in the FDT being required to boot and whatnot.

If it's not too hard I'll add it in my test scripts to keep it under check. 
Perhaps
we can even add it to QEMU testsuite.


Daniel