From: khali@linux-fr.org (Jean Delvare)
To: lm-sensors@vger.kernel.org
Subject: add eeprom i2c driver
Date: Thu, 19 May 2005 06:23:51 +0000 [thread overview]
Message-ID: <20030325231959.19cf9912.khali@linux-fr.org> (raw)
In-Reply-To: <20030325172024.GC15823@kroah.com>
> I would like to suggest completely re-writing the user interface to
> the EEPROM device. As an applications programmer, I should not have
> to open a separate sysfs file for each 16 bytes in the EEPROM. I
> should be able to open a single file and perform read/write/llseek
> operations on the file descriptor. I don't think that the actual data
> of the EEPROM device is something that belongs in sysfs, just as the
> data on a hard drive or from a serial port is not read through their
> sysfs counterparts. Furthermore, I think that checksum data is not
> something that should be in the kernel, but in the application that is
> using it. An application can open() /dev/eepromX (/dev/nvramX maybe?)
> and read the data and ensure that the checksum is OK. Not all
> applications will have the same checksum mechanism, location, etc.
The things were this way simply because the EEPROM driver started its
life as a component of the LM Sensors project. Actually, an EEPROM isn't
a sensor, and it was handled by LM Sensors simply because EEPROMs are
found on the I2C bus.
If we decide to change the interface for this driver, it means
(methinks) that it will have to leave the LM Sensors project completely,
since it won't be handled by sensors-detect nor libsensors/sensors. I
don't pretend it cause any sort of problem, I just want to make sure you
are aware of it.
Actually, it may be preferable to move the EEPROM driver away. If I
remember correctly, the LM Sensors team doesn't especially like this
driver ;) And I don't think anyone would look for an EEPROM driver
there.
We have a pair of scripts that dump some EEPROM information, written in
Perl. I can rewrite them if there's a new interface to the data, and
eventually they'll leave the LM Sensors project if there's no reason to
keep them there.
Mark, Phil, feel free to speak your mind if I'm wrong!
--
Jean Delvare
http://www.ensicaen.ismra.fr/~delvare/
next prev parent reply other threads:[~2005-05-19 6:23 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-03-25 14:42 add eeprom i2c driver Jan Dittmer
2003-03-25 17:20 ` Greg KH
2005-05-19 6:23 ` Greg KH
2003-03-25 19:33 ` Jan Dittmer
2005-05-19 6:23 ` Jan Dittmer
2003-03-26 9:42 ` Jan Dittmer
2005-05-19 6:23 ` Jan Dittmer
2003-03-26 21:29 ` Greg KH
2005-05-19 6:23 ` Greg KH
2003-03-28 20:49 ` Pavel Machek
2005-05-19 6:23 ` Pavel Machek
2005-05-19 6:23 ` Jean Delvare [this message]
2005-05-19 6:23 ` Mark Studebaker
2005-05-19 6:23 ` Mark Studebaker
2005-05-19 6:23 ` Deepak Saxena
2003-03-25 21:32 ` Greg KH
2005-05-19 6:23 ` Greg KH
2003-03-25 21:50 ` Deepak Saxena
2005-05-19 6:23 ` Deepak Saxena
2005-05-19 6:23 ` Greg KH
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=20030325231959.19cf9912.khali@linux-fr.org \
--to=khali@linux-fr.org \
--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.