> This change is really completely unrelated to a CPLD or specific SoC. > The commit description is just confusing. It should simply state that > it documents the existing fanX_fault attribute. Understood. Just to confirm should I change fanY_fault to fanX_fault in documentation as well as the patch description? For instance: /sys/class/hwmon/hwmonX/fanX_fault It seems that the documentation around it uses fanY_ format. Thanks, -Nick Hawkins