Re: [PATCH 2/2] ARM: l2c: honour status = "disabled" on the L2 cache node

From: Agustin Santiago Isasmendi

Date: Wed Nov 11 2026 - 10:24:31 EST


On Wed, Nov 11, 2026 at 14:56, Arnd Bergmann wrote:
> It is a little strange to use the MMIO registers of a status="disabled"
> device to turn it off, and it would be nice if we could require
> the firmware to leave it in a consistent state instead, but I assume
> you can't do that for the one device you are interested in because
> you don't have the firmware sources, right?

On the H3600 the firmware leaves it off (control register 0 at kernel
entry, cold and warm; the bootloader never touches the controller), so
this board only needs the early return. To be honest I also prefer
the firmware to be responsible for leaving the cache off when the
device tree disables it; I went the conservative way for a firmware I
do not control.

If you agree, v2 drops the disable path and patch 1, keeps just the
early return, and adds a line to the binding saying so. Otherwise I
keep the check and add the comment.

Agustin