On 16/05/2022 11:15, Holger Brunck wrote: >> On 11/05/2022 19:40, Guenter Roeck wrote: >>>> Me neither. Just pick one matching the device actually using this >>>> property. If all of them are using it, how about keeping old "national" >>>> or it's owner "ti"? If not all of them are using it, then this would >>>> need "allOf:if:then" restricting where the property is (not) applicable. >>>> >>> >>> It is only applicable for tmp451, tmp461, and adt7461[a], so it looks >>> like "allOf:if:then" will be needed. >> >> Yes, please. >> > > ok I will have a look. > > I noticed that tmp461 is missing at all in the yaml file, so this needs to be > added too. Should this go into a separate patch? That one was about adding extended range, so yes - adding more compatibles or other features should be in separate patches. Best regards, Krzysztof
