[PATCH] Staging: rts5139: sd_cprm: fix coding style issues

Dan Carpenter dan.carpenter at oracle.com
Tue May 29 12:24:16 UTC 2012


On Tue, May 29, 2012 at 12:30:19PM +0100, Adnan Ali wrote:
> This Commit fixes coding style issues including long lines,
> braces with single statment if condition, deprecated min().
> 
> Signed-off-by: Adnan Ali <adnan.ali at codethink.co.uk>
> ---
>  drivers/staging/rts5139/sd_cprm.c |    8 ++++----
>  1 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/staging/rts5139/sd_cprm.c b/drivers/staging/rts5139/sd_cprm.c
> index f8c6071..ff336d3 100644
> --- a/drivers/staging/rts5139/sd_cprm.c
> +++ b/drivers/staging/rts5139/sd_cprm.c
> @@ -201,9 +201,8 @@ RTY_SEND_CMD:
>  			if (buf[1] & 0x80)
>  				TRACE_RET(chip, STATUS_FAIL);
>  		}
> -		if (buf[1] & 0x7F) {
> +		if (buf[1] & 0x7F)
>  			TRACE_RET(chip, STATUS_FAIL);
> -		}
>  		if (buf[2] & 0xF8)
>  			TRACE_RET(chip, STATUS_FAIL);
>  
> @@ -224,7 +223,8 @@ RTY_SEND_CMD:
>  	return STATUS_SUCCESS;
>  }
>  
> -static int ext_sd_get_rsp(struct rts51x_chip *chip, int len, u8 *rsp, u8 rsp_type)
> +static int ext_sd_get_rsp(struct rts51x_chip *chip, int len,
> +					 u8 *rsp, u8 rsp_type)

Don't right justify things here.  Line the 's' from struct with the
'u' from u8.

>  {
>  	int retval, rsp_len;
>  	u16 reg_addr;
> @@ -844,7 +844,7 @@ int sd_pass_thru_mode(struct scsi_cmnd *srb, struct rts51x_chip *chip)
>  
>  	buf[15] = chip->max_lun;
>  
> -	len = min(18, (int)scsi_bufflen(srb));
> +	len = min_t(int, 18, scsi_bufflen(srb));

It would be better to use "unsigned" instead of "int".  I don't
think it matters either way, but scsi_bufflen() returns unsigned and
it is a pain to audit this to see if it can return anything larger
than INT_MAX.  If we use "unsigned" here, then we don't need to
audit it.  More free time for everyone.  A change like that would
need to go in a separate commit.

regards,
dan carpenter

>  	rts51x_set_xfer_buf(buf, len, srb);
>  
>  	return TRANSPORT_GOOD;
> -- 
> 1.7.4.1
> 
> _______________________________________________
> devel mailing list
> devel at linuxdriverproject.org
> http://driverdev.linuxdriverproject.org/mailman/listinfo/devel



More information about the devel mailing list