<div dir="ltr"><div>Doing some code changes I encountered this comment in PSUSensor.</div><div dir="ltr"><div><a href="https://github.com/openbmc/dbus-sensors/blob/dfad1ffcf4d41d6b5c0fd578d1817c0540631e29/src/PSUSensorMain.cpp#L762" target="_blank">https://github.com/openbmc/dbus-sensors/blob/dfad1ffcf4d41d6b5c0fd578d1817c0540631e29/src/PSUSensorMain.cpp#L762</a><br></div><div>I didn't verify it but it looks like the mechanism for disabling the selected sensor is already there.</div><div><br></div><div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">czw., 6 sie 2020 o 01:43 Jason Ling <<a href="mailto:jasonling@google.com" target="_blank">jasonling@google.com</a>> napisał(a):<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><b>Problem:</b></div>There is a use case where temp1_input should be omitted from being exposed to dbus.<div>A concrete example is if you have a temp sensor with 10 channels but only want to expose 2..10.</div><div><br></div><div>Currently dbus/hwmontemp doesn't allow this.</div><div><br></div><div><b>Solution:</b></div><div>In order to maintain backwards compatibility I am proposing an OmitList attribute that hwmontemp will attempt to retrieve.<br></div><div>If the "Name"s of any temp sensor appears in the list, it will be skip sensor creation.</div><div><br></div><div>I am proposing a list to support other use cases such as...</div><div><br></div><div>* you're doing BMC development and for whatever reason want to temporarily suppress a temperature and do some tests..you can add it to this list and then remove it instead of deleting and re-inserting.<br></div><div><br></div><div>* lets you have non-contiguous temp sensors exposed (e.g temp2_input, temp5_input, temp7_input) . There is a better solution to this; but for now this enables this use-case.</div><div><br></div><div><b>etc..</b></div><div>It's a simple feature; plan to have something within O(hours).</div></div>
</blockquote></div></div>