Re: [PATCH 0/3] Rename sifive L2 cache to sifive CCACHE

From: Ben Dooks
Date: Tue Aug 30 2022 - 04:00:17 EST


On 29/08/2022 07:21, Zong Li wrote:
Since composible cache may be L3 cache if private L2 cache exists, we
should use its original name "composible cache" to prevent confusion.

This patchset contains the modification which is related to ccache, such
as DT binding and EDAC driver.

Greentime Hu (1):
soc: sifive: l2 cache: Rename SiFive L2 cache to composible cache.

Zong Li (2):
dt-bindings: sifive-ccache: rename SiFive L2 cache to composible cache
EDAC/sifive: use sifive_ccache instead of sifive_l2

...ifive-l2-cache.yaml => sifive-ccache.yaml} | 6 +-
drivers/edac/Kconfig | 2 +-
drivers/edac/sifive_edac.c | 12 +-
drivers/soc/sifive/Kconfig | 7 +-
drivers/soc/sifive/Makefile | 2 +-
drivers/soc/sifive/sifive_ccache.c | 221 ++++++++++++++++
drivers/soc/sifive/sifive_l2_cache.c | 237 ------------------
include/soc/sifive/sifive_ccache.h | 16 ++
include/soc/sifive/sifive_l2_cache.h | 16 --
9 files changed, 253 insertions(+), 266 deletions(-)
rename Documentation/devicetree/bindings/riscv/{sifive-l2-cache.yaml => sifive-ccache.yaml} (92%)
create mode 100644 drivers/soc/sifive/sifive_ccache.c
delete mode 100644 drivers/soc/sifive/sifive_l2_cache.c
create mode 100644 include/soc/sifive/sifive_ccache.h
delete mode 100644 include/soc/sifive/sifive_l2_cache.h

Is this series available on a git branch anywhere, I have at least
one other cleanup I was doing in a similar move to rename and make
it usable as the l3 cache.

I gave the series a quick review and did not find any issues so far.

--
Ben