Added comment to mutex declarations, to resolve checkpatch.pl checks.
Signed-off-by: Rachit Dhar rchtdhr@gmail.com --- 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 e509fdc715db..1f2faa6d545d 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 gb_channel->active */ };
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 gb_lights->lights */ };
static void gb_lights_channel_free(struct gb_channel *channel);
Hey Rachit, Thanks for the patch. And the sending corrections along the path.
On Wed Feb 4, 2026 at 6:31 PM WET, Rachit Dhar wrote:
Added comment to mutex declarations, to resolve checkpatch.pl checks.
Signed-off-by: Rachit Dhar rchtdhr@gmail.com
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 e509fdc715db..1f2faa6d545d 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 gb_channel->active */
}; 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 gb_lights->lights */
};
This LGTM.
Acked-by: Rui Miguel Silva rui.silva@linaro.org
Cheers, Rui
static void gb_lights_channel_free(struct gb_channel *channel);
2.43.0
-
Rachit Dhar -
Rui Miguel Silva