| Message ID | 20231005155618.700312-6-peter.griffin@linaro.org (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Add minimal Tensor/GS101 SoC support and Oriole/Pixel6 board | expand |
On 05/10/2023 17:56, Peter Griffin wrote: > Add the "google,gs101-wdt" and "google,gs201-wdt" compatibles to the > dt-schema documentation. > > gs101 SoC has two CPU clusters and each cluster has its own dedicated > watchdog timer (similar to exynos850 and exynosautov9 SoCs). > > These WDT instances are controlled using different bits in PMU > registers. > > Signed-off-by: Peter Griffin <peter.griffin@linaro.org> > --- > .../devicetree/bindings/watchdog/samsung-wdt.yaml | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/watchdog/samsung-wdt.yaml b/Documentation/devicetree/bindings/watchdog/samsung-wdt.yaml > index 8fb6656ba0c2..30f5949037fc 100644 > --- a/Documentation/devicetree/bindings/watchdog/samsung-wdt.yaml > +++ b/Documentation/devicetree/bindings/watchdog/samsung-wdt.yaml > @@ -24,6 +24,8 @@ properties: > - samsung,exynos7-wdt # for Exynos7 > - samsung,exynos850-wdt # for Exynos850 > - samsung,exynosautov9-wdt # for Exynosautov9 > + - google,gs101-wdt # for Google gs101 > + - google,gs201-wdt # for Google gs101 Alphanumerical order. Best regards, Krzysztof
On 10/05/2023, Peter Griffin wrote: > Add the "google,gs101-wdt" and "google,gs201-wdt" compatibles to the > dt-schema documentation. > > gs101 SoC has two CPU clusters and each cluster has its own dedicated > watchdog timer (similar to exynos850 and exynosautov9 SoCs). > > These WDT instances are controlled using different bits in PMU > registers. > > Signed-off-by: Peter Griffin <peter.griffin@linaro.org> > --- > .../devicetree/bindings/watchdog/samsung-wdt.yaml | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/watchdog/samsung-wdt.yaml b/Documentation/devicetree/bindings/watchdog/samsung-wdt.yaml > index 8fb6656ba0c2..30f5949037fc 100644 > --- a/Documentation/devicetree/bindings/watchdog/samsung-wdt.yaml > +++ b/Documentation/devicetree/bindings/watchdog/samsung-wdt.yaml > @@ -24,6 +24,8 @@ properties: > - samsung,exynos7-wdt # for Exynos7 > - samsung,exynos850-wdt # for Exynos850 > - samsung,exynosautov9-wdt # for Exynosautov9 > + - google,gs101-wdt # for Google gs101 > + - google,gs201-wdt # for Google gs101 For "google,gs201-wdt the comment should be "for Google gs201". Regards, Will > > reg: > maxItems: 1 > @@ -42,13 +44,13 @@ properties: > samsung,cluster-index: > $ref: /schemas/types.yaml#/definitions/uint32 > description: > - Index of CPU cluster on which watchdog is running (in case of Exynos850) > + Index of CPU cluster on which watchdog is running (in case of Exynos850 or Google gsx01) > > samsung,syscon-phandle: > $ref: /schemas/types.yaml#/definitions/phandle > description: > Phandle to the PMU system controller node (in case of Exynos5250, > - Exynos5420, Exynos7 and Exynos850). > + Exynos5420, Exynos7, Exynos850 and gsx01). > > required: > - compatible > @@ -69,6 +71,8 @@ allOf: > - samsung,exynos7-wdt > - samsung,exynos850-wdt > - samsung,exynosautov9-wdt > + - google,gs101-wdt > + - google,gs201-wdt > then: > required: > - samsung,syscon-phandle > @@ -79,6 +83,8 @@ allOf: > enum: > - samsung,exynos850-wdt > - samsung,exynosautov9-wdt > + - google,gs101-wdt > + - google,gs201-wdt > then: > properties: > clocks: > -- > 2.42.0.582.g8ccd20d70d-goog >
diff --git a/Documentation/devicetree/bindings/watchdog/samsung-wdt.yaml b/Documentation/devicetree/bindings/watchdog/samsung-wdt.yaml index 8fb6656ba0c2..30f5949037fc 100644 --- a/Documentation/devicetree/bindings/watchdog/samsung-wdt.yaml +++ b/Documentation/devicetree/bindings/watchdog/samsung-wdt.yaml @@ -24,6 +24,8 @@ properties: - samsung,exynos7-wdt # for Exynos7 - samsung,exynos850-wdt # for Exynos850 - samsung,exynosautov9-wdt # for Exynosautov9 + - google,gs101-wdt # for Google gs101 + - google,gs201-wdt # for Google gs101 reg: maxItems: 1 @@ -42,13 +44,13 @@ properties: samsung,cluster-index: $ref: /schemas/types.yaml#/definitions/uint32 description: - Index of CPU cluster on which watchdog is running (in case of Exynos850) + Index of CPU cluster on which watchdog is running (in case of Exynos850 or Google gsx01) samsung,syscon-phandle: $ref: /schemas/types.yaml#/definitions/phandle description: Phandle to the PMU system controller node (in case of Exynos5250, - Exynos5420, Exynos7 and Exynos850). + Exynos5420, Exynos7, Exynos850 and gsx01). required: - compatible @@ -69,6 +71,8 @@ allOf: - samsung,exynos7-wdt - samsung,exynos850-wdt - samsung,exynosautov9-wdt + - google,gs101-wdt + - google,gs201-wdt then: required: - samsung,syscon-phandle @@ -79,6 +83,8 @@ allOf: enum: - samsung,exynos850-wdt - samsung,exynosautov9-wdt + - google,gs101-wdt + - google,gs201-wdt then: properties: clocks:
Add the "google,gs101-wdt" and "google,gs201-wdt" compatibles to the dt-schema documentation. gs101 SoC has two CPU clusters and each cluster has its own dedicated watchdog timer (similar to exynos850 and exynosautov9 SoCs). These WDT instances are controlled using different bits in PMU registers. Signed-off-by: Peter Griffin <peter.griffin@linaro.org> --- .../devicetree/bindings/watchdog/samsung-wdt.yaml | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-)