From: Ajay Singh <ajay.kat...@microchip.com>

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

Signed-off-by: Ajay Singh <ajay.kat...@microchip.com>
---
 drivers/staging/wilc1000/sdio.c | 34 ++-------------------------------
 drivers/staging/wilc1000/wlan.c |  4 +---
 2 files changed, 3 insertions(+), 35 deletions(-)

diff --git a/drivers/staging/wilc1000/sdio.c b/drivers/staging/wilc1000/sdio.c
index 66706efc5711..212affc4b9c1 100644
--- a/drivers/staging/wilc1000/sdio.c
+++ b/drivers/staging/wilc1000/sdio.c
@@ -415,28 +415,12 @@ 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
                 **/
@@ -444,6 +428,7 @@ static int wilc_sdio_write(struct wilc *wilc, u32 addr, u8 
*buf, u32 size)
                cmd.address = 0;
        }
 
+       size = ALIGN(size, 4);
        nblk = size / block_size;
        nleft = size % block_size;
 
@@ -551,28 +536,12 @@ 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
                 **/
@@ -580,6 +549,7 @@ static int wilc_sdio_read(struct wilc *wilc, u32 addr, u8 
*buf, u32 size)
                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 9dfabd1af4e7..db4ef175ccee 100644
--- 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;
-- 
2.24.0
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to