Thread (40 messages) 40 messages, 6 authors, 2025-01-22

Re: [PATCH v1 00/21] hwmon: Fix the type of 'config' in struct hwmon_channel_info to u64

From: "lihuisong (C)" <lihuisong@huawei.com>
Date: 2025-01-22 03:36:20
Also in: arm-scmi, linux-arm-kernel, linux-hwmon, linux-media, linux-rdma, linux-rtc, lkml, platform-driver-x86

在 2025/1/22 11:19, Guenter Roeck 写道:
On 1/21/25 18:52, lihuisong (C) wrote:
quoted
quoted
Those are not hwmon attributes and should not be (or have been) exposed
as sysfs attributes in the first place, but (if really wanted/needed)
through debugfs files. Even _if_ exposed as sysfs attributes they 
should
not have the power1_ prefix (except maybe for the last one).
I plan to accept the suggestion Armin proposed that acpi_power_meter 
can use the "extra_groups" parameter for these "not generic hwmon 
power attributes".
Should we drop the power1_ prefix? But this will lead to the change 
of these attributes.
Do you mean 'power1_is_battery' can be added to hwmon power 
attributes in hwmon.h?
No; power1_is_battery is still not a hardware monitoring attribute,
much less one to standardize. Also, don't change any attribute names.
They are wrong, but they are the ABI for this driver.
Ok, it's clear now.
.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help