x1000: GPIO refactor

The GPIO API was pretty clunky and pin settings were decentralized,
making it hard to see what was happening and making GPIO stuff look
like a mess, frankly.

Instead of passing clunky (port, pin) pairs everywhere, GPIOs are now
identified with a single int. The extra overhead should be minimal as
GPIO configuration is generally not on a performance-critical path.

Pin assignments are now mostly consolidated in gpio-target.h and put
in various tables so gpio_init() can assign most pins at boot time.

Most drivers no longer need to touch GPIOs and basic pin I/O stuff
can happen without config since pins are put into the right state.
IRQ pins still need to be configured manually before use.

Change-Id: Ic5326284b0b2a2f613e9e76a41cb50e24af3aa47

1 files changed, 0 insertions, 2 deletions

diff --git a/firmware/target/mips/ingenic_x1000/pcm-x1000.c b/firmware/target/mips/ingenic_x1000/pcm-x1000.c
index fd5e9d20c8..a3da3411f2 100644
--- a/firmware/target/mips/ingenic_x1000/pcm-x1000.c
+++ b/firmware/target/mips/ingenic_x1000/pcm-x1000.c
@@ -28,7 +28,6 @@
 #include "panic.h"
 #include "dma-x1000.h"
 #include "irq-x1000.h"
-#include "gpio-x1000.h"
 #include "x1000/aic.h"
 #include "x1000/cpm.h"
 
@@ -55,7 +54,6 @@ void pcm_play_dma_init(void)
      * on Ingenic's datasheets but I'm not sure what they are. Probably safe to
      * assume they are not useful to Rockbox... */
     jz_writef(CPM_CLKGR, AIC(0));
-    gpio_config(GPIO_B, 0x1f, GPIO_DEVICE(1));
 
     /* Configure AIC with some sane defaults */
     jz_writef(AIC_CFG, RST(1));