Even if it's not critical, the avoidance of checking the error code from devm_mutex_init() call today diminishes the point of using devm variant of it. Tomorrow it may even leak something. Enforce all callers checking the return value through the compiler. The series should go through the locking tree. Signed-off-by: Thomas Weißschuh <linux@xxxxxxxxxxxxxx> --- Changes in v2: - Rebase on 6.14-rc1 - Fix up leds-1202 driver - Link to v1: https://lore.kernel.org/r/20241202-must_check-devm_mutex_init-v1-1-e60eb97b8c72@xxxxxxxxxxxxxx --- Thomas Weißschuh (2): leds: st1202: Check for error code from devm_mutex_init() call locking/mutex: Mark devm_mutex_init() as __must_check drivers/leds/leds-st1202.c | 4 +++- include/linux/mutex.h | 17 ++++++++--------- 2 files changed, 11 insertions(+), 10 deletions(-) --- base-commit: 2014c95afecee3e76ca4a56956a936e23283f05b change-id: 20241031-must_check-devm_mutex_init-cac583bda8fe Best regards, -- Thomas Weißschuh <linux@xxxxxxxxxxxxxx>
