Include sensor monitor for our platform

[Kumar Thangavel]

Classification: Confidential

Hi Matt/Matthew,

            We wanted to enable the sensor monitors like "ShutdownAlaramMonitor" and "ThesholdAlaramLogger" to our system to monitor the sensor values of our fan.

            We created phosphor-fan_bbappend files and try to enable the sensor-monitor like "control", "monitor".
            But still sensor-monitor is not enabled.

           Could you please provide any suggestions on this.

Thanks,
Kumar.

-----Original Message-----
From: Kumar Thangavel
Sent: Friday, April 30, 2021 7:58 PM
To: Patrick Williams <patrick at stwcx.xyz>; Matthew Barth <msbarth at linux.ibm.com>; M: Matt Spinler <spinler at us.ibm.com>
Cc: openbmc at lists.ozlabs.org; Manish Kumar Thakur <manishkumar-thakur at hcl.com>; Velumani T-ERS,HCLTech <velumanit at hcl.com>; Amithash Prasad <amithash at fb.com>; sdasari at fb.com
Subject: RE: Include sensor monitor for our platform

Classification: Confidential

Thanks Patrick.

Added MAINTAINERS.

Thanks,
Kumar.

-----Original Message-----
From: Patrick Williams <patrick at stwcx.xyz>
Sent: Friday, April 30, 2021 7:39 PM
To: Kumar Thangavel <thangavel.k at hcl.com>
Cc: openbmc at lists.ozlabs.org; Manish Kumar Thakur <manishkumar-thakur at hcl.com>; Velumani T-ERS,HCLTech <velumanit at hcl.com>; Amithash Prasad <amithash at fb.com>; sdasari at fb.com
Subject: Re: Include sensor monitor for our platform

On Fri, Apr 30, 2021 at 09:37:17AM +0000, Kumar Thangavel wrote:
>           We wanted to enable the sensor monitors like "ShutdownAlaramMonitor" and "ThesholdAlaramLogger" to our system to monitor the sensor values of our fan.
>            These monitors were created under phosphor-fan-presence.

I don't think there is anyone using the monitoring of phosphor-fan-presence with the control of phosphor-pid-control, but that doesn't mean it can't be done.

If you look at the phosphor-fan recipe you'll see that the default PACKAGECONFIG is "presence control monitor".  I think you'll want to bbappend this to only enable "monitor" (and not control/presence).  This should configure the package so only the monitoring parts are enabled at compile time.

You may want to cc the MAINTAINERS of phosphor-fan-presence for any detailed questions you run into when giving this a try (and copy the list).

```
M:  Matthew Barth <msbarth at linux.ibm.com> <msbarth!>
M:  Matt Spinler <spinler at us.ibm.com> <mspinler[m]!> ```

--
Patrick Williams
::DISCLAIMER::
________________________________
The contents of this e-mail and any attachment(s) are confidential and intended for the named recipient(s) only. E-mail transmission is not guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or may contain viruses in transmission. The e mail and its contents (with or without referred errors) shall therefore not attach any liability on the originator or HCL or its affiliates. Views or opinions, if any, presented in this email are solely those of the author and may not necessarily reflect the views or opinions of HCL or its affiliates. Any form of reproduction, dissemination, copying, disclosure, modification, distribution and / or publication of this message without the prior written consent of authorized representative of HCL is strictly prohibited. If you have received this email in error please delete it and notify the sender immediately. Before opening any email and/or attachments, please check them for viruses and other defects.
________________________________