From nobody Sun May 10 12:49:38 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B8073C433F5 for ; Wed, 4 May 2022 10:30:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1348121AbiEDKeW (ORCPT ); Wed, 4 May 2022 06:34:22 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43496 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1348108AbiEDKeS (ORCPT ); Wed, 4 May 2022 06:34:18 -0400 Received: from mail-wm1-x32e.google.com (mail-wm1-x32e.google.com [IPv6:2a00:1450:4864:20::32e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 973BD192B3 for ; Wed, 4 May 2022 03:30:40 -0700 (PDT) Received: by mail-wm1-x32e.google.com with SMTP id p189so590512wmp.3 for ; Wed, 04 May 2022 03:30:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20210112.gappssmtp.com; s=20210112; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=vVvWe9CoZBTdNyE3eHbun0xPih5mbPe7IlFMtVIhZ0k=; b=G/k6F7kYqnZzccb/RHvYUNQ+cmItTgNY2NoAzmvysDGNT9sYZQ+gnOOEp9NrTXesP/ nllwOMFbfQzmDY79kENJAS3Cr9of3xravd1Cs8QaC52GH4DTYELD1BuPp6aH1ejniSMl Yow1xWQpSz9DCQCsNcZ0RDnRkY2dmPELeMhJqERNvHAk3ikwaLajXlK74wdgNvBVVCAQ sYuB6Omwr3P9ADIRWwesA2VNLz2rPDHMkaQnzdSaeu16Lg72w72CMKjpCqGWD1Yk6dM8 sq26YYAW10WDCeouuFJPPsH5FIX930Xq5sDhKa3tiLBIsc2jJiR7vQ5myGG3u/LDmzCn CFgw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=vVvWe9CoZBTdNyE3eHbun0xPih5mbPe7IlFMtVIhZ0k=; b=3D+iP50xnRVcsM8aNo4DUGDqKkXi2rrE6sPZGrDfXB2c32ERsQw++Px7bi1o20kyq0 Yu0KU7QWPXdmW8Yu3JIHO5BShy1UwoXBh901X7mZIDUbdrcy4FAxH6Qul1/b1Y4NyZu7 iaVKP17gf8sbeCuFg2kOfsa12b0/vR5rjy0bta14e8rQZJXhIyDjLPcJnLhen679bZRI nwBqjXn1Cna463bXXgqGQPQNUXkFmdYL/nRZQveCNhxKt1PYTJaR/l3msXgR3v9WDMj0 hqplgSGP3XTL3RUvbg9BJcjA2gI6aMAHqhaEX/J4EKEOvAhT6hLcCrs6fk0iOrCHf1Zx B0kA== X-Gm-Message-State: AOAM530qHgtqDF+pdmyMr6XF3L7KOMEBPAwlSe4x6ohRDF03awgDKlNN h8++qOBFF8Z2383kloxcAhmeVQ== X-Google-Smtp-Source: ABdhPJynx1ZvRXZ75QK4hcCQuYNk4l01zutV1yAb+3OC1A919J6T4sHa6uQUS5Htut6lA6azQH8elg== X-Received: by 2002:a7b:cc17:0:b0:38d:af7:3848 with SMTP id f23-20020a7bcc17000000b0038d0af73848mr7164990wmh.41.1651660239194; Wed, 04 May 2022 03:30:39 -0700 (PDT) Received: from localhost.localdomain (laubervilliers-658-1-213-31.w90-63.abo.wanadoo.fr. [90.63.244.31]) by smtp.googlemail.com with ESMTPSA id k6-20020a05600c1c8600b0039429bfebebsm7521395wms.3.2022.05.04.03.30.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 04 May 2022 03:30:38 -0700 (PDT) From: Corentin Labbe To: jdelvare@suse.com, linux@roeck-us.net Cc: linux-hwmon@vger.kernel.org, linux-kernel@vger.kernel.org, Corentin Labbe Subject: [PATCH v2 1/2] hwmon: acpi_power_meter: fix style issue Date: Wed, 4 May 2022 10:30:27 +0000 Message-Id: <20220504103028.493155-2-clabbe@baylibre.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20220504103028.493155-1-clabbe@baylibre.com> References: <20220504103028.493155-1-clabbe@baylibre.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Fix style issues found by checkpatch. Signed-off-by: Corentin Labbe --- drivers/hwmon/acpi_power_meter.c | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/hwmon/acpi_power_meter.c b/drivers/hwmon/acpi_power_me= ter.c index c405a5869581..d2545a1be9fc 100644 --- a/drivers/hwmon/acpi_power_meter.c +++ b/drivers/hwmon/acpi_power_meter.c @@ -481,7 +481,7 @@ static struct sensor_template meter_attrs[] =3D { RO_SENSOR_TEMPLATE("power1_average_interval_max", show_val, 1), RO_SENSOR_TEMPLATE("power1_is_battery", show_val, 5), RW_SENSOR_TEMPLATE(POWER_AVG_INTERVAL_NAME, show_avg_interval, - set_avg_interval, 0), + set_avg_interval, 0), {}, }; =20 @@ -530,6 +530,7 @@ static void remove_domain_devices(struct acpi_power_met= er_resource *resource) =20 for (i =3D 0; i < resource->num_domain_devices; i++) { struct acpi_device *obj =3D resource->domain_devices[i]; + if (!obj) continue; =20 @@ -580,7 +581,7 @@ static int read_domain_devices(struct acpi_power_meter_= resource *resource) } =20 resource->holders_dir =3D kobject_create_and_add("measures", - &resource->acpi_dev->dev.kobj); + &resource->acpi_dev->dev.kobj); if (!resource->holders_dir) { res =3D -ENOMEM; goto exit_free; @@ -590,7 +591,7 @@ static int read_domain_devices(struct acpi_power_meter_= resource *resource) =20 for (i =3D 0; i < pss->package.count; i++) { struct acpi_device *obj; - union acpi_object *element =3D &(pss->package.elements[i]); + union acpi_object *element =3D &pss->package.elements[i]; =20 /* Refuse non-references */ if (element->type !=3D ACPI_TYPE_LOCAL_REFERENCE) @@ -603,7 +604,7 @@ static int read_domain_devices(struct acpi_power_meter_= resource *resource) continue; =20 res =3D sysfs_create_link(resource->holders_dir, &obj->dev.kobj, - kobject_name(&obj->dev.kobj)); + kobject_name(&obj->dev.kobj)); if (res) { acpi_dev_put(obj); resource->domain_devices[i] =3D NULL; @@ -788,7 +789,7 @@ static int read_capabilities(struct acpi_power_meter_re= source *resource) str =3D &resource->model_number; =20 for (i =3D 11; i < 14; i++) { - union acpi_object *element =3D &(pss->package.elements[i]); + union acpi_object *element =3D &pss->package.elements[i]; =20 if (element->type !=3D ACPI_TYPE_STRING) { res =3D -EINVAL; @@ -868,8 +869,7 @@ static int acpi_power_meter_add(struct acpi_device *dev= ice) if (!device) return -EINVAL; =20 - resource =3D kzalloc(sizeof(struct acpi_power_meter_resource), - GFP_KERNEL); + resource =3D kzalloc(sizeof(*resource), GFP_KERNEL); if (!resource) return -ENOMEM; =20 @@ -884,7 +884,8 @@ static int acpi_power_meter_add(struct acpi_device *dev= ice) if (res) goto exit_free; =20 - resource->trip[0] =3D resource->trip[1] =3D -1; + resource->trip[0] =3D -1; + resource->trip[1] =3D -1; =20 res =3D setup_attrs(resource); if (res) --=20 2.35.1 From nobody Sun May 10 12:49:38 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2D138C433EF for ; Wed, 4 May 2022 10:30:55 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1348148AbiEDKe0 (ORCPT ); Wed, 4 May 2022 06:34:26 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43504 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1348111AbiEDKeS (ORCPT ); Wed, 4 May 2022 06:34:18 -0400 Received: from mail-wr1-x432.google.com (mail-wr1-x432.google.com [IPv6:2a00:1450:4864:20::432]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 51CBCE0C2 for ; Wed, 4 May 2022 03:30:42 -0700 (PDT) Received: by mail-wr1-x432.google.com with SMTP id v12so1377031wrv.10 for ; Wed, 04 May 2022 03:30:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20210112.gappssmtp.com; s=20210112; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=QRwF0dGsfdTHRlN/d1PJ9h1Zv3dHY26YISg3X11/2Y4=; b=3pFNEsQYO7nwxiiZ5Mb0jj/wh11cUGNC8gBKQOvPlpd8O88BCKvUK47+J1isla5ODQ xRySKPBmY2bRziXe6UB9zOMIhCf299RuMwgEU1zvUfa6DtgdZylgypqwSxbjfoqgeOfi F2ux7j53w/aNVcGzUjuHypDR6oKJAmECeMJpznqrO5iyrlM3ZkHk8Hr4fVODTg7MOyC9 ugg3xM62oIk5Md6bsIQwcCdtjT1CAZe+MBnfbblwm2UXrRFaJU12alPG1BBblWOHqnVr Tho1Xy7lTHFl7SKwEOGRtuxEAOWbfD53FzNYzmq/EZ7iOtWnHool2nLREkV9fiqSSDRB vKgQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=QRwF0dGsfdTHRlN/d1PJ9h1Zv3dHY26YISg3X11/2Y4=; b=iENn+m5QqbO+tf8wvyX1ieexw76/W1zPdTeSPLyc2aH5mwLbE5BPDJra09YXxzr1qk 9QErYQ71huenyLYLauWHbpwV4QDvCN99zc7dHikser91VSU9dkq7DJm5YiybUUZZQQ7G aD5jjHDXZDgyVZaUHvbN2fSN5TlSa1l+cWDcscRC+Z+X+52+FUZenhA/ccyMFcUOA6D5 sDU41kwHE27rSZ3ViQ5+Kwys1kjPOo2DmND4JSrg3cLtVJcUXj7q+6OwiuO62pkP3EUJ 7sP4BH+G5Lrz4wyGZWSOsU1kPCdKhEIr/UvrJ+TULRJF0E1Z3Xqs/Ie9lkagJM5NzbTS kjeQ== X-Gm-Message-State: AOAM532tzBo2RdG2+V6p29c6yYTYYVrZbPdUS0wpNb49DmkUww4Ikyny 6EpL8aARKb3rmYeCpgyYjZYMig== X-Google-Smtp-Source: ABdhPJxviVh0ZaERJTsydWaKnV4yo/EAWUS7IF9jWniuSgINgUFoJ7tfwAPb2wg+IlAT3wClYJwB4A== X-Received: by 2002:a05:6000:1acd:b0:20c:7201:9267 with SMTP id i13-20020a0560001acd00b0020c72019267mr6593519wry.41.1651660240359; Wed, 04 May 2022 03:30:40 -0700 (PDT) Received: from localhost.localdomain (laubervilliers-658-1-213-31.w90-63.abo.wanadoo.fr. [90.63.244.31]) by smtp.googlemail.com with ESMTPSA id k6-20020a05600c1c8600b0039429bfebebsm7521395wms.3.2022.05.04.03.30.39 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 04 May 2022 03:30:39 -0700 (PDT) From: Corentin Labbe To: jdelvare@suse.com, linux@roeck-us.net Cc: linux-hwmon@vger.kernel.org, linux-kernel@vger.kernel.org, Corentin Labbe Subject: [PATCH v2 2/2] hwmon: acpi_power_meter: convert to hwmon_device_register_with_info Date: Wed, 4 May 2022 10:30:28 +0000 Message-Id: <20220504103028.493155-3-clabbe@baylibre.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20220504103028.493155-1-clabbe@baylibre.com> References: <20220504103028.493155-1-clabbe@baylibre.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Booting lead to a hwmon_device_register() is deprecated. Please convert the= driver to use hwmon_device_register_with_info(). So let's convert the driver to use hwmon_device_register_with_info(). Signed-off-by: Corentin Labbe Reported-by: kernel test robot --- drivers/hwmon/acpi_power_meter.c | 509 +++++++++++++------------------ 1 file changed, 219 insertions(+), 290 deletions(-) diff --git a/drivers/hwmon/acpi_power_meter.c b/drivers/hwmon/acpi_power_me= ter.c index d2545a1be9fc..fa3417786356 100644 --- a/drivers/hwmon/acpi_power_meter.c +++ b/drivers/hwmon/acpi_power_meter.c @@ -23,7 +23,8 @@ #define ACPI_POWER_METER_DEVICE_NAME "Power Meter" #define ACPI_POWER_METER_CLASS "pwr_meter_resource" =20 -#define NUM_SENSORS 17 +#define TRIP_MIN 0 +#define TRIP_MAX 1 =20 #define POWER_METER_CAN_MEASURE (1 << 0) #define POWER_METER_CAN_TRIP (1 << 1) @@ -38,11 +39,6 @@ #define METER_NOTIFY_CAPPING 0x83 #define METER_NOTIFY_INTERVAL 0x84 =20 -#define POWER_AVERAGE_NAME "power1_average" -#define POWER_CAP_NAME "power1_cap" -#define POWER_AVG_INTERVAL_NAME "power1_average_interval" -#define POWER_ALARM_NAME "power1_alarm" - static int cap_in_hardware; static bool force_cap_on; =20 @@ -85,8 +81,6 @@ struct acpi_power_meter_resource { u64 avg_interval; int sensors_valid; unsigned long sensors_last_updated; - struct sensor_device_attribute sensors[NUM_SENSORS]; - int num_sensors; s64 trip[2]; int num_domain_devices; struct acpi_device **domain_devices; @@ -122,47 +116,32 @@ static int update_avg_interval(struct acpi_power_mete= r_resource *resource) return 0; } =20 -static ssize_t show_avg_interval(struct device *dev, - struct device_attribute *devattr, - char *buf) +static int acpi_power_average_interval_read(struct acpi_power_meter_resour= ce *resource) { - struct acpi_device *acpi_dev =3D to_acpi_device(dev); - struct acpi_power_meter_resource *resource =3D acpi_dev->driver_data; - mutex_lock(&resource->lock); update_avg_interval(resource); mutex_unlock(&resource->lock); =20 - return sprintf(buf, "%llu\n", resource->avg_interval); + return resource->avg_interval; } =20 -static ssize_t set_avg_interval(struct device *dev, - struct device_attribute *devattr, - const char *buf, size_t count) +static int set_average_interval(struct acpi_power_meter_resource *resource= , long val) { - struct acpi_device *acpi_dev =3D to_acpi_device(dev); - struct acpi_power_meter_resource *resource =3D acpi_dev->driver_data; union acpi_object arg0 =3D { ACPI_TYPE_INTEGER }; struct acpi_object_list args =3D { 1, &arg0 }; - int res; - unsigned long temp; unsigned long long data; acpi_status status; =20 - res =3D kstrtoul(buf, 10, &temp); - if (res) - return res; - - if (temp > resource->caps.max_avg_interval || - temp < resource->caps.min_avg_interval) + if (val > resource->caps.max_avg_interval || + val < resource->caps.min_avg_interval) return -EINVAL; - arg0.integer.value =3D temp; + arg0.integer.value =3D val; =20 mutex_lock(&resource->lock); status =3D acpi_evaluate_integer(resource->acpi_dev->handle, "_PAI", &args, &data); if (ACPI_SUCCESS(status)) - resource->avg_interval =3D temp; + resource->avg_interval =3D val; mutex_unlock(&resource->lock); =20 if (ACPI_FAILURE(status)) { @@ -175,7 +154,7 @@ static ssize_t set_avg_interval(struct device *dev, if (data) return -EINVAL; =20 - return count; + return 0; } =20 /* Cap functions */ @@ -196,46 +175,33 @@ static int update_cap(struct acpi_power_meter_resourc= e *resource) return 0; } =20 -static ssize_t show_cap(struct device *dev, - struct device_attribute *devattr, - char *buf) +static int acpi_power_cap_read(struct acpi_power_meter_resource *resource, + long *val) { - struct acpi_device *acpi_dev =3D to_acpi_device(dev); - struct acpi_power_meter_resource *resource =3D acpi_dev->driver_data; - mutex_lock(&resource->lock); update_cap(resource); mutex_unlock(&resource->lock); =20 - return sprintf(buf, "%llu\n", resource->cap * 1000); + return resource->cap * 1000; } =20 -static ssize_t set_cap(struct device *dev, struct device_attribute *devatt= r, - const char *buf, size_t count) +static int set_cap(struct acpi_power_meter_resource *resource, long val) { - struct acpi_device *acpi_dev =3D to_acpi_device(dev); - struct acpi_power_meter_resource *resource =3D acpi_dev->driver_data; union acpi_object arg0 =3D { ACPI_TYPE_INTEGER }; struct acpi_object_list args =3D { 1, &arg0 }; - int res; - unsigned long temp; unsigned long long data; acpi_status status; =20 - res =3D kstrtoul(buf, 10, &temp); - if (res) - return res; - - temp =3D DIV_ROUND_CLOSEST(temp, 1000); - if (temp > resource->caps.max_cap || temp < resource->caps.min_cap) + val =3D DIV_ROUND_CLOSEST(val, 1000); + if (val > resource->caps.max_cap || val < resource->caps.min_cap) return -EINVAL; - arg0.integer.value =3D temp; + arg0.integer.value =3D val; =20 mutex_lock(&resource->lock); status =3D acpi_evaluate_integer(resource->acpi_dev->handle, "_SHL", &args, &data); if (ACPI_SUCCESS(status)) - resource->cap =3D temp; + resource->cap =3D val; mutex_unlock(&resource->lock); =20 if (ACPI_FAILURE(status)) { @@ -248,7 +214,7 @@ static ssize_t set_cap(struct device *dev, struct devic= e_attribute *devattr, if (data) return -EINVAL; =20 - return count; + return 0; } =20 /* Power meter trip points */ @@ -263,12 +229,12 @@ static int set_acpi_trip(struct acpi_power_meter_reso= urce *resource) acpi_status status; =20 /* Both trip levels must be set */ - if (resource->trip[0] < 0 || resource->trip[1] < 0) + if (resource->trip[TRIP_MIN] < 0 || resource->trip[TRIP_MAX] < 0) return 0; =20 /* This driver stores min, max; ACPI wants max, min. */ - arg_objs[0].integer.value =3D resource->trip[1]; - arg_objs[1].integer.value =3D resource->trip[0]; + arg_objs[0].integer.value =3D resource->trip[TRIP_MAX]; + arg_objs[1].integer.value =3D resource->trip[TRIP_MIN]; =20 status =3D acpi_evaluate_integer(resource->acpi_dev->handle, "_PTP", &args, &data); @@ -285,30 +251,18 @@ static int set_acpi_trip(struct acpi_power_meter_reso= urce *resource) return 0; } =20 -static ssize_t set_trip(struct device *dev, struct device_attribute *devat= tr, - const char *buf, size_t count) +static int set_trip(struct acpi_power_meter_resource *resource, long val, = int triptype) { - struct sensor_device_attribute *attr =3D to_sensor_dev_attr(devattr); - struct acpi_device *acpi_dev =3D to_acpi_device(dev); - struct acpi_power_meter_resource *resource =3D acpi_dev->driver_data; int res; - unsigned long temp; =20 - res =3D kstrtoul(buf, 10, &temp); - if (res) - return res; - - temp =3D DIV_ROUND_CLOSEST(temp, 1000); + val =3D DIV_ROUND_CLOSEST(val, 1000); =20 mutex_lock(&resource->lock); - resource->trip[attr->index - 7] =3D temp; + resource->trip[triptype] =3D val; res =3D set_acpi_trip(resource); mutex_unlock(&resource->lock); =20 - if (res) - return res; - - return count; + return res; } =20 /* Power meter */ @@ -337,33 +291,26 @@ static int update_meter(struct acpi_power_meter_resou= rce *resource) return 0; } =20 -static ssize_t show_power(struct device *dev, - struct device_attribute *devattr, - char *buf) +static int acpi_power_power_read(struct acpi_power_meter_resource *resourc= e, + long *val) { - struct acpi_device *acpi_dev =3D to_acpi_device(dev); - struct acpi_power_meter_resource *resource =3D acpi_dev->driver_data; - mutex_lock(&resource->lock); update_meter(resource); mutex_unlock(&resource->lock); =20 - return sprintf(buf, "%llu\n", resource->power * 1000); + *val =3D resource->power * 1000; + return 0; } =20 /* Miscellaneous */ -static ssize_t show_str(struct device *dev, - struct device_attribute *devattr, - char *buf) +static ssize_t show_str(struct device *dev, int index, char *buf) { - struct sensor_device_attribute *attr =3D to_sensor_dev_attr(devattr); - struct acpi_device *acpi_dev =3D to_acpi_device(dev); - struct acpi_power_meter_resource *resource =3D acpi_dev->driver_data; + struct acpi_power_meter_resource *resource =3D dev_get_drvdata(dev); acpi_string val; int ret; =20 mutex_lock(&resource->lock); - switch (attr->index) { + switch (index) { case 0: val =3D resource->model_number; break; @@ -375,7 +322,7 @@ static ssize_t show_str(struct device *dev, break; default: WARN(1, "Implementation error: unexpected attribute index %d\n", - attr->index); + index); val =3D ""; break; } @@ -384,141 +331,138 @@ static ssize_t show_str(struct device *dev, return ret; } =20 -static ssize_t show_val(struct device *dev, - struct device_attribute *devattr, - char *buf) +static ssize_t power1_is_battery_show(struct device *dev, + struct device_attribute *attr, + char *buf) { - struct sensor_device_attribute *attr =3D to_sensor_dev_attr(devattr); struct acpi_device *acpi_dev =3D to_acpi_device(dev); struct acpi_power_meter_resource *resource =3D acpi_dev->driver_data; - u64 val =3D 0; + int val; =20 - switch (attr->index) { - case 0: - val =3D resource->caps.min_avg_interval; + if (resource->caps.flags & POWER_METER_IS_BATTERY) + val =3D 1; + else + val =3D 0; + return sprintf(buf, "%d\n", val); +} + +static ssize_t power1_model_number_show(struct device *dev, + struct device_attribute *attr, + char *buf) +{ + return show_str(dev, 0, buf); +} + +static ssize_t power1_serial_number_show(struct device *dev, + struct device_attribute *attr, + char *buf) +{ + return show_str(dev, 1, buf); +} + +static ssize_t power1_oem_info_show(struct device *dev, + struct device_attribute *attr, + char *buf) +{ + return show_str(dev, 2, buf); +} + +static int acpi_power_read(struct device *dev, enum hwmon_sensor_types typ= e, + u32 attr, int channel, long *val) +{ + struct acpi_power_meter_resource *resource =3D dev_get_drvdata(dev); + + switch (attr) { + case hwmon_power_average: + return acpi_power_power_read(resource, val); + case hwmon_power_average_interval_min: + *val =3D resource->caps.min_avg_interval; break; - case 1: - val =3D resource->caps.max_avg_interval; + case hwmon_power_average_interval_max: + *val =3D resource->caps.max_avg_interval; break; - case 2: - val =3D resource->caps.min_cap * 1000; + case hwmon_power_cap_min: + *val =3D resource->caps.min_cap * 1000; break; - case 3: - val =3D resource->caps.max_cap * 1000; + case hwmon_power_cap_max: + *val =3D resource->caps.max_cap * 1000; break; - case 4: + case hwmon_power_cap: + *val =3D acpi_power_cap_read(resource, val); + break; + case hwmon_power_cap_hyst: if (resource->caps.hysteresis =3D=3D UNKNOWN_HYSTERESIS) - return sprintf(buf, "unknown\n"); + return -EINVAL; =20 - val =3D resource->caps.hysteresis * 1000; + *val =3D resource->caps.hysteresis * 1000; break; - case 5: - if (resource->caps.flags & POWER_METER_IS_BATTERY) - val =3D 1; - else - val =3D 0; - break; - case 6: + case hwmon_power_alarm: if (resource->power > resource->cap) - val =3D 1; + *val =3D 1; else - val =3D 0; + *val =3D 0; break; - case 7: - case 8: - if (resource->trip[attr->index - 7] < 0) - return sprintf(buf, "unknown\n"); - - val =3D resource->trip[attr->index - 7] * 1000; + case hwmon_power_average_min: + if (resource->trip[TRIP_MIN] < 0) + return -EINVAL; + *val =3D resource->trip[TRIP_MIN] * 1000; + break; + case hwmon_power_average_max: + if (resource->trip[TRIP_MAX] < 0) + return -EINVAL; + *val =3D resource->trip[TRIP_MAX] * 1000; + break; + case hwmon_power_average_interval: + *val =3D acpi_power_average_interval_read(resource); + break; + case hwmon_power_accuracy: + *val =3D resource->caps.accuracy / 1000; break; default: WARN(1, "Implementation error: unexpected attribute index %d\n", - attr->index); - break; + attr); + return -EOPNOTSUPP; } =20 - return sprintf(buf, "%llu\n", val); -} - -static ssize_t show_accuracy(struct device *dev, - struct device_attribute *devattr, - char *buf) -{ - struct acpi_device *acpi_dev =3D to_acpi_device(dev); - struct acpi_power_meter_resource *resource =3D acpi_dev->driver_data; - unsigned int acc =3D resource->caps.accuracy; - - return sprintf(buf, "%u.%u%%\n", acc / 1000, acc % 1000); + return 0; } =20 -static ssize_t show_name(struct device *dev, - struct device_attribute *devattr, - char *buf) +static int acpi_power_write(struct device *dev, enum hwmon_sensor_types ty= pe, + u32 attr, int channel, long val) { - return sprintf(buf, "%s\n", ACPI_POWER_METER_NAME); -} - -#define RO_SENSOR_TEMPLATE(_label, _show, _index) \ - { \ - .label =3D _label, \ - .show =3D _show, \ - .index =3D _index, \ - } - -#define RW_SENSOR_TEMPLATE(_label, _show, _set, _index) \ - { \ - .label =3D _label, \ - .show =3D _show, \ - .set =3D _set, \ - .index =3D _index, \ + struct acpi_power_meter_resource *resource =3D dev_get_drvdata(dev); + + switch (attr) { + case hwmon_power_average_min: + return set_trip(resource, TRIP_MIN, val); + case hwmon_power_average_max: + return set_trip(resource, TRIP_MAX, val); + case hwmon_power_cap: + if (resource->caps.configurable_cap) + return set_cap(resource, val); + else + return -EINVAL; + case hwmon_power_average_interval: + return set_average_interval(resource, val); + default: + return -EOPNOTSUPP; } +} =20 -/* Sensor descriptions. If you add a sensor, update NUM_SENSORS above! */ -static struct sensor_template meter_attrs[] =3D { - RO_SENSOR_TEMPLATE(POWER_AVERAGE_NAME, show_power, 0), - RO_SENSOR_TEMPLATE("power1_accuracy", show_accuracy, 0), - RO_SENSOR_TEMPLATE("power1_average_interval_min", show_val, 0), - RO_SENSOR_TEMPLATE("power1_average_interval_max", show_val, 1), - RO_SENSOR_TEMPLATE("power1_is_battery", show_val, 5), - RW_SENSOR_TEMPLATE(POWER_AVG_INTERVAL_NAME, show_avg_interval, - set_avg_interval, 0), - {}, -}; - -static struct sensor_template misc_cap_attrs[] =3D { - RO_SENSOR_TEMPLATE("power1_cap_min", show_val, 2), - RO_SENSOR_TEMPLATE("power1_cap_max", show_val, 3), - RO_SENSOR_TEMPLATE("power1_cap_hyst", show_val, 4), - RO_SENSOR_TEMPLATE(POWER_ALARM_NAME, show_val, 6), - {}, -}; - -static struct sensor_template ro_cap_attrs[] =3D { - RO_SENSOR_TEMPLATE(POWER_CAP_NAME, show_cap, 0), - {}, -}; - -static struct sensor_template rw_cap_attrs[] =3D { - RW_SENSOR_TEMPLATE(POWER_CAP_NAME, show_cap, set_cap, 0), - {}, -}; - -static struct sensor_template trip_attrs[] =3D { - RW_SENSOR_TEMPLATE("power1_average_min", show_val, set_trip, 7), - RW_SENSOR_TEMPLATE("power1_average_max", show_val, set_trip, 8), - {}, -}; - -static struct sensor_template misc_attrs[] =3D { - RO_SENSOR_TEMPLATE("name", show_name, 0), - RO_SENSOR_TEMPLATE("power1_model_number", show_str, 0), - RO_SENSOR_TEMPLATE("power1_oem_info", show_str, 2), - RO_SENSOR_TEMPLATE("power1_serial_number", show_str, 1), - {}, +static DEVICE_ATTR_RO(power1_is_battery); +static DEVICE_ATTR_RO(power1_model_number); +static DEVICE_ATTR_RO(power1_oem_info); +static DEVICE_ATTR_RO(power1_serial_number); + +static struct attribute *acpi_power_attrs[] =3D { + &dev_attr_power1_is_battery.attr, + &dev_attr_power1_model_number.attr, + &dev_attr_power1_oem_info.attr, + &dev_attr_power1_serial_number.attr, + NULL }; =20 -#undef RO_SENSOR_TEMPLATE -#undef RW_SENSOR_TEMPLATE +ATTRIBUTE_GROUPS(acpi_power); =20 /* Read power domain data */ static void remove_domain_devices(struct acpi_power_meter_resource *resour= ce) @@ -621,55 +565,52 @@ static int read_domain_devices(struct acpi_power_mete= r_resource *resource) return res; } =20 -/* Registration and deregistration */ -static int register_attrs(struct acpi_power_meter_resource *resource, - struct sensor_template *attrs) -{ - struct device *dev =3D &resource->acpi_dev->dev; - struct sensor_device_attribute *sensors =3D - &resource->sensors[resource->num_sensors]; - int res =3D 0; - - while (attrs->label) { - sensors->dev_attr.attr.name =3D attrs->label; - sensors->dev_attr.attr.mode =3D 0444; - sensors->dev_attr.show =3D attrs->show; - sensors->index =3D attrs->index; - - if (attrs->set) { - sensors->dev_attr.attr.mode |=3D 0200; - sensors->dev_attr.store =3D attrs->set; - } - - sysfs_attr_init(&sensors->dev_attr.attr); - res =3D device_create_file(dev, &sensors->dev_attr); - if (res) { - sensors->dev_attr.attr.name =3D NULL; - goto error; - } - sensors++; - resource->num_sensors++; - attrs++; - } - -error: - return res; -} - -static void remove_attrs(struct acpi_power_meter_resource *resource) +static umode_t acpi_power_is_visible(const void *data, + enum hwmon_sensor_types type, + u32 attr, int channel) { - int i; + const struct acpi_power_meter_resource *resource =3D data; =20 - for (i =3D 0; i < resource->num_sensors; i++) { - if (!resource->sensors[i].dev_attr.attr.name) - continue; - device_remove_file(&resource->acpi_dev->dev, - &resource->sensors[i].dev_attr); + switch (attr) { + case hwmon_power_average_min: + case hwmon_power_average_max: + if (resource->caps.flags & POWER_METER_CAN_TRIP) + return 0644; + break; + case hwmon_power_average: + case hwmon_power_accuracy: + case hwmon_power_average_interval_min: + case hwmon_power_average_interval_max: + if (resource->caps.flags & POWER_METER_CAN_MEASURE) + return 0444; + break; + case hwmon_power_average_interval: + if (resource->caps.flags & POWER_METER_CAN_MEASURE) + return 0644; + break; + case hwmon_power_cap: + if (!can_cap_in_hardware()) + return 0; + if (!(resource->caps.flags & POWER_METER_CAN_CAP)) + return 0; + if (resource->caps.configurable_cap) + return 0644; + return 0444; + break; + case hwmon_power_cap_min: + case hwmon_power_cap_max: + case hwmon_power_cap_hyst: + case hwmon_power_cap_alarm: + if (!can_cap_in_hardware()) + return 0; + if (resource->caps.flags & POWER_METER_CAN_CAP) + return 0444; + break; + default: + break; } =20 - remove_domain_devices(resource); - - resource->num_sensors =3D 0; + return 0; } =20 static int setup_attrs(struct acpi_power_meter_resource *resource) @@ -680,47 +621,11 @@ static int setup_attrs(struct acpi_power_meter_resour= ce *resource) if (res) return res; =20 - if (resource->caps.flags & POWER_METER_CAN_MEASURE) { - res =3D register_attrs(resource, meter_attrs); - if (res) - goto error; + if (resource->caps.flags & POWER_METER_CAN_CAP && !can_cap_in_hardware())= { + dev_warn(&resource->acpi_dev->dev, + "Ignoring unsafe software power cap!\n"); } - - if (resource->caps.flags & POWER_METER_CAN_CAP) { - if (!can_cap_in_hardware()) { - dev_warn(&resource->acpi_dev->dev, - "Ignoring unsafe software power cap!\n"); - goto skip_unsafe_cap; - } - - if (resource->caps.configurable_cap) - res =3D register_attrs(resource, rw_cap_attrs); - else - res =3D register_attrs(resource, ro_cap_attrs); - - if (res) - goto error; - - res =3D register_attrs(resource, misc_cap_attrs); - if (res) - goto error; - } - -skip_unsafe_cap: - if (resource->caps.flags & POWER_METER_CAN_TRIP) { - res =3D register_attrs(resource, trip_attrs); - if (res) - goto error; - } - - res =3D register_attrs(resource, misc_attrs); - if (res) - goto error; - - return res; -error: - remove_attrs(resource); - return res; + return 0; } =20 static void free_capabilities(struct acpi_power_meter_resource *resource) @@ -795,7 +700,6 @@ static int read_capabilities(struct acpi_power_meter_re= source *resource) res =3D -EINVAL; goto error; } - *str =3D kcalloc(element->string.length + 1, sizeof(u8), GFP_KERNEL); if (!*str) { @@ -836,20 +740,20 @@ static void acpi_power_meter_notify(struct acpi_devic= e *device, u32 event) if (res) break; =20 - remove_attrs(resource); + remove_domain_devices(resource); setup_attrs(resource); break; case METER_NOTIFY_TRIP: - sysfs_notify(&device->dev.kobj, NULL, POWER_AVERAGE_NAME); + hwmon_notify_event(&device->dev, hwmon_power, hwmon_power_average, 0); break; case METER_NOTIFY_CAP: - sysfs_notify(&device->dev.kobj, NULL, POWER_CAP_NAME); + hwmon_notify_event(&device->dev, hwmon_power, hwmon_power_cap, 0); break; case METER_NOTIFY_INTERVAL: - sysfs_notify(&device->dev.kobj, NULL, POWER_AVG_INTERVAL_NAME); + hwmon_notify_event(&device->dev, hwmon_power, hwmon_power_average_interv= al, 0); break; case METER_NOTIFY_CAPPING: - sysfs_notify(&device->dev.kobj, NULL, POWER_ALARM_NAME); + hwmon_notify_event(&device->dev, hwmon_power, hwmon_power_alarm, 0); dev_info(&device->dev, "Capping in progress.\n"); break; default: @@ -861,6 +765,28 @@ static void acpi_power_meter_notify(struct acpi_device= *device, u32 event) dev_name(&device->dev), event, 0); } =20 +static const struct hwmon_channel_info *acpi_power_info[] =3D { + HWMON_CHANNEL_INFO(power, + HWMON_P_AVERAGE | HWMON_P_AVERAGE_INTERVAL | + HWMON_P_AVERAGE_MIN | HWMON_P_AVERAGE_MAX | + HWMON_P_CAP | HWMON_P_CAP_HYST | + HWMON_P_CAP_MIN | HWMON_P_CAP_MAX | + HWMON_P_ACCURACY + ), + NULL, +}; + +static const struct hwmon_ops acpi_power_hwmon_ops =3D { + .is_visible =3D acpi_power_is_visible, + .read =3D acpi_power_read, + .write =3D acpi_power_write, +}; + +static const struct hwmon_chip_info acpi_power_chip_info =3D { + .ops =3D &acpi_power_hwmon_ops, + .info =3D acpi_power_info, +}; + static int acpi_power_meter_add(struct acpi_device *device) { int res; @@ -891,7 +817,10 @@ static int acpi_power_meter_add(struct acpi_device *de= vice) if (res) goto exit_free_capability; =20 - resource->hwmon_dev =3D hwmon_device_register(&device->dev); + resource->hwmon_dev =3D hwmon_device_register_with_info(&device->dev, + ACPI_POWER_METER_NAME, + resource, &acpi_power_chip_info, + acpi_power_groups); if (IS_ERR(resource->hwmon_dev)) { res =3D PTR_ERR(resource->hwmon_dev); goto exit_remove; @@ -901,7 +830,7 @@ static int acpi_power_meter_add(struct acpi_device *dev= ice) goto exit; =20 exit_remove: - remove_attrs(resource); + remove_domain_devices(resource); exit_free_capability: free_capabilities(resource); exit_free: @@ -920,7 +849,7 @@ static int acpi_power_meter_remove(struct acpi_device *= device) resource =3D acpi_driver_data(device); hwmon_device_unregister(resource->hwmon_dev); =20 - remove_attrs(resource); + remove_domain_devices(resource); free_capabilities(resource); =20 kfree(resource); --=20 2.35.1