Re: [PATCH -next] crypto: ccp - Fix non static symbol warning
On Mon, Oct 17, 2016 at 03:08:50PM +, Wei Yongjun wrote: > From: Wei Yongjun> > Fixes the following sparse warning: > > drivers/crypto/ccp/ccp-dev.c:44:6: warning: > symbol 'ccp_error_codes' was not declared. Should it be static? > > Signed-off-by: Wei Yongjun Patch applied. Thanks. -- Email: Herbert Xu 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-crypto" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH -next] crypto: ccp - Fix non static symbol warning
On 10/17/2016 10:08 AM, Wei Yongjun wrote: > From: Wei Yongjun> > Fixes the following sparse warning: > > drivers/crypto/ccp/ccp-dev.c:44:6: warning: > symbol 'ccp_error_codes' was not declared. Should it be static? > > Signed-off-by: Wei Yongjun > --- > drivers/crypto/ccp/ccp-dev.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/crypto/ccp/ccp-dev.c b/drivers/crypto/ccp/ccp-dev.c > index cafa633..c25515a 100644 > --- a/drivers/crypto/ccp/ccp-dev.c > +++ b/drivers/crypto/ccp/ccp-dev.c > @@ -41,7 +41,7 @@ struct ccp_tasklet_data { > }; > > /* Human-readable error strings */ > -char *ccp_error_codes[] = { > +static char *ccp_error_codes[] = { > "", > "ERR 01: ILLEGAL_ENGINE", > "ERR 02: ILLEGAL_KEY_ID", > Yes, excellent. Thank you. Acked-by: Gary R Hook -- This is my day job. Follow me at: IG/Twitter/Facebook: @grhookphoto IG/Twitter/Facebook: @grhphotographer -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH -next] crypto: ccp - Fix non static symbol warning
On 10/17/2016 10:08 AM, Wei Yongjun wrote: From: Wei YongjunFixes the following sparse warning: drivers/crypto/ccp/ccp-dev.c:44:6: warning: symbol 'ccp_error_codes' was not declared. Should it be static? Signed-off-by: Wei Yongjun Excellent. Thank you. Acked-by: Gary R Hook -- This is my day job. Follow me at: IG/Twitter/Facebook: @grhookphoto IG/Twitter/Facebook: @grhphotographer -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH -next] crypto: ccp - Fix non static symbol warning
From: Wei YongjunFixes the following sparse warning: drivers/crypto/ccp/ccp-dev.c:44:6: warning: symbol 'ccp_error_codes' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/crypto/ccp/ccp-dev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/crypto/ccp/ccp-dev.c b/drivers/crypto/ccp/ccp-dev.c index cafa633..c25515a 100644 --- a/drivers/crypto/ccp/ccp-dev.c +++ b/drivers/crypto/ccp/ccp-dev.c @@ -41,7 +41,7 @@ struct ccp_tasklet_data { }; /* Human-readable error strings */ -char *ccp_error_codes[] = { +static char *ccp_error_codes[] = { "", "ERR 01: ILLEGAL_ENGINE", "ERR 02: ILLEGAL_KEY_ID", -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH -next] crypto: ccp - Fix non static symbol warning
On Fri, Aug 12, 2016 at 12:00:09AM +, Wei Yongjun wrote: > Fixes the following sparse warning: > > drivers/crypto/ccp/ccp-dev.c:62:14: warning: > symbol 'ccp_increment_unit_ordinal' was not declared. Should it be static? > > Signed-off-by: Wei YongjunPatch applied. Thanks. -- Email: Herbert Xu 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-crypto" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH -next] crypto: ccp - Fix non static symbol warning
Fixes the following sparse warning: drivers/crypto/ccp/ccp-dev.c:62:14: warning: symbol 'ccp_increment_unit_ordinal' was not declared. Should it be static? Signed-off-by: Wei Yongjun--- drivers/crypto/ccp/ccp-dev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/crypto/ccp/ccp-dev.c b/drivers/crypto/ccp/ccp-dev.c index 38a98d8..5d36eef 100644 --- a/drivers/crypto/ccp/ccp-dev.c +++ b/drivers/crypto/ccp/ccp-dev.c @@ -59,7 +59,7 @@ static struct ccp_device *ccp_rr; /* Ever-increasing value to produce unique unit numbers */ static atomic_t ccp_unit_ordinal; -unsigned int ccp_increment_unit_ordinal(void) +static unsigned int ccp_increment_unit_ordinal(void) { return atomic_inc_return(_unit_ordinal); } -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html