lkml.org 
[lkml]   [2020]   [Dec]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.10 344/717] platform/x86: intel-vbtn: Fix SW_TABLET_MODE always reporting 1 on some HP x360 models
    Date
    From: Hans de Goede <hdegoede@redhat.com>

    [ Upstream commit a4327979a19e8734ddefbd8bcbb73bd9905b69cd ]

    Some HP x360 models have an ACPI VGBS method which sets bit 4 instead of
    bit 6 when NOT in tablet mode at boot. Inspecting all the DSDTs in my DSDT
    collection shows only one other model, the Medion E1239T ever setting bit 4
    and it always sets this together with bit 6.

    So lets treat bit 4 as a second bit which when set indicates the device not
    being in tablet-mode, as we already do for bit 6.

    While at it also prefix all VGBS constant defines with "VGBS_".

    Note this wrokaround was first added to the kernel as
    commit d823346876a9 ("platform/x86: intel-vbtn: Fix SW_TABLET_MODE always
    reporting 1 on the HP Pavilion 11 x360").
    After commit 8169bd3e6e19 ("platform/x86: intel-vbtn: Switch to an
    allow-list for SW_TABLET_MODE reporting") got added to the kernel this
    was reverted, because with the new allow-list approach the workaround
    was no longer necessary for the model on which the issue was first
    reported.

    But it turns out that the workaround is still necessary because some
    affected models report a chassis-type of 31 which is on the allow-list.

    BugLink: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1894017
    Fixes: 21d64817c724 ("platform/x86: intel-vbtn: Revert "Fix SW_TABLET_MODE always reporting 1 on the HP Pavilion 11 x360"")
    Signed-off-by: Hans de Goede <hdegoede@redhat.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/platform/x86/intel-vbtn.c | 12 ++++++++----
    1 file changed, 8 insertions(+), 4 deletions(-)

    diff --git a/drivers/platform/x86/intel-vbtn.c b/drivers/platform/x86/intel-vbtn.c
    index 0419c8001fe33..123401f04b55a 100644
    --- a/drivers/platform/x86/intel-vbtn.c
    +++ b/drivers/platform/x86/intel-vbtn.c
    @@ -15,9 +15,13 @@
    #include <linux/platform_device.h>
    #include <linux/suspend.h>

    +/* Returned when NOT in tablet mode on some HP Stream x360 11 models */
    +#define VGBS_TABLET_MODE_FLAG_ALT 0x10
    /* When NOT in tablet mode, VGBS returns with the flag 0x40 */
    -#define TABLET_MODE_FLAG 0x40
    -#define DOCK_MODE_FLAG 0x80
    +#define VGBS_TABLET_MODE_FLAG 0x40
    +#define VGBS_DOCK_MODE_FLAG 0x80
    +
    +#define VGBS_TABLET_MODE_FLAGS (VGBS_TABLET_MODE_FLAG | VGBS_TABLET_MODE_FLAG_ALT)

    MODULE_LICENSE("GPL");
    MODULE_AUTHOR("AceLan Kao");
    @@ -72,9 +76,9 @@ static void detect_tablet_mode(struct platform_device *device)
    if (ACPI_FAILURE(status))
    return;

    - m = !(vgbs & TABLET_MODE_FLAG);
    + m = !(vgbs & VGBS_TABLET_MODE_FLAGS);
    input_report_switch(priv->input_dev, SW_TABLET_MODE, m);
    - m = (vgbs & DOCK_MODE_FLAG) ? 1 : 0;
    + m = (vgbs & VGBS_DOCK_MODE_FLAG) ? 1 : 0;
    input_report_switch(priv->input_dev, SW_DOCK, m);
    }

    --
    2.27.0


    \
     
     \ /
      Last update: 2020-12-28 15:19    [W:3.285 / U:0.244 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site