kernel.git/drivers/hwmon/ftsteutates.c, branch linux-5.1.y

hwmon: (ftsteutates) Use permission specific SENSOR[_DEVICE]_ATTR variants

2018-12-16T23:13:46Z

Use SENSOR[_DEVICE]_ATTR[_2]_{RO,RW,WO} to simplify the source code, to improve readbility, and to reduce the chance of inconsistencies. Also replace any remaining S_ in the driver with octal values. The conversion was done automatically with coccinelle. The semantic patches and the scripts used to generate this commit log are available at https://github.com/groeck/coccinelle-patches/hwmon/. This patch does not introduce functional changes. It was verified by compiling the old and new files and comparing text and data sizes. Signed-off-by: Guenter Roeck

hwmon: (ftsteutates) constify i2c_device_id

2017-08-30T00:44:23Z

i2c_device_id are not supposed to change at runtime. All functions working with i2c_device_id provided by work with const i2c_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav Signed-off-by: Guenter Roeck

hwmon: (ftsteutates) Fix clearing alarm sysfs entries

2017-08-13T15:24:01Z

sysfs store functions should return the number of bytes written. Returning zero results in an endless loop. Fixes: 08426eda58e0 ("hwmon: Add driver for FTS BMC chip "Teutates"") Signed-off-by: Thilo Cestonaro [groeck: Clean up documentation change and description] Signed-off-by: Guenter Roeck

hwmon: (ftsteutates) Add i2c detect functionality

2016-09-09T04:34:16Z

Signed-off-by: Thilo Cestonaro Signed-off-by: Guenter Roeck

hwmon: (ftsteutates) Fix potential memory access error

2016-07-31T22:02:51Z

Using set_bit() to set a bit in an integer is not a good idea, since the function expects an unsigned long as argument, which can be 64 bit wide. Coverity reports this problem as >>> CID 1364488: Memory - illegal accesses (INCOMPATIBLE_CAST) >>> Pointer "&ret" points to an object whose effective type is "int" >>> (32 bits, signed) but is dereferenced as a wider "unsigned +long" (64 bits, unsigned). This may lead to memory corruption. 245 set_bit(1, (unsigned long *)&ret); Just use BIT instead. Cc: Thilo Cestonaro Fixes: 08426eda58e0 ("hwmon: Add driver for FTS BMC chip "Teutates"") Signed-off-by: Guenter Roeck

hwmon: (ftsteutates) Remove unused including

2016-07-20T14:12:12Z

Remove including that don't need it. Signed-off-by: Wei Yongjun Signed-off-by: Guenter Roeck

hwmon: Add driver for FTS BMC chip "Teutates"

2016-07-20T13:29:54Z

This driver implements hardware monitoring and watchdog support for the FTS BMC Chip "Teutates". Signed-off-by: Thilo Cestonaro [groeck: Updated subject and description; fixed dependencies] Signed-off-by: Guenter Roeck