[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 17/24] pnv/psi: Correct the pnv-psi* devices not to be sysbus
From: |
Markus Armbruster |
Subject: |
Re: [PATCH 17/24] pnv/psi: Correct the pnv-psi* devices not to be sysbus devices |
Date: |
Tue, 19 May 2020 08:30:00 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
Cédric Le Goater <address@hidden> writes:
> On 5/18/20 7:04 AM, Markus Armbruster wrote:
>> pnv_chip_power8_instance_init() creates a "pnv-psi-POWER8" sysbus
>> device in a way that leaves it unplugged.
>> pnv_chip_power9_instance_init() and pnv_chip_power10_instance_init()
>> do the same for "pnv-psi-POWER9" and "pnv-psi-POWER10", respectively.
>>
>> These devices aren't actually sysbus devices. Correct that.
>
> I might have done things wrong regarding sysbus in the PowerNV machine.
>
> For some devices (PHBs), I have added :
>
> qdev_set_parent_bus(DEVICE(...), sysbus_get_default());
It's not wrong.
My next series will rework how devices get plugged into their buses.
> Should we do the same for the PSI device ?
No, because the PSI device is not a sysbus device.