dbus-sensors:hwmontemp: additional attribute proposal

[Zbyszek]

Doing some code changes I encountered this comment in PSUSensor.
https://github.com/openbmc/dbus-sensors/blob/dfad1ffcf4d41d6b5c0fd578d1817c0540631e29/src/PSUSensorMain.cpp#L762
I didn't verify it but it looks like the mechanism for disabling the
selected sensor is already there.


czw., 6 sie 2020 o 01:43 Jason Ling <jasonling at google.com> napisał(a):

> *Problem:*
> There is a use case where temp1_input should be omitted from being exposed
> to dbus.
> A concrete example is if you have a temp sensor with 10 channels but only
> want to expose 2..10.
>
> Currently dbus/hwmontemp doesn't allow this.
>
> *Solution:*
> In order to maintain backwards compatibility I am proposing an OmitList
> attribute that hwmontemp will attempt to retrieve.
> If the "Name"s of any temp sensor appears in the list, it will be skip
> sensor creation.
>
> I am proposing a list to support other use cases such as...
>
> * 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.
>
> * 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.
>
> *etc..*
> It's a simple feature; plan to have something within O(hours).
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ozlabs.org/pipermail/openbmc/attachments/20200806/1d38a8cf/attachment-0001.htm>