On 17/12/24 11:01, Krzysztof Kozlowski wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > On 17/12/2024 06:00, Charan.Pedumuru@xxxxxxxxxxxxx wrote: >>>> + - items: >>>> + - enum: >>>> + - atmel,at91sam9260-matrix >>>> + - atmel,at91sam9261-matrix >>>> + - atmel,at91sam9263-matrix >>>> + - atmel,at91sam9rl-matrix >>>> + - atmel,at91sam9g45-matrix >>>> + - atmel,at91sam9n12-matrix >>>> + - atmel,at91sam9x5-matrix >>>> + - atmel,sama5d3-matrix >>>> + - const: syscon >>>> + - items: >>>> + - const: microchip,sam9x60-matrix >>>> + - const: atmel,at91sam9x5-matrix >>>> + - const: syscon >>>> + - items: >>>> + - const: microchip,sam9x7-matrix >>> So that's just enum with previous group. >> Yes, as both boards have 2 compatibles one after another, I defined them >> as two different groups to resolve dt_check warnings. > My comment stands. I understand your "Yes" means you will fix it. Yes, I will remove the other group and define both `microchip,sam9x60-matrix` and `microchip,sam9x7-matrix` in enum under items and add const. > > > Best regards, > Krzysztof -- Best Regards, Charan.