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
