[Skiboot] [PATCH v3 5/6] iohub: Add HUB ID for everest systems

[Nicholas Piggin]

On Thu Feb 6, 2025 at 11:56 PM AEST, Mahesh Salgaonkar wrote:
> From: Aditya Gupta <adityag at linux.ibm.com>
>
> Everest's hub id is 0x52, which OPAL earlier didn't recognise:
>
>     [  574.179390090,6] CEC: HUB FRU 0 is CPU Card
>     [  574.179430286,6] CEC:   2 chips in FRU
>     [  574.179464930,7] CEC:   IO Hub Chip #0 OK
>     [  574.179497312,7] CEC:   PChip: 0 HUB ID: 0052 [EC=0x20] Hub#=0)
>     [  574.179543358,3] CEC:     Hub ID 0x0052 unsupported !    <--------
>
> Due to not recognising the HUB id, it doesn't initialise the PCI slots.
>
> Define 0x52 as Everest's hub id, so OPAL initialises PCIe slots also for
> Everest
>
> Signed-off-by: Aditya Gupta <adityag at linux.ibm.com>

Acked-by: Nicholas Piggin <npiggin at gmail.com>

> ---
>  hdata/iohub.c | 4 ++++
>  hdata/spira.h | 1 +
>  2 files changed, 5 insertions(+)
>
> diff --git a/hdata/iohub.c b/hdata/iohub.c
> index 40712bc94d..9bb34d7b67 100644
> --- a/hdata/iohub.c
> +++ b/hdata/iohub.c
> @@ -843,6 +843,10 @@ static void io_parse_fru(const void *sp_iohubs)
>  			prlog(PR_INFO, "CEC:     Rainier !\n");
>  			io_add_p9(hub, sp_iohubs);
>  			break;
> +		case CECHUB_HUB_EVEREST:
> +			prlog(PR_INFO, "CEC:     Everest !\n");
> +			io_add_p9(hub, sp_iohubs);
> +			break;
>  		case CECHUB_HUB_DENALI:
>  			prlog(PR_INFO, "CEC:     Denali !\n");
>  			io_add_p9(hub, sp_iohubs);
> diff --git a/hdata/spira.h b/hdata/spira.h
> index 9db9d599f4..5281493bb2 100644
> --- a/hdata/spira.h
> +++ b/hdata/spira.h
> @@ -613,6 +613,7 @@ struct cechub_io_hub {
>  #define CECHUB_HUB_AXONE_HOPPER		0x0040	/* axone+hopper */
>  #define CECHUB_HUB_RAINIER		0x0050
>  #define CECHUB_HUB_DENALI		0x0051
> +#define CECHUB_HUB_EVEREST		0x0052
>  	__be32		ec_level;
>  	__be32		aff_dom2;	/* HDAT < v9.x only */
>  	__be32		aff_dom3;	/* HDAT < v9.x only */