Received-SPF: pass (google.com: domain of linux-crypto-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH] crypto: sa2ul - Fix build warnings
To:     Herbert Xu <herbert@gondor.apana.org.au>,
        Linux Crypto Mailing List <linux-crypto@vger.kernel.org>,
        <j-keerthy@ti.com>
References: <20200723074350.GA3233@gondor.apana.org.au>
From:   Tero Kristo <t-kristo@ti.com>
Message-ID: <4e5eee05-c956-448f-10ea-06102852e979@ti.com>
Date:   Thu, 23 Jul 2020 12:57:23 +0300
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.8.0
MIME-Version: 1.0
In-Reply-To: <20200723074350.GA3233@gondor.apana.org.au>
Content-Type: text/plain; charset="utf-8"; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-crypto-owner@vger.kernel.org
Precedence: bulk

On 23/07/2020 10:43, Herbert Xu wrote:
> This patch fixes a bunch of initialiser warnings.
> 
> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>

Looks ok to me, however I never saw any build warnings with the code 
myself. Which compiler/version produces them?

-Tero

> 
> diff --git a/drivers/crypto/sa2ul.c b/drivers/crypto/sa2ul.c
> index ebcdffcdb686..fc3a8268e2c8 100644
> --- a/drivers/crypto/sa2ul.c
> +++ b/drivers/crypto/sa2ul.c
> @@ -916,7 +916,7 @@ static int sa_cipher_setkey(struct crypto_skcipher *tfm, const u8 *key,
>   static int sa_aes_cbc_setkey(struct crypto_skcipher *tfm, const u8 *key,
>   			     unsigned int keylen)
>   {
> -	struct algo_data ad = { 0 };
> +	struct algo_data ad = {};
>   	/* Convert the key size (16/24/32) to the key size index (0/1/2) */
>   	int key_idx = (keylen >> 3) - 2;
>   
> @@ -936,7 +936,7 @@ static int sa_aes_cbc_setkey(struct crypto_skcipher *tfm, const u8 *key,
>   static int sa_aes_ecb_setkey(struct crypto_skcipher *tfm, const u8 *key,
>   			     unsigned int keylen)
>   {
> -	struct algo_data ad = { 0 };
> +	struct algo_data ad = {};
>   	/* Convert the key size (16/24/32) to the key size index (0/1/2) */
>   	int key_idx = (keylen >> 3) - 2;
>   
> @@ -954,7 +954,7 @@ static int sa_aes_ecb_setkey(struct crypto_skcipher *tfm, const u8 *key,
>   static int sa_3des_cbc_setkey(struct crypto_skcipher *tfm, const u8 *key,
>   			      unsigned int keylen)
>   {
> -	struct algo_data ad = { 0 };
> +	struct algo_data ad = {};
>   
>   	ad.mci_enc = mci_cbc_3des_enc_array;
>   	ad.mci_dec = mci_cbc_3des_dec_array;
> @@ -968,7 +968,7 @@ static int sa_3des_cbc_setkey(struct crypto_skcipher *tfm, const u8 *key,
>   static int sa_3des_ecb_setkey(struct crypto_skcipher *tfm, const u8 *key,
>   			      unsigned int keylen)
>   {
> -	struct algo_data ad = { 0 };
> +	struct algo_data ad = {};
>   
>   	ad.mci_enc = mci_ecb_3des_enc_array;
>   	ad.mci_dec = mci_ecb_3des_dec_array;
> @@ -1233,7 +1233,7 @@ static int sa_cipher_run(struct skcipher_request *req, u8 *iv, int enc)
>   	struct sa_tfm_ctx *ctx =
>   	    crypto_skcipher_ctx(crypto_skcipher_reqtfm(req));
>   	struct crypto_alg *alg = req->base.tfm->__crt_alg;
> -	struct sa_req sa_req = { 0 };
> +	struct sa_req sa_req = {};
>   	int ret;
>   
>   	if (!req->cryptlen)
> @@ -1344,7 +1344,7 @@ static int sa_sha_run(struct ahash_request *req)
>   {
>   	struct sa_tfm_ctx *ctx = crypto_ahash_ctx(crypto_ahash_reqtfm(req));
>   	struct sa_sha_req_ctx *rctx = ahash_request_ctx(req);
> -	struct sa_req sa_req = { 0 };
> +	struct sa_req sa_req = {};
>   	size_t auth_len;
>   
>   	auth_len = req->nbytes;
> @@ -1566,7 +1566,7 @@ static int sa_sha_export(struct ahash_request *req, void *out)
>   
>   static int sa_sha1_cra_init(struct crypto_tfm *tfm)
>   {
> -	struct algo_data ad = { 0 };
> +	struct algo_data ad = {};
>   	struct sa_tfm_ctx *ctx = crypto_tfm_ctx(tfm);
>   
>   	sa_sha_cra_init_alg(tfm, "sha1");
> @@ -1582,7 +1582,7 @@ static int sa_sha1_cra_init(struct crypto_tfm *tfm)
>   
>   static int sa_sha256_cra_init(struct crypto_tfm *tfm)
>   {
> -	struct algo_data ad = { 0 };
> +	struct algo_data ad = {};
>   	struct sa_tfm_ctx *ctx = crypto_tfm_ctx(tfm);
>   
>   	sa_sha_cra_init_alg(tfm, "sha256");
> @@ -1598,7 +1598,7 @@ static int sa_sha256_cra_init(struct crypto_tfm *tfm)
>   
>   static int sa_sha512_cra_init(struct crypto_tfm *tfm)
>   {
> -	struct algo_data ad = { 0 };
> +	struct algo_data ad = {};
>   	struct sa_tfm_ctx *ctx = crypto_tfm_ctx(tfm);
>   
>   	sa_sha_cra_init_alg(tfm, "sha512");
> @@ -1842,7 +1842,7 @@ static int sa_aead_setauthsize(struct crypto_aead *tfm, unsigned int authsize)
>   static int sa_aead_cbc_sha1_setkey(struct crypto_aead *authenc,
>   				   const u8 *key, unsigned int keylen)
>   {
> -	struct algo_data ad = { 0 };
> +	struct algo_data ad = {};
>   
>   	ad.ealg_id = SA_EALG_ID_AES_CBC;
>   	ad.aalg_id = SA_AALG_ID_HMAC_SHA1;
> @@ -1855,7 +1855,7 @@ static int sa_aead_cbc_sha1_setkey(struct crypto_aead *authenc,
>   static int sa_aead_cbc_sha256_setkey(struct crypto_aead *authenc,
>   				     const u8 *key, unsigned int keylen)
>   {
> -	struct algo_data ad = { 0 };
> +	struct algo_data ad = {};
>   
>   	ad.ealg_id = SA_EALG_ID_AES_CBC;
>   	ad.aalg_id = SA_AALG_ID_HMAC_SHA2_256;
> @@ -1869,7 +1869,7 @@ static int sa_aead_run(struct aead_request *req, u8 *iv, int enc)
>   {
>   	struct crypto_aead *tfm = crypto_aead_reqtfm(req);
>   	struct sa_tfm_ctx *ctx = crypto_aead_ctx(tfm);
> -	struct sa_req sa_req = { 0 };
> +	struct sa_req sa_req = {};
>   	size_t auth_size, enc_size;
>   
>   	enc_size = req->cryptlen;
> 

--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki