[PATCH] Accessing GPIO GPB_XY and not losing the LED state
werner at openmoko.org
Thu Apr 10 00:52:41 CEST 2008
Holger Freyther wrote:
> So is the approach sane, implementation as well, comments?
I think that's the least evil way to fix this. Still touches code at
a lot of places :-(
One question: why change s3c2410_gpio_setpin to neo1973_gpb_set_pin,
and not something_setpin ? That seems to be a gratuituous confusion.
I'd also to suggest to do something like
BUG_ON(base != S3C24XX_GPIO_BASE(pin));
By the way, the vibrator doesn't need shadowing.
> - Fix the gsm reset handling: GTA01_GPIO_MODEM_RST != GTA02_GPIO_MODEM_RST
Ah, so we have a bug there as well. And it's a nasty one, because it
could start myths like "GSM firmware download only works from outside
the phone". Why hide it in this big change ? Make the slaying of each
bug a celebration, well deserving of having its own patch :-)
More information about the openmoko-kernel