]> git.proxmox.com Git - mirror_qemu.git/commit
pnv_phb4_pec: Simplify/align code to parent user-created PHBs
authorFrederic Barrat <fbarrat@linux.ibm.com>
Thu, 2 Mar 2023 16:37:15 +0000 (17:37 +0100)
committerDaniel Henrique Barboza <danielhb413@gmail.com>
Fri, 3 Mar 2023 19:50:17 +0000 (16:50 -0300)
commitddf0676f1ade90026483a91823d86db4096a40ef
treef42b8484903ee30153b1d8087e1c5c64b5bc3b77
parentfa9dc22aecf881bb7f7e27360a06334bc219ca6f
pnv_phb4_pec: Simplify/align code to parent user-created PHBs

When instantiating a user-created PHB on P9/P10, we don't really have
a reason any more to go through an indirection in pnv_chip_add_phb()
in pnv.c, we can go straight to the right function in
pnv_phb4_pec.c. That way, default PHBs and user-created PHBs are all
handled in the same file.  This patch also renames pnv_phb4_get_pec()
to pnv_pec_add_phb() to better reflect that it "hooks" a PHB to a PEC.

For P8, the PHBs are parented to the chip directly, so it makes sense
to keep calling pnv_chip_add_phb() in pnv.c, to also be consistent
with where default PHBs are handled. The only change here is that,
since that function is now only used for P8, we can refine the return
type.

So overall, the PnvPHB front-end now has a pnv_phb_user_get_parent()
function which handles the parenting of the user-created PHBs by
calling the right function in the right file based on the processor
version. It's also easily extensible if we ever need to support a
different parent object.

Signed-off-by: Frederic Barrat <fbarrat@linux.ibm.com>
Message-Id: <20230302163715.129635-5-fbarrat@linux.ibm.com>
Reviewed-by: Daniel Henrique Barboza <danielhb413@gmail.com>
Signed-off-by: Daniel Henrique Barboza <danielhb413@gmail.com>
hw/pci-host/pnv_phb.c
hw/pci-host/pnv_phb4_pec.c
hw/ppc/pnv.c
include/hw/pci-host/pnv_phb4.h
include/hw/ppc/pnv.h