commit 2a7ab9982cb648dba8305ad4ec02e721426e69f6
author Manorit Chawdhry <m-chawdhry@ti.com> Fri May 05 15:53:59 2023 +0530
committer Tom Rini <trini@konsulko.com> Thu Jun 01 12:32:03 2023 -0400
tree 625de9f129de4a3e4d4029b6a7b15cff5574302a
parent e45fcb0e76b51d1e805f077ec160f99ff2edfa36

Revert "arm: mach-k3: common: don't reconfigure background firewalls"

This reverts commit b8ebf24e7f4afb5093a31bdf122e1ed0781e5c3c.

This patch seems to be fundamentally wrong and requires a different way
on how the background firewalls should be configured so revert the patch

Signed-off-by: Manorit Chawdhry <m-chawdhry@ti.com>

arch/arm/mach-k3/common.c

diff --git a/arch/arm/mach-k3/common.c b/arch/arm/mach-k3/common.c
index 3c85cae..b40e930 100644
--- a/arch/arm/mach-k3/common.c
+++ b/arch/arm/mach-k3/common.c
@@ -580,10 +580,7 @@
 
 			fwl_ops->get_fwl_region(ti_sci, &region);
 
-			/* Don't disable the background regions */
-			if (region.control != 0 &&
-			    ((region.control & K3_BACKGROUND_FIREWALL_BIT) ==
-			     0)) {
+			if (region.control != 0) {
 				pr_debug("Attempting to disable firewall %5d (%25s)\n",
 					 region.fwl_id, fwl_data[i].name);
 				region.control = 0;

arch/arm/mach-k3/common.h

diff --git a/arch/arm/mach-k3/common.h b/arch/arm/mach-k3/common.h
index e7e59f5..130f502 100644
--- a/arch/arm/mach-k3/common.h
+++ b/arch/arm/mach-k3/common.h
@@ -9,10 +9,6 @@
 #include <asm/armv7_mpu.h>
 #include <asm/hardware.h>
 
-#define J721E  0xbb64
-#define J7200  0xbb6d
-#define K3_BACKGROUND_FIREWALL_BIT BIT(8)
-
 struct fwl_data {
 	const char *name;
 	u16 fwl_id;