Received-SPF: pass (google.com: domain of linux-kernel+bounces-157731-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Date: Wed, 24 Apr 2024 22:20:00 +0100
From: Conor Dooley <conor@kernel.org>
To: Alexandre Ghiti <alex@ghiti.fr>
Cc: Samuel Holland <samuel.holland@sifive.com>,
	linux-riscv@lists.infradead.org,
	Paul Walmsley <paul.walmsley@sifive.com>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/4] cache: sifive_ccache: Auxiliary device support
Message-ID: <20240424-recolor-deviation-29a5dd4e9499@spud>
References: <20240410232211.438277-1-samuel.holland@sifive.com>
 <d738dfc0-66af-4dad-bd60-a68c900bf807@ghiti.fr>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
	protocol="application/pgp-signature"; boundary="xuJOMISUXHsgfGn1"
Content-Disposition: inline
In-Reply-To: <d738dfc0-66af-4dad-bd60-a68c900bf807@ghiti.fr>


--xuJOMISUXHsgfGn1
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Apr 24, 2024 at 10:19:18PM +0200, Alexandre Ghiti wrote:
> Hi Samuel,
>=20
> 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.
> >=20
> > [1]: https://lore.kernel.org/linux-riscv/20240216000837.1868917-3-samue=
l.holland@sifive.com/
> >=20
> >=20
> > 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
> >=20
> >   drivers/cache/Kconfig              |  1 +
> >   drivers/cache/sifive_ccache.c      | 94 +++++++++++++++++++++---------
> >   include/soc/sifive/sifive_ccache.h |  8 +++
> >   3 files changed, 74 insertions(+), 29 deletions(-)
> >=20
>=20
> I took a quick look and I'm not sure if the whole patchset is a fix for 6=
=2E9
> or if only patch 1 is?

Only the first patch is IMO, but it is a fix for a patch I applied so
it'll go this week via the soc tree with those auto-update fixes.

--xuJOMISUXHsgfGn1
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZil3gAAKCRB4tDGHoIJi
0tEpAQDcI7yoAdCPL60ujB1LzolN4SJJmStybk3ftfXpKvZmLwD/WvBnGvK5ezjN
smI6d1HvJ3B7DEPaIjmTZJUFrQB1TA4=
=7AyF
-----END PGP SIGNATURE-----

--xuJOMISUXHsgfGn1--