Include sensor monitor for our platform

Include sensor monitor for our platform

[Kumar Thangavel]

[-- Attachment #1: Type: text/plain, Size: 1640 bytes --]

Classification: Confidential
Hi All,

          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.

           We did not have any bbappend files for phosphor-fan-presence. We enabled the fan feature using phosphor-pid-control.
           So, Could you please provide any pointers or examples to enable this feature for our platform.

Thanks,
Kumar.
::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.
________________________________

[-- Attachment #2: Type: text/html, Size: 4125 bytes --]

Re: Include sensor monitor for our platform

[Patrick Williams]

[-- Attachment #1: Type: text/plain, Size: 1061 bytes --]

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@linux.ibm.com> <msbarth!>
M:  Matt Spinler <spinler@us.ibm.com> <mspinler[m]!>
```

-- 
Patrick Williams

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

RE: Include sensor monitor for our platform

[Kumar Thangavel]

Classification: Confidential

Thanks Patrick.

Added MAINTAINERS.

Thanks,
Kumar.

-----Original Message-----
From: Patrick Williams <patrick@stwcx.xyz>
Sent: Friday, April 30, 2021 7:39 PM
To: Kumar Thangavel <thangavel.k@hcl.com>
Cc: openbmc@lists.ozlabs.org; Manish Kumar Thakur <manishkumar-thakur@hcl.com>; Velumani T-ERS,HCLTech <velumanit@hcl.com>; Amithash Prasad <amithash@fb.com>; sdasari@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@linux.ibm.com> <msbarth!>
M:  Matt Spinler <spinler@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.
________________________________

RE: 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@stwcx.xyz>; Matthew Barth <msbarth@linux.ibm.com>; M: Matt Spinler <spinler@us.ibm.com>
Cc: openbmc@lists.ozlabs.org; Manish Kumar Thakur <manishkumar-thakur@hcl.com>; Velumani T-ERS,HCLTech <velumanit@hcl.com>; Amithash Prasad <amithash@fb.com>; sdasari@fb.com
Subject: RE: Include sensor monitor for our platform

Classification: Confidential

Thanks Patrick.

Added MAINTAINERS.

Thanks,
Kumar.

-----Original Message-----
From: Patrick Williams <patrick@stwcx.xyz>
Sent: Friday, April 30, 2021 7:39 PM
To: Kumar Thangavel <thangavel.k@hcl.com>
Cc: openbmc@lists.ozlabs.org; Manish Kumar Thakur <manishkumar-thakur@hcl.com>; Velumani T-ERS,HCLTech <velumanit@hcl.com>; Amithash Prasad <amithash@fb.com>; sdasari@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@linux.ibm.com> <msbarth!>
M:  Matt Spinler <spinler@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.
________________________________