[Skiboot] [PATCH 6/7] astbmc: Don't do P8 PSI or DT fixups on P9

Michael Neuling mikey at neuling.org
Sun Jan 15 07:06:00 AEDT 2017


On Fri, 2017-01-13 at 17:56 +1100, Oliver O'Halloran wrote:
> We do some P8 specific hacks on BMC platforms to work around Hostboot
> not initialising the PSI BAR and to fix some problems in the hostboot
> provided device tree. These shouldn't be done on P9.

I think we may need to do some of the fixups on p9.  astbmc_fixup_dt_mbox() adds
the mbox stuff that cyrils mbox code keys off later. 

I'm not sure the best way to fix it. 

Mikey

> 
> Signed-off-by: Oliver O'Halloran <oohall at gmail.com>
> Cc: Benjamin Herrenschmidt <benh at kernel.crashing.org>
> ---
> Ben, I'm guessing the we need the PSI BAR setup on P8 for interrupts.
> Do we need to do anything similar on P9?
> ---
>  platforms/astbmc/common.c | 14 ++++++++++----
>  1 file changed, 10 insertions(+), 4 deletions(-)
> 
> diff --git a/platforms/astbmc/common.c b/platforms/astbmc/common.c
> index ce8edeac043c..e78b5fa39ab7 100644
> --- a/platforms/astbmc/common.c
> +++ b/platforms/astbmc/common.c
> @@ -321,11 +321,17 @@ static void astbmc_fixup_psi_bar(void)
>  
>  void astbmc_early_init(void)
>  {
> -	/* Hostboot's device-tree isn't quite right yet */
> -	astbmc_fixup_dt();
> +	/*
> +	 * On P9 we don't have a HB supplied devicetree and we have a
> +	 * different PSI BAR hack here is P8 specific.
> +	 */
> +	if (proc_gen == proc_gen_p8) {
> +		/* Hostboot's device-tree isn't quite right yet */
> +		astbmc_fixup_dt();
>  
> -	/* Hostboot forgets to populate the PSI BAR */
> -	astbmc_fixup_psi_bar();
> +		/* Hostboot forgets to populate the PSI BAR */
> +		astbmc_fixup_psi_bar();
> +	}
>  
>  	/* Send external interrupts to me */
>  	psi_set_external_irq_policy(EXTERNAL_IRQ_POLICY_SKIBOOT);


More information about the Skiboot mailing list