Could phosphor-hwmon support peci-sensors

[James Feist]

On 7/8/19 10:52 AM, Brad Bishop wrote:
> On Mon, Jul 08, 2019 at 12:33:21PM +0000, Duke Du (杜祥嘉) wrote:
>> Hi All,
>>  I have some problems with phosphor-hwmon.
>> Could package phosphor-hwmon supports peci sensors such as cpu and 
>> dimm tempatures ?
>> If yes, could you give me some hints that how do I write configuration 
>> for peci sensors, please :).
>>
>> Thanks.
>> Duke, Du
> 
> phosphor-hwmon should work with anything that implements the hwmon ABI. 
> So I think to make PECI sensors work with phosphor-hwmon, you need hwmon 
> drivers for PECI sensors.  I don't know if anyone is working on that.
> 
> I suspect the other sensor application - dbus-sensors supports PECI just 
> fine.  But I don't know how it works, if it can be used without entity 
> manager,  and/or if it can be run in parallel with phosphor-hwmon.

+ Jae, CPU Sensor Author

https://github.com/openbmc/dbus-sensors/blob/master/src/CPUSensorMain.cpp

CPU sensor currently supports PECI sensors 
https://github.com/openbmc/dbus-sensors/blob/master/src/CPUSensor.cpp

https://github.com/openbmc/entity-manager/blob/f861da89ed03f9ec556c5ed6ac819988c0c1f498/configurations/WFT%20Baseboard.json#L2181

It does require entity-manager currently, but it could be run in 
parallel with phosphor-hwmon.

-James