Align the allocation size with DMA_MINALIGN to make sure we do not
flush/invalidate data from following allocations.

Signed-off-by: Neil Armstrong <neil.armstr...@linaro.org>
---
 drivers/ufs/ufs.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/drivers/ufs/ufs.c b/drivers/ufs/ufs.c
index e4400f319a7..bc86903245c 100644
--- a/drivers/ufs/ufs.c
+++ b/drivers/ufs/ufs.c
@@ -634,7 +634,9 @@ static int ufshcd_memory_alloc(struct ufs_hba *hba)
        /* Allocate one Transfer Request Descriptor
         * Should be aligned to 1k boundary.
         */
-       hba->utrdl = memalign(1024, sizeof(struct utp_transfer_req_desc));
+       hba->utrdl = memalign(1024,
+                             ALIGN(sizeof(struct utp_transfer_req_desc),
+                                   ARCH_DMA_MINALIGN));
        if (!hba->utrdl) {
                dev_err(hba->dev, "Transfer Descriptor memory allocation 
failed\n");
                return -ENOMEM;
@@ -643,7 +645,9 @@ static int ufshcd_memory_alloc(struct ufs_hba *hba)
        /* Allocate one Command Descriptor
         * Should be aligned to 1k boundary.
         */
-       hba->ucdl = memalign(1024, sizeof(struct utp_transfer_cmd_desc));
+       hba->ucdl = memalign(1024,
+                            ALIGN(sizeof(struct utp_transfer_cmd_desc),
+                                  ARCH_DMA_MINALIGN));
        if (!hba->ucdl) {
                dev_err(hba->dev, "Command descriptor memory allocation 
failed\n");
                return -ENOMEM;

-- 
2.34.1

Reply via email to