watchdog: mtx1-wdt: fix GPIO toggling
authorFlorian Fainelli <florian@openwrt.org>
Wed, 15 Jun 2011 17:15:41 +0000 (19:15 +0200)
committerWim Van Sebroeck <wim@iguana.be>
Tue, 28 Jun 2011 07:42:30 +0000 (07:42 +0000)
commit2ea4e76e997019ae25ac3417aa46e31ddf7ecb17
tree347392d5b9cfc8d5fd97c636050b01d1a338ff1a
parent9b19d40aa3ebaf1078779da10555da2ab8512422
watchdog: mtx1-wdt: fix GPIO toggling

Commit e391be76 (MIPS: Alchemy: Clean up GPIO registers and accessors)
changed the way the GPIO was toggled. Prior to this patch, we would
always actively drive the GPIO output to either 0 or 1, this patch
drove the GPIO active to 0, and put the GPIO in tristate to drive it
to 1, unfortunately this does not work, revert back to active driving.

Using a signed variable (gstate) to hold the gpio state and using a bit-
wise operation on it also resulted in toggling value from 1 to -2 since
the variable is signed. This value was then passed on to gpio_direction_
output, which always perform a if (value) ... to set the value to the
gpio, so we were always writing a 1 to this GPIO instead of 1 -> 0 -> 1 ...

Signed-off-by: Florian Fainelli <florian@openwrt.org>
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
Cc: stable <stable@kernel.org>
drivers/watchdog/mtx-1_wdt.c