[PATCH v3 2/2] scsi: arcmsr: use round_up() instead of logical operation

[ching Huang]

From: ching Huang <ching2...@areca.com.tw>

Use round_up() instead of logical operation.

Reported-by: Martin K. Petersen <martin.peter...@oracle.com>
Signed-off-by: ching Huang <ching2...@areca.com.tw>
---

diff --git a/drivers/scsi/arcmsr/arcmsr_hba.c b/drivers/scsi/arcmsr/arcmsr_hba.c
index 55d85c9..1e358d9 100644
--- a/drivers/scsi/arcmsr/arcmsr_hba.c
+++ b/drivers/scsi/arcmsr/arcmsr_hba.c
@@ -644,13 +644,12 @@ static void arcmsr_hbaF_assign_regAddr(struct 
AdapterControlBlock *acb)
        struct MessageUnit_F __iomem *pmuF;
 
        memset(acb->dma_coherent2, 0xff, acb->completeQ_size);
-       acb->message_wbuffer = (uint32_t *)((((unsigned long)acb->dma_coherent2 
+
-               acb->completeQ_size + 3) >> 2) << 2);
+       acb->message_wbuffer = (uint32_t *)round_up((unsigned 
long)acb->dma_coherent2 +
+               acb->completeQ_size, 4);
        acb->message_rbuffer = ((void *)acb->message_wbuffer) + 0x100;
        acb->msgcode_rwbuffer = ((void *)acb->message_wbuffer) + 0x200;
        memset((void *)acb->message_wbuffer, 0, MESG_RW_BUFFER_SIZE);
-       host_buffer_dma = ((acb->dma_coherent_handle2 + acb->completeQ_size +
-               3) >> 2) << 2;
+       host_buffer_dma = round_up(acb->dma_coherent_handle2 + 
acb->completeQ_size, 4);
        pmuF = acb->pmuF;
        /* host buffer low address, bit0:1 all buffer active */
        writel(lower_32_bits(host_buffer_dma | 1), &pmuF->inbound_msgaddr0);