From f617868678c230a080df4061b17a40b19a3bd048 Mon Sep 17 00:00:00 2001 From: Varun Wadekar Date: Fri, 6 Jul 2018 10:39:32 -0700 Subject: [PATCH] Tegra: fiq_glue: remove bakery locks from interrupt handler This patch removes usage of bakery_locks from the FIQ handler, as it creates unnecessary dependency whenever the watchdog timer interrupt fires. All operations inside the interrupt handler are 'reads', so no need for serialization. Change-Id: I3f675e610e4dabc5b1435fdd24bc28e424f5a8e4 Signed-off-by: Varun Wadekar --- plat/nvidia/tegra/common/tegra_fiq_glue.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/plat/nvidia/tegra/common/tegra_fiq_glue.c b/plat/nvidia/tegra/common/tegra_fiq_glue.c index 8e198ae76..dee99fb13 100644 --- a/plat/nvidia/tegra/common/tegra_fiq_glue.c +++ b/plat/nvidia/tegra/common/tegra_fiq_glue.c @@ -1,5 +1,6 @@ /* * Copyright (c) 2016-2020, ARM Limited and Contributors. All rights reserved. + * Copyright (c) 2020, NVIDIA Corporation. All rights reserved. * * SPDX-License-Identifier: BSD-3-Clause */ @@ -12,7 +13,6 @@ #include #include #include -#include #include #include @@ -25,8 +25,6 @@ /* Legacy FIQ used by earlier Tegra platforms */ #define LEGACY_FIQ_PPI_WDT 28U -static DEFINE_BAKERY_LOCK(tegra_fiq_lock); - /******************************************************************************* * Static variables ******************************************************************************/ @@ -57,8 +55,6 @@ static uint64_t tegra_fiq_interrupt_handler(uint32_t id, */ irq = plat_ic_get_pending_interrupt_id(); - bakery_lock_get(&tegra_fiq_lock); - /* * Jump to NS world only if the NS world's FIQ handler has * been registered @@ -107,8 +103,6 @@ static uint64_t tegra_fiq_interrupt_handler(uint32_t id, plat_ic_end_of_interrupt(irq); } - bakery_lock_release(&tegra_fiq_lock); - return 0; }