All of lore.kernel.org
 help / color / mirror / Atom feed
From: mhoffman@lightlink.com (Mark M. Hoffman)
To: lm-sensors@vger.kernel.org
Subject: Problem with eeprom module in 2.9.0
Date: Thu, 19 May 2005 06:25:29 +0000	[thread overview]
Message-ID: <20050106223520.GD5303@jupiter.solarsys.private> (raw)
In-Reply-To: <41DDAE4F.9030606@earthlink.net>

Hi:

* Jean Delvare <khali@linux-fr.org> [2005-01-06 23:12:16 +0100]:
> OK, the big change here is most probably the support (or attempt
> thereof) of I2C block read in i2c-i801. As far as I know you are the
> first tester of this feature. The eeprom driver does make use of this
> feature when possible, and it doesn't seem to work as intended for you.
> 
> What I find strange is that the lm93 driver does make use of the block
> transfers too, and they seem to work for you (until the eeprom driver
> trashes the whole bus). This suggests that the bug isn't in the i2c-i801
> driver but it the eeprom driver, or maybe the eeprom and lm93 driver do
> not use i2c block reads in the same way.

MDS just added the I2C variant of the block transfer.  But lm93 uses the
SMBus variant... so it is possible that it's a problem in i2c-i801.

Of course last time I said something like that, there was egg all over
my face... :)

> What's the PCI ID of this one? Maybe block transfers work OK for the
> E7501 but not for the E7520.
> 
> Can you please try the attached patch? It disables block reads in the
> eeprom driver. If it solves the problem for you then we at least know
> what is triggering the bug (although I still can't get why lm93 doesn't
> trigger it).

See above.  But the eeprom patch you gave is still a useful test.

I really should get a 2.4 kernel / distro back on my ICH5 machine so I
can see what's going on there.  I'm short on time this week and will
be traveling next week... so it may have to wait a while.

Regards,

-- 
Mark M. Hoffman
mhoffman@lightlink.com

  reply	other threads:[~2005-05-19  6:25 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-05-19  6:25 Problem with eeprom module in 2.9.0 David Knierim
2005-05-19  6:25 ` Mark M. Hoffman [this message]
2005-05-19  6:25 ` Jean Delvare
2005-05-19  6:25 ` Mark Studebaker
2005-05-19  6:25 ` Mark M. Hoffman
2005-05-19  6:25 ` Mark Studebaker
2005-05-19  6:25 ` Mark M. Hoffman
2005-05-19  6:25 ` Mark M. Hoffman

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=20050106223520.GD5303@jupiter.solarsys.private \
    --to=mhoffman@lightlink.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.