> -----Original Message----- > From: Yehezkel Bernat [mailto:yehezkel...@gmail.com] > Sent: Sunday, July 8, 2018 2:56 AM > To: Mika Westerberg > Cc: lu...@wunner.de; LKML; Andreas Noever; michael.ja...@intel.com; > r...@rjwysocki.net; christ...@kellner.me; Limonciello, Mario > Subject: Re: [PATCH 5/5] thunderbolt: Add support for runtime PM > > On Sun, Jul 8, 2018 at 10:31 AM Mika Westerberg > <mika.westerb...@linux.intel.com> wrote: > > > > On Sat, Jul 07, 2018 at 11:14:01PM +0200, Lukas Wunner wrote: > > > > > > Because I'm fairly certain that > > > I do not get a PME for the Light Ridge in my MacBook Pro, but I'll test > > > this once more and modify negotiate_os_control() to grant PME control > > > to the OS. > > > > I think in case of Apple hardware, they handle the in some different > > means than PME (possibly part of chipset driver or ACPI method/event). > > In addition to what already mentioned, many things have changed around power > management during Alpine Ridge development, some of them came later as FW > updates (and BIOS changes). Comparing Alpine ridge to Light Ridge here is > comparing oranges to, well, Apples.
In practice I don't anticipate anyone in the industry outside of Apple wiring up TBT with RTD3 unless it's at least Alpine Ridge or newer.
