public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Roger Leigh <rleigh@whinlatter.ukfsn.org>
To: benh@kernel.crashing.org
Cc: linux-kernel@vger.kernel.org
Subject: Re: radeonfb i2c regression post-2.6.18.
Date: Wed, 21 Nov 2007 23:56:05 +0000	[thread overview]
Message-ID: <877ikbf90q.fsf@hardknott.home> (raw)
In-Reply-To: <1195675089.6970.96.camel@pasglop> (Benjamin Herrenschmidt's message of "Thu\, 22 Nov 2007 06\:58\:09 +1100")

[-- Attachment #1: Type: text/plain, Size: 1129 bytes --]

Benjamin Herrenschmidt <benh@kernel.crashing.org> writes:

>> > Can you try the patch from Jean that I pasted below and let us know if
>> > it helps ? It looks like the releasing of the i2c lines may have been
>> > done backward.
>> 
>> This patch fixes the problem.  The monitor stays powered on during the
>> switch to the framebuffer.
>
> Excellent ! That saves me having to test myself :-)
>
> As far as I'm concerned, that's an Ack for the patch.

Fantastic, thanks!  I've copied this to Debian bugs 433236 and 426124
which were about this problem.

BTW, the framebuffer penguin logo looked a little wierd (low number of
colours, odd colours), though on my powerpc it has always looked odd
(wrong colours).  Could there be some endianness bug in the fblogo
code?  I'll check it with other video options when I next have a few
minutes.


Thanks again,
Roger

-- 
  .''`.  Roger Leigh
 : :' :  Debian GNU/Linux             http://people.debian.org/~rleigh/
 `. `'   Printing on GNU/Linux?       http://gutenprint.sourceforge.net/
   `-    GPG Public Key: 0x25BFB848   Please GPG sign your mail.

[-- Attachment #2: Type: application/pgp-signature, Size: 188 bytes --]

  parent reply	other threads:[~2007-11-21 23:59 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-18 23:16 radeonfb i2c regression post-2.6.18 Benjamin Herrenschmidt
2007-11-21  9:42 ` Roger Leigh
2007-11-21 19:58   ` Benjamin Herrenschmidt
2007-11-21 20:13     ` Michael Buesch
2007-11-21 20:17       ` Benjamin Herrenschmidt
2007-11-21 23:56     ` Roger Leigh [this message]
2007-11-22  0:00       ` Benjamin Herrenschmidt
2007-11-23 16:00 ` Michael Buesch
2007-11-23 22:29   ` Jean Delvare
2007-11-24  1:11     ` Benjamin Herrenschmidt
2007-11-24  9:29       ` Jean Delvare
2007-11-24 14:18     ` Michael Buesch
2007-11-24 22:20       ` Jean Delvare
2007-11-24 22:25         ` Benjamin Herrenschmidt
  -- strict thread matches above, loose matches on Subject: below --
2007-11-18 21:58 Roger Leigh

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=877ikbf90q.fsf@hardknott.home \
    --to=rleigh@whinlatter.ukfsn.org \
    --cc=benh@kernel.crashing.org \
    --cc=linux-kernel@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox