From: Daniel Vetter <daniel.vetter@ffwll.ch>
To: Henrik Rydberg <rydberg@euromail.se>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
Henrik Rydberg <rydberg@bitmath.se>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
dri-devel@lists.freedesktop.org
Subject: Re: Linux 3.7-rc6
Date: Thu, 22 Nov 2012 21:32:42 +0100 [thread overview]
Message-ID: <CAKMK7uH1=vA7YCh9eZSyW=ewF3kYkdgT45LhYqnigPg0whe5aA@mail.gmail.com> (raw)
In-Reply-To: <20121122182325.GA777@polaris.bitmath.org>
On Thu, Nov 22, 2012 at 7:23 PM, Henrik Rydberg <rydberg@euromail.se> wrote:
>> >> My apologies for the long delay in answering, I've somehow mixed up
>> >> different bugreports and thought I've sent you a patch to test
>> >> already. Anyway, please test
>> >>
>> >> https://patchwork.kernel.org/patch/1728111/
>> >
>> > Tested-by: Henrik Rydberg <rydberg@euromail.se>
>>
>> Can you please boot with drm.debug=0xe added to your kernel cmdline
>> with that patch applied and attach the full dmesg?
>
> Are you looking for something in particular? The patch obviously works
> because edp_bpp is never set. The reason seems to be
>
> [ 1.634759] [drm:intel_parse_bios], VBT signature missing
>
> In the source, a few lines above that message, we have something that
> looks suspciciously like an off-by-one error:
>
> /* Scour memory looking for the VBT signature */
> for (i = 0; i + 4 < size; i++) {
>
> If that matters or not, I do not know. Any more tests would have to
> wait until tomorrow.
Yeah, the utter lack of a vbt fits very nicely, thanks for checking,
I've merged the patch into drm-intel-fixes and will forward it for
inclusion into 3.7 rsn.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
next prev parent reply other threads:[~2012-11-22 20:32 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-17 2:12 Linux 3.7-rc6 Linus Torvalds
2012-11-20 10:39 ` Henrik Rydberg
2012-11-21 11:55 ` Daniel Vetter
2012-11-21 17:55 ` Henrik Rydberg
2012-11-22 11:18 ` Henrik Rydberg
2012-11-22 13:10 ` Daniel Vetter
2012-11-22 18:23 ` Henrik Rydberg
2012-11-22 20:32 ` Daniel Vetter [this message]
2012-11-22 20:53 ` Henrik Rydberg
2012-11-23 9:53 ` Jani Nikula
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='CAKMK7uH1=vA7YCh9eZSyW=ewF3kYkdgT45LhYqnigPg0whe5aA@mail.gmail.com' \
--to=daniel.vetter@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=linux-kernel@vger.kernel.org \
--cc=rydberg@bitmath.se \
--cc=rydberg@euromail.se \
--cc=torvalds@linux-foundation.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;
as well as URLs for NNTP newsgroup(s).