linux-kernel - Re: [PATCH 1/3] ASoC: cs4271: Fix cs4271 I2C and SPI drivers automatic module loading

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4240deca-6dc5-4353-afd8-9b4d2819570b@sirena.org.uk>
Date: Wed, 22 Oct 2025 19:01:49 +0100
From: Mark Brown <broonie@...nel.org>
To: Alexander Sverdlin <alexander.sverdlin@...il.com>
Cc: Javier Martinez Canillas <javierm@...hat.com>,
	Wolfram Sang <wsa@...-dreams.de>,
	Herve Codina <herve.codina@...tlin.com>,
	David Rhodes <david.rhodes@...rus.com>,
	Richard Fitzgerald <rf@...nsource.cirrus.com>,
	Liam Girdwood <lgirdwood@...il.com>, Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>,
	Jaroslav Kysela <perex@...ex.cz>, Takashi Iwai <tiwai@...e.com>,
	Nikita Shubin <nikita.shubin@...uefel.me>,
	Axel Lin <axel.lin@...ics.com>,
	Brian Austin <brian.austin@...rus.com>, linux-sound@...r.kernel.org,
	patches@...nsource.cirrus.com, devicetree@...r.kernel.org,
	linux-kernel@...r.kernel.org,
	Thomas Petazzoni <thomas.petazzoni@...tlin.com>
Subject: Re: [PATCH 1/3] ASoC: cs4271: Fix cs4271 I2C and SPI drivers
 automatic module loading

On Wed, Oct 22, 2025 at 07:46:26PM +0200, Alexander Sverdlin wrote:
> On Wed, 2025-10-22 at 15:56 +0100, Mark Brown wrote:

> > > Now declaring "of:" to be the new I2C bus prefix for uevents starting from
> > > Linux v4.18 sounds strange.

> > I think a robust solution would involve having the OF aliases namespaced
> > by bus, or just not using the OF aliases but potentially having
> > collisions if two vendors pick the same device name.

> But this sounds like the situation before the above mentioned commit
> af503716ac14, when both i2c and spi were symmetrically namespaced with
> i2c: and spi: respectively and contained the "compatible" stripped of the
> vendor prefix.

> And I must admit that I had more understanding for the prior state of things.

Right, it is.  The issue with this situation is that if the compatible
strings for a device are not simply vendor,part or if two vendors happen
to end up with the same part name (Wondermedia and Wolfson used to both
use wmNNNN for example, there were actual collisions though not on the
same bus type) we will have trouble figuring things out.  We don't have
a robust mechanism for translating from a compatible string to a
platform registration style name for a device.

Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)