From nobody Tue Dec 9 03:52:34 2025 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) (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 870A87081A for ; Thu, 13 Nov 2025 00:15:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.133.124 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762992923; cv=none; b=Y3Enu8aD1Gj6t/K6vsaWB5BDxlzmYc+Lj916GDe0ZjbgGJ7/5rVINc7AYDC0N3lI6K6KBSD1cCua5/8ZcFqj1sTQV2XcAfl+AJ5Ufsev2bbJ22grkOvIGne8WkXcU/RRYCHGNbwEAlMLGEVdMpXhCMxqsPtOpzzt2H7cHScASUE= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762992923; c=relaxed/simple; bh=iFZsqRH5eByoiejBJe2tCpiSPFSMdpwxoI77YoyNZXg=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=V2sWWjVHioya2K58b99rH7CIw5tL3+NfkT5pKtkm5mJEl0cqQzmh4apZGONsBh+EOCklRiN3Q8XoaQqzUOXG28VoCFFq3nqSHR4kTGcAoAvIVWbri9swjddJ22FA2lDQn7eUesnUQFeJi8FqWg9e2UWZ1eHSOvQelK+MddN110o= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=YaXiyMzl; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b=DAQSbhmH; arc=none smtp.client-ip=170.10.133.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="YaXiyMzl"; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b="DAQSbhmH" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1762992919; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=+jraoxNmzCzqfQSJ06TY8sylDjJVSc0Y8i9xrHNp7CU=; b=YaXiyMzlTwPqGMpgc5eG83ZCjewJrPDw6X6NugAiccE3WwtFhxfSVZYaxStf+RxSSDA0O1 zkY0LpIPkya3lYaQDD9f2zoOZXUkrxqPCa7LaIrAKD/dmEHnwZipmU3Cug4Bq+pKMZDdcI dEKggD8T12UvY4QtxnfXJ2Q6GU3PgBc= Received: from mail-wm1-f70.google.com (mail-wm1-f70.google.com [209.85.128.70]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-284-K5FGzC7BPHWhlzKfqFb0Rw-1; Wed, 12 Nov 2025 19:15:17 -0500 X-MC-Unique: K5FGzC7BPHWhlzKfqFb0Rw-1 X-Mimecast-MFC-AGG-ID: K5FGzC7BPHWhlzKfqFb0Rw_1762992917 Received: by mail-wm1-f70.google.com with SMTP id 5b1f17b1804b1-47788165c97so1545865e9.0 for ; Wed, 12 Nov 2025 16:15:17 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=google; t=1762992916; x=1763597716; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=+jraoxNmzCzqfQSJ06TY8sylDjJVSc0Y8i9xrHNp7CU=; b=DAQSbhmH4LFfl+xKyVqupVADAuiQCm5i9F0DH6OFP7I/TliDuFJX2EnCsG8UUXsY3Y VZSPC4q34Q9CDDeR8BfcEDMmKYXm4mp2kZACoa4Elnv8L5GfXS23b1Qqr0zX/FEum9YF chVRKzWj354uCTjY05w5ctSV8gY4ktqdpZSoSAu1ClXDEvXSZ4ymWSju59Ulkq/pRTv7 Yo2fv2quRqcyx6Q0KAFGfU9JfHdvIg1ZKTTuoAdT2Qvn/7K7FSJkUlspKNz59eZvYDCL PP83+AMPE+8HcFyAWkDUTJLQbd73KoXG0zYpk1z82d07LUn2o9KEzXtEC65Bu5/UmkcM tCIQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1762992917; x=1763597717; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=+jraoxNmzCzqfQSJ06TY8sylDjJVSc0Y8i9xrHNp7CU=; b=GTyeo22dXDoJ+prrTUuR8PrqvJISNg5VB8cS6dKgzINQpVCwRFa2LF+AxhySACnFhn YRx+/rXdAELJ1KegHwvyy/endeNA2W/uKmVk+4noMCacBpu8RZcKzm6BWVSaoHoYXdjx d4fEJfIQTEYfXStvwIWGmwpplv+YMStqefnTUpq5Zb+ncfSWsbnNyTszhXdZ2Tnyd62o UwC8zbZ0Uo6++vdlUm6ngWt8nNfsh7UXqy/M6e7zGxBOEaV72Ouj2vqq2BwMiQG09V9p dI+SYeJnK+FGHHaULTqNYjeZBteaifg+9t/h38ce+azp7BZFLdy0A5/aWLFXX1LnZrDB 3Ahg== X-Forwarded-Encrypted: i=1; AJvYcCXDouUmlaUUayrooSIKBRwQlVnDHtDOxzpqiDP15vHw1HPU1qK7upCfFfUkHS7Im+6p81+qqJrBgqscnBQ=@vger.kernel.org X-Gm-Message-State: AOJu0Yyai3xT674jRePhdzq7y9o4MLxLO2U9qhi3Ybg0onUDxv6hJdrg vKYfEpaBA7vjwzgmtfWVUmZn+vp1qP+4cD6imGhtzcQrIGRI20MvwOpaeHGKPfC7E4kmbY6AoQI yRR6HKBXXVwWh0/k+WaiWHfJ+YsM0xyVkT0hUidBlcBgidLEDiWCZA74LyHGoRo1SwQ== X-Gm-Gg: ASbGncuAw7ZA0LpYIpUvPHipuewe1bZqCBsXqJlYUU5deErGiuTk0/9mw99YMlFzeSP qJEfrEaUrRmESCxTl398Yp7opDj6prxl8TI+tmD/gunj+Is5BVQxnHjv4BMXGmOkUkcxB0qaaDx lmII3Te7b1uzLJoIVFQfl99KFy7Ccd/4NGPBK4SYMiZFn4BFje5DAVAqCLaJCKR8UpSJWanMVQx mpgNwRY6DNmPSbBQuPOZfr4OTg7ryRf7U2s1PChLDIBqHesQrzT/wQN4bDSy0cZiMtRtIbfCbHi xaciCoL1tYF7gJKklePztyIQ3OUsaUoob4nGzSa5DPKMx1yaIiYc3SzTXuHgbQGdeiCq+dHIfl0 cjsV5v24kOcij/Q== X-Received: by 2002:a05:600c:138a:b0:477:19b7:d3c0 with SMTP id 5b1f17b1804b1-4778703d63emr49826545e9.2.1762992916536; Wed, 12 Nov 2025 16:15:16 -0800 (PST) X-Google-Smtp-Source: AGHT+IFqqQ7G2RU6cj93XJDUMviHL5Kr7pxCjAO+RqdjhI94usTzWHkoFMclayr8p3zMJ/v1JLLkIQ== X-Received: by 2002:a05:600c:138a:b0:477:19b7:d3c0 with SMTP id 5b1f17b1804b1-4778703d63emr49826415e9.2.1762992916150; Wed, 12 Nov 2025 16:15:16 -0800 (PST) Received: from sissix.lzampier.com ([2a06:5900:814a:ab00:3725:2991:6cf3:b3aa]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-42b53f0b589sm432983f8f.23.2025.11.12.16.15.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 12 Nov 2025 16:15:15 -0800 (PST) From: Lucas Zampieri To: linux-input@vger.kernel.org Cc: Lucas Zampieri , linux-kernel@vger.kernel.org, Jiri Kosina , Benjamin Tissoires , Sebastian Reichel , Bastien Nocera , linux-pm@vger.kernel.org Subject: [RFC PATCH v2 2/3] HID: input: Refactor battery code to use struct hid_battery Date: Thu, 13 Nov 2025 00:15:04 +0000 Message-ID: <20251113001508.713574-3-lzampier@redhat.com> X-Mailer: git-send-email 2.51.1 In-Reply-To: <20251113001508.713574-1-lzampier@redhat.com> References: <20251113001508.713574-1-lzampier@redhat.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Refactor the battery handling code to use the newly introduced struct hid_battery internally, replacing direct access to individual power_supply and state fields. The legacy dev->battery and dev->battery_* fields are maintained and synchronized for backward compatibility. This refactoring prepares the code for supporting multiple batteries per device in a subsequent patch. Signed-off-by: Lucas Zampieri --- drivers/hid/hid-input.c | 125 ++++++++++++++++++++++++++-------------- 1 file changed, 83 insertions(+), 42 deletions(-) diff --git a/drivers/hid/hid-input.c b/drivers/hid/hid-input.c index e56e7de53279..0e71efea9da3 100644 --- a/drivers/hid/hid-input.c +++ b/drivers/hid/hid-input.c @@ -454,7 +454,8 @@ static int hidinput_get_battery_property(struct power_s= upply *psy, enum power_supply_property prop, union power_supply_propval *val) { - struct hid_device *dev =3D power_supply_get_drvdata(psy); + struct hid_battery *bat =3D power_supply_get_drvdata(psy); + struct hid_device *dev =3D bat->dev; int value; int ret =3D 0; @@ -465,13 +466,13 @@ static int hidinput_get_battery_property(struct power= _supply *psy, break; case POWER_SUPPLY_PROP_CAPACITY: - if (dev->battery_status !=3D HID_BATTERY_REPORTED && - !dev->battery_avoid_query) { + if (bat->status !=3D HID_BATTERY_REPORTED && + !bat->avoid_query) { value =3D hidinput_query_battery_capacity(dev); if (value < 0) return value; } else { - value =3D dev->battery_capacity; + value =3D bat->capacity; } val->intval =3D value; @@ -482,20 +483,20 @@ static int hidinput_get_battery_property(struct power= _supply *psy, break; case POWER_SUPPLY_PROP_STATUS: - if (dev->battery_status !=3D HID_BATTERY_REPORTED && - !dev->battery_avoid_query) { + if (bat->status !=3D HID_BATTERY_REPORTED && + !bat->avoid_query) { value =3D hidinput_query_battery_capacity(dev); if (value < 0) return value; - dev->battery_capacity =3D value; - dev->battery_status =3D HID_BATTERY_QUERIED; + bat->capacity =3D value; + bat->status =3D HID_BATTERY_QUERIED; } - if (dev->battery_status =3D=3D HID_BATTERY_UNKNOWN) + if (bat->status =3D=3D HID_BATTERY_UNKNOWN) val->intval =3D POWER_SUPPLY_STATUS_UNKNOWN; else - val->intval =3D dev->battery_charge_status; + val->intval =3D bat->charge_status; break; case POWER_SUPPLY_PROP_SCOPE: @@ -513,8 +514,9 @@ static int hidinput_get_battery_property(struct power_s= upply *psy, static int hidinput_setup_battery(struct hid_device *dev, unsigned report_= type, struct hid_field *field, bool is_percentage) { + struct hid_battery *bat; struct power_supply_desc *psy_desc; - struct power_supply_config psy_cfg =3D { .drv_data =3D dev, }; + struct power_supply_config psy_cfg; unsigned quirks; s32 min, max; int error; @@ -530,16 +532,22 @@ static int hidinput_setup_battery(struct hid_device *= dev, unsigned report_type, if (quirks & HID_BATTERY_QUIRK_IGNORE) return 0; - psy_desc =3D kzalloc(sizeof(*psy_desc), GFP_KERNEL); - if (!psy_desc) + bat =3D kzalloc(sizeof(*bat), GFP_KERNEL); + if (!bat) return -ENOMEM; + psy_desc =3D kzalloc(sizeof(*psy_desc), GFP_KERNEL); + if (!psy_desc) { + error =3D -ENOMEM; + goto err_free_bat; + } + psy_desc->name =3D kasprintf(GFP_KERNEL, "hid-%s-battery", strlen(dev->uniq) ? dev->uniq : dev_name(&dev->dev)); if (!psy_desc->name) { error =3D -ENOMEM; - goto err_free_mem; + goto err_free_desc; } psy_desc->type =3D POWER_SUPPLY_TYPE_BATTERY; @@ -559,63 +567,85 @@ static int hidinput_setup_battery(struct hid_device *= dev, unsigned report_type, if (quirks & HID_BATTERY_QUIRK_FEATURE) report_type =3D HID_FEATURE_REPORT; - dev->battery_min =3D min; - dev->battery_max =3D max; - dev->battery_report_type =3D report_type; - dev->battery_report_id =3D field->report->id; - dev->battery_charge_status =3D POWER_SUPPLY_STATUS_DISCHARGING; + /* Initialize battery structure */ + bat->dev =3D dev; + bat->min =3D min; + bat->max =3D max; + bat->report_type =3D report_type; + bat->report_id =3D field->report->id; + bat->charge_status =3D POWER_SUPPLY_STATUS_DISCHARGING; + bat->status =3D HID_BATTERY_UNKNOWN; /* * Stylus is normally not connected to the device and thus we * can't query the device and get meaningful battery strength. * We have to wait for the device to report it on its own. */ - dev->battery_avoid_query =3D report_type =3D=3D HID_INPUT_REPORT && - field->physical =3D=3D HID_DG_STYLUS; + bat->avoid_query =3D report_type =3D=3D HID_INPUT_REPORT && + field->physical =3D=3D HID_DG_STYLUS; if (quirks & HID_BATTERY_QUIRK_AVOID_QUERY) - dev->battery_avoid_query =3D true; + bat->avoid_query =3D true; - dev->battery =3D power_supply_register(&dev->dev, psy_desc, &psy_cfg); - if (IS_ERR(dev->battery)) { - error =3D PTR_ERR(dev->battery); + psy_cfg.drv_data =3D bat; + bat->ps =3D power_supply_register(&dev->dev, psy_desc, &psy_cfg); + if (IS_ERR(bat->ps)) { + error =3D PTR_ERR(bat->ps); hid_warn(dev, "can't register power supply: %d\n", error); goto err_free_name; } - power_supply_powers(dev->battery, &dev->dev); + power_supply_powers(bat->ps, &dev->dev); + + /* Maintain legacy single battery fields for backward compatibility */ + dev->battery =3D bat->ps; + dev->battery_min =3D bat->min; + dev->battery_max =3D bat->max; + dev->battery_report_type =3D bat->report_type; + dev->battery_report_id =3D bat->report_id; + dev->battery_charge_status =3D bat->charge_status; + dev->battery_status =3D bat->status; + dev->battery_avoid_query =3D bat->avoid_query; + return 0; err_free_name: kfree(psy_desc->name); -err_free_mem: +err_free_desc: kfree(psy_desc); - dev->battery =3D NULL; +err_free_bat: + kfree(bat); return error; } static void hidinput_cleanup_battery(struct hid_device *dev) { + struct hid_battery *bat; const struct power_supply_desc *psy_desc; if (!dev->battery) return; + bat =3D power_supply_get_drvdata(dev->battery); psy_desc =3D dev->battery->desc; power_supply_unregister(dev->battery); kfree(psy_desc->name); kfree(psy_desc); + kfree(bat); dev->battery =3D NULL; } -static bool hidinput_update_battery_charge_status(struct hid_device *dev, +static bool hidinput_update_battery_charge_status(struct hid_battery *bat, unsigned int usage, int value) { switch (usage) { case HID_BAT_CHARGING: - dev->battery_charge_status =3D value ? - POWER_SUPPLY_STATUS_CHARGING : - POWER_SUPPLY_STATUS_DISCHARGING; + bat->charge_status =3D value ? + POWER_SUPPLY_STATUS_CHARGING : + POWER_SUPPLY_STATUS_DISCHARGING; + /* Update legacy field for backward compatibility */ + if (bat->dev->battery =3D=3D bat->ps) + bat->dev->battery_charge_status =3D bat->charge_status; return true; } @@ -625,32 +655,43 @@ static bool hidinput_update_battery_charge_status(str= uct hid_device *dev, static void hidinput_update_battery(struct hid_device *dev, unsigned int u= sage, int value) { + struct hid_battery *bat; int capacity; if (!dev->battery) return; - if (hidinput_update_battery_charge_status(dev, usage, value)) { - power_supply_changed(dev->battery); + bat =3D power_supply_get_drvdata(dev->battery); + + if (hidinput_update_battery_charge_status(bat, usage, value)) { + power_supply_changed(bat->ps); return; } if ((usage & HID_USAGE_PAGE) =3D=3D HID_UP_DIGITIZER && value =3D=3D 0) return; - if (value < dev->battery_min || value > dev->battery_max) + if (value < bat->min || value > bat->max) return; capacity =3D hidinput_scale_battery_capacity(dev, value); - if (dev->battery_status !=3D HID_BATTERY_REPORTED || - capacity !=3D dev->battery_capacity || - ktime_after(ktime_get_coarse(), dev->battery_ratelimit_time)) { - dev->battery_capacity =3D capacity; - dev->battery_status =3D HID_BATTERY_REPORTED; - dev->battery_ratelimit_time =3D + if (bat->status !=3D HID_BATTERY_REPORTED || + capacity !=3D bat->capacity || + ktime_after(ktime_get_coarse(), bat->ratelimit_time)) { + bat->capacity =3D capacity; + bat->status =3D HID_BATTERY_REPORTED; + bat->ratelimit_time =3D ktime_add_ms(ktime_get_coarse(), 30 * 1000); - power_supply_changed(dev->battery); + + /* Update legacy fields for backward compatibility */ + if (dev->battery =3D=3D bat->ps) { + dev->battery_capacity =3D bat->capacity; + dev->battery_status =3D bat->status; + dev->battery_ratelimit_time =3D bat->ratelimit_time; + } + + power_supply_changed(bat->ps); } } #else /* !CONFIG_HID_BATTERY_STRENGTH */ -- 2.51.1