[PATCH] staging: ion: Fix checkpatch.pl coding style issues

Greg KH gregkh at linuxfoundation.org
Thu Sep 22 07:12:17 UTC 2016


On Thu, Sep 22, 2016 at 09:57:16AM +0300, Antti Keränen wrote:
> This patch fixes two coding style issues found with checkpatch.pl. Both
> of them are warnings.

Ah, then this should be two separate patches :)

Also, always use scripts/get_maintainer.pl to determine who to cc: on a
patch.

> 
> uapi/ion.h had an unaligned block comment. ion/hisilicon/hi6220_ion.c
> had unnecessary braces around a single statement block.
> 
> One warning in uapi/ion.h was left in place that wanted a typedef
> (ion_user_handle_t) to be removed.  I think the typedef makes sense so
> I left it intact.

Again, this paragraph makes no sense, don't talk about what you did not
do, why would that matter in a changelog comment?

thanks,

greg k-h


More information about the devel mailing list