Received-SPF: pass (google.com: domain of linux-kernel+bounces-157678-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Message-ID: <d738dfc0-66af-4dad-bd60-a68c900bf807@ghiti.fr>
Date: Wed, 24 Apr 2024 22:19:18 +0200
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 0/4] cache: sifive_ccache: Auxiliary device support
Content-Language: en-US
To: Samuel Holland <samuel.holland@sifive.com>,
 Conor Dooley <conor@kernel.org>, linux-riscv@lists.infradead.org
Cc: Paul Walmsley <paul.walmsley@sifive.com>, linux-kernel@vger.kernel.org
References: <20240410232211.438277-1-samuel.holland@sifive.com>
From: Alexandre Ghiti <alex@ghiti.fr>
In-Reply-To: <20240410232211.438277-1-samuel.holland@sifive.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit

Hi Samuel,

On 11/04/2024 01:22, Samuel Holland wrote:
> As of commit c90847bcbfb6 ("cache: sifive_ccache: Partially convert to a
> platform driver"), the cache subsystem binds a platform driver to the
> Composable Cache's DT node. This prevents the perf subsystem from doing
> the same for the new PMU driver[1]. To allow using both drivers at the
> same time without conflicts or module linkage dependencies, attach the
> PMU driver to the auxiliary device bus. While at it, prepare to use the
> auxiliary device bus for the EDAC driver as well, which significantly
> simplifies that driver. The actual EDAC driver conversion has to wait
> another development cycle to avoid dependencies between git trees.
>
> [1]: https://lore.kernel.org/linux-riscv/20240216000837.1868917-3-samuel.holland@sifive.com/
>
>
> Samuel Holland (4):
>    cache: sifive_ccache: Silence unused variable warning
>    cache: sifive_ccache: Use of_iomap() helper
>    cache: sifive_ccache: Export base address for child drivers
>    cache: sifive_ccache: Add EDAC and PMU as auxiliary devices
>
>   drivers/cache/Kconfig              |  1 +
>   drivers/cache/sifive_ccache.c      | 94 +++++++++++++++++++++---------
>   include/soc/sifive/sifive_ccache.h |  8 +++
>   3 files changed, 74 insertions(+), 29 deletions(-)
>

I took a quick look and I'm not sure if the whole patchset is a fix for 
6.9 or if only patch 1 is?

Alex