Adding sensors for facebook

[James Feist]

On 11/14/2018 09:23 AM, Jae Hyun Yoo wrote:
> On 11/14/2018 9:00 AM, James Feist wrote:
>> On 11/13/2018 04:08 PM, Vijay Khemka wrote:
>>> Hi James,
>>> Thanks for this detailed information. I have quick questions though 
>>> about dbus-sensors, is this intel's features only not a common one. 
>>> So I need to copy whole recipes to our repo. Also I see following 
>>> compile issue.
>> We can submit a patch to move in into common, although I think you 
>> should still be able to access it, we pull in code from meta-phosphor, 
>> meta-openembedded, etc. meta-intel is just another meta layer. Let me 
>> know if you can't and we can start the process.
>>
>>>
>>> /home/vijaykhemka/proj/openbmc/lf/new-gh/openbmc/build/workspace/sources/dbus-sensors/sensors/src/CPUSensorMain.cpp:18:10: 
>>> fatal error: linux/peci-ioctl.h: No such file or directory
>>>
>>> And I can't find peci-ioctl.h in my kernel tree as well.
>>
>> Jae,
>>
>> Can you point them to the latest peci kernel patch?
>>
> 
> Hi Vijay,
> 
> I'm still trying to upstream the PECI implementation into linux kernel
> tree. Meanwhile, you could back port this patch series for test.
> 
> https://lkml.org/lkml/2018/10/15/670

Joel,

Is it possible to add this into the openbmc/linux tree?

Thanks,

James

> 
> Thanks,
> 
> Jae
> 
>> It hasn't fully been through the kernel review process but it is 
>> probably good enough to be merged into openbmc/linux. Until this 
>> happens you probably need to apply it as a patch downstream.
>>
>> Thanks,
>>
>> James
>>