All of lore.kernel.org
 help / color / mirror / Atom feed
From: Aaron Williams <awilliams@marvell.com>
To: Emily Shaffer <emilyshaffer@google.com>
Cc: Lei YU <mine260309@gmail.com>,
	OpenBMC Maillist <openbmc@lists.ozlabs.org>
Subject: Re: [EXT] Re: OpenBMC Sensors
Date: Wed, 30 Jan 2019 13:28:38 -0800	[thread overview]
Message-ID: <32964718.XoMeP9TR8m@flash> (raw)
In-Reply-To: <CAJoAoZmZhxFNuvRYKQ04iTYZVrHgoukbM4HdTaPP6kCZppMhDA@mail.gmail.com>

Hi Emily,

That's what I need the temperature for. We have two controllers, one that 
monitors the core temperature (a TI TMP421) and one that controls the fans 
(ADT7462). In order to maintain the thermal envelope the TMP421 needs to be 
monitored to adjust the fan speed through the ADT7462.

Further complicating things is the fact that these sensors disappear when the 
host is powered down.

-Aaron

On Wednesday, January 30, 2019 9:49:32 AM PST Emily Shaffer wrote:
> External Email
> 
> ----------------------------------------------------------------------
> Hi Aaron,
> 
> Note that you only really need to worry about sending the temperature via
> IPMI if you want to send it somewhere besides the BMC.  If you plan to do
> internal thermal control (BMC reads temperature, BMC adjusts fans
> accordingly) you probably don't need IPMI config and can get away with
> setting it up as far as DBus in the sensor architecture doc Lei sent.
> 
> Emily
> 
> On Thu, Jan 24, 2019 at 1:27 AM Lei YU <mine260309@gmail.com> wrote:
> > On Thu, Jan 24, 2019 at 5:13 PM Aaron Williams <awilliams@marvell.com>
> > 
> > wrote:
> > > I see how to set up the hwmon portion defining the devices based on the
> > 
> > device
> > 
> > > tree, but I am unsure how to go about configuring the YAML and other
> > 
> > files for
> > 
> > > this.
> > 
> > For sensors' config, please refer to
> > https://github.com/openbmc/docs/blob/master/sensor-architecture.md
> > And it looks you are looking for fan controls, then you could refer to:
> > 
> > https://github.com/mine260309/openbmc-intro/blob/master/Porting_Guide.md#f
> > ans
> > 
> > (I really need to submit my porting guide to openbmc/docs)

  reply	other threads:[~2019-01-30 21:28 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-24  9:03 OpenBMC Sensors Aaron Williams
2019-01-24  9:26 ` Lei YU
2019-01-30 17:49   ` Emily Shaffer
2019-01-30 21:28     ` Aaron Williams [this message]
2019-01-30 21:42       ` [EXT] " Emily Shaffer
2019-01-30 21:47         ` Aaron Williams
2019-01-30 21:48           ` Emily Shaffer
     [not found]             ` <2342439.TUI2J960l9@flash>
     [not found]               ` <CAJoAoZnV+u8kJKV7TkPxU_b1hdmJ3nbFyD6rVMSd9ohPSjDHig@mail.gmail.com>
2019-01-31 15:30                 ` Patrick Venture
2019-02-01  1:05                   ` Aaron Williams
2019-02-01  5:17                   ` Aaron Williams
2019-02-01 17:07                     ` Patrick Venture
2019-02-01 18:04                     ` Vijay Khemka
2019-02-01 19:32                       ` Matthew Barth

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=32964718.XoMeP9TR8m@flash \
    --to=awilliams@marvell.com \
    --cc=emilyshaffer@google.com \
    --cc=mine260309@gmail.com \
    --cc=openbmc@lists.ozlabs.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.