Messages in this thread | | | Date | Mon, 21 Mar 2022 12:10:05 +0100 | From | Krzysztof Kozlowski <> | Subject | Re: [PATCH v7 1/2] dt-bindings: watchdog: Add watchdog yaml file for Sunplus SP7021 |
| |
On Fri, Mar 04, 2022 at 04:12:08PM +0800, Xiantao Hu wrote: > This adds the documentation for the devicetree bindings of the Sunplus > SP7021 watchdog driver, found from SP7021 SoCs and newer. > > Reviewed-by: Rob Herring <robh@kernel.org> > Signed-off-by: Xiantao Hu <xt.hu@cqplus1.com> > --- > .../bindings/watchdog/sunplus,sp7021-wdt.yaml | 47 +++++++++++++++++++ > MAINTAINERS | 6 +++ > 2 files changed, 53 insertions(+) > create mode 100644 Documentation/devicetree/bindings/watchdog/sunplus,sp7021-wdt.yaml
Please use scripts/get_maintainer.pl on recent kernel (newest mainline or recent linux-next) to get the list of people to CC.
> > diff --git a/Documentation/devicetree/bindings/watchdog/sunplus,sp7021-wdt.yaml b/Documentation/devicetree/bindings/watchdog/sunplus,sp7021-wdt.yaml > new file mode 100644 > index 000000000..9a9bfe150 > --- /dev/null > +++ b/Documentation/devicetree/bindings/watchdog/sunplus,sp7021-wdt.yaml > @@ -0,0 +1,47 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +# Copyright (C) Sunplus Co., Ltd. 2021 > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/watchdog/sunplus,sp7021-wdt.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Sunplus SoCs Watchdog Device Tree Bindings
s/Device Tree Bindings//
> + > +maintainers: > + - XianTao Hu <xt.hu@cqplus1.com> > + > +allOf: > + - $ref: watchdog.yaml# > + > +properties: > + compatible: > + const: sunplus,sp7021-wdt > + > + reg: > + items: > + - description: watchdog registers regions > + - description: miscellaneous control registers regions > + > + clocks: > + maxItems: 1 > + > + resets: > + maxItems: 1 > + > +required: > + - compatible > + - reg > + - clocks > + - resets > + > +additionalProperties: false > + > +examples: > + - | > + watchdog: watchdog@9c000630 { > + compatible = "sunplus,sp7021-wdt"; > + reg = <0x9c000630 0x08>, <0x9C000274 0x04>;
lowercase hex (Second reg)
Can be fixed while applying, so in such case: Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Best regards, Krzysztof
| |