Re: [PATCH 1/8] Staging: bcm: IPv6Protocol.c: Fix coding style

[Kevin McKinney <klmckinney1@xxxxxxxxx>]

On Tue, Apr 03, 2012 at 12:35:24PM +0100, Max Tottenham wrote:
> The first in a series of 8 patches to fix IPv6Protocol.c
> This first patch fixes formatting issues for braced statements
> (e.g. if/for/while)
> 
> Signed-off-by: Max Tottenham <mtottenh@xxxxxxxxx>
> ---
>  drivers/staging/bcm/IPv6Protocol.c |  158 ++++++++++++++----------------------
>  1 files changed, 62 insertions(+), 96 deletions(-)
> 
> diff --git a/drivers/staging/bcm/IPv6Protocol.c b/drivers/staging/bcm/IPv6Protocol.c
> index 5b4fd37..cd5f1c3 100644
> --- a/drivers/staging/bcm/IPv6Protocol.c
> +++ b/drivers/staging/bcm/IPv6Protocol.c
> @@ -9,20 +9,18 @@ static UCHAR * GetNextIPV6ChainedHeader(UCHAR **ppucPayload,UCHAR *pucNextHeader
>  	UCHAR *pucRetHeaderPtr = NULL;
>  	UCHAR *pucPayloadPtr = NULL;
>  	USHORT  usNextHeaderOffset = 0 ;
> -    PMINI_ADAPTER Adapter = GET_BCM_ADAPTER(gblpnetdev);
> +	PMINI_ADAPTER Adapter = GET_BCM_ADAPTER(gblpnetdev);
>  
> -	if((NULL == ppucPayload) || (*pusPayloadLength == 0) || (*bParseDone))
> -	{
> +	if ((NULL == ppucPayload) || (*pusPayloadLength == 0) ||
> +		(*bParseDone)) {
>  		*bParseDone = TRUE;
>  		return NULL;
> -
>  	}
>  

These patches look good to me. Thanks Max!

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