All of lore.kernel.org
 help / color / mirror / Atom feed
From: Vincent Pelletier <plr.vincent-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
Cc: Ben Skeggs <bskeggs-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
Subject: Bisect: i2c regression in 63daa9589cb88617d9cc8a0b0282564682a975b7
Date: Sun, 1 Jan 2012 22:38:25 +0100	[thread overview]
Message-ID: <201201012238.25620.plr.vincent@gmail.com> (raw)

Hi.

Updating nouveau module to a newer git revision, I noticed I lost hwmon.

I ran a bisect, which identified 63daa9589cb88617d9cc8a0b0282564682a975b7 as 
being the first bad commit.

Adding printk traces, I saw that probing i2c bus 3 for sensors always failed 
with -EIO (no ack from slave). After reading the code, remembering how i2c 
works and reading sample code on wikipedia, I couldn't identify any 
algorythmic problem.

So I gave a try at changing delays. I doubled T_RISEFALL and T_HOLD, and the 
adt7473 chip could be detected & used again. I haven't tested other 
combination, nor tried to link those delays with any datasheet.

My card is a GTX 280.

Regards,
-- 
Vincent Pelletier

             reply	other threads:[~2012-01-01 21:38 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-01 21:38 Vincent Pelletier [this message]
     [not found] ` <201201012238.25620.plr.vincent-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2012-01-02 22:40   ` Bisect: i2c regression in 63daa9589cb88617d9cc8a0b0282564682a975b7 Ben Skeggs
2012-01-03  8:22     ` Vincent Pelletier
2012-01-20 21:32     ` Vincent Pelletier

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=201201012238.25620.plr.vincent@gmail.com \
    --to=plr.vincent-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
    --cc=bskeggs-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.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.