Re: [PATCH 9/9] staging: csr: Move else keyword up to follow after }

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, May 31, 2013 at 10:23:52PM -0700, Lisa Nguyen wrote:
> Resolved a closing brace issue generated by checkpatch.pl to
> meet kernel coding standards in sme_userspace.c
> 
> Signed-off-by: Lisa Nguyen <lisa@xxxxxxxxxxxxxxx>
> ---
>  drivers/staging/csr/sme_userspace.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/csr/sme_userspace.c b/drivers/staging/csr/sme_userspace.c
> index 7e9db10..7076d2b 100644
> --- a/drivers/staging/csr/sme_userspace.c
> +++ b/drivers/staging/csr/sme_userspace.c
> @@ -230,8 +230,7 @@ unifi_ta_indicate_protocol(void *ospriv,
>  			    packet_type,
>  			    direction,
>  			    *src_addr);
> -    }
> -    else {
> +    } else {

Oh...  It wouldn't have violated the "one thing per patch" to do
that in the previous patch.

regards,
dan carpenter

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux