<div dir="ltr">ack,<div>sounds like I'm better off just moving temp sensors over to being handled by PSUSensors.</div><div><br></div><div>Slightly different topic:</div><div>What about making the device/type lists in PSUSensors extendable using JSON?</div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Aug 7, 2020 at 9:24 AM James Feist <<a href="mailto:james.feist@linux.intel.com">james.feist@linux.intel.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 8/6/2020 3:52 PM, Jason Ling wrote:<br>
> Assuming that you mean "Omit Name attribute from the sensor <br>
> configuration definition and then change hwmontemp to require any Name.*"<br>
> This won't work since Entity-Manager requires Name (tried it, <br>
> entity-manager does indeed complain about not finding name).<br>
> <br>
> My rationale for an omit list vs permit list<br>
> (1) if it's a permit list then everytime you add another temp you want <br>
> to monitor you need to add to this list..if you want to drop a temp then <br>
> you have to modify the list again.<br>
> (2) General assumption is that the primary use case is to display all <br>
> named temperatures which means a permit list is typically large<br>
> (3) adding a permit list also breaks all existing code. Everyone has to <br>
> go back into their json config and add all the sensor values to the list.<br>
> <br>
> My rationale for using the value for the "Name" attribute rather than <br>
> labels or referencing sysfs attributes<br>
> (1) Looking at just the config , it's obvious as to what you're omitting.<br>
> (2) If it's label base, a label change in a driver would mean a breakage <br>
> in the userspace daemon. Not a big deal; but it can be annoying.<br>
> (3) if it's sysfs attribute based then it's my opinion that it's not as <br>
> readable.<br>
> <br>
<br>
I'm not a huge fan of this as the PSU sensor already has a way of <br>
handling this, and it adds a new way of handling it. I'd rather follow <br>
what is already there. It's already confusing enough that hwmontemp and <br>
psu do things in slightly different ways.<br>
<br>
> <br>
</blockquote></div>