linux-kernel - Re: [PATCH] Staging: fsl-mc: Changed allocation types into the types that the kernel prefers

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Mon, 7 Nov 2016 11:31:03 +0100
From:   Greg KH <gregkh@...uxfoundation.org>
To:     Shiva Kerdel <shiva@...ev.nl>
Cc:     German.Rivera@...escale.com, devel@...verdev.osuosl.org,
        nipun.gupta@....com, stuart.yoder@....com,
        linux-kernel@...r.kernel.org, german.rivera@....com,
        treding@...dia.com, bharat.bhushan@....com, itai.katz@....com
Subject: Re: [PATCH] Staging: fsl-mc: Changed allocation types into the types
 that the kernel prefers

On Mon, Nov 07, 2016 at 12:12:01AM +0100, Shiva Kerdel wrote:
> Fixed reports from checkpatch that pointed out that the code was not
> using the preferred types of the kernel.
> 
> Signed-off-by: Shiva Kerdel <shiva@...ev.nl>
> ---
>  drivers/staging/fsl-mc/bus/dpmcp.h      | 80 ++++++++++++++++-----------------
>  drivers/staging/fsl-mc/include/mc-bus.h |  4 +-
>  drivers/staging/fsl-mc/include/mc.h     |  2 +-
>  3 files changed, 43 insertions(+), 43 deletions(-)
> 
> diff --git a/drivers/staging/fsl-mc/bus/dpmcp.h b/drivers/staging/fsl-mc/bus/dpmcp.h
> index fe79d4d..684c5b5 100644
> --- a/drivers/staging/fsl-mc/bus/dpmcp.h
> +++ b/drivers/staging/fsl-mc/bus/dpmcp.h
> @@ -39,7 +39,7 @@
>  struct fsl_mc_io;
>  
>  int dpmcp_open(struct fsl_mc_io *mc_io,
> -	       uint32_t cmd_flags,
> +	       u32 cmd_flags,
>  	       int dpmcp_id,
>  	       uint16_t *token);
>  
> @@ -47,7 +47,7 @@ int dpmcp_open(struct fsl_mc_io *mc_io,
>  #define DPMCP_GET_PORTAL_ID_FROM_POOL (-1)
>  
>  int dpmcp_close(struct fsl_mc_io *mc_io,
> -		uint32_t cmd_flags,
> +		u32 cmd_flags,
>  		uint16_t token);
>  
>  /**
> @@ -60,16 +60,16 @@ struct dpmcp_cfg {
>  };
>  
>  int dpmcp_create(struct fsl_mc_io	*mc_io,
> -		 uint32_t		cmd_flags,
> +		 u32			cmd_flags,
>  		 const struct dpmcp_cfg	*cfg,
> -		uint16_t		*token);
> +		 uint16_t		*token);

Why did you change the indentation of this line?

Please only do one type of thing, like change the unit32_t types to u32,
in one patch, then do the uint64_t, and then, you forgot the uint16_t
changes :(

Please fix up and make a patch series for this.

thanks,

greg k-h