[Skiboot] [PATCH v2 1/2] phb4: set PHB CMPM registers for tunneled operations
Philippe Bergheaud
felix at linux.vnet.ibm.com
Fri Dec 1 23:55:24 AEDT 2017
On 01/12/2017 13:34, Philippe Bergheaud wrote:
> P9 supports PCI tunneled operations (atomics and as_notify) that require
> setting the PHB ASN Compare/Mask register with a 16-bit indication.
>
> This register is currently initialized by enable_capi_mode(). But, as
> tunneled operations may also work in PCI mode, the ASN Compare/Mask
> register should rather be initialized in phb4_init_ioda3().
>
> This patch also adds "ibm,phb-indications" to the device tree, to tell
> Linux the values of CAPI, ASN, and NBW indications, when supported.
>
> Signed-off-by: Philippe Bergheaud <felix at linux.vnet.ibm.com>
> ---
> Changelog:
>
> v2: Do not let Linux choose indication values.
> Let skiboot tell Linux via the device tree.
> Initialize the PHB ASN indication in PCI mode.
Linux patches:
https://patchwork.ozlabs.org/patch/843482/
https://patchwork.ozlabs.org/patch/843483/
> ---
> hw/phb4.c | 30 ++++++++++++++++++++++--------
> include/phb4-regs.h | 4 ++--
> 2 files changed, 24 insertions(+), 10 deletions(-)
>
> diff --git a/hw/phb4.c b/hw/phb4.c
> index cfadf59d..94e67e9f 100644
> --- a/hw/phb4.c
> +++ b/hw/phb4.c
> @@ -3695,6 +3695,13 @@ static void phb4_init_capp_errors(struct phb4 *p)
> out_be64(p->regs + 0x0cb0, 0x35777073ff000000ull);
> }
>
> +#define CAPIIND 0x0200
> +#define CAPIMASK 0xFE00
> +#define ASNIND 0x0400
> +#define ASNMASK 0xF7FF
> +#define NBWIND 0x0300
> +#define NBWMASK 0xF700
> +
> /* Power Bus Common Queue Registers
> * All PBCQ and PBAIB registers are accessed via SCOM
> * NestBase = 4010C00 for PEC0
> @@ -3707,7 +3714,7 @@ static void phb4_init_capp_errors(struct phb4 *p)
> * Some registers are shared amongst all of the stacks and will only
> * have 1 copy. Other registers are implemented one per stack.
> * Registers that are duplicated will have an additional offset
> - * of “StackBase” so that they have a unique address.
> + * of 'StackBase' so that they have a unique address.
> * Stackoffset = 00000040 for Stack0
> * = 00000080 for Stack1
> * = 000000C0 for Stack2
> @@ -3785,16 +3792,13 @@ static int64_t enable_capi_mode(struct phb4 *p, uint64_t pe_number,
>
> /*
> * Bit [0:7] XSL_DSNCTL[capiind]
> - * Init_25 - CAPI Compare/Mask
> + * Init_26 - CAPI Compare/Mask
> */
> out_be64(p->regs + PHB_CAPI_CMPM,
> - 0x0200FE0000000000Ull | PHB_CAPI_CMPM_ENABLE);
> + ((u64)CAPIIND << 48) |
> + ((u64)CAPIMASK << 32) | PHB_CAPI_CMPM_ENABLE);
>
> if (!(p->rev == PHB4_REV_NIMBUS_DD10)) {
> - /* Init_24 - ASN Compare/Mask */
> - out_be64(p->regs + PHB_PBL_ASN_CMPM,
> - 0x0400FF0000000000Ull | PHB_PBL_ASN_ENABLE);
> -
> /* PBCQ Tunnel Bar Register
> * Write Tunnel register to match PSL TNR register
> */
> @@ -4116,7 +4120,10 @@ static void phb4_init_ioda3(struct phb4 *p)
> /* See enable_capi_mode() */
>
> /* Init_25 - ASN Compare/Mask */
> - /* See enable_capi_mode() */
> + if (p->rev != PHB4_REV_NIMBUS_DD10)
> + out_be64(p->regs + PHB_ASN_CMPM,
> + ((u64)ASNIND << 48) |
> + ((u64)ASNMASK << 32) | PHB_ASN_CMPM_ENABLE);
>
> /* Init_26 - CAPI Compare/Mask */
> /* See enable_capi_mode() */
> @@ -4716,6 +4723,13 @@ static void phb4_add_properties(struct phb4 *p)
>
> /* Indicate to Linux that CAPP timebase sync is supported */
> dt_add_property_string(np, "ibm,capp-timebase-sync", NULL);
> +
> + /* Tell Linux Compare/Mask indication values */
> + if (p->rev == PHB4_REV_NIMBUS_DD10)
> + dt_add_property_cells(np, "ibm,phb-indications", CAPIIND, 0, 0);
> + else
> + dt_add_property_cells(np, "ibm,phb-indications", CAPIIND,
> + ASNIND, NBWIND);
> }
>
> static bool phb4_calculate_windows(struct phb4 *p)
> diff --git a/include/phb4-regs.h b/include/phb4-regs.h
> index 0d8aa48b..34d822a0 100644
> --- a/include/phb4-regs.h
> +++ b/include/phb4-regs.h
> @@ -71,8 +71,8 @@
> #define PHB_PEST_BAR 0x1a8
> #define PHB_PEST_BAR_ENABLE PPC_BIT(0)
> #define PHB_PEST_BASE_ADDRESS PPC_BITMASK(8,51)
> -#define PHB_PBL_ASN_CMPM 0x1C0
> -#define PHB_PBL_ASN_ENABLE PPC_BIT(63)
> +#define PHB_ASN_CMPM 0x1C0
> +#define PHB_ASN_CMPM_ENABLE PPC_BIT(63)
> #define PHB_CAPI_CMPM 0x1C8
> #define PHB_CAPI_CMPM_ENABLE PPC_BIT(63)
> #define PHB_M64_UPPER_BITS 0x1f0
More information about the Skiboot
mailing list