[Skiboot] [PATCH v5 0/6] Power11 enablement

[Nicholas Piggin]

All looks good to me.

Thanks,
Nick

On Mon Mar 3, 2025 at 5:02 PM AEST, Mahesh Salgaonkar wrote:
> This series adds Power11 support. Much of the code has been reused from
> Power10.
>
> With this patchset the skiboot lid size (skiboot.lid.xz.stb) is 513K.
> Older BMC based systems had skiboot limit of 512K. Hence, in order to
> reduce the LID size we can use following build options:
> 	make CONFIG_FSP=0                           <= 459K
> 	make CONFIG_FSP=0 CONFIG_P8=0               <= 425K
>
> On latest openbmc firmware this restriction has been lifted and new size limit
> is 1MB.  However, after manually flashing the skiboot lid on BMC, pldmd service
> needs to be restarted to refresh the new file attributes. This makes sure that
> hostboot does not fail during 'ISTEP 20. 1 - host_load_payload' step.
>
>     # ln -sf ./skiboot.lid.xz.stb 81e00660.lid
>     # systemctl restart pldmd
>
> Changes in v5:
> - patch 3/6: Move qemu,powernv11 check to if case that enables direct controls.
> - See v4 at https://lists.ozlabs.org/pipermail/skiboot/2025-February/019111.html
>
> Changes in v4:
> - Moved psi.c and xscom.c hunks from patch 3/6 to patch 1/6
> - See v3 at https://lists.ozlabs.org/pipermail/skiboot/2025-February/019086.html
>
> Changes in v3:
> - Add helper function to get proc_gen number
> - Print unknown processor detected in case of -ve proc_gen number
> - Make proc_gen checks consistent all over.
> - See v2 at https://lists.ozlabs.org/pipermail/skiboot/2025-February/019063.html
>
> Aditya Gupta (2):
>   plat/qemu: add support for Power11 platform
>   iohub: Add HUB ID for everest systems
>
> Mahesh Salgaonkar (4):
>   Initial Power11 enablement
>   cpufeatures: Add Power11 support
>   external/mambo: skiboot.tcl add Power11 config
>   platform: Identify correct bmc platform based on bmc hw version
>
>  asm/head.S                       | 12 +++++--
>  asm/misc.S                       |  4 ++-
>  core/affinity.c                  |  2 +-
>  core/chip.c                      | 13 +++----
>  core/cpu.c                       | 18 +++++++++-
>  core/cpufeatures.c               | 62 ++++++++++++++++++--------------
>  core/direct-controls.c           | 18 +++++-----
>  core/fast-reboot.c               |  4 +--
>  core/hmi.c                       | 12 ++++---
>  core/init.c                      | 10 +++---
>  core/mce.c                       |  2 +-
>  core/platform.c                  | 14 +++++++-
>  external/mambo/skiboot.tcl       | 28 +++++++++++++++
>  hdata/fsp.c                      |  2 +-
>  hdata/i2c.c                      |  5 +--
>  hdata/iohub.c                    |  7 +++-
>  hdata/spira.c                    | 15 ++++++--
>  hdata/spira.h                    |  1 +
>  hw/chiptod.c                     |  4 ++-
>  hw/fsp/fsp-occ.c                 |  1 +
>  hw/fsp/fsp-psi.c                 |  1 +
>  hw/homer.c                       |  1 +
>  hw/imc.c                         |  5 ++-
>  hw/lpc.c                         |  7 ++--
>  hw/occ.c                         |  4 ++-
>  hw/p8-i2c.c                      |  2 +-
>  hw/phb4.c                        |  2 +-
>  hw/phys-map.c                    |  3 ++
>  hw/prd.c                         |  1 +
>  hw/psi.c                         |  7 +++-
>  hw/sbe.c                         |  2 +-
>  hw/slw.c                         |  7 ++--
>  hw/vas.c                         |  2 +-
>  hw/xive2.c                       |  4 +--
>  hw/xscom.c                       | 42 ++++++++++++++++------
>  include/chip.h                   |  1 +
>  include/phb4.h                   |  2 +-
>  include/processor.h              |  6 ++++
>  include/skiboot.h                |  1 +
>  platforms/ibm-fsp/hostservices.c |  1 +
>  platforms/qemu/qemu.c            | 23 ++++++++++++
>  41 files changed, 265 insertions(+), 93 deletions(-)