<div dir="ltr"><div dir="ltr">Hello,<div><br></div><div>Thank you for reviewing my patch. I received your feedback:</div><div>> Are you sure about all of these?  Please document them individually and</div>> put the proof of that in the changelog text.<div><br></div><div>I'd like to clarify a few points to ensure I address your concerns correctly:</div><div><br></div><div>1. By "document them individually", do you mean I should submit separate patches for each mutex documentation change?</div><div><br></div><div>2. Regarding the "proof" for the changelog text, could you please elaborate on what kind of proof you 're looking for? Should I include:</div><div> - References to the code where these mutexes are used?</div><div> - Explanations of the race conditions these mutexes prevent?</div><div> - Citations from the driver's design documentation (if available)?</div><div><br></div><div>3. Is there a preferred format or level of detail you'd like to see in the changelog for each documentation change?</div><div><br></div><div>Thank you for your time and assistance.</div><div><br></div><div>Regards,</div><div>Riyan Dhiman</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jul 31, 2024 at 1:14 PM Greg KH <<a href="mailto:gregkh@linuxfoundation.org">gregkh@linuxfoundation.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed, Jul 31, 2024 at 01:06:05PM +0530, Riyan Dhiman wrote:<br>
> Adhere to Linux kernel coding style<br>
> <br>
> Reported by checkpatch:<br>
> <br>
> CHECK: mutex definition without comment<br>
> <br>
> Signed-off-by: Riyan Dhiman <<a href="mailto:riyandhiman14@gmail.com" target="_blank">riyandhiman14@gmail.com</a>><br>
> ---<br>
>  drivers/staging/vme_user/vme_bridge.h | 4 ++++<br>
>  1 file changed, 4 insertions(+)<br>
> <br>
> diff --git a/drivers/staging/vme_user/vme_bridge.h b/drivers/staging/vme_user/vme_bridge.h<br>
> index 9bdc41bb6602..0b1f05944f0d 100644<br>
> --- a/drivers/staging/vme_user/vme_bridge.h<br>
> +++ b/drivers/staging/vme_user/vme_bridge.h<br>
> @@ -28,6 +28,7 @@ struct vme_master_resource {<br>
>  struct vme_slave_resource {<br>
>       struct list_head list;<br>
>       struct vme_bridge *parent;<br>
> +     /* Locking for VME slave resources */<br>
<br>
Are you sure about all of these?  Please document them individually and<br>
put the proof of that in the changelog text.<br>
<br>
thanks,<br>
<br>
greg k-h<br>
</blockquote></div></div>