| Message ID | 20200313090104.20750-1-digetx@gmail.com |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [v1] ARM: tegra: Correct PL310 Auxiliary Control Register initialization | expand |
Le ven. 13 mars 2020 à 10:02, Dmitry Osipenko <digetx@gmail.com> a écrit : > > The PL310 Auxiliary Control Register shouldn't have the "Full line of > zero" optimization bit being set before L2 cache is enabled. The L2X0 > driver takes care of enabling the optimization by itself. > > This patch fixes a noisy error message on Tegra20 and Tegra30 telling > that cache optimization is erroneously enabled without enabling it for > the CPU: > > L2C-310: enabling full line of zeros but not enabled in Cortex-A9 > > Cc: <stable@vger.kernel.org> > Signed-off-by: Dmitry Osipenko <digetx@gmail.com> Tested-by: Nicolas Chauvet <kwizart@gmail.com> This was tested on paz00 (tegra20). Also tested that suspend still works. Thanks for fixing this long standing issue!
On Fri, Mar 13, 2020 at 12:01:04PM +0300, Dmitry Osipenko wrote: > The PL310 Auxiliary Control Register shouldn't have the "Full line of > zero" optimization bit being set before L2 cache is enabled. The L2X0 > driver takes care of enabling the optimization by itself. > > This patch fixes a noisy error message on Tegra20 and Tegra30 telling > that cache optimization is erroneously enabled without enabling it for > the CPU: > > L2C-310: enabling full line of zeros but not enabled in Cortex-A9 > > Cc: <stable@vger.kernel.org> > Signed-off-by: Dmitry Osipenko <digetx@gmail.com> > --- > arch/arm/mach-tegra/tegra.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Applied to for-5.8/arm/core, thanks. Thierry
diff --git a/arch/arm/mach-tegra/tegra.c b/arch/arm/mach-tegra/tegra.c index f1ce2857a251..b620b0651157 100644 --- a/arch/arm/mach-tegra/tegra.c +++ b/arch/arm/mach-tegra/tegra.c @@ -107,8 +107,8 @@ static const char * const tegra_dt_board_compat[] = { }; DT_MACHINE_START(TEGRA_DT, "NVIDIA Tegra SoC (Flattened Device Tree)") - .l2c_aux_val = 0x3c400001, - .l2c_aux_mask = 0xc20fc3fe, + .l2c_aux_val = 0x3c400000, + .l2c_aux_mask = 0xc20fc3ff, .smp = smp_ops(tegra_smp_ops), .map_io = tegra_map_common_io, .init_early = tegra_init_early,
The PL310 Auxiliary Control Register shouldn't have the "Full line of zero" optimization bit being set before L2 cache is enabled. The L2X0 driver takes care of enabling the optimization by itself. This patch fixes a noisy error message on Tegra20 and Tegra30 telling that cache optimization is erroneously enabled without enabling it for the CPU: L2C-310: enabling full line of zeros but not enabled in Cortex-A9 Cc: <stable@vger.kernel.org> Signed-off-by: Dmitry Osipenko <digetx@gmail.com> --- arch/arm/mach-tegra/tegra.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)