Re: [PATCH] spidev: Make probe to fail early if a spidev compatible is used

From: Mark Brown
Date: Tue Nov 23 2021 - 09:55:45 EST

On Fri, Nov 19, 2021 at 09:32:32AM +0100, Javier Martinez Canillas wrote:
> On 11/19/21 08:40, Uwe Kleine-König wrote:

> > Not this affects also devices that use

> > compatible = "myvender,devicename", "spidev";

> This is indeed a corner case and I'm less sure what the kernel should do
> about it. I just learned now that of_device_is_compatible() return value

TBH I feel like that falls into the same bucket as any other uses of
spidev so I'm not overly worried. Grepping around it looks like we have
no examples of this in tree, only a few plain spidevs in DTs for older
platforms that were most likely converted from board files and *probably*
aren't too relevant at this point.

> Or just consider DT nodes with a general "spidev" compatible string to
> also not be valid. I would lean towards this one I think.

Yes, I think so. Your other options are worth exploring if it turns out
to be an issue but hopefully it's not.

Attachment: signature.asc
Description: PGP signature