diff options
author | Roger Shimizu <rogershimizu@gmail.com> | 2016-01-21 09:38:44 -0500 |
---|---|---|
committer | Gregory CLEMENT <gregory.clement@free-electrons.com> | 2016-01-25 08:48:57 -0500 |
commit | 144e08abe80080c9c2cf0a06e40f1bc8150674eb (patch) | |
tree | 56935df661090a9e4d28e8c00e4563ba3a66948e | |
parent | 293de94cd66cded940849545f47c701cc36c0931 (diff) |
ARM: dts: kirkwood: gpio pin fixes for linkstation ls-wxl/wsxl
For kirkwood, gpio pins starts from 32 are in the 2nd bank, so it should be
converted to "gpio1 <pin minus 32>" in dts file.
e.g. gpio 40 should be "gpio1 8"
Besides, a few other pin fixes for ls-wxl/wsxl, to match with mpp pin
definition:
- gpio-leds: "lswxl:blue:power" pin
- gpio-leds: "lswxl:red:func" pin
- gpio-leds: "lswxl:red:hdderr0" pin
- gpio-leds: "lswxl:red:hdderr1" pin
- gpio-fan: low/high/alarm pin
The pin/bank issue was found when discussing Debian Bug #810894
[https://bugs.debian.org/810894#47]
Fixes: e54e4b1b622e ("ARM: dts: add buffalo linkstation ls-wxl/wsxl")
Reported-by: Arnaud Patard (Rtp) <arnaud.patard@rtp-net.org>
Signed-off-by: Roger Shimizu <rogershimizu@gmail.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
-rw-r--r-- | arch/arm/boot/dts/kirkwood-lswxl.dts | 29 |
1 files changed, 15 insertions, 14 deletions
diff --git a/arch/arm/boot/dts/kirkwood-lswxl.dts b/arch/arm/boot/dts/kirkwood-lswxl.dts index f5db16a08597..0e93f6d19259 100644 --- a/arch/arm/boot/dts/kirkwood-lswxl.dts +++ b/arch/arm/boot/dts/kirkwood-lswxl.dts | |||
@@ -1,7 +1,8 @@ | |||
1 | /* | 1 | /* |
2 | * Device Tree file for Buffalo Linkstation LS-WXL/WSXL | 2 | * Device Tree file for Buffalo Linkstation LS-WXL/WSXL |
3 | * | 3 | * |
4 | * Copyright (C) 2015, rogershimizu@gmail.com | 4 | * Copyright (C) 2015, 2016 |
5 | * Roger Shimizu <rogershimizu@gmail.com> | ||
5 | * | 6 | * |
6 | * This program is free software; you can redistribute it and/or | 7 | * This program is free software; you can redistribute it and/or |
7 | * modify it under the terms of the GNU General Public License | 8 | * modify it under the terms of the GNU General Public License |
@@ -156,21 +157,21 @@ | |||
156 | button@1 { | 157 | button@1 { |
157 | label = "Function Button"; | 158 | label = "Function Button"; |
158 | linux,code = <KEY_OPTION>; | 159 | linux,code = <KEY_OPTION>; |
159 | gpios = <&gpio1 41 GPIO_ACTIVE_LOW>; | 160 | gpios = <&gpio1 9 GPIO_ACTIVE_LOW>; |
160 | }; | 161 | }; |
161 | 162 | ||
162 | button@2 { | 163 | button@2 { |
163 | label = "Power-on Switch"; | 164 | label = "Power-on Switch"; |
164 | linux,code = <KEY_RESERVED>; | 165 | linux,code = <KEY_RESERVED>; |
165 | linux,input-type = <5>; | 166 | linux,input-type = <5>; |
166 | gpios = <&gpio1 42 GPIO_ACTIVE_LOW>; | 167 | gpios = <&gpio1 10 GPIO_ACTIVE_LOW>; |
167 | }; | 168 | }; |
168 | 169 | ||
169 | button@3 { | 170 | button@3 { |
170 | label = "Power-auto Switch"; | 171 | label = "Power-auto Switch"; |
171 | linux,code = <KEY_ESC>; | 172 | linux,code = <KEY_ESC>; |
172 | linux,input-type = <5>; | 173 | linux,input-type = <5>; |
173 | gpios = <&gpio1 43 GPIO_ACTIVE_LOW>; | 174 | gpios = <&gpio1 11 GPIO_ACTIVE_LOW>; |
174 | }; | 175 | }; |
175 | }; | 176 | }; |
176 | 177 | ||
@@ -185,12 +186,12 @@ | |||
185 | 186 | ||
186 | led@1 { | 187 | led@1 { |
187 | label = "lswxl:blue:func"; | 188 | label = "lswxl:blue:func"; |
188 | gpios = <&gpio1 36 GPIO_ACTIVE_LOW>; | 189 | gpios = <&gpio1 4 GPIO_ACTIVE_LOW>; |
189 | }; | 190 | }; |
190 | 191 | ||
191 | led@2 { | 192 | led@2 { |
192 | label = "lswxl:red:alarm"; | 193 | label = "lswxl:red:alarm"; |
193 | gpios = <&gpio1 49 GPIO_ACTIVE_LOW>; | 194 | gpios = <&gpio1 17 GPIO_ACTIVE_LOW>; |
194 | }; | 195 | }; |
195 | 196 | ||
196 | led@3 { | 197 | led@3 { |
@@ -200,23 +201,23 @@ | |||
200 | 201 | ||
201 | led@4 { | 202 | led@4 { |
202 | label = "lswxl:blue:power"; | 203 | label = "lswxl:blue:power"; |
203 | gpios = <&gpio1 8 GPIO_ACTIVE_LOW>; | 204 | gpios = <&gpio1 7 GPIO_ACTIVE_LOW>; |
204 | }; | 205 | }; |
205 | 206 | ||
206 | led@5 { | 207 | led@5 { |
207 | label = "lswxl:red:func"; | 208 | label = "lswxl:red:func"; |
208 | gpios = <&gpio1 5 GPIO_ACTIVE_LOW>; | 209 | gpios = <&gpio1 2 GPIO_ACTIVE_LOW>; |
209 | default-state = "keep"; | 210 | default-state = "keep"; |
210 | }; | 211 | }; |
211 | 212 | ||
212 | led@6 { | 213 | led@6 { |
213 | label = "lswxl:red:hdderr0"; | 214 | label = "lswxl:red:hdderr0"; |
214 | gpios = <&gpio1 2 GPIO_ACTIVE_LOW>; | 215 | gpios = <&gpio0 8 GPIO_ACTIVE_LOW>; |
215 | }; | 216 | }; |
216 | 217 | ||
217 | led@7 { | 218 | led@7 { |
218 | label = "lswxl:red:hdderr1"; | 219 | label = "lswxl:red:hdderr1"; |
219 | gpios = <&gpio1 3 GPIO_ACTIVE_LOW>; | 220 | gpios = <&gpio1 14 GPIO_ACTIVE_LOW>; |
220 | }; | 221 | }; |
221 | }; | 222 | }; |
222 | 223 | ||
@@ -225,15 +226,15 @@ | |||
225 | pinctrl-0 = <&pmx_fan_low &pmx_fan_high &pmx_fan_lock>; | 226 | pinctrl-0 = <&pmx_fan_low &pmx_fan_high &pmx_fan_lock>; |
226 | pinctrl-names = "default"; | 227 | pinctrl-names = "default"; |
227 | 228 | ||
228 | gpios = <&gpio0 47 GPIO_ACTIVE_LOW | 229 | gpios = <&gpio1 16 GPIO_ACTIVE_LOW |
229 | &gpio0 48 GPIO_ACTIVE_LOW>; | 230 | &gpio1 15 GPIO_ACTIVE_LOW>; |
230 | 231 | ||
231 | gpio-fan,speed-map = <0 3 | 232 | gpio-fan,speed-map = <0 3 |
232 | 1500 2 | 233 | 1500 2 |
233 | 3250 1 | 234 | 3250 1 |
234 | 5000 0>; | 235 | 5000 0>; |
235 | 236 | ||
236 | alarm-gpios = <&gpio1 49 GPIO_ACTIVE_HIGH>; | 237 | alarm-gpios = <&gpio1 8 GPIO_ACTIVE_HIGH>; |
237 | }; | 238 | }; |
238 | 239 | ||
239 | restart_poweroff { | 240 | restart_poweroff { |
@@ -256,7 +257,7 @@ | |||
256 | enable-active-high; | 257 | enable-active-high; |
257 | regulator-always-on; | 258 | regulator-always-on; |
258 | regulator-boot-on; | 259 | regulator-boot-on; |
259 | gpio = <&gpio0 37 GPIO_ACTIVE_HIGH>; | 260 | gpio = <&gpio1 5 GPIO_ACTIVE_HIGH>; |
260 | }; | 261 | }; |
261 | hdd_power0: regulator@2 { | 262 | hdd_power0: regulator@2 { |
262 | compatible = "regulator-fixed"; | 263 | compatible = "regulator-fixed"; |