[PATCH 2/2] staging: greybus: light: add comments to mutex definitions
From: contectforbusiness
Date: Fri Sep 04 2026 - 02:07:56 EST
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@xxxxxxxxx>
---
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);