From: Matthew Garrett <mjg59@srcf.ucam.org>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: platform-driver-x86@vger.kernel.org
Subject: Re: [PATCH] x86,mrst: Intel Medfield over-current detection patch
Date: Mon, 20 Dec 2010 16:10:33 +0000 [thread overview]
Message-ID: <20101220161033.GB25502@srcf.ucam.org> (raw)
In-Reply-To: <20101213112147.15213.12742.stgit@bob.linux.org.uk>
On Mon, Dec 13, 2010 at 11:53:41AM +0000, Alan Cox wrote:
> From: Durgadoss R <durgadoss.r@intel.com>
>
> This patch enables over current detection on the Intel Medfield
> Platform. This driver provides sysfs interfaces to configure current
> thresholds. When the current being drawn on the platform exceeds
> the configured threshold, an interrupt is generated. This interrupt
> can be used to bring the current consumption down by taking necessary
> hardware or software actions.
>
> Note that while we use hwmon style helpers and interface this is not an hwmon
> device and hwmon has no common API with this hardware. After discussion with
> the hwmon maintainers it was decided to make this an x86 platform driver.
Hm. Do you have a summary or a pointer to that discussion? This really
does seem to be a hardware monitoring device, and so hwmon would seem
the more natural home for it - I'd like some idea of what the
justification was.
--
Matthew Garrett | mjg59@srcf.ucam.org
next prev parent reply other threads:[~2010-12-20 16:10 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-13 11:53 [PATCH] x86,mrst: Intel Medfield over-current detection patch Alan Cox
2010-12-20 16:10 ` Matthew Garrett [this message]
2010-12-20 16:26 ` Alan Cox
2010-12-20 16:31 ` Matthew Garrett
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=20101220161033.GB25502@srcf.ucam.org \
--to=mjg59@srcf.ucam.org \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=platform-driver-x86@vger.kernel.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.