Re: [PATCH 2/2] staging: media: ipu3: code style fix - missing a blank line after declarations

From: Greg Kroah-Hartman
Date: Sun Feb 21 2021 - 03:21:48 EST


On Sun, Feb 21, 2021 at 11:12:36AM +0300, Nikolay Kyx wrote:
> This patch fixes the following checkpatch.pl warning:
>
> WARNING: Missing a blank line after declarations
>
> in file ipu3-css-fw.h
>
> Signed-off-by: Nikolay Kyx <knv418@xxxxxxxxx>
> ---
>
> Additionally some style warnings remain valid here and could be fixed by
> another patch.
>
> drivers/staging/media/ipu3/ipu3-css-fw.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/staging/media/ipu3/ipu3-css-fw.h b/drivers/staging/media/ipu3/ipu3-css-fw.h
> index 79ffa7045139..3c078f15a295 100644
> --- a/drivers/staging/media/ipu3/ipu3-css-fw.h
> +++ b/drivers/staging/media/ipu3/ipu3-css-fw.h
> @@ -148,6 +148,7 @@ union imgu_fw_union {
> struct imgu_fw_info {
> size_t header_size; /* size of fw header */
> u32 type __aligned(8); /* enum imgu_fw_type */
> +
> union imgu_fw_union info; /* Binary info */
> struct imgu_abi_blob_info blob; /* Blob info */
> /* Dynamic part */

With your knowledge of C, does this change look correct?

thanks,

greg k-h