This is the third iteration of this series which enables sensors support for the Congatec Board Controller. The cgbc_hwmon_compute_curr_channel() macro which computed the channel for current sensor was removed. To get the channel id of a currrent sensor (from the Board Controller point of view) we just have to add a constant. Regards, Thomas Signed-off-by: Thomas Richard <thomas.richard@xxxxxxxxxxx> --- Changes in v3: - remove the cgbc_hwmon_compute_curr_channel() macro. - Link to v2: https://lore.kernel.org/r/20241108-congatec-board-controller-hwmon-v2-0-16e337398527@xxxxxxxxxxx Changes in v2: - hwmon: use unsigned int type instead of u8 in struct cgbc_hwmon_sensor and struct cgbc_hwmon_data. - hwmon: in cgbc_hwmon_probe_sensors() no need to request data for the first sensor as the Board Controller returns data of the first sensors with the number of sensors. - hwmon: fix typos in comments and improve them. - hwmon: remove dead code in cgbc_hwmon_read() and in cgbc_hwmon_read_string() (deadcode was the 'return -ENODEV'). - hwmon: remove useless platform_set_drvdata(). - hwmon: channel id always refers to the same sensor. - hwmon: add a enum cgbc_sensor_types. - Link to v1: https://lore.kernel.org/r/20241104-congatec-board-controller-hwmon-v1-0-871e4cd59d8e@xxxxxxxxxxx --- Thomas Richard (2): hwmon: Add Congatec Board Controller monitoring driver mfd: cgbc: add a hwmon cell MAINTAINERS | 1 + drivers/hwmon/Kconfig | 9 ++ drivers/hwmon/Makefile | 1 + drivers/hwmon/cgbc-hwmon.c | 304 +++++++++++++++++++++++++++++++++++++++++++++ drivers/mfd/cgbc-core.c | 1 + 5 files changed, 316 insertions(+) --- base-commit: 1ffec08567f426a1c593e038cadc61bdc38cb467 change-id: 20240809-congatec-board-controller-hwmon-e9e63d957d33 Best regards, -- Thomas Richard <thomas.richard@xxxxxxxxxxx>