Re: [RFC PATCH 2/3] platform/x86: Add Uniwill laptop driver

[Werner Sembach <wse@tuxedocomputers.com>]

Hi,

Am 17.06.25 um 23:50 schrieb Armin Wolf:
> Am 16.06.25 um 14:32 schrieb Werner Sembach:
>
>> Hi,
>>
>> Am 15.06.25 um 19:59 schrieb Armin Wolf:
[...]
>>> +The EC behind the ``GetSetULong`` method is used by the OEM software 
>>> supplied by the manufacturer.
>>> +Reverse-engineering of this software is difficult since it uses an 
>>> obfuscator, however some parts
>>> +are not obfuscated.
>> Iirc Christoffer said there is also another non wmi, but acpi, get/set cmd 
>> that can write only specific regions but is faster. Just wanted to leave this 
>> here for potential future optimizations if you are interested.
>
> What interface are you referring to exactly?

I think he was referring to these 2 methods in the dsdt:

     Scope (_SB)
     {
         Device (INOU)
         {
             [...]

             Method (ECRR, 1, NotSerialized)
             {
                 Local0 = (0xFE410000 + Arg0)
                 Local1 = MMRW (Local0, Zero, Zero, Zero)
                 Return (Local1)
             }

             Method (ECRW, 2, NotSerialized)
             {
                 Local0 = (0xFE410000 + Arg0)
                 MMRW (Local0, One, Zero, Arg1)
             }

             [...]
         }
     }

They can read/write the 0x04xx and 0x07xx addresses but not the 0x18xx ones 
iirc, don't know about the 0x0fxx ones.

I think 0x18xx was never intended to be written probably, but we use it to 
implement a custom fan curve option in tuxedo-drivers on older devices that 
officially don't support it.

Best regards,

Werner