From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932494AbeA2U1x (ORCPT ); Mon, 29 Jan 2018 15:27:53 -0500 Received: from mail-wr0-f195.google.com ([209.85.128.195]:39418 "EHLO mail-wr0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754481AbeA2U1t (ORCPT ); Mon, 29 Jan 2018 15:27:49 -0500 X-Google-Smtp-Source: AH8x227EVU0+T4EnI0oDId8hWMRTw+9e+MBjZEche+CW7Gcw0Rd4ef9dnuRFeMSqu4g2maNwZprCMQ== From: Marco Martin To: platform-driver-x86@vger.kernel.org Cc: mario_limonciello@dell.com, andy@infradead.org, pali.rohar@gmail.com, dvhart@infradead.org, mjg59@srcf.ucam.org, linux-kernel@vger.kernel.org, bhush94@gmail.com, notmart@gmail.com, =?UTF-8?q?Stefan=20Br=C3=BCns?= Subject: [PATCH] Support intel-vbtn based tablet mode switch Date: Mon, 29 Jan 2018 21:27:43 +0100 Message-Id: <1517257663-2171-1-git-send-email-notmart@gmail.com> X-Mailer: git-send-email 2.7.4 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On some laptop like the Dell Inspiron 7000 series tablet mode switch implemented in Intel ACPI, the events to enter and exit the tablet mode are 0xCC and 0xCD This initializes the tablet/laptop mode at the correct value if the system booted in tablet mode (or the vbtn module loaded with the device in tablet mode) CC: platform-driver-x86@vger.kernel.org CC: Matthew Garrett CC: "Pali Rohár" CC: Darren Hart CC: Mario Limonciello CC: Andy Shevchenko CC: Stefan Brüns Signed-off-by: Marco Martin Reviewed-by: Mario Limonciello Acked-by: Pali Rohár --- drivers/platform/x86/intel-vbtn.c | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+) diff --git a/drivers/platform/x86/intel-vbtn.c b/drivers/platform/x86/intel-vbtn.c index 5fc4315..019ff21 100644 --- a/drivers/platform/x86/intel-vbtn.c +++ b/drivers/platform/x86/intel-vbtn.c @@ -26,6 +26,9 @@ #include #include +/* When NOT in tablet mode, VGBS returns with the flag 0x40 */ +#define TABLET_MODE_FLAG 0x40 + MODULE_LICENSE("GPL"); MODULE_AUTHOR("AceLan Kao"); @@ -108,6 +111,7 @@ static void notify_handler(acpi_handle handle, u32 event, void *context) static int intel_vbtn_probe(struct platform_device *device) { + struct acpi_buffer vgbs_output = { ACPI_ALLOCATE_BUFFER, NULL }; acpi_handle handle = ACPI_HANDLE(&device->dev); struct intel_vbtn_priv *priv; acpi_status status; @@ -130,6 +134,22 @@ static int intel_vbtn_probe(struct platform_device *device) return err; } + status = acpi_evaluate_object(handle, "VGBS", NULL, &vgbs_output); + /* VGBS being present and returning something means + * we have a tablet mode switch + */ + if (ACPI_SUCCESS(status)) { + union acpi_object *obj = vgbs_output.pointer; + + if (obj && obj->type == ACPI_TYPE_INTEGER) { + input_report_switch(priv->input_dev, + SW_TABLET_MODE, + !(obj->integer.value & TABLET_MODE_FLAG)); + } + } + + kfree(vgbs_output.pointer); + status = acpi_install_notify_handler(handle, ACPI_DEVICE_NOTIFY, notify_handler, -- 2.7.4