[Skiboot] [RFC PATCH 00/11] Device tree updates

[Vasant Hegde]

On 09/28/2016 06:09 PM, Oliver O'Halloran wrote:
> On Wed, Sep 28, 2016 at 10:05 PM, Vasant Hegde
> <hegdevasant at linux.vnet.ibm.com> wrote:
>> This patchset makes various device tree updates for P9. This is based on
>> latest HDAT spec (v10.3g).
>>
>> I've verified this patchset on P8 tuleta box. It booted fine. Hopefully
>> I've not broken anything on P8.
>
> Should be fine provided the hdat_to_dt tests pass. You can run them
> with `make hdata-check`

Yep. I know . hdata-check is passed ..

>
>> I will check with FW to see whether we can verify these changes on simulators.
>
> Currently there is no code in hostboot to generate the new HDAT
> fields. I'm not sure what the ETA is either. I've got some code for
> parsing the new fields, but I didn't want to post until they had some
> testing.

Yep. I think they have brought up FSP model .. We can verify couple of things 
with FSP model.
And of course for fields like LPC we need hostboot side changes.


>
>>
>>
>> TODO:
>>    - Parse SP ntuple and create LPC node
>>    - Create PNOR node
>>      On p8 BMC based system hostboot DT provides PNOR information.
>>      I do not see anyone using this information. But we may just want
>>      to add these nodes.
>
> I've got a patch for the LPC IO.

Cool!

>
>>    - Create BMC sensors node
>>    - Secure boot changes (secure boot and TPM)
>
> Talk to Claudio about the TPM stuff. I'm not sure if the DT bindings
> for the TPM are finalised yet.

I've seen his secureboot patchset. Let it get merged. Then we can look into P9 
DT creation :-)
Should be straight forward to add these changes.

@Claudio,
   I hope you have reviewed HDAT spec and made sure it contains all the 
information required for secure boot.


>
>>    - PCIA changes
>
> I think I've got patches for this. I'll check tomorrow.

Only change we need here is interrupt-controller reg property.. They have 
removed those fields from HDAT. I haven't looked into it in detail.

-Vasant