From: greg@kroah.com (Greg KH)
To: lm-sensors@vger.kernel.org
Subject: add eeprom i2c driver
Date: Thu, 19 May 2005 06:23:51 +0000 [thread overview]
Message-ID: <20030326213723.GC26886@kroah.com> (raw)
In-Reply-To: <20030325172024.GC15823@kroah.com>
On Tue, Mar 25, 2003 at 06:17:00PM -0500, Mark Studebaker wrote:
> Jean described the history of the driver well.
>
> I would call it more of a "demonstration" driver than something
> that is exceptionally useful.
> (the CVS continues that tradition, I added I2C Block read capability to the
> driver, more as a demonstration and block read test than anything else).
>
> It's also useful to us as a diagnostic tool with users -
> (if eeprom works then the i2c bus works. if eeprom is the only
> thing found on the i2c bus then the sensors must be on the ISA bus).
Well, that's a good reason for it to stick around, right?
> As such, I'm not really sentimental about the existing interface in /proc
> or what it could be in sysfs, or whether it goes into the kernel at all.
> If the debate makes it more trouble than
> it's worth to get into the kernel, it can stay out of the kernel
> as far as I'm concerned.
>
> For Greg, it's probably best to start with lm75, not eeprom, to review
> the changes W.R.T. sysfs.
Good point, I've sent my patch for lm75 to the list.
It looks like the binary files for sysfs support isn't as easy when you
are in the driver model, so it will take a bit longer to add that
support (need to add some driver core wrapper support first). But that
is the way the eeprom driver should move to.
thanks,
greg k-h
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 ` Greg KH [this message]
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 ` Mark Studebaker
2005-05-19 6:23 ` Jean Delvare
2005-05-19 6:23 ` Mark Studebaker
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=20030326213723.GC26886@kroah.com \
--to=greg@kroah.com \
--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.