[PATCH] staging:media:atomisp:i2c removed unnecessary white space before comma in memset()

Varsha Rao rvarsha016 at gmail.com
Mon Mar 27 13:01:34 UTC 2017


On Mon, Mar 27, 2017 at 5:12 PM,  <vaibhavddit at gmail.com> wrote:
> From: Vaibhav Kothari <vaibhavddit at gmail.com>

"From: .." should not be included in the patch. The subject is still
incorrect, there should be a space after each colon. The subject
should give overview of changes made in the patch.
It is preferred to fix single checkpatch issue in a patch.

> - Fixing up check-patch error & Warnings
> - Added blank line between declaration and defination
>   at various places

The commit message is not proper. There are no full stops at the
end of any sentence. Also change defination to definition.
Check git log for example.

static int gc2235_read_reg(struct i2c_client *client,
>                 return -EINVAL;
>         }
>
> -       memset(msg, 0 , sizeof(msg));
> +       memset(msg, 0, sizeof(msg));

This change is not reflected in commit message.

>         if (is_init == 0) {
>                 /* force gc2235 to do a reset in res change, otherwise it
> -               * can not output normal after switching res. and it is not
> -               * necessary for first time run up after power on, for the sack
> -               * of performance
> -               */
> +                * can not output normal after switching res. and it is not
> +                * necessary for first time run up after power on, for the sack
> +                * of performance
> +                */

Even this change fixes a different issue.

Send patch for a single checkpatch issue.

https://kernelnewbies.org/FirstKernelPatch#submit%2Ba%2Bpatch
Check the First Kernel Patch page.

Thanks,
Varsha Rao


More information about the devel mailing list