linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Antonino A. Daplas" <adaplas@gmail.com>
To: Andrew Morton <akpm@osdl.org>
Cc: linux-fbdev-devel@lists.sourceforge.net, mano@roarinelk.homelinux.net
Subject: Re: [PATCH] i810fb: Do not probe the third i2c bus by default
Date: Sun, 29 Jan 2006 18:39:44 +0800	[thread overview]
Message-ID: <43DC9B70.3080004@gmail.com> (raw)
In-Reply-To: <20060129001631.597a7766.akpm@osdl.org>

Andrew Morton wrote:
> "Antonino A. Daplas" <adaplas@gmail.com> wrote:
>>
>>  Some time before 2.6.15, a third DDC channel was added to i810fb.
>>  On systems where these ddc pins are not connected, the probe
>>  takes about 10 seconds.
>>
>>  Add a boot/module  option for i810fb to explicitly probe for the 3rd
>>  ddc bus if needed.
> 
> argh.  We really shouldn't be adding options like this.  People just won't

Me too.

> know that the option exists and they'll sit there wondering why their
> kernel doesn't work right.  Or we invert the sense of the option and they
> sit there wondering why the bootup is so slow.

Well the 3rd i2c bus is undocumented, even by Intel. But it's supposed to
exist to allow attaching of daughter cards. I added support for this on the
prodding of Nicholas Boichat. As far as I know, nobody uses this 3rd bus.

> 
> Is there not any way in which to fix this so things just work?  A 10-second
> timeout sounds crazy.

The only way to know if this bus exists is to probe it, unfortunately. And
because it's undocumented, I don't which chipset type has it.  So anyone who
has knowledge of this undocumented feature will probably have the energy
to search for the proper boot option, or know where to mail me :-)

Tony
 




-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642

      reply	other threads:[~2006-01-29 10:40 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-01-29  2:02 [PATCH] i810fb: Do not probe the third i2c bus by default Antonino A. Daplas
2006-01-29  8:16 ` Andrew Morton
2006-01-29 10:39   ` Antonino A. Daplas [this message]

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=43DC9B70.3080004@gmail.com \
    --to=adaplas@gmail.com \
    --cc=akpm@osdl.org \
    --cc=linux-fbdev-devel@lists.sourceforge.net \
    --cc=mano@roarinelk.homelinux.net \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).