On 5/2/24 08:27, Aditya Gupta wrote:
> Power10 DD1.0 was dropped in:
>
> commit 8f054d9ee825 ("ppc: Drop support for POWER9 and POWER10 DD1 chips")
>
> Use the newer Power10 DD2 chips cfam id.
>
> Cc: Cédric Le Goater <clg@kaod.org>
> Cc: David Gibson <david@gibson.dropbear.id.au>
> Cc: Frédéric Barrat <fbarrat@linux.ibm.com>
> Cc: Laurent Vivier <lvivier@redhat.com>
> Cc: Mahesh J Salgaonkar <mahesh@linux.ibm.com>
> Cc: Madhavan Srinivasan <maddy@linux.ibm.com>
> Cc: Nicholas Piggin <npiggin@gmail.com>
> Cc: Paolo Bonzini <pbonzini@redhat.com>
> Cc: Thomas Huth <thuth@redhat.com>
> Signed-off-by: Aditya Gupta <adityag@linux.ibm.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Thanks,
C.
> ---
> hw/ppc/pnv.c | 2 +-
> tests/qtest/pnv-xscom.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c
> index 6e3a5ccdec76..06a4e4d13948 100644
> --- a/hw/ppc/pnv.c
> +++ b/hw/ppc/pnv.c
> @@ -2090,7 +2090,7 @@ static void pnv_chip_power10_class_init(ObjectClass *klass, void *data)
> PnvChipClass *k = PNV_CHIP_CLASS(klass);
> static const int i2c_ports_per_engine[PNV10_CHIP_MAX_I2C] = {14, 14, 2, 16};
>
> - k->chip_cfam_id = 0x120da04900008000ull; /* P10 DD1.0 (with NX) */
> + k->chip_cfam_id = 0x220da04980000000ull; /* P10 DD2.0 (with NX) */
> k->cores_mask = POWER10_CORE_MASK;
> k->chip_pir = pnv_chip_pir_p10;
> k->intc_create = pnv_chip_power10_intc_create;
> diff --git a/tests/qtest/pnv-xscom.h b/tests/qtest/pnv-xscom.h
> index 6f62941744a6..5aa1701ea768 100644
> --- a/tests/qtest/pnv-xscom.h
> +++ b/tests/qtest/pnv-xscom.h
> @@ -56,7 +56,7 @@ static const PnvChip pnv_chips[] = {
> .chip_type = PNV_CHIP_POWER10,
> .cpu_model = "POWER10",
> .xscom_base = 0x000603fc00000000ull,
> - .cfam_id = 0x120da04900008000ull,
> + .cfam_id = 0x220da04980000000ull,
> .first_core = 0x0,
> .num_i2c = 4,
> },