All of lore.kernel.org
 help / color / mirror / Atom feed
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>

  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 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.