linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andreas Noever <andreas.noever@gmail.com>
To: Matthew Garrett <mjg59@srcf.ucam.org>
Cc: Adam Goode <adam@spicenitz.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: Thunderbolt hotplug not working on MacMini7,1
Date: Fri, 24 Apr 2015 13:41:25 +0200	[thread overview]
Message-ID: <CAMxnaaViiKDYpD5mZtATp3NZ9ebuCxXTC5kgvJvRDHsfYxo37w@mail.gmail.com> (raw)
In-Reply-To: <20150423181246.GA23441@srcf.ucam.org>

On Thu, Apr 23, 2015 at 8:12 PM, Matthew Garrett <mjg59@srcf.ucam.org> wrote:
> On Thu, Apr 23, 2015 at 12:08:01PM +0200, Andreas Noever wrote:
>> Hi Adam,
>>
>> On my system (MacBookPro10,1 - 4 channel TB1) the bridges and the
>> controller both use 0x1547 and are only differentiated by
>> subvendor/subdevice.
>
> Do they have the same PCI class?
No, 0604 for the bridges and 0880 for the device.

It looks like the only reason that the bridges do not have a
subsystem/subvendor set is because there is no such field in the pci
bridge header. Instead they put 2222:1111 into the SSVID capability:

06:06.0 PCI bridge [0604]: Intel Corporation Unknown device
[8086:1547] (rev 03) (prog-if 00 [Normal decode])
Flags: bus master, fast devsel, latency 0
Bus: primary=06, secondary=08, subordinate=08, sec-latency=0
Capabilities: [80] Power Management version 3
Capabilities: [88] Message Signalled Interrupts: Mask- 64bit+ Queue=0/0 Enable+
Capabilities: [ac] Subsystem: Unknown device [2222:1111]
Capabilities: [c0] Express Downstream Port (Slot+), MSI 00
Capabilities: [100] #8086

07:00.0 System peripheral [0880]: Intel Corporation Unknown device
[8086:1547] (rev 03)
Subsystem: Unknown device [2222:1111]
Flags: bus master, fast devsel, latency 0, IRQ 17
Memory at c1d00000 (32-bit, non-prefetchable)
Memory at c1d40000 (32-bit, non-prefetchable)
Capabilities: [80] Power Management version 3
Capabilities: [88] Message Signalled Interrupts: Mask- 64bit+ Queue=0/0 Enable+
Capabilities: [ac] Subsystem: Unknown device [2222:1111]
Capabilities: [c0] Express Endpoint, MSI 00
Capabilities: [a0] MSI-X: Enable- Mask- TabSize=16
Capabilities: [100] #8086

pci_setup_device actually reads this capability and puts it into
dev->subsystem_vendor/device. So we might actually get bound to the
bridges (if the pcieport driver is unavailable). I'll post a patch to
bind to the class code instead. Good idea!

Andreas

      reply	other threads:[~2015-04-24 11:41 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CADN=WdrsLoertuPi0Tabb6dmdRAj0d200q=5awq-xL9yE8W=hw@mail.gmail.com>
2015-04-22  4:05 ` Fwd: Thunderbolt hotplug not working on MacMini7,1 Adam Goode
2015-04-23  2:56   ` Adam Goode
2015-04-23  7:24     ` Greg KH
2015-04-23 10:08     ` Andreas Noever
2015-04-23 13:28       ` Adam Goode
2015-04-23 15:10         ` Adam Goode
2015-04-23 16:12           ` Andreas Noever
2015-04-23 17:15             ` Adam Goode
2015-04-24  4:50               ` Adam Goode
2015-04-24 11:46                 ` Andreas Noever
2015-04-24 18:39                   ` Adam Goode
2015-04-25  4:58                     ` Adam Goode
2015-04-26  2:41                       ` Adam Goode
2015-04-26  3:00                         ` Adam Goode
2015-05-04 13:00                           ` Adam Goode
2015-05-06 10:18                             ` Andreas Noever
2015-04-23 18:12       ` Matthew Garrett
2015-04-24 11:41         ` Andreas Noever [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=CAMxnaaViiKDYpD5mZtATp3NZ9ebuCxXTC5kgvJvRDHsfYxo37w@mail.gmail.com \
    --to=andreas.noever@gmail.com \
    --cc=adam@spicenitz.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mjg59@srcf.ucam.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).