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] [PATCH] sensors-detect: Check for 1-register-only
Date: Fri, 09 May 2008 17:05:41 +0000	[thread overview]
Message-ID: <20080509190541.408024c2@hyperion.delvare> (raw)
In-Reply-To: <20080508224145.41cfa3ef@hyperion.delvare>

On Fri, 09 May 2008 18:56:30 +0200, achim wrote:
> Am Freitag, den 09.05.2008, 17:24 +0200 schrieb Jean Delvare:
> > Hi Achim,
> 
> > I see this, and I think I understand why. The address exclusion logic
> > assumes that the original list contains _all_ addresses. This is no
> > longer the case. When I restricted sensors-detect to only probe
> > addresses where are least one hardware monitoring chip is known, I
> > forgot about this optimization which turned into a bug. Thanks for
> > reporting. This patch should fix it:
> > 
> > Can you please test it?
> > 
> > Thanks,
> 
> Yeah, the patch works. I tested sensors-detect with all known adresses
> of the board excluded and it did not messup the smbus chip.

OK, good. I'll commit this patch later today then.

> Afterward i tried it with only 0x4c excluded and the smbus chip still
> works. With no exclusions lm75 detection hangs at 0x4c and afterwards i
> only get XX's from dumps.

By "hangs" you mean it takes several seconds, or does it really stop?
Are you certain that it's the LM75 detection causing problem? We check
for no less than 33 different devices at address 0x4c...

Anyway, most certainly the problem is with word transactions on this
device you have at 0x4c. That won't be easy to work around, but I'll
take a look anyway, maybe I'll have an idea. If I do, I'll ask you -
again - to help with testing. Thanks *a lot* for all the help you are
providing in this respect.

-- 
Jean Delvare

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

  parent reply	other threads:[~2008-05-09 17:05 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-08 20:41 [lm-sensors] [PATCH] sensors-detect: Check for 1-register-only Jean Delvare
2008-05-09  5:03 ` Hans de Goede
2008-05-09  5:06 ` Hans de Goede
2008-05-09  6:46 ` Hans de Goede
2008-05-09 10:43 ` Jean Delvare
2008-05-09 10:47 ` Hans de Goede
2008-05-09 11:51 ` achim
2008-05-09 12:14 ` Jean Delvare
2008-05-09 15:29 ` Jean Delvare
2008-05-09 17:05 ` Jean Delvare [this message]
2008-05-09 17:31 ` achim
2008-05-09 21:02 ` Jean Delvare
2008-05-09 21:16 ` Ludovic Lebègue
2008-05-10 16:40 ` Jean Delvare
2008-05-11 16:23 ` Jean Delvare
2008-05-12 10:53 ` Achim Gottinger

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=20080509190541.408024c2@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.