atheros: fix kernel warning about usage of reset gpio
authorGabor Juhos <juhosg@openwrt.org>
Wed, 25 Sep 2013 17:51:12 +0000 (17:51 +0000)
committerGabor Juhos <juhosg@openwrt.org>
Wed, 25 Sep 2013 17:51:12 +0000 (17:51 +0000)
During a reset the gpiolib warns about using the reset gpio
without explicit requesting it before:

[  147.930000] ------------[ cut here ]------------
[  147.930000] WARNING: at drivers/gpio/gpiolib.c:103 gpio_ensure_requested+0x68/0xcc()
[  147.930000] autorequest GPIO-5
[  147.930000] Modules linked in: ath5k ath mac80211 ipv6 cfg80211 compat arc4 crypto_blkcipher aead gpio_button_hotplug
[  147.930000] Call Trace:
[  147.930000] [<8004e3bc>] dump_stack+0x8/0x34
[  147.930000] [<80058430>] warn_slowpath_common+0x78/0xa4
[  147.930000] [<800584e4>] warn_slowpath_fmt+0x2c/0x38
[  147.930000] [<80166f18>] gpio_ensure_requested+0x68/0xcc
[  147.930000] [<80167264>] gpio_direction_output+0xdc/0x168
[  147.930000] [<8004aba8>] ar2315_restart+0x28/0x54
[  147.930000] [<80069778>] sys_reboot+0x178/0x1c0
[  147.930000] [<80043670>] stack_done+0x20/0x40
[  147.930000]
[  147.930000] ---[ end trace 2c7a53d8549562d6 ]---

This is fixed with this patch by using another gpiolib api call
which includes the missing request.

Signed-off-by: Michael Heimpold <mhei@heimpold.de>
Signed-off-by: Gabor Juhos <juhosg@openwrt.org>
SVN-Revision: 38183

target/linux/atheros/patches-3.10/100-board.patch
target/linux/atheros/patches-3.8/100-board.patch

index 5fb3e00c0c0dd1fc8f6e386a3e07682c4252b5e3..96be80d044c3deff1d5edd92a540966435274c95 100644 (file)
 +      /* Cold reset does not work on the AR2315/6, use the GPIO reset bits a workaround.
 +       * give it some time to attempt a gpio based hardware reset
 +       * (atheros reference design workaround) */
-+      gpio_direction_output(AR2315_RESET_GPIO, 0);
++      gpio_request_one(AR2315_RESET_GPIO, GPIOF_OUT_INIT_LOW, "Reset");
 +      mdelay(100);
 +
 +      /* Some boards (e.g. Senao EOC-2610) don't implement the reset logic
index 8663156ade0c843ba073aa53687d468f2f94fd40..32667e982543100eb6d97ac9301529586f07654d 100644 (file)
 +      /* Cold reset does not work on the AR2315/6, use the GPIO reset bits a workaround.
 +       * give it some time to attempt a gpio based hardware reset
 +       * (atheros reference design workaround) */
-+      gpio_direction_output(AR2315_RESET_GPIO, 0);
++      gpio_request_one(AR2315_RESET_GPIO, GPIOF_OUT_INIT_LOW, "Reset");
 +      mdelay(100);
 +
 +      /* Some boards (e.g. Senao EOC-2610) don't implement the reset logic