Skip to content

Commit 179c4ac

Browse files
jhovoldWim Van Sebroeck
authored andcommitted
dt-bindings: watchdog: qcom,pm8916-wdt: add parent spmi node to example
The PM8916 watchdog is part of an SPMI PMIC, which lives on an SPMI bus. Add a parent SPMI bus node with an '#address-cells' of 2 and '#size-cells' of 0 instead of relying on the fact that the default number of register cells happen to match (i.e. 1 + 1). Signed-off-by: Johan Hovold <[email protected]> Reviewed-by: Krzysztof Kozlowski <[email protected]> Reviewed-by: Wim Van Sebroeck <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Guenter Roeck <[email protected]> Signed-off-by: Wim Van Sebroeck <[email protected]>
1 parent 1cb113c commit 179c4ac

File tree

1 file changed

+19
-14
lines changed

1 file changed

+19
-14
lines changed

Documentation/devicetree/bindings/watchdog/qcom,pm8916-wdt.yaml

Lines changed: 19 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -30,22 +30,27 @@ examples:
3030
#include <dt-bindings/interrupt-controller/irq.h>
3131
#include <dt-bindings/spmi/spmi.h>
3232
33-
pmic@0 {
34-
compatible = "qcom,pm8916", "qcom,spmi-pmic";
35-
reg = <0x0 SPMI_USID>;
36-
#address-cells = <1>;
33+
spmi {
34+
#address-cells = <2>;
3735
#size-cells = <0>;
3836
39-
pon@800 {
40-
compatible = "qcom,pm8916-pon";
41-
reg = <0x800>;
42-
mode-bootloader = <0x2>;
43-
mode-recovery = <0x1>;
44-
45-
watchdog {
46-
compatible = "qcom,pm8916-wdt";
47-
interrupts = <0x0 0x8 6 IRQ_TYPE_EDGE_RISING>;
48-
timeout-sec = <60>;
37+
pmic@0 {
38+
compatible = "qcom,pm8916", "qcom,spmi-pmic";
39+
reg = <0x0 SPMI_USID>;
40+
#address-cells = <1>;
41+
#size-cells = <0>;
42+
43+
pon@800 {
44+
compatible = "qcom,pm8916-pon";
45+
reg = <0x800>;
46+
mode-bootloader = <0x2>;
47+
mode-recovery = <0x1>;
48+
49+
watchdog {
50+
compatible = "qcom,pm8916-wdt";
51+
interrupts = <0x0 0x8 6 IRQ_TYPE_EDGE_RISING>;
52+
timeout-sec = <60>;
53+
};
4954
};
5055
};
5156
};

0 commit comments

Comments
 (0)