On 05/10/2023 17:56, Peter Griffin wrote: > GS101 has three different SYSREG controllers, add dedicated > compatibles for them to the documentation. > > Signed-off-by: Peter Griffin <peter.griffin@xxxxxxxxxx> > --- > .../bindings/soc/samsung/samsung,exynos-sysreg.yaml | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/Documentation/devicetree/bindings/soc/samsung/samsung,exynos-sysreg.yaml b/Documentation/devicetree/bindings/soc/samsung/samsung,exynos-sysreg.yaml > index 163e912e9cad..02f580d6489b 100644 > --- a/Documentation/devicetree/bindings/soc/samsung/samsung,exynos-sysreg.yaml > +++ b/Documentation/devicetree/bindings/soc/samsung/samsung,exynos-sysreg.yaml > @@ -50,6 +50,13 @@ properties: > - samsung,exynosautov9-peric1-sysreg > - const: samsung,exynosautov9-sysreg > - const: syscon > + - items: > + - enum: > + - google,gs101-peric0-sysreg > + - google,gs101-peric1-sysreg > + - google,gs101-apm-sysreg > + - const: google,gs101-sysreg Please drop this one compatible. Exynos has it only for backwards compatibility. Also, please put entire list ("items") before such entry for samsung,exynos5433-sysreg, so everything is more-or-less ordered alphabetically, by the fallback compatible. > + - const: syscon > > reg: > maxItems: 1 Best regards, Krzysztof