Re: [PATCH v3 1/1] hw/arm/aspeed:Add vpd data for Rainier machine

[Ninad Palsule <ninad@linux.vnet.ibm.com>]

Hello Cedric,


On 5/25/23 1:37 AM, Cédric Le Goater wrote:
> [ ... ]
>
>> However, regarding Cédric's log above, a reboot is expected on the first
>> boot of a fresh image when there's valid VPD available. For the first
>> boot of a fresh image we configure the kernel with a minimal devicetree
>> that allows us to read the VPD data. This determines the system we're
>> actually on and configures an appropriate devicetree for subsequent
>> boots. We then reboot to pick up the new devicetree.
>
> Yes. Then, the behavior looks correct under QEMU :
>
>   https://www.kaod.org/qemu/aspeed/rainier/rainer.log
>
> Here are the services which still have with some issues :
>
> * clear-once.service                loaded failed failed          
> Clear one time boot overrides
> * ncsi-linkspeed@eth0.service       loaded failed failed          Set 
> eth0 gigabit link speed
> * ncsi-linkspeed@eth1.service       loaded failed failed          Set 
> eth1 gigabit link speed
> * obmc-flash-bios-init.service      loaded failed failed          
> Setup Host FW directories
> * system-vpd.service                loaded failed failed          
> System VPD Collection
> * trace-enable.service              loaded failed failed          
> Enable Linux trace events in the boot loader

On my system system-vpd.service was active. I couldn't run your build on 
my development machine.

I am still checking it.

>
> C.
>
>
>
>