Add a comment to the mutex definition in struct gb_camera to describe what it protects, satisfying a checkpatch.pl CHECK.
Signed-off-by: Jose A. Perez de Azpillaga azpijr@gmail.com --- Hi,
I noticed a CHECK for alignment in this file on line 267. However, fixing it pushed the line over 100 characters. I felt that adding extreme indentation might add more noise than value, so I've only included the mutex comment fix here.
I'd appreciate your guidance on whether you prefer strict alignment even if it breaks the 100-column rule in these specific cases. --- drivers/staging/greybus/camera.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/greybus/camera.c b/drivers/staging/greybus/camera.c index 62b55bb28408..a6f3637b8871 100644 --- a/drivers/staging/greybus/camera.c +++ b/drivers/staging/greybus/camera.c @@ -53,7 +53,7 @@ struct gb_camera { struct gb_connection *data_connection; u16 data_cport_id;
- struct mutex mutex; + struct mutex mutex; /* Protects camera state */ enum gb_camera_state state;
struct { -- 2.53.0