<div dir="ltr">Thanks Bruce for your response.<div><br></div><div>Please find below the entitymanager hwmon sensor configuration path for our system.</div><div><a href="https://github.com/openbmc/entity-manager/blob/master/configurations/FBYV2.json#L434">https://github.com/openbmc/entity-manager/blob/master/configurations/FBYV2.json#L434</a><br></div><div><br></div><div>Thanks,</div><div>Kumar.</div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Jan 1, 2022 at 1:42 AM Bruce Mitchell <<a href="mailto:bruce.mitchell@linux.vnet.ibm.com">bruce.mitchell@linux.vnet.ibm.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 12/31/2021 09:30, Bruce Mitchell wrote:<br>
> On 12/20/2021 10:29, Ed Tanous wrote:<br>
>> On Sun, Dec 19, 2021 at 9:11 PM Kumar Thangavel<br>
>> <<a href="mailto:kumarthangavel.hcl@gmail.com" target="_blank">kumarthangavel.hcl@gmail.com</a>> wrote:<br>
>>><br>
>>> Hi All,<br>
>>><br>
>>> Gentle Reminder on this.<br>
>>><br>
>>> Thanks,<br>
>>> Kumar.<br>
>>><br>
>>> On Thu, Dec 16, 2021 at 1:56 PM Kumar Thangavel <br>
>>> <<a href="mailto:kumarthangavel.hcl@gmail.com" target="_blank">kumarthangavel.hcl@gmail.com</a>> wrote:<br>
>>>><br>
>>>> Hi All,<br>
>>>><br>
>>>><br>
>>>> In dbus-sensors repo, I saw the patch for changes in Hwmon Temp Sensor.<br>
>>>> <a href="https://github.com/openbmc/dbus-sensors/commit/544e7dc58cecef898fe7d684c3ba0fbce178cf2d#diff-bfc8d7ed4ade4b3519ad3b7fe091f9c79a60faf8a26f410c1568e5341b22b7ab" rel="noreferrer" target="_blank">https://github.com/openbmc/dbus-sensors/commit/544e7dc58cecef898fe7d684c3ba0fbce178cf2d#diff-bfc8d7ed4ade4b3519ad3b7fe091f9c79a60faf8a26f410c1568e5341b22b7ab</a> <br>
>>>><br>
>><br>
>> Bruce, this is your patch.  Could you take a look at the below and<br>
>> provide your input?<br>
>><br>
>>>><br>
>>>><br>
>>>> While testing those changes in my platform, I found that each sensor <br>
>>>> in hwmon temp has been creating twice.<br>
>>>> After analyzed the patch, for below 2 paths SP_OUTLET_TEMP sensor is <br>
>>>> created twice.<br>
>>>><br>
>>>><br>
>>>> /sys/class/hwmon/hwmon2/temp2_input<br>
>>>> /sys/class/hwmon/hwmon2/temp1_input<br>
>>>><br>
>>>><br>
>>>> Before your change, hwmon temp sensor will check whether the path is <br>
>>>> already exists or not.<br>
>>>> If it exists, it will skip and proceed with next path.<br>
>>>> <a href="https://github.com/openbmc/dbus-sensors/commit/544e7dc58cecef898fe7d684c3ba0fbce178cf2d#diff-bfc8d7ed4ade4b3519ad3b7fe091f9c79a60faf8a26f410c1568e5341b22b7abL96" rel="noreferrer" target="_blank">https://github.com/openbmc/dbus-sensors/commit/544e7dc58cecef898fe7d684c3ba0fbce178cf2d#diff-bfc8d7ed4ade4b3519ad3b7fe091f9c79a60faf8a26f410c1568e5341b22b7abL96</a> <br>
>>>><br>
>>>><br>
>>>><br>
>>>> Could you please explain why it is creating twice in hwmon temp ?<br>
>>>><br>
>>>><br>
>>>> Journalctl log before your changes applied :<br>
>>>><br>
>>>><br>
>>>> Dec 31 16:01:32 yosemitev2 hwmontempsensor[286]: Path : <br>
>>>> "/sys/class/hwmon/hwmon2/temp2_input"<br>
>>>> Dec 31 16:01:32 yosemitev2 hwmontempsensor[286]: Type : <br>
>>>> xyz.openbmc_project.Configuration.TMP421<br>
>>>> Dec 31 16:01:32 yosemitev2 hwmontempsensor[286]: Name : SP_OUTLET_TEMP<br>
>>>> Dec 31 16:01:32 yosemitev2 hwmontempsensor[286]: Dbus path : <br>
>>>> /xyz/openbmc_project/sensors/temperature/SP_OUTLET_TEMP<br>
>>>> Dec 31 16:01:32 yosemitev2 hwmontempsensor[286]: Dbus path : <br>
>>>> /xyz/openbmc_project/sensors/temperature/SP_OUTLET_REMOTE_TEMP<br>
>>>> Dec 31 16:01:32 yosemitev2 hwmontempsensor[286]: Path : <br>
>>>> "/sys/class/hwmon/hwmon2/temp1_input"<br>
>>>><br>
>>>><br>
>>>><br>
>>>> Journalctl log after your changes applied :<br>
>>>><br>
>>>><br>
>>>> Dec 31 16:01:54 yosemitev2 hwmontempsensor[327]: Path : <br>
>>>> "/sys/class/hwmon/hwmon2/temp2_input"<br>
>>>> Dec 31 16:01:54 yosemitev2 hwmontempsensor[327]: Sensor type : <br>
>>>> xyz.openbmc_project.Configuration.TMP421<br>
>>>> Dec 31 16:01:54 yosemitev2 hwmontempsensor[327]: Name : SP_OUTLET_TEMP<br>
>>>> Dec 31 16:01:54 yosemitev2 hwmontempsensor[327]: Path : <br>
>>>> /xyz/openbmc_project/sensors/temperature/SP_OUTLET_TEMP<br>
>>>> Dec 31 16:01:54 yosemitev2 hwmontempsensor[327]: Path : <br>
>>>> /xyz/openbmc_project/sensors/temperature/SP_OUTLET_REMOTE_TEMP<br>
>>>><br>
>>>><br>
>>>> Dec 31 16:01:54 yosemitev2 hwmontempsensor[327]: Path : <br>
>>>> "/sys/class/hwmon/hwmon2/temp1_input"<br>
>>>> Dec 31 16:01:54 yosemitev2 hwmontempsensor[327]: Sensor type : <br>
>>>> xyz.openbmc_project.Configuration.TMP421<br>
>>>> Dec 31 16:01:54 yosemitev2 hwmontempsensor[327]: Name : SP_OUTLET_TEMP<br>
>>>> Dec 31 16:01:54 yosemitev2 hwmontempsensor[327]: Path : <br>
>>>> /xyz/openbmc_project/sensors/temperature/SP_OUTLET_TEMP<br>
>>>> Dec 31 16:01:54 yosemitev2 hwmontempsensor[327]: Path : <br>
>>>> /xyz/openbmc_project/sensors/temperature/SP_OUTLET_REMOTE_TEMP<br>
>>>><br>
>>>><br>
>>>><br>
>>>> Thanks,<br>
>>>><br>
>>>> Kumar.<br>
> <br>
> I will give it quick look today.  Also today is my last day at IBM.<br>
> <br>
<br>
The problem I had to solve was that I was adding devices<br>
(DPS310 and Si7020 type) that has more than one sensor type<br>
per device, in this case under /sys/bus/iio/devices .<br>
So the loop cannot continue after finding the first sensor of<br>
the device.  I have not seen p10bmc /sys/class/hwmon temperature<br>
sensors being created twice, in this case the TMP75.<br>
<br>
What is the entity-manager configuration for your system?<br>
I suspect that the dbus-sensors Hwmon does not take into account<br>
some detail that is needs to related to entity-manager.<br>
<br>
<br>
</blockquote></div>