[Skiboot] [PATCH v2 10/12] hdat: Add P9 compatible property

[Vasant Hegde]

On 12/20/2016 12:11 PM, Stewart Smith wrote:
> Vasant Hegde <hegdevasant at linux.vnet.ibm.com> writes:
>> Add p9 compatible property for various nodes.
>>
>> Signed-off-by: Vasant Hegde <hegdevasant at linux.vnet.ibm.com>
>> ---
>>   hdata/spira.c | 10 ++++++++++
>>   1 file changed, 10 insertions(+)
>>
>> diff --git a/hdata/spira.c b/hdata/spira.c
>> index 2066426..590e929 100644
>> --- a/hdata/spira.c
>> +++ b/hdata/spira.c
>> @@ -258,6 +258,10 @@ static struct dt_node *add_xscom_node(uint64_t base, uint32_t hw_id,
>>   		dt_add_property_strings(node, "compatible",
>>   					"ibm,xscom", "ibm,power8-xscom");
>>   		break;
>> +	case proc_gen_p9:
>> +		dt_add_property_strings(node, "compatible",
>> +					"ibm,xscom", "ibm,power9-xscom");
>> +		break;
>>   	default:
>>   		dt_add_property_strings(node, "compatible", "ibm,xscom");
>>   	}
>> @@ -574,6 +578,9 @@ static void add_chiptod_node(unsigned int chip_id, int flags)
>>   	case proc_gen_p8:
>>   		compat_str = "ibm,power8-chiptod";
>>   		break;
>> +	case proc_gen_p9:
>> +		compat_str = "ibm,power9-chiptod";
>> +		break;
>>   	default:
>>   		return;
>>   	}
>> @@ -710,6 +717,9 @@ static void add_nx_node(u32 gcid)
>>   	case proc_gen_p8:
>>   		cp_str = "ibm,power8-nx";
>>   		break;
>> +	case proc_gen_p9:
>> +		cp_str = "ibm,power9-nx";
>> +		break;
>
> is the subsequent adding of ibm,power-nx accurate? I haven't been
> keeping track on if the same icswx invocation is being carried across to
> p9....

I"m not sure how accurate it is. But spec doesn't talk about anything and if its 
not functional it will not be added. Hence I just added p9 property here.

-Vasant

>