Message ID | 20231209233106.147416-4-peter.griffin@linaro.org |
---|---|
State | Not Applicable, archived |
Headers | show |
Series | Add minimal Tensor/GS101 SoC support and Oriole/Pixel6 board | expand |
Context | Check | Description |
---|---|---|
robh/checkpatch | success | |
robh/patch-applied | success | |
robh/dtbs-check | warning | build log |
robh/dt-meta-schema | success |
On 10/12/2023 00:30, Peter Griffin wrote: > GS101 has three different SYSREG controllers, add dedicated > compatibles for them to the documentation. > > Reviewed-by: Sam Protsenko <semen.protsenko@linaro.org> > Signed-off-by: Peter Griffin <peter.griffin@linaro.org> > --- > .../bindings/soc/samsung/samsung,exynos-sysreg.yaml | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/Documentation/devicetree/bindings/soc/samsung/samsung,exynos-sysreg.yaml b/Documentation/devicetree/bindings/soc/samsung/samsung,exynos-sysreg.yaml > index 2de4301a467d..127f4ffde76a 100644 > --- a/Documentation/devicetree/bindings/soc/samsung/samsung,exynos-sysreg.yaml > +++ b/Documentation/devicetree/bindings/soc/samsung/samsung,exynos-sysreg.yaml > @@ -22,6 +22,12 @@ properties: > - tesla,fsd-fsys1-sysreg > - tesla,fsd-peric-sysreg > - const: syscon > + - items: > + - enum: > + - google,gs101-apm-sysreg > + - google,gs101-peric0-sysreg > + - google,gs101-peric1-sysreg This should be part of the first enum. No need for new list for every new SoC. I'll fix it while applying. Best regards, Krzysztof
On Sat, 09 Dec 2023 23:30:49 +0000, Peter Griffin wrote: > GS101 has three different SYSREG controllers, add dedicated > compatibles for them to the documentation. > > Applied, thanks! [03/20] dt-bindings: soc: google: exynos-sysreg: add dedicated SYSREG compatibles to GS101 https://git.kernel.org/krzk/linux/c/d9232785858eafde8553932f96fb7e25c2191ed2 Best regards,
Hi Krzysztof, On Sun, 10 Dec 2023 at 13:49, Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > > On 10/12/2023 00:30, Peter Griffin wrote: > > GS101 has three different SYSREG controllers, add dedicated > > compatibles for them to the documentation. > > > > Reviewed-by: Sam Protsenko <semen.protsenko@linaro.org> > > Signed-off-by: Peter Griffin <peter.griffin@linaro.org> > > --- > > .../bindings/soc/samsung/samsung,exynos-sysreg.yaml | 6 ++++++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/soc/samsung/samsung,exynos-sysreg.yaml b/Documentation/devicetree/bindings/soc/samsung/samsung,exynos-sysreg.yaml > > index 2de4301a467d..127f4ffde76a 100644 > > --- a/Documentation/devicetree/bindings/soc/samsung/samsung,exynos-sysreg.yaml > > +++ b/Documentation/devicetree/bindings/soc/samsung/samsung,exynos-sysreg.yaml > > @@ -22,6 +22,12 @@ properties: > > - tesla,fsd-fsys1-sysreg > > - tesla,fsd-peric-sysreg > > - const: syscon > > + - items: > > + - enum: > > + - google,gs101-apm-sysreg > > + - google,gs101-peric0-sysreg > > + - google,gs101-peric1-sysreg > > This should be part of the first enum. No need for new list for every > new SoC. I'll fix it while applying. Noted, thanks! Peter
diff --git a/Documentation/devicetree/bindings/soc/samsung/samsung,exynos-sysreg.yaml b/Documentation/devicetree/bindings/soc/samsung/samsung,exynos-sysreg.yaml index 2de4301a467d..127f4ffde76a 100644 --- a/Documentation/devicetree/bindings/soc/samsung/samsung,exynos-sysreg.yaml +++ b/Documentation/devicetree/bindings/soc/samsung/samsung,exynos-sysreg.yaml @@ -22,6 +22,12 @@ properties: - tesla,fsd-fsys1-sysreg - tesla,fsd-peric-sysreg - const: syscon + - items: + - enum: + - google,gs101-apm-sysreg + - google,gs101-peric0-sysreg + - google,gs101-peric1-sysreg + - const: syscon - items: - enum: - samsung,exynos5433-cam0-sysreg