On 20/05/2024 19:14, Conor Dooley wrote: > On Sat, May 18, 2024 at 10:07:17PM +0200, Krzysztof Kozlowski wrote: >> Add dedicated binding for the AM654 MCU SCM system controller registers, >> already used in the DTS to properly describe its children. >> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> >> --- >> .../soc/ti/ti,am654-system-controller.yaml | 60 ++++++++++++++++++++++ >> 1 file changed, 60 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/soc/ti/ti,am654-system-controller.yaml b/Documentation/devicetree/bindings/soc/ti/ti,am654-system-controller.yaml >> new file mode 100644 >> index 000000000000..e79803e586ca >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/soc/ti/ti,am654-system-controller.yaml >> @@ -0,0 +1,60 @@ >> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/soc/ti/ti,am654-system-controller.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: TI AM654 System Controller Registers R/W > > "R/W"? Is that a copy-paste mistake from the register map for this SoC? Other TI syscon bindings had it. I'll drop it. Best regards, Krzysztof