[Feature Request] Allow Fan and Sensor to utilise hwmonX/name file value for persistence about fan2go HOT 4 CLOSED

@eldorel try the latest 0.2.4 build of fan2go and simply put "corsaircpro" in the platform fields. It should pick up your commander every time.

from fan2go.

I realized I didn't provide a reference to the hwmon spec for my statement about 'name' being the only mandatory file.

https://www.kernel.org/doc/html/latest/hwmon/sysfs-interface.html#attribute-access

All entries (except name) are optional, and should only be created in a given driver if the chip has the feature.

Naming and data format standards for sysfs files — The Linux Kernel documentation

from fan2go.

Please don't mix the description of separate issues. It just causes confusion when reading both of your issues. I have edited your post accordingly.

The identifier shuffling around seems to be specific to USB HID devices, maybe we can get a more unique identifier from the USB device itself 🤔 The current "fix" was implemented by @elais and allows to use a regex for the platform config argument. So you can just use corsaircpro* if you just have one device.

I am not sure what your understanding of the "platform" is, but it is a generated string that is already built using the name file exposed by hwmon. It just has additional stuff attached to it to be able to distinguish multiple instances with the same name (and to be as similar to the sensors command as possible).

So, although it might be a little more intuitive to use name instead of platform, adding a name: config option would be pretty redundant.

It might be a good idea to create an FAQ section or Wiki where such cases are described in more detail.

from fan2go.

@eldorel try the latest 0.2.4 build of fan2go and simply put "corsaircpro" in the platform fields. It should pick up your commander every time.

Perfect, thank you very much.

from fan2go.