From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752441Ab0FXXmj (ORCPT ); Thu, 24 Jun 2010 19:42:39 -0400 Received: from imr1.ericy.com ([198.24.6.9]:48020 "EHLO imr1.ericy.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751312Ab0FXXmi (ORCPT ); Thu, 24 Jun 2010 19:42:38 -0400 X-Greylist: delayed 2113 seconds by postgrey-1.27 at vger.kernel.org; Thu, 24 Jun 2010 19:42:38 EDT Date: Thu, 24 Jun 2010 16:40:58 -0700 From: Guenter Roeck To: "Ira W. Snyder" CC: Randy Dunlap , Jean Delvare , Andrew Morton , Hans de Goede , Mark Brown , Samuel Ortiz , Jonathan Cameron , "lm-sensors@lm-sensors.org" , "linux-kernel@vger.kernel.org" , "linux-doc@vger.kernel.org" Subject: Re: [PATCH v3 3/4] hwmon: Update sysfs-interface to include new attributes Message-ID: <20100624234058.GA25719@ericsson.com> References: <1277416861-32494-1-git-send-email-guenter.roeck@ericsson.com> <1277416861-32494-4-git-send-email-guenter.roeck@ericsson.com> <20100624233240.GA9934@ovro.caltech.edu> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20100624233240.GA9934@ovro.caltech.edu> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Jun 24, 2010 at 07:32:40PM -0400, Ira W. Snyder wrote: > On Thu, Jun 24, 2010 at 03:01:00PM -0700, Guenter Roeck wrote: > > Added _lcrit and _crit to voltage attributes. > > Added _lcrit to temperature attributes. > > > > Signed-off-by: Guenter Roeck > > --- > > Documentation/hwmon/sysfs-interface | 21 ++++++++++++++++++++- > > 1 files changed, 20 insertions(+), 1 deletions(-) > > > > diff --git a/Documentation/hwmon/sysfs-interface b/Documentation/hwmon/sysfs-interface > > index d4e2917..7f60c0d 100644 > > --- a/Documentation/hwmon/sysfs-interface > > +++ b/Documentation/hwmon/sysfs-interface > > @@ -107,10 +107,24 @@ in[0-*]_min Voltage min value. > > Unit: millivolt > > RW > > > > +in[0-*]_lcrit Voltage critical min value. > > + Unit: millivolt > > + RW > > + If voltage drops to or below this limit, the system may > > + take drastic action such as power down or reset. At the very > > + least, it should report a fault. > > + > > in[0-*]_max Voltage max value. > > Unit: millivolt > > RW > > > > +in[0-*]_crit Voltage critical max value. > > + Unit: millivolt > > + RW > > + If voltage reaches or exceeds this limit, the system may > > + take drastic action such as power down or reset. At the very > > + least, it should report a fault. > > + > > The critical voltages API looks good to me. > > I have a hwmon chip driver for the TI INA209 (which Jean NAK'd) that > could use these limits. I'll submit it again soon with changes to use > this new set of sysfs attributes. > > It would be great to see a patch to libsensors and the sensors program > to allow it to display these limits too. > I am working on it. I'll send the patch to Jean as soon as the kernel changes have been approved. Guenter > Thanks, > Ira > > > in[0-*]_input Voltage input value. > > Unit: millivolt > > RO > > @@ -284,11 +298,16 @@ temp[1-*]_input Temperature input value. > > Unit: millidegree Celsius > > RO > > > > -temp[1-*]_crit Temperature critical value, typically greater than > > +temp[1-*]_crit Temperature critical max value, typically greater than > > corresponding temp_max values. > > Unit: millidegree Celsius > > RW > > > > +temp[1-*]_lcrit Temperature critical min value, typically lower than > > + corresponding temp_min values. > > + Unit: millidegree Celsius > > + RW > > + > > temp[1-*]_crit_hyst > > Temperature hysteresis value for critical limit. > > Unit: millidegree Celsius > > -- > > 1.7.0.87.g0901d > > From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Date: Thu, 24 Jun 2010 23:40:58 +0000 Subject: Re: [lm-sensors] [PATCH v3 3/4] hwmon: Update sysfs-interface to Message-Id: <20100624234058.GA25719@ericsson.com> List-Id: References: <1277416861-32494-1-git-send-email-guenter.roeck@ericsson.com> <1277416861-32494-4-git-send-email-guenter.roeck@ericsson.com> <20100624233240.GA9934@ovro.caltech.edu> In-Reply-To: <20100624233240.GA9934@ovro.caltech.edu> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: "Ira W. Snyder" Cc: Randy Dunlap , Jean Delvare , Andrew Morton , Hans de Goede , Mark Brown , Samuel Ortiz , Jonathan Cameron , "lm-sensors@lm-sensors.org" , "linux-kernel@vger.kernel.org" , "linux-doc@vger.kernel.org" On Thu, Jun 24, 2010 at 07:32:40PM -0400, Ira W. Snyder wrote: > On Thu, Jun 24, 2010 at 03:01:00PM -0700, Guenter Roeck wrote: > > Added _lcrit and _crit to voltage attributes. > > Added _lcrit to temperature attributes. > > > > Signed-off-by: Guenter Roeck > > --- > > Documentation/hwmon/sysfs-interface | 21 ++++++++++++++++++++- > > 1 files changed, 20 insertions(+), 1 deletions(-) > > > > diff --git a/Documentation/hwmon/sysfs-interface b/Documentation/hwmon/sysfs-interface > > index d4e2917..7f60c0d 100644 > > --- a/Documentation/hwmon/sysfs-interface > > +++ b/Documentation/hwmon/sysfs-interface > > @@ -107,10 +107,24 @@ in[0-*]_min Voltage min value. > > Unit: millivolt > > RW > > > > +in[0-*]_lcrit Voltage critical min value. > > + Unit: millivolt > > + RW > > + If voltage drops to or below this limit, the system may > > + take drastic action such as power down or reset. At the very > > + least, it should report a fault. > > + > > in[0-*]_max Voltage max value. > > Unit: millivolt > > RW > > > > +in[0-*]_crit Voltage critical max value. > > + Unit: millivolt > > + RW > > + If voltage reaches or exceeds this limit, the system may > > + take drastic action such as power down or reset. At the very > > + least, it should report a fault. > > + > > The critical voltages API looks good to me. > > I have a hwmon chip driver for the TI INA209 (which Jean NAK'd) that > could use these limits. I'll submit it again soon with changes to use > this new set of sysfs attributes. > > It would be great to see a patch to libsensors and the sensors program > to allow it to display these limits too. > I am working on it. I'll send the patch to Jean as soon as the kernel changes have been approved. Guenter > Thanks, > Ira > > > in[0-*]_input Voltage input value. > > Unit: millivolt > > RO > > @@ -284,11 +298,16 @@ temp[1-*]_input Temperature input value. > > Unit: millidegree Celsius > > RO > > > > -temp[1-*]_crit Temperature critical value, typically greater than > > +temp[1-*]_crit Temperature critical max value, typically greater than > > corresponding temp_max values. > > Unit: millidegree Celsius > > RW > > > > +temp[1-*]_lcrit Temperature critical min value, typically lower than > > + corresponding temp_min values. > > + Unit: millidegree Celsius > > + RW > > + > > temp[1-*]_crit_hyst > > Temperature hysteresis value for critical limit. > > Unit: millidegree Celsius > > -- > > 1.7.0.87.g0901d > > _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors