staging: wilc1000: make use of ALIGN macro

Make use of 'ALIGN' macro to align the size data value.

Signed-off-by: Ajay Singh <ajay.kathat@microchip.com>
Link: https://lore.kernel.org/r/20200214172250.13026-5-ajay.kathat@microchip.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/staging/wilc1000/sdio.c b/drivers/staging/wilc1000/sdio.c
index 66706ef..212affc 100644 (file)
--- a/drivers/staging/wilc1000/sdio.c
+++ b/drivers/staging/wilc1000/sdio.c
@@ -416,34 +416,19 @@ static int wilc_sdio_write(struct wilc *wilc, u32 addr, u8 *buf, u32 size)
        cmd.read_write = 1;
        if (addr > 0) {
                /**
-                *      has to be word aligned...
-                **/
-               if (size & 0x3) {
-                       size += 4;
-                       size &= ~0x3;
-               }
-
-               /**
                 *      func 0 access
                 **/
                cmd.function = 0;
                cmd.address = 0x10f;
        } else {
                /**
-                *      has to be word aligned...
-                **/
-               if (size & 0x3) {
-                       size += 4;
-                       size &= ~0x3;
-               }
-
-               /**
                 *      func 1 access
                 **/
                cmd.function = 1;
                cmd.address = 0;
        }
 
+       size = ALIGN(size, 4);
        nblk = size / block_size;
        nleft = size % block_size;
 
@@ -552,34 +537,19 @@ static int wilc_sdio_read(struct wilc *wilc, u32 addr, u8 *buf, u32 size)
        cmd.read_write = 0;
        if (addr > 0) {
                /**
-                *      has to be word aligned...
-                **/
-               if (size & 0x3) {
-                       size += 4;
-                       size &= ~0x3;
-               }
-
-               /**
                 *      func 0 access
                 **/
                cmd.function = 0;
                cmd.address = 0x10f;
        } else {
                /**
-                *      has to be word aligned...
-                **/
-               if (size & 0x3) {
-                       size += 4;
-                       size &= ~0x3;
-               }
-
-               /**
                 *      func 1 access
                 **/
                cmd.function = 1;
                cmd.address = 0;
        }
 
+       size = ALIGN(size, 4);
        nblk = size / block_size;
        nleft = size % block_size;
 

diff --git a/drivers/staging/wilc1000/wlan.c b/drivers/staging/wilc1000/wlan.c
index 9dfabd1..db4ef17 100644 (file)
--- a/drivers/staging/wilc1000/wlan.c
+++ b/drivers/staging/wilc1000/wlan.c
@@ -508,9 +508,7 @@ int wilc_wlan_handle_txq(struct wilc *wilc, u32 *txq_count)
                        vmm_sz = HOST_HDR_OFFSET;
 
                vmm_sz += tqe->buffer_size;
-
-               if (vmm_sz & 0x3)
-                       vmm_sz = (vmm_sz + 4) & ~0x3;
+               vmm_sz = ALIGN(vmm_sz, 4);
 
                if ((sum + vmm_sz) > WILC_TX_BUFF_SIZE)
                        break;