[Skiboot] [PATCH v3 1/4] hdat: Add 'primary' property to master chip xscom node

Benjamin Herrenschmidt benh at kernel.crashing.org
Tue Mar 14 17:01:46 AEDT 2017


On Tue, 2017-03-14 at 10:31 +0530, Vasant Hegde wrote:
> Needed for SBE communication as some of the SBE MBOX commands
> has to be sent to master chip SBE only.

Is this well defined in HDAT that the primary chiptod is the same as
the "primary" chip/mbox ? Or did you just hack that up without asking
the HDAT/HostBoot guys ? :-)

> Signed-off-by: Vasant Hegde <hegdevasant at linux.vnet.ibm.com>
> ---
>  hdata/spira.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/hdata/spira.c b/hdata/spira.c
> index 512784f..362b7b2 100644
> --- a/hdata/spira.c
> +++ b/hdata/spira.c
> @@ -631,8 +631,10 @@ static void add_chiptod_node(unsigned int
> chip_id, int flags)
>  	dt_add_property_strings(node, "compatible", "ibm,power-
> chiptod",
>  			       compat_str);
>  
> -	if (flags & CHIPTOD_ID_FLAGS_PRIMARY)
> +	if (flags & CHIPTOD_ID_FLAGS_PRIMARY) {
> +		dt_add_property(xscom_node, "primary", NULL, 0);
>  		dt_add_property(node, "primary", NULL, 0);
> +	}
>  	if (flags & CHIPTOD_ID_FLAGS_SECONDARY)
>  		dt_add_property(node, "secondary", NULL, 0);
>  }


More information about the Skiboot mailing list