checkpatch complains about:
CHECK: struct mutex definition without comment
Add short comments describing what each mutex protects. The channel lock protects the channel state and lights_lock protects the lights array.
No functional change, just quiets checkpatch.
Signed-off-by: Vaibhav contectforbusiness@proton.me --- drivers/staging/greybus/light.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/greybus/light.c b/drivers/staging/greybus/light.c index cab02b5da..a036fa2a0 100644 --- a/drivers/staging/greybus/light.c +++ b/drivers/staging/greybus/light.c @@ -37,7 +37,7 @@ struct gb_channel { bool releasing; bool strobe_state; bool active; - struct mutex lock; + struct mutex lock; /* protects channel */ };
struct gb_light { @@ -59,7 +59,7 @@ struct gb_lights { struct gb_connection *connection; u8 lights_count; struct gb_light *lights; - struct mutex lights_lock; + struct mutex lights_lock; /* protects lights */ };
static void gb_lights_channel_free(struct gb_channel *channel);