Help with getting SW_TABLET_MODE reporting working on the MSI Summit E16 Flip ?

[jwrdegoede]

Short self introduction I'm a Linux developer mostly working on hw-enablement for Linux laptops. I'm also the subsystem maintainer for kernel drivers under: drivers/platform/x86

I have been working with a user to get SW_TABLET_MODE reporting working on their MSI Summit E16 Flip A12UCT: https://gitlab.freedesktop.org/libinput/libinput/-/issues/822

Looking at the ACPI tables of that model, their is an intel-hid ACPI device (check for HIDD in dsdt.dsl) which should report if the 2-in-1 is folded into tablet-mode or being used in laptop mode. Specifically the bit of ACPI code below should cause 0xcc / 0xcd ACPI notifies to be send which should then get picked up by the https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/platform/x86/intel/hid.c code and translated into SW_TABLET_MODE input events:

    Scope (_SB.PC00.LPCB)
    {
...
        Device (EC)
        {
...
            Method (_Q84, 0, NotSerialized)  // _Qxx: EC Query, xx=0x00-0xFF
            {
                SCIC = 0x84
                DBG8 = 0x84
                If ((IUCE == One))
                {
                    If (MYEC)
                    {
                        CONV = ISHS /* \_SB_.PC00.LPCB.EC__.ISHS */
                        If (((OSYS >= 0x07DF) && ^^^^HIDD.BTLD))
                        {
                            If ((CONV == 0x03))
                            {
                                UPBT (0x06, Zero)
                                Notify (^^^^HIDD, 0xCC) // Hardware-Specific
                            }
                            ElseIf ((CONV == One))
                            {
                                UPBT (0x06, One)
                                Notify (^^^^HIDD, 0xCD) // Hardware-Specific
                            }
                        }
                        ElseIf (CondRefOf (\_SB.PC00.GFX0.IUEH))
                        {
                            ^^^GFX0.IUEH (0x06)
                        }
                    }
                    Else
                    {
                        UPBT (0x06, One)
                        Notify (^^^^HIDD, 0xCD) // Hardware-Specific
                    }

                    Notify (LID0, 0x80) // Status Change
                }
                Else
                {
                    Notify (LID0, 0x80) // Status Change
                }
            }

But it seems that for some reason the EC is never sending 0x84 events, so this code never gets executed by the kernel's ACPI interpreter and we never get the 0xCC / 0xCD notifies on the ACPI HIDD device.

@dmitry-s93 since you have quite a bit of experience with the MSI EC on these devices, I wonder if you have any insights in this?

[rottenpants466]

The user was me Hans 👍 (You got the hotkeys working btw! this will surely help other people too!!)

If i may, a while ago i also tried getting some help with other 2 people in order to solve this issue and others (battery thresholds):

@ThePBone https://github.com/ThePBone/msi-ec-modern
@BeardOverflow https://github.com/BeardOverflow/msi-ec (Original MSI ec author, not as active)

This may be useful.

In the meantime i have been using MControlCenter, all the features work in my laptop! Kudos to dmitry!

Edit: this may be useful too: @musikid https://github.com/musikid/acpi_ec

[rottenpants466]

@jwrdegoede @teackot

So im still trying to find a way to fix this issue, i managed to install windows 11 on a separate ssd and i'm using this tool: https://github.com/ThePBone/MsiEcRamEditor

Im trying to see if i can find anything in the right column when i flip the screen but when then list updates it usually shows 3 or 4 new entries so i don't know which corresponds with the screen flip.

[timschneeb]

Im trying to see if i can find anything in the right column when i flip the screen but when then list updates it usually shows 3 or 4 new entries so i don't know which corresponds with the screen flip.

@rottenpants466 That's probably caused by the real-time CPU & GPU info that is updated constantly in EC memory and litters the whole log on the right side.

You can click on 'Log options > Exclude addresses' in the program and enter those addresses containing the real-time data to prevent the log from getting spammed with irrelevant data.

[teackot]

I can say for sure that those that update even when you don't flip your screen are not what we're looking for

E.g. 0x68 is CPU temp

[rottenpants466]

Thanks for responding.

So i noticed 2 new adresses that only appear when i flip the screen back and forth:

0xED
0xEE

EE always appears when i flip the screen, ED most of the time

[teackot]

They're subsequent so I think they may be related in some way

Looking at this picture I see that 0xEE had its bit1 flipped and 0xED has 2 flipped bits

ED most of the time

Can it be a rotation sensor? Try rotating your laptop without flipping the screen (edit: it isn't a rotation sensor)

[rottenpants466]

Rotation sensor works fine in Linux

Tried a couple of times ED only shows up like 80% of the time. EE always shows up. In the picture above the first ED/EE entries in red appeared when i flipped to tablet mode, the second ED/EE entries in red when i flipped back to "normal" mode

[teackot]

Hans wrote:

But it seems that for some reason the EC is never sending 0x84 event

He mentioned some 0x84 event and in the picture 0xED is getting set to 0x84 (in the second red pair)

In this issue in my EC table repo we figured out that 0xED gets changed when some sort of events are emitted (e.g. brightness up or down, USB-C connection)

Tried a couple of times ED only shows up like 80% of the time

I think 0xED is set to 0x84 when the screen is flipped. It doesn't show up all the time because the right column shows changes, so 0xED needs to first be set to some other value by another event for it to show up there

[teackot]

If ((CONV == 0x03))
{
UPBT (0x06, Zero)
Notify (^^^^HIDD, 0xCC) // Hardware-Specific
}
ElseIf ((CONV == One))
{
UPBT (0x06, One)
Notify (^^^^HIDD, 0xCD) // Hardware-Specific
}

CONV is compared to 0x03 and One (which I suppose means 0x01)

EC's 0xEE address is set to 0x03 and 0x01 when screen is flipped back and forth

[teackot]

@rottenpants466 can you please check if 0xED and/or 0xEE are changed in Linux when flipping the screen?

Specifically, you need to look for 0x84 in [0xED] and 0x01/0x03 in [0xEE]

[rottenpants466]

@rottenpants466 can you please check if 0xED and/or 0xEE are changed in Linux when flipping the screen?

Specifically, you need to look for 0x84 in [0xED] and 0x01/0x03 in [0xEE]

Will check

Thank you 👍

[rottenpants466]

Also since i'm testing this stuff on windows do you want me to test something else regarding the ec drivers @teackot ?

cheer

[teackot]

Also since i'm testing this stuff on windows do you want me to test something else regarding the ec drivers

I think we got all of your values on Linux, though you can test a couple more things: check if switching mute LED triggers the [0x2C] and if mic mute triggers [0x2B]

Also switching webcam should trigger [0x2E]