All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hans de Goede <j.w.r.degoede@hhs.nl>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] Help wanted: individual alarm files
Date: Sun, 01 Jul 2007 21:13:45 +0000	[thread overview]
Message-ID: <46881909.20109@hhs.nl> (raw)
In-Reply-To: <20070701220005.1dda9d8f@hyperion.delvare>

Hi All,

I'll take care of converting the "fscher" driver as my work PC has such a 
beast. I hope to be able todo this soon.

Ivo, if you are reading this, any chance you could try to convert the sis5595 
driver?

Regards,

Hans


Jean Delvare wrote:
> Hi all,
> 
> The future libsensors assumes that all hardware monitoring drivers
> follow the standard sysfs interface defined in
> Documentation/hwmon/sysfs-interface. They all mostly do already, except
> for one thing: alarms. Most drivers are still only exposing a single
> file with all alarms in a bit-vector, in chip-dependent order. In order
> to let the new libsenors display the alarms properly, we must add
> individual alarm files to these drivers. For some drivers, fault and/or
> beep files need to be added too.
> 
> Here's the current status, to the best of my knowledge.
> 
> Already have individual alarm files:
> abituguru.c
> coretemp.c
> ds1621.c
> dme1737.c
> f71805f.c
> lm63.c
> lm83.c
> lm85.c
> lm90.c
> pc87427.c
> smsc47m192.c
> vt1211.c
> w83627ehf.c
> w83793.c
> 
> Don't have alarms at all:
> ad7418.c
> adm1029.c
> ams
> applesmc.c
> atxp1.c
> hdaps.c
> k8temp.c
> lm70.c
> lm75.c
> max6650.c
> smsc47b397.c
> w83l785ts.c
> 
> Need to be converted:
> adm1021.c
> adm1025.c
> adm1026.c
> adm1031.c
> adm9240.c
> asb100.c
> fscher.c
> fscpos.c
> gl518sm.c
> gl520sm.c
> it87.c
> lm77.c
> lm78.c
> lm80.c
> lm87.c
> lm92.c
> max1619.c
> pc87360.c
> sis5595.c
> smsc47m1.c
> via686a.c
> vt8231.c
> w83627hf.c
> w83781d.c
> w83791d.c
> w83792d.c
> 
> As you can see, the list is quite long, so I need help. That's not a
> difficult job, but you need chip ship in question to test.
> 
> How to add individual alarm files:
> 
> * Read Documentation/hwmon/sysfs-interface, so that you know what are
> the valid names for individual alarm, fault and beep files.
> 
> * Take a look at this patch which adds individual alarm support to 4
> drivers:
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h-45771e6ea79f56a7d85e448f702f60ef86c228#patch2
> 
> * Do the same to your driver. To find out which alarm bit correspond to
> what, you can look at the defines used in the current version of
> libsensors:
> http://www.lm-sensors.org/browser/lm-sensors/branches/lm-sensors-3.0.0/kernel/include/sensors.h?revD73
> Or you can look at the datasheet for your chip, but it's likely to take
> longer.
> 
> * Test. You can test manually (change the limits and read the alarm
> files from sysfs) or you can use the next version of libsensors, which
> is in branch "lm-sensors-3.0.0" under SVN:
> svn checkout http://lm-sensors.org/svn/lm-sensors/branch/lm-sensors-3.0.0 lm-sensors-3
> 
> * Submit your patch to the lm-sensors list, I'll review it.
> 
> Thanks,


_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

  parent reply	other threads:[~2007-07-01 21:13 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-01 20:00 [lm-sensors] Help wanted: individual alarm files Jean Delvare
2007-07-01 21:05 ` Ivo Manca
2007-07-01 21:13 ` Hans de Goede [this message]
2007-07-02 12:01 ` Carlos Olalla Martínez
2007-07-05 11:27 ` JGong
2007-07-07 12:09 ` Jean Delvare
2007-07-09 15:57 ` Krzysztof Helt
2007-07-20  1:27 ` Charles Spirakis
2007-07-21 19:42 ` Jean Delvare
2007-10-07  9:49 ` Jean Delvare
2007-10-07 10:01 ` Jean Delvare
2007-10-07 16:39 ` Ivo Manca

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=46881909.20109@hhs.nl \
    --to=j.w.r.degoede@hhs.nl \
    --cc=lm-sensors@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.