Hi, Varshini, In my comment from v4 I meant to say: dt-bindings: clk: at91: add sam9x7 -> dt-bindings: clocks: at91sam9x5-sckc: add sam9x7 Suggestion was: s/dt-bindings: clk: at91/dt-bindings: clocks: at91sam9x5-sckc Anyway, I'll adjust it when applying. No need to resent just for this. On 03.07.2024 13:27, Varshini Rajendran wrote: > Add bindings for SAM9X7's slow clock controller. > > Signed-off-by: Varshini Rajendran <varshini.rajendran@xxxxxxxxxxxxx> > Acked-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx> Other than the title: Reviewed-by: Claudiu Beznea <claudiu.beznea@xxxxxxxxx> > --- > Changes in v5: > - Changed subject according to suggestion. > - Alphanumerically sorted entries. > - Updated Acked-by tag. > --- > .../devicetree/bindings/clock/atmel,at91sam9x5-sckc.yaml | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/clock/atmel,at91sam9x5-sckc.yaml b/Documentation/devicetree/bindings/clock/atmel,at91sam9x5-sckc.yaml > index 7be29877e6d2..c2283cd07f05 100644 > --- a/Documentation/devicetree/bindings/clock/atmel,at91sam9x5-sckc.yaml > +++ b/Documentation/devicetree/bindings/clock/atmel,at91sam9x5-sckc.yaml > @@ -18,7 +18,9 @@ properties: > - atmel,sama5d4-sckc > - microchip,sam9x60-sckc > - items: > - - const: microchip,sama7g5-sckc > + - enum: > + - microchip,sam9x7-sckc > + - microchip,sama7g5-sckc > - const: microchip,sam9x60-sckc > > reg: