crypto: ux500 - Fix COMPILE_TEST warnings
authorHerbert Xu <herbert@gondor.apana.org.au>
Thu, 5 Sep 2019 04:51:37 +0000 (14:51 +1000)
committerHerbert Xu <herbert@gondor.apana.org.au>
Mon, 9 Sep 2019 13:48:41 +0000 (23:48 +1000)
This patch fixes a number of warnings encountered when this driver
is built on a 64-bit platform with COMPILE_TEST.

Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/ux500/cryp/cryp_core.c
drivers/crypto/ux500/hash/hash_core.c

index e966e9a64501c7be950e5e75d3128acd7406b69f..1628ae7a14677210b99f2ac33dcb6021da63638b 100644 (file)
@@ -528,9 +528,9 @@ static int cryp_set_dma_transfer(struct cryp_ctx *ctx,
 
        dev_dbg(ctx->device->dev, "[%s]: ", __func__);
 
-       if (unlikely(!IS_ALIGNED((u32)sg, 4))) {
+       if (unlikely(!IS_ALIGNED((unsigned long)sg, 4))) {
                dev_err(ctx->device->dev, "[%s]: Data in sg list isn't "
-                       "aligned! Addr: 0x%08x", __func__, (u32)sg);
+                       "aligned! Addr: 0x%08lx", __func__, (unsigned long)sg);
                return -EFAULT;
        }
 
@@ -763,9 +763,9 @@ static int hw_crypt_noxts(struct cryp_ctx *ctx,
 
        ctx->outlen = ctx->datalen;
 
-       if (unlikely(!IS_ALIGNED((u32)indata, 4))) {
+       if (unlikely(!IS_ALIGNED((unsigned long)indata, 4))) {
                pr_debug(DEV_DBG_NAME " [%s]: Data isn't aligned! Addr: "
-                        "0x%08x", __func__, (u32)indata);
+                        "0x%08lx", __func__, (unsigned long)indata);
                return -EINVAL;
        }
 
index f1ebc3dfa21e005d4399a735807c81a35b6714b6..c172a695347777752c4a7fef17ce0cab36534074 100644 (file)
@@ -806,7 +806,7 @@ static int hash_process_data(struct hash_device_data *device_data,
                         * HW peripheral, otherwise we first copy data
                         * to a local buffer
                         */
-                       if ((0 == (((u32)data_buffer) % 4)) &&
+                       if (IS_ALIGNED((unsigned long)data_buffer, 4) &&
                            (0 == *index))
                                hash_processblock(device_data,
                                                  (const u32 *)data_buffer,
@@ -864,7 +864,8 @@ static int hash_dma_final(struct ahash_request *req)
        if (ret)
                return ret;
 
-       dev_dbg(device_data->dev, "%s: (ctx=0x%x)!\n", __func__, (u32) ctx);
+       dev_dbg(device_data->dev, "%s: (ctx=0x%lx)!\n", __func__,
+               (unsigned long)ctx);
 
        if (req_ctx->updated) {
                ret = hash_resume_state(device_data, &device_data->state);
@@ -969,7 +970,8 @@ static int hash_hw_final(struct ahash_request *req)
        if (ret)
                return ret;
 
-       dev_dbg(device_data->dev, "%s: (ctx=0x%x)!\n", __func__, (u32) ctx);
+       dev_dbg(device_data->dev, "%s: (ctx=0x%lx)!\n", __func__,
+               (unsigned long)ctx);
 
        if (req_ctx->updated) {
                ret = hash_resume_state(device_data, &device_data->state);
@@ -1272,8 +1274,8 @@ void hash_get_digest(struct hash_device_data *device_data,
        else
                loop_ctr = SHA256_DIGEST_SIZE / sizeof(u32);
 
-       dev_dbg(device_data->dev, "%s: digest array:(0x%x)\n",
-               __func__, (u32) digest);
+       dev_dbg(device_data->dev, "%s: digest array:(0x%lx)\n",
+               __func__, (unsigned long)digest);
 
        /* Copy result into digest array */
        for (count = 0; count < loop_ctr; count++) {