On 23/06/2023 22:30, Varshini Rajendran wrote: > Add documentation for SAM9X7 reset controller. > A nit, subject: drop second/last, redundant "bindings". The "dt-bindings" prefix is already stating that these are bindings. > Signed-off-by: Varshini Rajendran <varshini.rajendran@xxxxxxxxxxxxx> > --- > .../devicetree/bindings/reset/atmel,at91sam9260-reset.yaml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/reset/atmel,at91sam9260-reset.yaml b/Documentation/devicetree/bindings/reset/atmel,at91sam9260-reset.yaml > index 98465d26949e..593a13c277ab 100644 > --- a/Documentation/devicetree/bindings/reset/atmel,at91sam9260-reset.yaml > +++ b/Documentation/devicetree/bindings/reset/atmel,at91sam9260-reset.yaml > @@ -22,6 +22,7 @@ properties: > - atmel,at91sam9g45-rstc > - atmel,sama5d3-rstc > - microchip,sam9x60-rstc > + - microchip,sam9x7-rstc It does not look like you tested the DTS against bindings. Please run `make dtbs_check` (see Documentation/devicetree/bindings/writing-schema.rst or https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/ for instructions). Best regards, Krzysztof