On Fri, Jan 13, 2023 at 11:33:46AM +0100, Krzysztof Kozlowski wrote: > On some SoCs the watchdog device is actually mixed with timer, e.g. > the qcom,msm-timer on older Qualcomm SoCs where this is actually one > hardware block responsible for both system timer and watchdog. > > Allow calling such device nodes as "timer". > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > Reviewed-by: Rob Herring <robh@xxxxxxxxxx> > Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> > --- > > Changes since v1: > 1. Add tag. > > See also: > https://lore.kernel.org/linux-arm-msm/20221212163532.142533-1-krzysztof.kozlowski@xxxxxxxxxx/T/#t > > which causes warnings: > > qcom-msm8960-cdp.dtb: timer@200a000: $nodename:0: 'timer@200a000' does not match '^watchdog(@.*|-[0-9a-f])?$' > From schema: Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml > --- > Documentation/devicetree/bindings/watchdog/watchdog.yaml | 7 ++++++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/watchdog/watchdog.yaml b/Documentation/devicetree/bindings/watchdog/watchdog.yaml > index fccae0d00110..519b48889eb1 100644 > --- a/Documentation/devicetree/bindings/watchdog/watchdog.yaml > +++ b/Documentation/devicetree/bindings/watchdog/watchdog.yaml > @@ -14,9 +14,14 @@ description: | > This document describes generic bindings which can be used to > describe watchdog devices in a device tree. > > +select: > + properties: > + $nodename: > + pattern: "^watchdog(@.*|-[0-9a-f])?$" > + > properties: > $nodename: > - pattern: "^watchdog(@.*|-[0-9a-f])?$" > + pattern: "^(timer|watchdog)(@.*|-[0-9a-f])?$" > > timeout-sec: > description: > -- > 2.34.1 >
