From 19799fd8fb318682a8169d5e4fc32e41ae50c0b1 Mon Sep 17 00:00:00 2001 From: Amit Nagal Date: Tue, 10 Dec 2024 10:26:48 +0530 Subject: [PATCH] fix(versal-net): remove_redundant_lock_defs HW_ASSISTED_COHERENCY is always enabled on cortex a78 based versal-net platform. hence remove the redundant definitions for pm_client_lock_get and pm_client_lock_release. Change-Id: Ifcf1bca0d494cfeb4ca23f6c884dfa5a347f786b Signed-off-by: Amit Nagal --- plat/xilinx/versal_net/pm_service/pm_client.c | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/plat/xilinx/versal_net/pm_service/pm_client.c b/plat/xilinx/versal_net/pm_service/pm_client.c index cff400cf5..d072d667c 100644 --- a/plat/xilinx/versal_net/pm_service/pm_client.c +++ b/plat/xilinx/versal_net/pm_service/pm_client.c @@ -30,22 +30,6 @@ DEFINE_RENAME_SYSREG_RW_FUNCS(cpu_pwrctrl_val, S3_0_C15_C2_7) -/* - * ARM v8.2, the cache will turn off automatically when cpu - * power down. Therefore, there is no doubt to use the spin_lock here. - */ -#if !HW_ASSISTED_COHERENCY -DEFINE_BAKERY_LOCK(pm_client_secure_lock); -static inline void pm_client_lock_get(void) -{ - bakery_lock_get(&pm_client_secure_lock); -} - -static inline void pm_client_lock_release(void) -{ - bakery_lock_release(&pm_client_secure_lock); -} -#else spinlock_t pm_client_secure_lock; static inline void pm_client_lock_get(void) { @@ -56,7 +40,6 @@ static inline void pm_client_lock_release(void) { spin_unlock(&pm_client_secure_lock); } -#endif static const struct pm_ipi apu_ipi = { .local_ipi_id = IPI_LOCAL_ID,