From patchwork Mon Jan 29 16:36:21 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Szilard Fabian X-Patchwork-Id: 193588 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7301:2087:b0:106:209c:c626 with SMTP id gs7csp688305dyb; Mon, 29 Jan 2024 08:41:24 -0800 (PST) X-Google-Smtp-Source: AGHT+IETQc86RcOk+KbBB7F+7n3wmnasedDRxqmCElULL62486cf3EVvGHdAZ8u1oAJQgC/dtUpJ X-Received: by 2002:a05:620a:1001:b0:783:9022:b345 with SMTP id z1-20020a05620a100100b007839022b345mr4677876qkj.27.1706546484384; Mon, 29 Jan 2024 08:41:24 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1706546484; cv=pass; d=google.com; s=arc-20160816; b=C79shB+7EaixK9cUhxqzISxiFrDUrwT3IVYIev0xMOr4xnTH/kd2guqJWMHMp+uvBW x7dRqsgPeUSHKNLSxt2btGsDvdPhAwaeAuikwiwXe0cpw9UvK3Pmzw1tRQ6pcuBUKNCZ xSGTRBF32LsfXzRuYvJ2eDSL/0IgJU+cTnYK8dOjgHaluRc55cYQRBi5RSuvRxQYLAsf 9yIP8avgbJNDtxBRYMzvvKW+DX0c/LzhO8FNw3+kW8vR5dGSguePCt/oWtchiuMrx93z /t1ECkrC7eRYQk5rflqLzOu7FgDw5R7Jxc7opkNc6AXT9FvRZO4a4ybJA0ycngqgURHR 75Wg== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:feedback-id:message-id:subject:cc :from:to:date:dkim-signature; bh=Pz5S2tybGQXBc/sGlP5HtlTloY+yieTdkiBNNBw2Rds=; fh=wyr5kd+OHhec+fSqlVS0QwHY6kapvvv7KAguD+qAyuw=; b=KOQwOB/tK8x52wWunmgu1wRT5UAvw9053v2fHfMCMZ9FBYbKl91M+2Hn7wmnbalTcT gFUnx+Jzrwhl3bgSugwJuItMVUTdLLzXmja311f2t2MaeaY7oTMdzyvDRN9tyl8PV+WZ 23oSL6gOh0M8/ySJVnQGcB7BG4jgICInoVKC1YYl8f7gkiJN2ZnMD48S0AuEPVhMGj9J /33PXLtH3+7mHPi9h46xSwSaWY2QxV8HDkgGC2lQl8mcVj650b2P9q84KRn/9EGMhHKD p7J6IsGZ9EQok0/6r3+koeCbrbCbZCMhTOWm788gWxvYwlehZWD+dl55CYQy9P+XKIOd zEPw== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@bluemarch.art header.s=protonmail header.b=mdH6IaVD; arc=pass (i=1 spf=pass spfdomain=bluemarch.art dkim=pass dkdomain=bluemarch.art dmarc=pass fromdomain=bluemarch.art); spf=pass (google.com: domain of linux-kernel+bounces-43127-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-43127-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=bluemarch.art Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [147.75.199.223]) by mx.google.com with ESMTPS id b22-20020a05620a271600b00783f60700e5si4162688qkp.199.2024.01.29.08.41.24 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 29 Jan 2024 08:41:24 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-43127-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223; Authentication-Results: mx.google.com; dkim=pass header.i=@bluemarch.art header.s=protonmail header.b=mdH6IaVD; arc=pass (i=1 spf=pass spfdomain=bluemarch.art dkim=pass dkdomain=bluemarch.art dmarc=pass fromdomain=bluemarch.art); spf=pass (google.com: domain of linux-kernel+bounces-43127-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-43127-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=bluemarch.art Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 309D81C20473 for ; Mon, 29 Jan 2024 16:41:24 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id D08DF15A4AD; Mon, 29 Jan 2024 16:36:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bluemarch.art header.i=@bluemarch.art header.b="mdH6IaVD" Received: from mail-4323.proton.ch (mail-4323.proton.ch [185.70.43.23]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1C24E15A493; Mon, 29 Jan 2024 16:36:37 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.70.43.23 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706546202; cv=none; b=Y2lgSsLxp4gOdJGMcgfP0FaqQFuEjzFNYTa4f0FvEPlhcNhj5x10WBBaHWfsLPyC6Kq1CwQA1z+v8YgqKZRMHQKUKJMofcwPKo5RPqQ+caLsjlVI5v88ofieojq5rZ444kHglWr0FAXV6XPT8Ilypr2HaqJcAuaTb6h4jtdPJQA= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706546202; c=relaxed/simple; bh=l8roFfgwgVf/q6EsSpxj+S5HWm0zPj8ehUUy6GtK4j8=; h=Date:To:From:Cc:Subject:Message-ID:MIME-Version:Content-Type; b=iIR/VFbakn1ac71nQ2P5TpfJPMMQPVcUvRk/GpaiU/p0CaELi2gr4fkyV/Cnq87qH2VMlQ8JicrsT3arFPK3Go18OrFkwJZt4c80ZGR4c5NpYEKeFOilQNvqiOvr7E2JRvJgw9Q6ad6QfseRE0pdl0A0FEL5gl+FASZRzJwFq0c= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=bluemarch.art; spf=pass smtp.mailfrom=bluemarch.art; dkim=pass (2048-bit key) header.d=bluemarch.art header.i=@bluemarch.art header.b=mdH6IaVD; arc=none smtp.client-ip=185.70.43.23 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=bluemarch.art Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bluemarch.art DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bluemarch.art; s=protonmail; t=1706546188; x=1706805388; bh=Pz5S2tybGQXBc/sGlP5HtlTloY+yieTdkiBNNBw2Rds=; h=Date:To:From:Cc:Subject:Message-ID:Feedback-ID:From:To:Cc:Date: Subject:Reply-To:Feedback-ID:Message-ID:BIMI-Selector; b=mdH6IaVDjHg+RLsD1E8Ka3HL8wHZLvcxukkgQN3VQJwTKbXQ/HQ1WwsOKapXWCEh0 1BwJzVPCYBtyHtcgcTWj5VwHPC8Mkdi0vyDxx9Kg6fbp3GHFFWEcAsAqp2DrSLuX/q dtCA7C8dzMQ3fD1aToy6dfrUhJeMLXvusWGGp9RlyuGEJrd/O4ENN7fSAk+Bvmvnkq iE+PUiqEbQH/Qoek41mqsPZ0yBCDaLEBIi90crJkRJOdDwcjrulNmfzoSMP1ZJ2ldh hrfEVwRw+g31JeJTDTrQmwUaw7bOV3sKrUzxCVEXfp305dV3rtYMkO5oTQ9lZUazvJ ugwzxn9syRALg== Date: Mon, 29 Jan 2024 16:36:21 +0000 To: linux-kernel@vger.kernel.org, platform-driver-x86@vger.kernel.org, jwoithe@just42.net, hdegoede@redhat.com, ilpo.jarvinen@linux.intel.com From: Szilard Fabian Cc: Szilard Fabian Subject: [RFC PATCH] platform/x86/fujitsu-laptop: Add battery charge control support Message-ID: <20240129163502.161409-2-szfabian@bluemarch.art> Feedback-ID: 87830438:user:proton Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1789443686763452094 X-GMAIL-MSGID: 1789443686763452094 This patch adds battery charge control support on Fujitsu notebooks via the S006 method of the FUJ02E3 ACPI device. With this method it's possible to set charge_control_end_threshold between 50 and 100%. Tested on Lifebook E5411 and Lifebook U728. Sadly I can't test this patch on a dual battery one, but I didn't find any clue about independent battery charge control on dual battery Fujitsu notebooks either. And by that I mean checking the DSDT table of various Lifebook notebooks and reverse engineering FUJ02E3.dll. --- drivers/platform/x86/fujitsu-laptop.c | 95 +++++++++++++++++++++++++++ 1 file changed, 95 insertions(+) diff --git a/drivers/platform/x86/fujitsu-laptop.c b/drivers/platform/x86/fujitsu-laptop.c index 085e044e888e..bf3df74e4d63 100644 --- a/drivers/platform/x86/fujitsu-laptop.c +++ b/drivers/platform/x86/fujitsu-laptop.c @@ -49,6 +49,8 @@ #include #include #include +#include +#include #include #define FUJITSU_DRIVER_VERSION "0.6.0" @@ -97,6 +99,10 @@ #define BACKLIGHT_OFF (BIT(0) | BIT(1)) #define BACKLIGHT_ON 0 +/* FUNC interface - battery control interface */ +#define FUNC_S006_METHOD 0x1006 +#define CHARGE_CONTROL_RW 0x21 + /* Scancodes read from the GIRB register */ #define KEY1_CODE 0x410 #define KEY2_CODE 0x411 @@ -164,6 +170,91 @@ static int call_fext_func(struct acpi_device *device, return value; } +/* Battery charge control code */ + +static ssize_t charge_control_end_threshold_store(struct device *dev, + struct device_attribute *attr, + const char *buf, size_t count) +{ + int value, ret; + + ret = kstrtouint(buf, 10, &value); + if (ret) + return ret; + + if (value < 50 || value > 100) + return -EINVAL; + + int cc_end_value, s006_cc_return; + + cc_end_value = value * 0x100 + 0x20; + s006_cc_return = call_fext_func(fext, FUNC_S006_METHOD, + CHARGE_CONTROL_RW, cc_end_value, 0x0); + + /* + * The S006 0x21 method returns 0x00 in case the provided value + * is invalid. + */ + if (s006_cc_return == 0x00) + return -EINVAL; + + return count; +} + +static ssize_t charge_control_end_threshold_show(struct device *dev, + struct device_attribute *attr, + char *buf) +{ + int status; + status = call_fext_func(fext, FUNC_S006_METHOD, + CHARGE_CONTROL_RW, 0x21, 0x0); + + return sprintf(buf, "%d\n", status); +} + +static DEVICE_ATTR_RW(charge_control_end_threshold); + +/* ACPI battery hook */ + +static int fujitsu_battery_add(struct power_supply *battery, + struct acpi_battery_hook *hook) +{ + /* Check if there is an existing FUJ02E3 ACPI device. */ + if (fext == NULL) + return -ENODEV; + + /* + * Check if the S006 0x21 method exists by trying to get the current + * battery charge limit. + */ + int s006_cc_return; + s006_cc_return = call_fext_func(fext, FUNC_S006_METHOD, + CHARGE_CONTROL_RW, 0x21, 0x0); + if (s006_cc_return == UNSUPPORTED_CMD) + return -ENODEV; + + if (device_create_file(&battery->dev, + &dev_attr_charge_control_end_threshold)) + return -ENODEV; + + return 0; +} + +static int fujitsu_battery_remove(struct power_supply *battery, + struct acpi_battery_hook *hook) +{ + device_remove_file(&battery->dev, + &dev_attr_charge_control_end_threshold); + + return 0; +} + +static struct acpi_battery_hook battery_hook = { + .add_battery = fujitsu_battery_add, + .remove_battery = fujitsu_battery_remove, + .name = "Fujitsu Battery Extension", +}; + /* Hardware access for LCD brightness control */ static int set_lcd_level(struct acpi_device *device, int level) @@ -839,6 +930,8 @@ static int acpi_fujitsu_laptop_add(struct acpi_device *device) if (ret) goto err_free_fifo; + battery_hook_register(&battery_hook); + return 0; err_free_fifo: @@ -851,6 +944,8 @@ static void acpi_fujitsu_laptop_remove(struct acpi_device *device) { struct fujitsu_laptop *priv = acpi_driver_data(device); + battery_hook_unregister(&battery_hook); + fujitsu_laptop_platform_remove(device); kfifo_free(&priv->fifo);