This issue is more easily resolved by just changing !OSDW to OSDW for
that method. The default behavior it seems is for Linux to identify
itself as OSX, but there are a few methods where we don't want to
identify as being OSX. I'll try and come up with a list of methods of
where we don't want to identify as being OSX.
On Wed, Mar 2, 2016 at 2:07 PM, Leif Liddy <leif.linux@xxxxxxxxx> wrote:
> You're right, it's not evaluating the OSDW method properly. I tracked
> it down to this single change:
>
> change this:
>
> If (!OSDW ())
> {
> Return (UBUF) /* \_SB_.PCI0.SPI1.SPIT._CRS.UBUF */
> }
> Return (ABUF) /* \_SB_.PCI0.SPI1.SPIT._CRS.ABUF */
> }
>
> to:
>
> Return (UBUF) /* \_SB_.PCI0.SPI1.SPIT._CRS.UBUF */
>
>
> On Wed, Mar 2, 2016 at 12:42 PM, Lukas Wunner <lukas@xxxxxxxxx> wrote:
>> Hi,
>>
>> On Wed, Mar 02, 2016 at 04:13:16AM +0100, Leif Liddy wrote:
>>> I'll post more tomorrow, need to evaluate the changes I made --but I
>>> think the change that did it was changing this:
>>>
>>> Method (_CRS, 0, NotSerialized) // _CRS: Current Resource Settings
>>> {
>>> If (!OSDW ())
>>> {
>>> Return (WBUF) /* \_SB_.PCI0.SPI1.WBUF */
>>> }
>>>
>>> Return (ConcatenateResTemplate (RBUF, DBUF))
>>> }
>>>
>>> to:
>>>
>>> Method (_CRS, 0, NotSerialized) // _CRS: Current Resource Settings
>>> {
>>> Return (ConcatenateResTemplate (RBUF, DBUF))
>>> }
>>
>> If that is true then the fix is to change the OS we're reporting to ACPI:
>> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/drivers/acpi/osl.c#n141
>>
>> We had trouble with this before:
>> https://mjg59.dreamwidth.org/29954.html
>>
>> I've just grabbed the acpidump you've posted here:
>> https://bugzilla.kernel.org/attachment.cgi?id=200961&action=edit
>>
>> I'll take a look at it hopefully this afternoon to see what they've
>> changed in the OSDW method.
>>
>> Thanks a lot for your perseverance in tackling this issue.
>>
>> Lukas
--
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
