2010-02-10 01:31:22

by Richard Hartmann

[permalink] [raw]
Subject: [PATCH 1/1] CRYPTO: Fix checkpatch errors & warnings in cipher.c


Signed-off-by: Richard Hartmann <[email protected]>
---
crypto/compress.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/crypto/compress.c b/crypto/compress.c
index 1ee3570..c33f076 100644
--- a/crypto/compress.c
+++ b/crypto/compress.c
@@ -7,7 +7,7 @@
*
* This program is free software; you can redistribute it and/or modify it
* under the terms of the GNU General Public License as published by the Free
- * Software Foundation; either version 2 of the License, or (at your option)
+ * Software Foundation; either version 2 of the License, or (at your option)
* any later version.
*
*/
@@ -39,7 +39,7 @@ int crypto_init_compress_ops(struct crypto_tfm *tfm)

ops->cot_compress = crypto_compress;
ops->cot_decompress = crypto_decompress;
-
+
return 0;
}

--
1.6.6.1



2010-02-10 02:21:28

by Richard Hartmann

[permalink] [raw]
Subject: Re: [PATCH 1/1] CRYPTO: Fix checkpatch errors & warnings in cipher.c

Obviously, this should have been named

CRYPTO: Fix checkpatch errors & warnings in compress.c

Should I re-send the patch or is it less noise & hassle if I just leave
it this way?


Richard

On Wed, Feb 10, 2010 at 02:30, Richard Hartmann
<[email protected]> wrote:
>
> Signed-off-by: Richard Hartmann <[email protected]>
> ---
>  crypto/compress.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/crypto/compress.c b/crypto/compress.c
> index 1ee3570..c33f076 100644
> --- a/crypto/compress.c
> +++ b/crypto/compress.c
> @@ -7,7 +7,7 @@
>  *
>  * This program is free software; you can redistribute it and/or modify it
>  * under the terms of the GNU General Public License as published by the Free
> - * Software Foundation; either version 2 of the License, or (at your option)
> + * Software Foundation; either version 2 of the License, or (at your option)
>  * any later version.
>  *
>  */
> @@ -39,7 +39,7 @@ int crypto_init_compress_ops(struct crypto_tfm *tfm)
>
>        ops->cot_compress = crypto_compress;
>        ops->cot_decompress = crypto_decompress;
> -
> +
>        return 0;
>  }
>
> --
> 1.6.6.1
>
>

2010-02-11 04:46:11

by Herbert Xu

[permalink] [raw]
Subject: Re: [PATCH 1/1] CRYPTO: Fix checkpatch errors & warnings in cipher.c

On Wed, Feb 10, 2010 at 03:21:26AM +0100, Richard Hartmann wrote:
> Obviously, this should have been named
>
> CRYPTO: Fix checkpatch errors & warnings in compress.c
>
> Should I re-send the patch or is it less noise & hassle if I just leave
> it this way?

You don't have to resend it.

Thanks,
--
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <[email protected]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

2010-02-16 11:56:17

by Richard Hartmann

[permalink] [raw]
Subject: Re: [PATCH 1/1] CRYPTO: Fix checkpatch errors & warnings in cipher.c

On Thu, Feb 11, 2010 at 05:46, Herbert Xu <[email protected]> wrote:

>> Should I re-send the patch or is it less noise & hassle if I just leave
>> it this way?
>
> You don't have to resend it.

GregKH said that I should just re-send all patches after a week. OTOH, I
am not sure if this is not a little bit too agressive.

Thus, I wanted to check if I should wait some more or what the general
consensus of action in the crypto subtree is.

And yes, I am aware that janitor work, and thus my patches, is very low
priority.


Thanks,
Richard

2010-02-16 12:00:15

by Herbert Xu

[permalink] [raw]
Subject: Re: [PATCH 1/1] CRYPTO: Fix checkpatch errors & warnings in cipher.c

On Tue, Feb 16, 2010 at 12:56:15PM +0100, Richard Hartmann wrote:
>
> Thus, I wanted to check if I should wait some more or what the general
> consensus of action in the crypto subtree is.
>
> And yes, I am aware that janitor work, and thus my patches, is very low
> priority.

For the crypto tree at least I've got all your patches so you
don't have to resend.

For other trees please follow whatever their maintainer (or Greg)
says.

Cheers,
--
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <[email protected]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

2010-02-16 12:06:29

by Richard Hartmann

[permalink] [raw]
Subject: Re: [PATCH 1/1] CRYPTO: Fix checkpatch errors & warnings in cipher.c

On Tue, Feb 16, 2010 at 13:00, Herbert Xu <[email protected]> wrote:

> For the crypto tree at least I've got all your patches so you
> don't have to resend.

OK, thank you very much. Expect more patches, in this case :)

One more question on workflow, though.. I have been checking
http://git.kernel.org/?p=linux/kernel/git/herbert/crypto-2.6.git;a=summary
for updates, but obviously, this was the wrong place.

Where should I look for your commits so I don't have to have you
waste your time answering emails?


Thanks,
Richard

2010-02-16 12:21:33

by Herbert Xu

[permalink] [raw]
Subject: Re: [PATCH 1/1] CRYPTO: Fix checkpatch errors & warnings in cipher.c

On Tue, Feb 16, 2010 at 01:06:27PM +0100, Richard Hartmann wrote:
>
> One more question on workflow, though.. I have been checking
> http://git.kernel.org/?p=linux/kernel/git/herbert/crypto-2.6.git;a=summary
> for updates, but obviously, this was the wrong place.

You should check the cryptodev-2.6 tree. However, they won't be
there yet as I haven't pushed your changes out. Expect them to
be there by tomorrow.

Cheers,
--
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <[email protected]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt