[Skiboot] [PATCH 0/3] P9 core dts support

[Cédric Le Goater]

On 02/17/2017 10:49 PM, Rob Lippert wrote:
> On Thu, Feb 2, 2017 at 10:49 AM, Cédric Le Goater <clg at kaod.org> wrote:
>>
>> On 01/31/2017 08:31 AM, Cédric Le Goater wrote:
>>> On 01/31/2017 06:18 AM, Stewart Smith wrote:
>>>> Cédric Le Goater <clg at kaod.org> writes:
>>>>> Hello,
>>>>>
>>>>> Here is a short series adding some cleanups to the DTS framework
>>>>> before introducing support for the P9 cores. The P9 DTS were inherited
>>>>> from P8 and there is not much differences. Other chiplets also have
>>>>> DTS (cache, nest), their support will be added later on.
>>>>>
>>>>> I did not have the chance to validate the patchset on real HW and only
>>>>> did some xscom address validation under qemu and simics. So this is
>>>>> for review only for the moment, until I can hack a skiboot on a zaius
>>>>> or a witherspoon system.
>>>>
>>>> Perhaps Rob or Mikey can give a try (or delegate somebody to) and add a
>>>> Tested-by ?
>>>
>>> that'd be nice. It should not break anything really and
>>> if the distro is recent enough, the hwmon module should
>>> be autoloaded and should expose sysfs files under :
>>>
>>>       /sys/class/hwmon/hwmon*/temp*
>>
>> So I got access to a witherspoon and surprisingly, this code is
>> only reading zeroes. I need to re-check where the xscoms are going.
> 
> Cedric, any update on getting the P9 DTS sensors to work with skiboot?
> 
> We have not tested yet since I was waiting for confirmation on your side.

The skiboot framework is in place (with this patch) and the kernel 
module is creating the hwmon sysfs entries. I need to dig a little 
further to see why we read zeroes on the core DTS as I did not see 
any XSCOM errors.

I will try on another system next week. 

C.