[Skiboot] [PATCH 1/9] npu2: Split out common helper functions into separate file

Stewart Smith stewart at linux.vnet.ibm.com
Mon Jan 8 13:39:59 AEDT 2018


Alistair Popple <alistair at popple.id.au> writes:
> On Friday, 22 December 2017 3:22:32 PM AEDT Stewart Smith wrote:
>> Andrew Donnellan <andrew.donnellan at au1.ibm.com> writes:
>> > On 19/12/17 11:37, Alistair Popple wrote:
>> >>> +bool is_p9dd1(void)
>> >> 
>> >> Does OpenCAPI still need this? If not we should just drop it and DD1 support
>> >> along with it as it's very unlikely anything still works there.
>> >
>> > OpenCAPI has no DD1 support whatsoever. If you want to drop DD1 support 
>> > for NVLink then I have no objections!
>> 
>> I have no objection to this.
>> 
>> Anyone currently attempting NVLink on DD1 gets all the checkstops they deserve.
>
> Yep. I'm going to pull it out. Although it may just lead to xstops on boot for
> DD1 ... is that something we care about or should we still detect DD1 and
> disable NVLink (which would prevent any checkstops)?

Detect and disable sounds good. just skip all nvlink init on
dd1... although perhaps we'd have to rip some stuff out of the device
tree?
(or not add it)

-- 
Stewart Smith
OPAL Architect, IBM.



More information about the Skiboot mailing list