<div dir="ltr">Hi Ed,<div><br></div><div>The question was more of a general ask on whether dbus-sensors plans for this on record. If so, individual daemon should have the config option to ignore a device completely. Currently, I think PSUSensor has the ability, but HwmonTempSensor does not.</div><div><br></div><div>The reason behind it may be complicated. One is if we can fix the PSUSensor performance issue on time so that we can use it directly for PID control based on VR temperatures. And then, if we can't fix it on time, what work around can we have? Is it upstream-able or local patches? What's more, can we have different polling rates for temperature and voltage/current/power by using multiple daemon for the same device? Of course, ideally, we can go for a fast feature-enriched PSUSensor to take care of everything and deprecate HwmonTempSensor, but you know I have been talking about schedule for multiple times with you before...</div><div><br></div><div>Thanks.</div><div><br clear="all"><div><div dir="ltr" data-smartmail="gmail_signature"><div dir="ltr">- Alex Qiu</div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 11, 2020 at 4:02 PM Ed Tanous <<a href="mailto:ed@tanous.net" target="_blank">ed@tanous.net</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, Aug 10, 2020 at 5:09 PM Alex Qiu <<a href="mailto:xqiu@google.com" target="_blank">xqiu@google.com</a>> wrote:<br>
><br>
> Hi Ed and James,<br>
><br>
> Is it acceptable for a device to be listed in both HwmonTempSensor and PSUSensor?<br>
><br>
<br>
Can you lay out a little bit of info about why you would need the same<br>
type in both places?  My knee jerk reaction would be to say no, under<br>
the idea that we shouldn't be duplicating functionality and code<br>
between sensors, but if there are good technical reasons that amount<br>
to more than "one has bugs, the other doesn't" then I definitely would<br>
be for allowing it.<br>
</blockquote></div>