From: "Mantas Mikulėnas" <grawity@gmail.com>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Matthew Garrett <mjg@redhat.com>,
Bjorn Helgaas <bhelgaas@google.com>,
Seth Forshee <seth.forshee@canonical.com>,
Dave Airlie <airlied@gmail.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: Regression: Screen turns off when booting in EFI mode
Date: Tue, 19 Mar 2013 21:31:34 +0200 [thread overview]
Message-ID: <5148BD16.6040600@gmail.com> (raw)
In-Reply-To: <CA+55aFyOypoeXgBWP0d7KmL3YezmjwEgrB=ntZaZdYjdmFkQpA@mail.gmail.com>
On 2013-03-19 19:09, Linus Torvalds wrote:
> This is apparently still outstanding, and Mantas hadn't cc'd the
> people involved with the commit itself.
I completely forgot about that – sorry.
> Mantas, mind changing that "pcibios_add_device()" function so that
> instead of setting dev->rom/romlen, it just prints out the values
> (including the device address)? Plase also make it print out the
> "data->len" field in addition to the rom->xyz fields..
Which variable is the device address stored in? I'm mostly clueless
about the kernel beyond adding printk's, unfortunately.
But I tried printing out the other values, and this is what I have so far:
For 01:00.0 VGA compatible controller [0300]: Advanced Micro Devices
[AMD] nee ATI Robson CE [AMD Radeon HD 6300 Series] [1002:68e4]
pci 0000:01:00.0: [1002:68e4] type 00 class 0x030000
pci 0000:01:00.0: reg 10: [mem 0xc0000000-0xcfffffff 64bit pref]
pci 0000:01:00.0: reg 18: [mem 0xd0020000-0xd003ffff 64bit]
pci 0000:01:00.0: reg 20: [io 0xd000-0xd0ff]
pci 0000:01:00.0: reg 30: [mem 0xd0000000-0xd001ffff pref]
pci 0000:01:00.0: supports D1 D2
(dbg) -------------------------------------
(dbg) old dev->rom: 0x0000000000000000, dev->romlen: 0
(dbg) new dev->rom: 0x00000000029ad058, dev->romlen: 61952
(dbg) pa_data: 0x00000000029ad018 + offset: 64
(dbg) data->type: 3, data->len: 62000
(dbg) rom->segment: 0, rom->bus: 1
(dbg) rom->device: 0, rom->function: 0
(dbg) rom->vendor: 1002, rom->devid: 68e4 <-- radeon
(dbg) -------------------------------------
And for 05:00.5 Ethernet controller [0200]: JMicron Technology Corp.
JMC250 PCI Express Gigabit Ethernet Controller [197b:0250] (rev 03) –
haven't checked if it's affected too, but including anyway.
pci 0000:05:00.5: [197b:0250] type 00 class 0x020000
pci 0000:05:00.5: reg 10: [mem 0xd0200000-0xd0203fff]
pci 0000:05:00.5: reg 18: [io 0x9100-0x917f]
pci 0000:05:00.5: reg 1c: [io 0x9000-0x90ff]
pci 0000:05:00.5: PME# supported from D0 D3hot D3cold
(dbg) -------------------------------------
(dbg) old dev->rom: 0x0000000000000000, dev->romlen: 0
(dbg) new dev->rom: 0x00000000029bd058, dev->romlen: 40960
(dbg) pa_data: 0x00000000029bd018 + offset: 64
(dbg) data->type: 3, data->len: 41008
(dbg) rom->segment: 0, rom->bus: 5
(dbg) rom->device: 0, rom->function: 5
(dbg) rom->vendor: 197b, rom->devid: 250
(dbg) -------------------------------------
pci 0000:05:00.5: System wakeup disabled by ACPI
--
Mantas Mikulėnas <grawity@gmail.com>
next prev parent reply other threads:[~2013-03-19 19:31 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-02-21 23:09 Regression: Screen turns off when booting in EFI mode Mantas Mikulėnas
2013-02-21 23:54 ` Dave Airlie
2013-02-22 1:03 ` Mantas Mikulėnas
2013-03-09 21:42 ` Mantas Mikulėnas
2013-03-19 17:09 ` Linus Torvalds
2013-03-19 17:58 ` Linus Torvalds
2013-03-19 19:31 ` Mantas Mikulėnas [this message]
2013-03-19 19:59 ` Matthew Garrett
2013-03-19 20:20 ` Linus Torvalds
2013-03-19 21:16 ` Mantas Mikulėnas
2013-03-19 21:26 ` [PATCH] PCI: Use ROM images from firmware only if no other ROM source available Matthew Garrett
2013-03-25 23:52 ` Regression: Screen turns off when booting in EFI mode Dave Airlie
2013-03-25 23:55 ` Matthew Garrett
2013-03-27 19:29 ` Bjorn Helgaas
2013-03-27 20:02 ` Dave Airlie
2013-03-27 20:28 ` Bjorn Helgaas
2013-03-26 21:25 ` [PATCH 1/3] pci: Add PCI ROM helper for platform-provided ROM images Matthew Garrett
2013-03-26 21:25 ` [PATCH 2/3] nouveau: Attempt to use platform-provided ROM image Matthew Garrett
2013-03-26 21:25 ` [PATCH 3/3] radeon: " Matthew Garrett
2013-03-26 22:53 ` [PATCH 1/3] pci: Add PCI ROM helper for platform-provided ROM images Bjorn Helgaas
2013-03-26 22:55 ` Matthew Garrett
2013-03-27 20:33 ` Bjorn Helgaas
2013-03-27 20:34 ` Bjorn Helgaas
2013-03-28 8:48 ` Mantas Mikulėnas
2013-04-01 17:39 ` Bjorn Helgaas
2013-04-01 17:47 ` Matthew Garrett
2013-04-02 20:10 ` Bjorn Helgaas
2013-04-05 20:31 ` Chris Murphy
2013-04-05 20:35 ` Bjorn Helgaas
2013-04-05 22:28 ` Chris Murphy
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=5148BD16.6040600@gmail.com \
--to=grawity@gmail.com \
--cc=airlied@gmail.com \
--cc=bhelgaas@google.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mjg@redhat.com \
--cc=seth.forshee@canonical.com \
--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).