On 07/10/2022 13:39, Manjunatha Venkatesh wrote: > > On 9/14/2022 8:06 PM, Arnd Bergmann wrote: >> Caution: EXT Email >> >> On Wed, Sep 14, 2022, at 4:29 PM, Manjunatha Venkatesh wrote: >>> + >>> +properties: >>> + compatible: >>> + enum: >>> + - nxp,sr1xx >>> + >> You should not have wildcard names in the compatible string. >> Make this a specific model number without 'xx', and >> have the devices list their own name along with the oldest >> one they are compatible with. >> >> Arnd > This driver is common for both sr100 and sr150,so we have used sr1xx > naming convention or can we use name with highest version(sr150)? In general each device needs its compatible, so you would need two of them. However if one is compatible with the other, then express it as well. IOW, driver binds to one compatible, binding describes both (one as fallback). There are many, many of such examples in the kernel. Best regards, Krzysztof