<div dir="ltr"><div>Hi All,</div><div><br></div><div>  Please find the below patch to fix this issue.</div>  <a aria-label="Link https://gerrit.openbmc-project.xyz/c/openbmc/dbus-sensors/+/50153" title="https://gerrit.openbmc-project.xyz/c/openbmc/dbus-sensors/+/50153" href="https://gerrit.openbmc-project.xyz/c/openbmc/dbus-sensors/+/50153" rel="noopener noreferrer" target="_blank" tabindex="-1" style="box-sizing:border-box;outline-style:none;color:rgb(98,100,167);text-decoration-line:none;font-family:"Segoe UI",system-ui,"Apple Color Emoji","Segoe UI Emoji",sans-serif;font-size:14px">https://gerrit.openbmc-project.xyz/c/openbmc/dbus-sensors/+/50153</a><br><div><br></div><div>Thanks,</div><div>Kumar.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jan 5, 2022 at 12:08 AM Ed Tanous <<a href="mailto:edtanous@google.com">edtanous@google.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 Mon, Jan 3, 2022 at 1:09 PM Patrick Williams <<a href="mailto:patrick@stwcx.xyz" target="_blank">patrick@stwcx.xyz</a>> wrote:<br>
><br>
> Ed,<br>
><br>
> I think there are other commits needing the IIO functionality now. There is at least the pending commit for Humidity that we’d like to get merged. I think it is better to move forward with a fix rather than backwards with a revert at this point.<br>
<br>
Sure, if there are patches available to fix the regression, I'm happy<br>
to avoid the revert.  I'll let this sit another day or two to see if<br>
someone puts up a patch to fix the issue, but after that, we need to<br>
unbreak master, and in the absence of a patch to fix it, a revert is<br>
the next best option.<br>
<br>
><br>
> Kumar/Jayashree,<br>
><br>
> Can you dig into this and get to the bottom of it?<br>
><br>
> - Patrick<br>
> Sent from my iPhone<br>
><br>
> > On Jan 3, 2022, at 11:21 AM, Ed Tanous <<a href="mailto:ed@tanous.net" target="_blank">ed@tanous.net</a>> wrote:<br>
> ><br>
> > On Fri, Dec 31, 2021 at 9:31 AM Bruce Mitchell<br>
> > <<a href="mailto:bruce.mitchell@linux.vnet.ibm.com" target="_blank">bruce.mitchell@linux.vnet.ibm.com</a>> wrote:<br>
> >><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 <<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>
> >>> 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 in hwmon temp has been creating twice.<br>
> >>>>> After analyzed the patch, for below 2 paths SP_OUTLET_TEMP sensor is 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 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>
> >>>>> 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 : "/sys/class/hwmon/hwmon2/temp2_input"<br>
> >>>>> Dec 31 16:01:32 yosemitev2 hwmontempsensor[286]: Type : 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 : /xyz/openbmc_project/sensors/temperature/SP_OUTLET_TEMP<br>
> >>>>> Dec 31 16:01:32 yosemitev2 hwmontempsensor[286]: Dbus path : /xyz/openbmc_project/sensors/temperature/SP_OUTLET_REMOTE_TEMP<br>
> >>>>> Dec 31 16:01:32 yosemitev2 hwmontempsensor[286]: Path : "/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 : "/sys/class/hwmon/hwmon2/temp2_input"<br>
> >>>>> Dec 31 16:01:54 yosemitev2 hwmontempsensor[327]: Sensor type : 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 : /xyz/openbmc_project/sensors/temperature/SP_OUTLET_TEMP<br>
> >>>>> Dec 31 16:01:54 yosemitev2 hwmontempsensor[327]: Path : /xyz/openbmc_project/sensors/temperature/SP_OUTLET_REMOTE_TEMP<br>
> >>>>><br>
> >>>>><br>
> >>>>> Dec 31 16:01:54 yosemitev2 hwmontempsensor[327]: Path : "/sys/class/hwmon/hwmon2/temp1_input"<br>
> >>>>> Dec 31 16:01:54 yosemitev2 hwmontempsensor[327]: Sensor type : 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 : /xyz/openbmc_project/sensors/temperature/SP_OUTLET_TEMP<br>
> >>>>> Dec 31 16:01:54 yosemitev2 hwmontempsensor[327]: Path : /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>
> > Do you plan on continuing this beyond your tenure at IBM?  If not, is<br>
> > someone taking this feature over for you?  Given it's been a few weeks<br>
> > since this regression was reported, and I don't see any fixes in<br>
> > review, I'm leaning toward reverting it on master for the time being<br>
> > until someone puts up a patch to fix the issue.<br>
><br>
</blockquote></div>