public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Guenter Roeck <linux@roeck-us.net>
To: Andreas Werner <wernerandy@gmx.de>, linux-kernel@vger.kernel.org
Cc: khali@linux-fr.org, andreas.werner@men.de
Subject: Re: Question: hwmon: Adding addtitional sysfs entries
Date: Sat, 10 May 2014 05:32:13 -0700	[thread overview]
Message-ID: <536E1C4D.9030206@roeck-us.net> (raw)
In-Reply-To: <20140510132238.GA28364@awedesk.fritz.box>

On 05/10/2014 06:22 AM, Andreas Werner wrote:
> Hi,
> i am currently working on an implemenation of my Board Management Controller (BMC).
>
> This Controller is a MCR assembled on almost all of our Compact PCI or Compact PCI Serial
> Cards as well as on some other CPU Boards.
>
> The BMC includes LED´s, Watchdog, Voltage Monitoring and some other features.
> I have already implemented an MFD driver, LED and Watchdog driver.
>
> Currently i am working on the hwmon implemenation. This includes for the first step the
> voltage monitoring.
>
> The BMC has some other feature which are not supported by any of the subsystem.
>   - Errorcounter (Board status)
>   - Power Failure status
>   - Operating ours

Guess you mean hours here.

>   - Do software reset, do cold reset
>   - Set power modes
>   - Backplate Geographical Address
>
> These are all informations related to the Board where the BMC is assembled to.
> I think all those features could be part of the hwmon implementation.
>
Why ?

> Is it possible to add "custom" sysfs entries in my hwmon driver to support all these features?
> Does lm-sensor ingore those custom features or is it a problem for lm-sensors?
>
> What is the normal way to add those features?
>

The hwmon subsystem isn't really the correct location for this functionality.
Power management should be done with a regulator driver (possibly power).
The other attributes and functionality would better fit into the mfd core
driver.

You might want to tie the reset functions into the system reset functionality.

Guenter


  reply	other threads:[~2014-05-10 12:32 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-10 13:22 Question: hwmon: Adding addtitional sysfs entries Andreas Werner
2014-05-10 12:32 ` Guenter Roeck [this message]
2014-05-10 16:02   ` Andreas Werner
2014-05-10 15:32     ` Guenter Roeck
2014-05-14  6:44       ` Andreas Werner

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=536E1C4D.9030206@roeck-us.net \
    --to=linux@roeck-us.net \
    --cc=andreas.werner@men.de \
    --cc=khali@linux-fr.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=wernerandy@gmx.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox