diff options
author | Marian Balakowicz <m8@semihalf.com> | 2007-10-18 14:44:24 -0400 |
---|---|---|
committer | Grant Likely <grant.likely@secretlab.ca> | 2007-10-21 14:42:55 -0400 |
commit | d24bc314c964e900ddf30efa1ef44565cf0cf44c (patch) | |
tree | ee79394490a55998b355ea63acee96d14821a37a | |
parent | c5c01c9700e59a5b20d7bdd15808d33ab9d9e877 (diff) |
[POWERPC] Update device tree binding for mpc5200 gpt
Add 'fsl,' prefix to 'compatible' property for gpt nodes.
Add 'fsl,' prefix to empty, GPT0 specific 'has-wdt' property.
The fsl, prefix is being added to better match the convention of prefixing
manufacturer specific properties and values with the vendors name.
Signed-off-by: Marian Balakowicz <m8@semihalf.com>
Signed-off-by: Grant Likely <grant.likely@secretlab.ca>
-rw-r--r-- | Documentation/powerpc/mpc52xx-device-tree-bindings.txt | 4 | ||||
-rw-r--r-- | arch/powerpc/boot/dts/lite5200.dts | 26 | ||||
-rw-r--r-- | arch/powerpc/boot/dts/lite5200b.dts | 26 | ||||
-rw-r--r-- | drivers/watchdog/mpc5200_wdt.c | 3 |
4 files changed, 23 insertions, 36 deletions
diff --git a/Documentation/powerpc/mpc52xx-device-tree-bindings.txt b/Documentation/powerpc/mpc52xx-device-tree-bindings.txt index 5f7d536cb0c6..5e03610e186f 100644 --- a/Documentation/powerpc/mpc52xx-device-tree-bindings.txt +++ b/Documentation/powerpc/mpc52xx-device-tree-bindings.txt | |||
@@ -185,7 +185,7 @@ bestcomm@<addr> dma-controller mpc5200-bestcomm 5200 pic also requires | |||
185 | Recommended soc5200 child nodes; populate as needed for your board | 185 | Recommended soc5200 child nodes; populate as needed for your board |
186 | name device_type compatible Description | 186 | name device_type compatible Description |
187 | ---- ----------- ---------- ----------- | 187 | ---- ----------- ---------- ----------- |
188 | gpt@<addr> gpt mpc5200-gpt General purpose timers | 188 | gpt@<addr> gpt fsl,mpc5200-gpt General purpose timers |
189 | rtc@<addr> rtc mpc5200-rtc Real time clock | 189 | rtc@<addr> rtc mpc5200-rtc Real time clock |
190 | mscan@<addr> mscan mpc5200-mscan CAN bus controller | 190 | mscan@<addr> mscan mpc5200-mscan CAN bus controller |
191 | pci@<addr> pci mpc5200-pci PCI bridge | 191 | pci@<addr> pci mpc5200-pci PCI bridge |
@@ -213,7 +213,7 @@ cell-index int When multiple devices are present, is the | |||
213 | 5) General Purpose Timer nodes (child of soc5200 node) | 213 | 5) General Purpose Timer nodes (child of soc5200 node) |
214 | On the mpc5200 and 5200b, GPT0 has a watchdog timer function. If the board | 214 | On the mpc5200 and 5200b, GPT0 has a watchdog timer function. If the board |
215 | design supports the internal wdt, then the device node for GPT0 should | 215 | design supports the internal wdt, then the device node for GPT0 should |
216 | include the empty property 'has-wdt'. | 216 | include the empty property 'fsl,has-wdt'. |
217 | 217 | ||
218 | 6) PSC nodes (child of soc5200 node) | 218 | 6) PSC nodes (child of soc5200 node) |
219 | PSC nodes can define the optional 'port-number' property to force assignment | 219 | PSC nodes can define the optional 'port-number' property to force assignment |
diff --git a/arch/powerpc/boot/dts/lite5200.dts b/arch/powerpc/boot/dts/lite5200.dts index bc45f5fbb060..6731763f0282 100644 --- a/arch/powerpc/boot/dts/lite5200.dts +++ b/arch/powerpc/boot/dts/lite5200.dts | |||
@@ -70,18 +70,16 @@ | |||
70 | }; | 70 | }; |
71 | 71 | ||
72 | gpt@600 { // General Purpose Timer | 72 | gpt@600 { // General Purpose Timer |
73 | compatible = "mpc5200-gpt"; | 73 | compatible = "fsl,mpc5200-gpt"; |
74 | device_type = "gpt"; | ||
75 | cell-index = <0>; | 74 | cell-index = <0>; |
76 | reg = <600 10>; | 75 | reg = <600 10>; |
77 | interrupts = <1 9 0>; | 76 | interrupts = <1 9 0>; |
78 | interrupt-parent = <&mpc5200_pic>; | 77 | interrupt-parent = <&mpc5200_pic>; |
79 | has-wdt; | 78 | fsl,has-wdt; |
80 | }; | 79 | }; |
81 | 80 | ||
82 | gpt@610 { // General Purpose Timer | 81 | gpt@610 { // General Purpose Timer |
83 | compatible = "mpc5200-gpt"; | 82 | compatible = "fsl,mpc5200-gpt"; |
84 | device_type = "gpt"; | ||
85 | cell-index = <1>; | 83 | cell-index = <1>; |
86 | reg = <610 10>; | 84 | reg = <610 10>; |
87 | interrupts = <1 a 0>; | 85 | interrupts = <1 a 0>; |
@@ -89,8 +87,7 @@ | |||
89 | }; | 87 | }; |
90 | 88 | ||
91 | gpt@620 { // General Purpose Timer | 89 | gpt@620 { // General Purpose Timer |
92 | compatible = "mpc5200-gpt"; | 90 | compatible = "fsl,mpc5200-gpt"; |
93 | device_type = "gpt"; | ||
94 | cell-index = <2>; | 91 | cell-index = <2>; |
95 | reg = <620 10>; | 92 | reg = <620 10>; |
96 | interrupts = <1 b 0>; | 93 | interrupts = <1 b 0>; |
@@ -98,8 +95,7 @@ | |||
98 | }; | 95 | }; |
99 | 96 | ||
100 | gpt@630 { // General Purpose Timer | 97 | gpt@630 { // General Purpose Timer |
101 | compatible = "mpc5200-gpt"; | 98 | compatible = "fsl,mpc5200-gpt"; |
102 | device_type = "gpt"; | ||
103 | cell-index = <3>; | 99 | cell-index = <3>; |
104 | reg = <630 10>; | 100 | reg = <630 10>; |
105 | interrupts = <1 c 0>; | 101 | interrupts = <1 c 0>; |
@@ -107,8 +103,7 @@ | |||
107 | }; | 103 | }; |
108 | 104 | ||
109 | gpt@640 { // General Purpose Timer | 105 | gpt@640 { // General Purpose Timer |
110 | compatible = "mpc5200-gpt"; | 106 | compatible = "fsl,mpc5200-gpt"; |
111 | device_type = "gpt"; | ||
112 | cell-index = <4>; | 107 | cell-index = <4>; |
113 | reg = <640 10>; | 108 | reg = <640 10>; |
114 | interrupts = <1 d 0>; | 109 | interrupts = <1 d 0>; |
@@ -116,8 +111,7 @@ | |||
116 | }; | 111 | }; |
117 | 112 | ||
118 | gpt@650 { // General Purpose Timer | 113 | gpt@650 { // General Purpose Timer |
119 | compatible = "mpc5200-gpt"; | 114 | compatible = "fsl,mpc5200-gpt"; |
120 | device_type = "gpt"; | ||
121 | cell-index = <5>; | 115 | cell-index = <5>; |
122 | reg = <650 10>; | 116 | reg = <650 10>; |
123 | interrupts = <1 e 0>; | 117 | interrupts = <1 e 0>; |
@@ -125,8 +119,7 @@ | |||
125 | }; | 119 | }; |
126 | 120 | ||
127 | gpt@660 { // General Purpose Timer | 121 | gpt@660 { // General Purpose Timer |
128 | compatible = "mpc5200-gpt"; | 122 | compatible = "fsl,mpc5200-gpt"; |
129 | device_type = "gpt"; | ||
130 | cell-index = <6>; | 123 | cell-index = <6>; |
131 | reg = <660 10>; | 124 | reg = <660 10>; |
132 | interrupts = <1 f 0>; | 125 | interrupts = <1 f 0>; |
@@ -134,8 +127,7 @@ | |||
134 | }; | 127 | }; |
135 | 128 | ||
136 | gpt@670 { // General Purpose Timer | 129 | gpt@670 { // General Purpose Timer |
137 | compatible = "mpc5200-gpt"; | 130 | compatible = "fsl,mpc5200-gpt"; |
138 | device_type = "gpt"; | ||
139 | cell-index = <7>; | 131 | cell-index = <7>; |
140 | reg = <670 10>; | 132 | reg = <670 10>; |
141 | interrupts = <1 10 0>; | 133 | interrupts = <1 10 0>; |
diff --git a/arch/powerpc/boot/dts/lite5200b.dts b/arch/powerpc/boot/dts/lite5200b.dts index 6582c9a39b27..b540388c608c 100644 --- a/arch/powerpc/boot/dts/lite5200b.dts +++ b/arch/powerpc/boot/dts/lite5200b.dts | |||
@@ -70,18 +70,16 @@ | |||
70 | }; | 70 | }; |
71 | 71 | ||
72 | gpt@600 { // General Purpose Timer | 72 | gpt@600 { // General Purpose Timer |
73 | compatible = "mpc5200b-gpt","mpc5200-gpt"; | 73 | compatible = "fsl,mpc5200b-gpt","fsl,mpc5200-gpt"; |
74 | device_type = "gpt"; | ||
75 | cell-index = <0>; | 74 | cell-index = <0>; |
76 | reg = <600 10>; | 75 | reg = <600 10>; |
77 | interrupts = <1 9 0>; | 76 | interrupts = <1 9 0>; |
78 | interrupt-parent = <&mpc5200_pic>; | 77 | interrupt-parent = <&mpc5200_pic>; |
79 | has-wdt; | 78 | fsl,has-wdt; |
80 | }; | 79 | }; |
81 | 80 | ||
82 | gpt@610 { // General Purpose Timer | 81 | gpt@610 { // General Purpose Timer |
83 | compatible = "mpc5200b-gpt","mpc5200-gpt"; | 82 | compatible = "fsl,mpc5200b-gpt","fsl,mpc5200-gpt"; |
84 | device_type = "gpt"; | ||
85 | cell-index = <1>; | 83 | cell-index = <1>; |
86 | reg = <610 10>; | 84 | reg = <610 10>; |
87 | interrupts = <1 a 0>; | 85 | interrupts = <1 a 0>; |
@@ -89,8 +87,7 @@ | |||
89 | }; | 87 | }; |
90 | 88 | ||
91 | gpt@620 { // General Purpose Timer | 89 | gpt@620 { // General Purpose Timer |
92 | compatible = "mpc5200b-gpt","mpc5200-gpt"; | 90 | compatible = "fsl,mpc5200b-gpt","fsl,mpc5200-gpt"; |
93 | device_type = "gpt"; | ||
94 | cell-index = <2>; | 91 | cell-index = <2>; |
95 | reg = <620 10>; | 92 | reg = <620 10>; |
96 | interrupts = <1 b 0>; | 93 | interrupts = <1 b 0>; |
@@ -98,8 +95,7 @@ | |||
98 | }; | 95 | }; |
99 | 96 | ||
100 | gpt@630 { // General Purpose Timer | 97 | gpt@630 { // General Purpose Timer |
101 | compatible = "mpc5200b-gpt","mpc5200-gpt"; | 98 | compatible = "fsl,mpc5200b-gpt","fsl,mpc5200-gpt"; |
102 | device_type = "gpt"; | ||
103 | cell-index = <3>; | 99 | cell-index = <3>; |
104 | reg = <630 10>; | 100 | reg = <630 10>; |
105 | interrupts = <1 c 0>; | 101 | interrupts = <1 c 0>; |
@@ -107,8 +103,7 @@ | |||
107 | }; | 103 | }; |
108 | 104 | ||
109 | gpt@640 { // General Purpose Timer | 105 | gpt@640 { // General Purpose Timer |
110 | compatible = "mpc5200b-gpt","mpc5200-gpt"; | 106 | compatible = "fsl,mpc5200b-gpt","fsl,mpc5200-gpt"; |
111 | device_type = "gpt"; | ||
112 | cell-index = <4>; | 107 | cell-index = <4>; |
113 | reg = <640 10>; | 108 | reg = <640 10>; |
114 | interrupts = <1 d 0>; | 109 | interrupts = <1 d 0>; |
@@ -116,8 +111,7 @@ | |||
116 | }; | 111 | }; |
117 | 112 | ||
118 | gpt@650 { // General Purpose Timer | 113 | gpt@650 { // General Purpose Timer |
119 | compatible = "mpc5200b-gpt","mpc5200-gpt"; | 114 | compatible = "fsl,mpc5200b-gpt","fsl,mpc5200-gpt"; |
120 | device_type = "gpt"; | ||
121 | cell-index = <5>; | 115 | cell-index = <5>; |
122 | reg = <650 10>; | 116 | reg = <650 10>; |
123 | interrupts = <1 e 0>; | 117 | interrupts = <1 e 0>; |
@@ -125,8 +119,7 @@ | |||
125 | }; | 119 | }; |
126 | 120 | ||
127 | gpt@660 { // General Purpose Timer | 121 | gpt@660 { // General Purpose Timer |
128 | compatible = "mpc5200b-gpt","mpc5200-gpt"; | 122 | compatible = "fsl,mpc5200b-gpt","fsl,mpc5200-gpt"; |
129 | device_type = "gpt"; | ||
130 | cell-index = <6>; | 123 | cell-index = <6>; |
131 | reg = <660 10>; | 124 | reg = <660 10>; |
132 | interrupts = <1 f 0>; | 125 | interrupts = <1 f 0>; |
@@ -134,8 +127,7 @@ | |||
134 | }; | 127 | }; |
135 | 128 | ||
136 | gpt@670 { // General Purpose Timer | 129 | gpt@670 { // General Purpose Timer |
137 | compatible = "mpc5200b-gpt","mpc5200-gpt"; | 130 | compatible = "fsl,mpc5200b-gpt","fsl,mpc5200-gpt"; |
138 | device_type = "gpt"; | ||
139 | cell-index = <7>; | 131 | cell-index = <7>; |
140 | reg = <670 10>; | 132 | reg = <670 10>; |
141 | interrupts = <1 10 0>; | 133 | interrupts = <1 10 0>; |
diff --git a/drivers/watchdog/mpc5200_wdt.c b/drivers/watchdog/mpc5200_wdt.c index 9cfb97576623..11f6a111e75b 100644 --- a/drivers/watchdog/mpc5200_wdt.c +++ b/drivers/watchdog/mpc5200_wdt.c | |||
@@ -176,6 +176,8 @@ static int mpc5200_wdt_probe(struct of_device *op, const struct of_device_id *ma | |||
176 | 176 | ||
177 | has_wdt = of_get_property(op->node, "has-wdt", NULL); | 177 | has_wdt = of_get_property(op->node, "has-wdt", NULL); |
178 | if (!has_wdt) | 178 | if (!has_wdt) |
179 | has_wdt = of_get_property(op->node, "fsl,has-wdt", NULL); | ||
180 | if (!has_wdt) | ||
179 | return -ENODEV; | 181 | return -ENODEV; |
180 | 182 | ||
181 | wdt = kzalloc(sizeof(*wdt), GFP_KERNEL); | 183 | wdt = kzalloc(sizeof(*wdt), GFP_KERNEL); |
@@ -254,6 +256,7 @@ static int mpc5200_wdt_shutdown(struct of_device *op) | |||
254 | 256 | ||
255 | static struct of_device_id mpc5200_wdt_match[] = { | 257 | static struct of_device_id mpc5200_wdt_match[] = { |
256 | { .compatible = "mpc5200-gpt", }, | 258 | { .compatible = "mpc5200-gpt", }, |
259 | { .compatible = "fsl,mpc5200-gpt", }, | ||
257 | {}, | 260 | {}, |
258 | }; | 261 | }; |
259 | static struct of_platform_driver mpc5200_wdt_driver = { | 262 | static struct of_platform_driver mpc5200_wdt_driver = { |