All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jean Delvare <khali@linux-fr.org>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] dme1737 0-002e: Write to register 0x30 failed!
Date: Tue, 23 Oct 2007 12:08:49 +0000	[thread overview]
Message-ID: <20071023140849.7b8010f0@hyperion.delvare> (raw)
In-Reply-To: <1642847378@web.de>

Hi Juerg,

On Mon, 22 Oct 2007 09:02:37 -0700, Juerg Haefliger wrote:
> From the ACPI spec:
> Indicates that the transaction failed because the SMBus host
> reports that the SMBus is presently busy with some other
> transaction. For example, the Smart Battery might be
> sending charging information to the Smart Battery Charger.

Hmm, OK, that's really "bus busy" then. It may be related to this chip
at address 0x0d. Can't say more without detailed hardware
specifications of the system.

> > Depends on what kernel Juergen is running. Oleg Ryjkov has submitted
> > interesting patches that clean up this part of the i2c-nforce2 driver:
> > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;hA53549734cbdba24e9cf5eb200b70b7b1572e15
> > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;hÔ9584c4a37c7228e7778bcb60f79e7a08472fa8
> > These are already in Linus' tree for 2.6.24.
> 
> Hmm... These patches add abort functionality in case the controller is
> locked. I don't think this is our problem here. In Juergen's case, any
> subsequent transaction after one that fails succeeds so it's a
> transient problem and not a hard lock.

The second patch if completely unrelated, agreed. But the first patch
changes how the driver polls for transaction status. While it is not
related to Juergen's problem, the timing change involved could affect
the code you want to add to retry the transaction when the bus is busy.

-- 
Jean Delvare

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

  parent reply	other threads:[~2007-10-23 12:08 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-09-25 19:08 [lm-sensors] dme1737 0-002e: Write to register 0x30 failed! Juergen Bausa
2007-10-01 17:44 ` Juerg Haefliger
2007-10-01 19:22 ` Juergen Bausa
2007-10-01 21:39 ` Juerg Haefliger
2007-10-01 21:50 ` Jean Delvare
2007-10-08 18:31 ` Juergen Bausa
2007-10-09  7:03 ` Jean Delvare
2007-10-09 16:08 ` Juerg Haefliger
2007-10-09 19:41 ` Juergen Bausa
2007-10-10 17:12 ` Juerg Haefliger
2007-10-11  7:45 ` Jean Delvare
2007-10-17 18:56 ` Juergen Bausa
2007-10-17 19:43 ` Juerg Haefliger
2007-10-17 21:32 ` Jean Delvare
2007-10-18  4:53 ` Juerg Haefliger
2007-10-19 15:07 ` Jean Delvare
2007-10-20 19:28 ` Juergen Bausa
2007-10-20 19:39 ` Juergen Bausa
2007-10-21 18:40 ` Mark M. Hoffman
2007-10-21 19:14 ` Jean Delvare
2007-10-21 19:37 ` Jean Delvare
2007-10-22 16:02 ` Juerg Haefliger
2007-10-22 16:05 ` Juerg Haefliger
2007-10-23 12:08 ` Jean Delvare [this message]
2007-10-23 12:22 ` Jean Delvare
2007-10-23 20:44 ` Juergen Bausa
2007-10-23 21:54 ` Rudolf Marek
2007-10-24  8:48 ` Jean Delvare
2007-10-29 21:23 ` Rudolf Marek
2007-10-30  2:52 ` Juerg Haefliger
2007-10-31  2:40 ` Juerg Haefliger

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=20071023140849.7b8010f0@hyperion.delvare \
    --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.