diff options
author | Konstantin Porotchkin <kostap@marvell.com> | 2021-03-22 14:34:30 +0200 |
---|---|---|
committer | Manish Pandey <manish.pandey2@arm.com> | 2021-04-20 12:59:13 +0200 |
commit | dceac436f620e60cd0149194377871b225216079 (patch) | |
tree | c47f507f71fc42c8d9b95391af8b3e10afe0afb6 /plat | |
parent | 52c24e3077f6c68b31284b926fe7909fbee731b2 (diff) | |
download | trusted-firmware-a-dceac436f620e60cd0149194377871b225216079.tar.gz |
fix: plat: marvell: fix MSS loader for A8K family
Wrong brakets caused MSS FW load timeout error:
ERROR: MSS DMA failed (timeout)
ERROR: MSS FW chunk 0 load failed
ERROR: SCP Image load failed
This patch fixes the operator precedence in MSS FW load.
Signed-off-by: Konstantin Porotchkin <kostap@marvell.com>
Change-Id: I78c215606bde112f40429926c51f5fa1e4334c13
Diffstat (limited to 'plat')
-rw-r--r-- | plat/marvell/armada/common/mss/mss_scp_bootloader.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/plat/marvell/armada/common/mss/mss_scp_bootloader.c b/plat/marvell/armada/common/mss/mss_scp_bootloader.c index f669a777be..72a304a801 100644 --- a/plat/marvell/armada/common/mss/mss_scp_bootloader.c +++ b/plat/marvell/armada/common/mss/mss_scp_bootloader.c @@ -85,9 +85,9 @@ static int mss_iram_dma_load(uint32_t src_addr, uint32_t size, /* Poll DMA_ACK at MSS_DMACTLR until it is ready */ timeout = MSS_DMA_TIMEOUT; while (timeout > 0U) { - if ((mmio_read_32(MSS_DMA_CTRLR(mss_regs)) >> - (MSS_DMA_CTRLR_ACK_OFFSET & - MSS_DMA_CTRLR_ACK_MASK)) + if (((mmio_read_32(MSS_DMA_CTRLR(mss_regs)) >> + MSS_DMA_CTRLR_ACK_OFFSET) & + MSS_DMA_CTRLR_ACK_MASK) == MSS_DMA_CTRLR_ACK_READY) { break; } |