Skip to content

Commit

Permalink
lib: xlnx: Avoid mapping in MPU for region mapped by bsp
Browse files Browse the repository at this point in the history
Avoiding memory mapping in MPU config table for region that are already
mapped by bsp and no change in attribute property.

Signed-off-by: Rajiv Mohan <rajiv.mohan@amd.com>
Signed-off-by: Tanmay Shah <tanmay.shah@amd.com>
  • Loading branch information
rajimoha authored and arnopo committed Feb 19, 2024
1 parent 224cdea commit 3517a07
Showing 1 changed file with 54 additions and 1 deletion.
55 changes: 54 additions & 1 deletion lib/system/generic/xlnx/sys.c
@@ -1,6 +1,6 @@
/*
* Copyright (c) 2022, Xilinx Inc. and Contributors. All rights reserved.
* Copyright (c) 2022-2023 Advanced Micro Devices, Inc. All Rights Reserved.
* Copyright (c) 2022-2024 Advanced Micro Devices, Inc. All Rights Reserved.
*
* SPDX-License-Identifier: BSD-3-Clause
*/
Expand Down Expand Up @@ -72,12 +72,65 @@ void metal_weak metal_generic_default_poll(void)
metal_asm volatile("wfi");
}

/*
* VERSAL_NET is used since XMpu_Config structure is
* different for r52(versal net) and r5(zynqmp) to avoid build failure
*/
#ifdef VERSAL_NET
void *metal_machine_io_mem_map_versal_net(void *va, metal_phys_addr_t pa,
size_t size, unsigned int flags)
{
void *__attribute__((unused)) physaddr;
u32 req_end_addr = pa + size;
XMpu_Config mpu_config;
u32 req_addr = pa;
u32 mmap_req = 1;
u32 base_end_addr;
u32 cnt;

/* Get the MPU Config enties */
Xil_GetMPUConfig(mpu_config);

for (cnt = 0; cnt < MAX_POSSIBLE_MPU_REGS; cnt++) {

if (!mpu_config[cnt].flags & XMPU_VALID_REGION)
continue;

base_end_addr = mpu_config[cnt].Size + mpu_config[cnt].BaseAddress;

if (mpu_config[cnt].BaseAddress <= req_addr && base_end_addr >= req_end_addr) {
/*
* Mapping available for requested region in MPU table
* If no change in Attribute for region then additional
* mapping in MPU table is not required
*/
if (mpu_config[cnt].Attribute == flags) {
mmap_req = 0;
break;
}
}
}

/* if mapping is required we call Xil_MemMap to get the mapping done */
if (mmap_req == 1) {
physaddr = Xil_MemMap(pa, size, flags);
metal_assert(physaddr == (void *)pa);
}

return va;
}
#endif

void *metal_machine_io_mem_map(void *va, metal_phys_addr_t pa,
size_t size, unsigned int flags)
{
void *__attribute__((unused)) physaddr;

#ifdef VERSAL_NET
va = metal_machine_io_mem_map_versal_net(va, pa, size, flags);
#else
physaddr = Xil_MemMap(pa, size, flags);
metal_assert(physaddr == (void *)pa);
#endif
return va;
}

0 comments on commit 3517a07

Please sign in to comment.