From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-15.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2AF52C4320A for ; Tue, 10 Aug 2021 23:41:46 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 0F38A60EC0 for ; Tue, 10 Aug 2021 23:41:46 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235511AbhHJXmH (ORCPT ); Tue, 10 Aug 2021 19:42:07 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50856 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235242AbhHJXmG (ORCPT ); Tue, 10 Aug 2021 19:42:06 -0400 Received: from mail-wm1-x334.google.com (mail-wm1-x334.google.com [IPv6:2a00:1450:4864:20::334]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9D909C061765; Tue, 10 Aug 2021 16:41:43 -0700 (PDT) Received: by mail-wm1-x334.google.com with SMTP id o7-20020a05600c5107b0290257f956e02dso3095895wms.1; Tue, 10 Aug 2021 16:41:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=ukdhrGWWWaCyt50i/Wjrb+ecEVA10wbif+kzGW2VjsY=; b=Alx17CE/c8Aht+d4SNThpEE+szQQQ2Xi44XWtqZ635FPACfFZvzQFNuPwM8qoiiG60 MnCAsthmNz3RjFPe26xq1EmONEOOkvyPoOw8iiEIrDPie+tEwTPewnLrOMeSEDLtiFYk g/Jf6dPlYqL8YP7b++Ho+yJUgxyxgB+LTA13DOaWDed0yWmxUUImXPFlfey/FXWEfx3d Ir1DxjA/d7yYwe1CG9NQ/Lb1txGi5sbBfZ9apSx8TDE0y4JGo2Bu1ZGKt4bxhb6qhZt6 Pg9EhUhGa7olzlHnnRbX/a6FEnJtsR6LVraXyxednaIB+kAfcG/W3xvcRfhOEuLYK8fi Vh7Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=ukdhrGWWWaCyt50i/Wjrb+ecEVA10wbif+kzGW2VjsY=; b=gmhkctJgUZPqfGBD5UyHJDkInUJGyrmOLgy289k7SHJ3PZuv4bdQpv1i0PD7d0JJcg AipmhqhG0dtHOXhOnDfBoA6lYuOhoC/1e6mt1gBsqL0iCVJzdoT4URJlXmKhqxJ8wTZL Xg3Go1SYRa9vvYIfWF0clUYDYk+mUckZXS4b8oa9kiW3A9VjgMdLkI8soXYTkahiUjLO 05252Z/AnwxWnERkYbktD7oIxUpBJk0WlyyXXqmtJoZ+iPhIgg8Zs+AoSXX1xQD5Zt0p zatPxvStynTV5hN5BSr0CHHVrqiS/W6o8SAz/QBsxBLaUjmMcflYrKJKNb/ialAxZkPz hvfQ== X-Gm-Message-State: AOAM531UOLvMIU3Ivtg2/dMZ9NigBxin+MYbaxSuKnbojQmSUNBnhTu2 j6Yt6xhs/Le08RxtT7KhAgErG+9/2YR/Jh8qbBK0TA== X-Google-Smtp-Source: ABdhPJwEArkxbJsuGqYM55ik2vWOdSncVYsgvgpSVSdvLnaR0G1VR72oWsIPo45GFnGerzvuWQvaug== X-Received: by 2002:a7b:c7d1:: with SMTP id z17mr6989686wmk.50.1628638902201; Tue, 10 Aug 2021 16:41:42 -0700 (PDT) Received: from Blackbeard.. (static.214.212.76.144.clients.your-server.de. [144.76.212.214]) by smtp.gmail.com with ESMTPSA id u6sm10166427wrp.83.2021.08.10.16.41.40 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 10 Aug 2021 16:41:41 -0700 (PDT) From: JafarAkhondali To: Hans de Goede , jlee@suse.com, linux-kernel@vger.kernel.org, platform-driver-x86@vger.kernel.org, mgross@linux.intel.com Cc: JafarAkhondali Subject: [PATCH v4] platform/x86: acer-wmi: Add Turbo Mode support for Acer PH315-53 Date: Wed, 11 Aug 2021 04:09:28 +0430 Message-Id: <20210810233928.1389320-1-jafar.akhoondali@gmail.com> X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, The Acer Predator Helios series (usually denoted by PHxxx-yy) features a special key above the keyboard named "TURBO". The turbo key does 3 things: 1. Set all fan's speeds to TURBO mode 2. Overclocks the CPU and GPU in the safe range 3. Turn on an LED just below the turbo button All of these actions are done by WMI function calls, and there is no custom OC level for turbo. It acts as a flag for enabling turbo mode instead of telling processors to use 1.3x of power. I've run some benchmark tests and it worked fine: GpuTest 0.7.0 http://www.geeks3d.com Module: FurMark Normal mode Score: 7289 points (FPS: 121) Turbo mode Score: 7675 points (FPS: 127) Settings: - 1920x1080 fullscreen - antialiasing: Off - duration: 60000 ms Renderer: - GeForce RTX 2060/PCIe/SSE2 - OpenGL: 4.6.0 NVIDIA 460.32.03 This feature is presented by Acer officially and should not harm hardware in any case. A challenging part of implementing this feature is that calling overclocking the function requires knowing the exact count of fans for CPU and GPU for each model, which to the best of my knowledge is not available in the kernel. So after checking the official PredatorSense application methods, it turned out they have provided the software the list of fans in each model. I have access to the mentioned list, and all similar PH-iii-jj can be added easily by matching "DMI_PRODUCT_NAME". Creating a separate file for the gaming interface was not possible because the current WMI event GUID is needed for the turbo button, and it's not possible to register multiple functions on the same event GUID. Signed-off-by: JafarAkhondali --- Changes in v4: - Fix Indents - Make functions return early Changes in v3: - Remove usages of gaming_interface - Add ACPI output for u32 buffer length - Remove set_u64 and get_u64 functions - Remove unrelated whitespace changes for to this patch Changes in v2: - Fix formatting problems drivers/platform/x86/acer-wmi.c | 183 ++++++++++++++++++++++++++++++++ 1 file changed, 183 insertions(+) diff --git a/drivers/platform/x86/acer-wmi.c b/drivers/platform/x86/acer-wmi.c index 85db9403cc14..183faa6db4b2 100644 --- a/drivers/platform/x86/acer-wmi.c +++ b/drivers/platform/x86/acer-wmi.c @@ -60,6 +60,11 @@ MODULE_LICENSE("GPL"); #define ACER_WMID_GET_THREEG_METHODID 10 #define ACER_WMID_SET_THREEG_METHODID 11 +#define ACER_WMID_SET_GAMING_LED_METHODID 2 +#define ACER_WMID_GET_GAMING_LED_METHODID 4 +#define ACER_WMID_SET_GAMING_FAN_BEHAVIOR 14 +#define ACER_WMID_SET_GAMING_MISC_SETTING_METHODID 22 + /* * Acer ACPI method GUIDs */ @@ -68,6 +73,7 @@ MODULE_LICENSE("GPL"); #define WMID_GUID1 "6AF4F258-B401-42FD-BE91-3D4AC2D7C0D3" #define WMID_GUID2 "95764E09-FB56-4E83-B31A-37761F60994A" #define WMID_GUID3 "61EF69EA-865C-4BC3-A502-A0DEBA0CB531" +#define WMID_GUID4 "7A4DDFE7-5B5D-40B4-8595-4408E0CC7F56" /* * Acer ACPI event GUIDs @@ -81,6 +87,7 @@ MODULE_ALIAS("wmi:676AA15E-6A47-4D9F-A2CC-1E6D18D14026"); enum acer_wmi_event_ids { WMID_HOTKEY_EVENT = 0x1, WMID_ACCEL_OR_KBD_DOCK_EVENT = 0x5, + WMID_GAMING_TURBO_KEY_EVENT = 0x7, }; static const struct key_entry acer_wmi_keymap[] __initconst = { @@ -215,6 +222,9 @@ struct hotkey_function_type_aa { #define ACER_CAP_THREEG BIT(4) #define ACER_CAP_SET_FUNCTION_MODE BIT(5) #define ACER_CAP_KBD_DOCK BIT(6) +#define ACER_CAP_TURBO_OC BIT(7) +#define ACER_CAP_TURBO_LED BIT(8) +#define ACER_CAP_TURBO_FAN BIT(9) /* * Interface type flags @@ -301,6 +311,9 @@ struct quirk_entry { u8 mailled; s8 brightness; u8 bluetooth; + u8 turbo; + u8 cpu_fans; + u8 gpu_fans; }; static struct quirk_entry *quirks; @@ -312,6 +325,10 @@ static void __init set_quirks(void) if (quirks->brightness) interface->capability |= ACER_CAP_BRIGHTNESS; + + if (quirks->turbo) + interface->capability |= ACER_CAP_TURBO_OC | ACER_CAP_TURBO_LED + | ACER_CAP_TURBO_FAN; } static int __init dmi_matched(const struct dmi_system_id *dmi) @@ -340,6 +357,12 @@ static struct quirk_entry quirk_acer_travelmate_2490 = { .mailled = 1, }; +static struct quirk_entry quirk_acer_predator_ph315_53 = { + .turbo = 1, + .cpu_fans = 1, + .gpu_fans = 1, +}; + /* This AMW0 laptop has no bluetooth */ static struct quirk_entry quirk_medion_md_98300 = { .wireless = 1, @@ -507,6 +530,15 @@ static const struct dmi_system_id acer_quirks[] __initconst = { }, .driver_data = &quirk_acer_travelmate_2490, }, + { + .callback = dmi_matched, + .ident = "Acer Predator PH315-53", + .matches = { + DMI_MATCH(DMI_SYS_VENDOR, "Acer"), + DMI_MATCH(DMI_PRODUCT_NAME, "Predator PH315-53"), + }, + .driver_data = &quirk_acer_predator_ph315_53, + }, { .callback = set_force_caps, .ident = "Acer Aspire Switch 10E SW3-016", @@ -1344,6 +1376,93 @@ static struct wmi_interface wmid_v2_interface = { .type = ACER_WMID_v2, }; +/* + * WMID Gaming interface + */ + +static acpi_status +WMI_gaming_execute_u64(u32 method_id, u64 in, u64 *out) +{ + struct acpi_buffer input = { (acpi_size) sizeof(u64), (void *)(&in) }; + struct acpi_buffer result = { ACPI_ALLOCATE_BUFFER, NULL }; + union acpi_object *obj; + u32 tmp = 0; + acpi_status status; + + status = wmi_evaluate_method(WMID_GUID4, 0, method_id, &input, &result); + + if (ACPI_FAILURE(status)) + return status; + obj = (union acpi_object *) result.pointer; + + if (obj) { + if (obj->type == ACPI_TYPE_BUFFER) { + if (obj->buffer.length == sizeof(u32)) + tmp = *((u32 *) obj->buffer.pointer); + else if (obj->buffer.length == sizeof(u64)) + tmp = *((u64 *) obj->buffer.pointer); + } else if (obj->type == ACPI_TYPE_INTEGER) { + tmp = (u64) obj->integer.value; + } + } + + if (out) + *out = tmp; + + kfree(result.pointer); + + return status; +} + +static acpi_status WMID_gaming_set_u64(u64 value, u32 cap) +{ + u32 method_id = 0; + + if (!(interface->capability & cap)) + return AE_BAD_PARAMETER; + + switch (cap) { + case ACER_CAP_TURBO_LED: + method_id = ACER_WMID_SET_GAMING_LED_METHODID; + break; + case ACER_CAP_TURBO_FAN: + method_id = ACER_WMID_SET_GAMING_FAN_BEHAVIOR; + break; + case ACER_CAP_TURBO_OC: + method_id = ACER_WMID_SET_GAMING_MISC_SETTING_METHODID; + break; + default: + return AE_BAD_PARAMETER; + } + + return WMI_gaming_execute_u64(method_id, value, NULL); +} + +static acpi_status WMID_gaming_get_u64(u64 *value, u32 cap) +{ + acpi_status status; + u64 result; + u64 input; + u32 method_id; + + if (!(interface->capability & cap)) + return AE_BAD_PARAMETER; + + switch (cap) { + case ACER_CAP_TURBO_LED: + method_id = ACER_WMID_GET_GAMING_LED_METHODID; + input = 0x1; + break; + default: + return AE_BAD_PARAMETER; + } + status = WMI_gaming_execute_u64(method_id, input, &result); + if (ACPI_SUCCESS(status)) + *value = (u64) result; + + return status; +} + /* * Generic Device (interface-independent) */ @@ -1575,6 +1694,66 @@ static int acer_gsensor_event(void) return 0; } +/* + * Predator series turbo button + */ +static int acer_toggle_turbo(void) +{ + /* Get current state from turbo button */ + u64 turbo_led_state, gpu_fan_config1, gpu_fan_config2; + u8 i; + + if (ACPI_FAILURE(WMID_gaming_get_u64(&turbo_led_state, ACER_CAP_TURBO_LED))) + return -1; + + if (turbo_led_state) { + // turns off turbo led + WMID_gaming_set_u64(0x1, ACER_CAP_TURBO_LED); + + // set FAN mode to auto + if (quirks->cpu_fans > 0) + gpu_fan_config2 |= 1; + for (i = 0; i < (quirks->cpu_fans + quirks->gpu_fans); ++i) + gpu_fan_config2 |= 1 << (i + 1); + for (i = 0; i < quirks->gpu_fans; ++i) + gpu_fan_config2 |= 1 << (i + 3); + if (quirks->cpu_fans > 0) + gpu_fan_config1 |= 1; + for (i = 0; i < (quirks->cpu_fans + quirks->gpu_fans); ++i) + gpu_fan_config1 |= 1 << (2 * i + 2); + for (i = 0; i < quirks->gpu_fans; ++i) + gpu_fan_config1 |= 1 << (2 * i + 6); + WMID_gaming_set_u64(gpu_fan_config2 | gpu_fan_config1 << 16, ACER_CAP_TURBO_FAN); + + // set OC to normal + WMID_gaming_set_u64(0x5, ACER_CAP_TURBO_OC); + WMID_gaming_set_u64(0x7, ACER_CAP_TURBO_OC); + } else { + // turn on turbo led + WMID_gaming_set_u64(0x10001, ACER_CAP_TURBO_LED); + + // set FAN to turbo mode + if (quirks->cpu_fans > 0) + gpu_fan_config2 |= 1; + for (i = 0; i < (quirks->cpu_fans + quirks->gpu_fans); ++i) + gpu_fan_config2 |= 1 << (i + 1); + for (i = 0; i < quirks->gpu_fans; ++i) + gpu_fan_config2 |= 1 << (i + 3); + if (quirks->cpu_fans > 0) + gpu_fan_config1 |= 2; + for (i = 0; i < (quirks->cpu_fans + quirks->gpu_fans); ++i) + gpu_fan_config1 |= 2 << (2 * i + 2); + for (i = 0; i < quirks->gpu_fans; ++i) + gpu_fan_config1 |= 2 << (2 * i + 6); + WMID_gaming_set_u64(gpu_fan_config2 | gpu_fan_config1 << 16, ACER_CAP_TURBO_FAN); + + // set OC to turbo mode + WMID_gaming_set_u64(0x205, ACER_CAP_TURBO_OC); + WMID_gaming_set_u64(0x207, ACER_CAP_TURBO_OC); + } + return turbo_led_state; +} + /* * Switch series keyboard dock status */ @@ -1872,6 +2051,10 @@ static void acer_wmi_notify(u32 value, void *context) acer_gsensor_event(); acer_kbd_dock_event(&return_value); break; + case WMID_GAMING_TURBO_KEY_EVENT: + if (return_value.key_num == 0x4) + acer_toggle_turbo(); + break; default: pr_warn("Unknown function number - %d - %d\n", return_value.function, return_value.key_num); -- 2.30.2