<div dir="ltr"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Oct 20, 2020 at 8:46 PM Matt Spinler <<a href="mailto:mspinler@linux.ibm.com">mspinler@linux.ibm.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><br>
<br>
On 10/19/2020 10:23 AM, Thu Ba Nguyen wrote:<br>
> Thanks for your reply Matt Spinler, Can you show me the discussion <br>
> threads? I also...<br>
> This Message Is From an External Sender<br>
> This message came from outside your organization.<br>
><br>
> Thanks for your reply Matt Spinler,<br>
><br>
> Can you show me the discussion threads?<br>
<br>
Sure: <a href="https://lists.ozlabs.org/pipermail/openbmc/2019-October/018967.html" rel="noreferrer" target="_blank">https://lists.ozlabs.org/pipermail/openbmc/2019-October/018967.html</a><br>
<br>
><br>
> I also thought about the solution for that features:<br>
> In the current hwmon we support GPIOCHIP + GPIO option which used to <br>
> enable sensors to read. In the hwmon code, we just set that pin and <br>
> wait before reading.<br>
> I think we can support a similar option named GPIOENABLE + GPIOV. When <br>
> the status of Gpio pin defind in GPIOEANBLE match with GPIOV.<br>
> That sensors will be read and update to Dbus.<br>
> If not it will be removed from DBus until the GPIO pin math GPIOV.<br>
> Maybe we can have many different solutions.<br>
<br>
As Ed mentioned, I think a good direction to start with is how <br>
dbus-sensors handles it, so we  can have<br>
common behavior.  I believe they look at the host state D-Bus property <br>
and still keep the sensor<br>
on D-Bus even when power is off.<br></blockquote><div> </div><div>[Thu] Yah, I think  using <span style="color:rgb(0,0,0);font-family:-webkit-standard;font-size:medium">the host state D-Bus property</span><span style="color:rgb(0,0,0);font-family:-webkit-standard;font-size:medium"> to verify host state.</span></div><div><font color="#000000" face="-webkit-standard" size="3"><span style="caret-color: rgb(0, 0, 0);">Then handling the host sensors based on this status is a better solution.</span></font></div><div><font color="#000000" face="-webkit-standard" size="3"><span style="caret-color: rgb(0, 0, 0);">Keeping the sensors on Dbus can cause the value or the status of warning/critical value of sensors</span></font></div><div><font color="#000000" face="-webkit-standard" size="3"><span style="caret-color: rgb(0, 0, 0);">is out update. I expected that the host sensors will be removed from Dbus when the host is off.</span></font></div><div><font color="#000000" face="-webkit-standard" size="3"><span style="caret-color: rgb(0, 0, 0);"><br></span></font></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
><br>
> If you don't mind, can you tell me how IBM supports that features?<br>
<br>
We lucked out out in that the driver was only loaded when power was on.<br></blockquote><div>[Thu] What will be displayed when you open health --> sensors page in OpenBmc web? </div><div>Are the host sensors still there and value is "na" or some things like that?</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<br>
><br>
> Regards.<br>
> Thu Nguyen.<br>
><br>
> On Mon, Oct 19, 2020 at 9:16 PM Matt Spinler <<a href="mailto:mspinler@linux.ibm.com" target="_blank">mspinler@linux.ibm.com</a> <br>
> <mailto:<a href="mailto:mspinler@linux.ibm.com" target="_blank">mspinler@linux.ibm.com</a>>> wrote:<br>
><br>
><br>
><br>
>     On 10/18/2020 8:58 AM, Thu Ba Nguyen wrote:<br>
>     > Dear, I'm supporting the host sensors for Ampere Computing LLC<br>
>     > platform. We are...<br>
>     > This Message Is From an External Sender<br>
>     > This message came from outside your organization.<br>
>     ><br>
>     > Dear,<br>
>     ><br>
>     > I'm supporting the host sensors for Ampere Computing LLC platform.<br>
>     > We are using phosphor-hwmon to update values of sensors and<br>
>     monitoring<br>
>     > sensors warning/errors base on threshold setting.<br>
>     ><br>
>     > There are some sensors which are turned off when host Off. It<br>
>     can be<br>
>     > the sensors reported by host or voltage/temperature/power sensors<br>
>     >  which use the same power source with host.<br>
>     ><br>
>     > I researched in openBmc sensor-architecture documents but can't<br>
>     find<br>
>     > any option to enable/disable sensors base on one status or GPIO<br>
>     pins.<br>
>     > I can't use REMOVERCS.<br>
>     ><br>
>     > Research in phosphor-hwmon code, I don't see the answer too.<br>
>     ><br>
>     > Do we have any options/solution to Enable/Disable some sensors when<br>
>     > Host On/Off?<br>
><br>
>     Hi,<br>
>     The phosphor-hwmon code doesn't support that yet.  It has been<br>
>     discussed<br>
>     before but nobody<br>
>     has implemented it.<br>
><br>
>     ><br>
>     > Thanks.<br>
>     > Thu Nguyen.<br>
><br>
<br>
</blockquote></div></div></div>