hwmon: Relax name attribute validation for new APIs
authorGuenter Roeck <linux@roeck-us.net>
Sat, 28 Jan 2017 03:35:57 +0000 (19:35 -0800)
committerGuenter Roeck <linux@roeck-us.net>
Tue, 31 Jan 2017 11:52:15 +0000 (03:52 -0800)
commit74d3b6419772e49563877b9a3c502b763d24b075
tree92361dbe382a4c6f4b19db2f0704a8d220624a88
parentf172841573c41428b422b541bd5f50af0144c330
hwmon: Relax name attribute validation for new APIs

While invalid name attributes are really not desirable and do mess up
libsensors, enforcing valid names has the detrimental effect of driving
users away from using the new hardware monitoring API, especially those
registering name attributes violating the ABI restrictions. Another
undesirable side effect is that this violation and the resulting error
may only be discovered some time after a conversion to the new API,
which in turn may trigger a revert of that conversion.

To solve the problem, relax validation and only issue a warning instead
of returning an error if a name attribute violating the ABI is provided.
This lets callers continue to provide invalid name attributes while
notifying them about it.

Many thanks are due to Dmitry Torokhov for the idea.

Reviewed-by: Jean Delvare <jdelvare@suse.de>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
drivers/hwmon/hwmon.c