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

From: Mark Brown

Date: Wed Oct 22 2025 - 14:01:56 EST


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.

Attachment: signature.asc
Description: PGP signature