On Wed, Feb 14, 2018 at 4:39 PM, <Mario.Limonciello@xxxxxxxx> wrote: >> -----Original Message----- >> From: rjwysocki@xxxxxxxxx [mailto:rjwysocki@xxxxxxxxx] On Behalf Of Rafael J. >> Wysocki >> Sent: Wednesday, February 14, 2018 2:46 AM >> To: Limonciello, Mario <Mario_Limonciello@xxxxxxxx> >> Cc: ACPI Devel Maling List <linux-acpi@xxxxxxxxxxxxxxx>; Len Brown >> <lenb@xxxxxxxxxx>; Rafael J . Wysocki <rjw@xxxxxxxxxxxxx>; Mika Westerberg >> <mika.westerberg@xxxxxxxxxxxxxxx> >> Subject: Re: [PATCH] ACPICA: Recognize the Windows 10 RS3 OSI string >> >> On Tue, Feb 13, 2018 at 7:00 PM, Mario Limonciello >> <mario.limonciello@xxxxxxxx> wrote: >> > Dell uses this string to activate Thunderbolt native mode on supported >> > machines. >> > >> > Signed-off-by: Mario Limonciello <mario.limonciello@xxxxxxxx> >> > --- >> > Thunderbolt Native mode requires other patches to be applied otherwise >> > this patch will cause problems. > > I should have been clearer - problems with thunderbolt driver in Linux. > > If you apply this patch without the rest of Mika's series there will be problems > with Thunderbolt native mode in Linux. Right. > The patches you already reviewed in >> https://marc.info/?l=linux-acpi&m=151853954129169&w=2 > > Are the more important ones, but a few of the fixes that came in the second > series help with some of the corner cases (for example cold boot ICM timeout). OK > Mika might have a few more comments to add here from the testing he did > to make those patches. > >> > >> > This patch should only be applied with: >> > 1) Acpiphp platches from >> > https://marc.info/?l=linux-acpi&m=151853954129169&w=2 >> > and >> > >> > 2) Thunderbolt patches from >> > https://lkml.org/lkml/2018/2/13/685 >> >> Thanks for these pointers, they help a lot. >> >> > This has also been submited into ACPICA: >> > https://github.com/acpica/acpica/pull/359 >> >> Due to the dependencies outside of upstream ACPICA, I don't think that >> this patch is suitable for upstream ACPICA at all. > > Actually ACPICA did merge it already: > https://github.com/acpica/acpica/commit/24f7791789c49ec02eb024106c85d23322be5672 So there appear to be differing opinions. :-) -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html