Re: [PATCH] ACPICA: Recognize the Windows 10 RS3 OSI string

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux