On 12/17/24 16:07, Ivy Huang wrote:
From: Sumit Gupta <sumitg@xxxxxxxxxx> Access to safety cluster engine (SCE) fabric registers was blocked by firewall after the introduction of Functional Safety Island in Tegra234. After that, any access by software to SCE registers is correctly resulting in the internal bus error. However, when CPUs try accessing the SCE-fabric registers to print error info, another firewall error occurs as the fabric registers are also firewall protected. This results in a second error to be printed. Disable the SCE fabric node to avoid printing the misleading error. The first error info will be printed by the interrupt from the fabric causing the actual access. Cc: stable@xxxxxxxxxxxxxxx Fixes: 302e154000ec ("arm64: tegra: Add node for CBB 2.0 on Tegra234") Signed-off-by: Sumit Gupta <sumitg@xxxxxxxxxx> Signed-off-by: Ivy Huang <yijuh@xxxxxxxxxx>
I think disabling this block is the right thing to do. No need to remove it entirely. While at the moment I do not expect it to be used, I don't think anyone could say with certainty that it will never be used again.
Reviewed-by: Brad Griffis <bgriffis@xxxxxxxxxx>