Re: [PATCH 02/19] crypto: khazad - Fix checkpatch errors

[Herbert Xu]

On Fri, Feb 19, 2010 at 01:21:41AM +0100, Richard Hartmann wrote:
> 
> Signed-off-by: Richard Hartmann <richih.mailinglist@xxxxxxxxx>
> ---
>  crypto/khazad.c |   71 +++++++++++++++++++++++++++----------------------------
>  1 files changed, 35 insertions(+), 36 deletions(-)
>
> @@ -810,25 +810,25 @@ static void khazad_crypt(const u64 roundKey[KHAZAD_ROUNDS + 1],
>  	state = be64_to_cpu(*src) ^ roundKey[0];
>  
>  	for (r = 1; r < KHAZAD_ROUNDS; r++) {
> -		state = T0[(int)(state >> 56)       ] ^
> -			T1[(int)(state >> 48) & 0xff] ^
> -			T2[(int)(state >> 40) & 0xff] ^
> -			T3[(int)(state >> 32) & 0xff] ^
> -			T4[(int)(state >> 24) & 0xff] ^
> -			T5[(int)(state >> 16) & 0xff] ^
> -			T6[(int)(state >>  8) & 0xff] ^
> -			T7[(int)(state      ) & 0xff] ^
> +		state = T0[(int)(state >> 56)] ^
> +			T1[(int)(state >> 48)&0xff] ^
> +			T2[(int)(state >> 40)&0xff] ^
> +			T3[(int)(state >> 32)&0xff] ^
> +			T4[(int)(state >> 24)&0xff] ^
> +			T5[(int)(state >> 16)&0xff] ^
> +			T6[(int)(state >>  8)&0xff] ^
> +			T7[(int)(state      )&0xff] ^
>  			roundKey[r];
> -    	}
> +		}

This indentation change is bogus.

> @@ -865,7 +865,6 @@ static struct crypto_alg khazad_alg = {
>  static int __init khazad_mod_init(void)
>  {
>  	int ret = 0;
> -	
>  	ret = crypto_register_alg(&khazad_alg);
>  	return ret;
>  }

What's this for?

Please review the all the rest of the patches for similar errors
and resubmit.

Cheers,
-- 
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <herbert@xxxxxxxxxxxxxxxxxxx>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/