update GPIO driver

.gitignore

@@ -46,3 +46,5 @@ sw/device/lib/drivers/**/*_structs.h
 
 # openroad
 flow/OpenROAD-flow-scripts
+
+run.sh

run.sh

@@ -0,0 +1,28 @@
+#! usr/bin/bash# This script is to be run in the x-heep base directory (aka BASE)
+	# It takes only one argument: the application name.
+	# It takes for granted that myApplication.c is located in
+	# BASE/sw/applications/myApplication/# Run this script from the BASE with
+	# $ sudo bash pynq-run <name-of-your-app># The pynq-z2 board must be
+	#   - Powered
+	#   - Loaded with the appropriate bitstream (use vivado)
+	#   - Connected to the EPFL programmer
+	#   - With sw1=0(outside) and sw0=1(inside)
+#echo Will build for $1
+echo Will run application $1
+
+make app-clean 
+	# Compile and build the binaries to be loaded in the flash.
+	# To use different LINKER options, just modify this script.
+	# (remember to also change the make flash-load command below)
+make app PROJECT=$1 LINKER=flash_load TARGET=pynq-z2
+
+
+
+	# To use the iceprog
+
+( cd sw/vendor/yosyshq_icestorm/iceprog && make clean && make all)
+
+# Make and load the flash programmable binary into the iceprog
+make flash-prog &&
+	# Open picocom to communicate with the board through UART
+picocom -b 115200 -r -l --imap lfcrlf /dev/ttyUSB2

sw/applications/blinky_freertos/main.c

@@ -225,9 +225,6 @@ int8_t intr_flag = 0;
 /* Temporal counter to store blinking status */
 int8_t intr_blink = 0;
 
-/* GPIO struct */
-gpio_t gpio;
-
 /****************************************************************************/
 /**                                                                        **/
 /*                           EXPORTED FUNCTIONS                             */
@@ -251,25 +248,22 @@ void system_init(void)
     soc_ctrl.base_addr = mmio_region_from_addr((uintptr_t)SOC_CTRL_START_ADDRESS);
     uint32_t freq_hz = soc_ctrl_get_frequency(&soc_ctrl);
 
-    // Set GPIOs
-    gpio_params_t gpio_params;
-    gpio_result_t gpio_res;
-    gpio_params.base_addr = mmio_region_from_addr((uintptr_t)GPIO_START_ADDRESS);
-    gpio_res = gpio_init(gpio_params, &gpio);
-    if (gpio_res != kGpioOk) 
-	{
-      printf("GPIO Failed\n;");
-    }
 
-    gpio_res = gpio_output_set_enabled(&gpio, GPIO_LD5_R, true);
-    if (gpio_res != kGpioOk) printf("Failed\n;");
-	gpio_write(&gpio, GPIO_LD5_R, false);
-	gpio_res = gpio_output_set_enabled(&gpio, GPIO_LD5_B, true);
-    if (gpio_res != kGpioOk) printf("Failed\n;");
-	gpio_write(&gpio, GPIO_LD5_B, false);
-	gpio_res = gpio_output_set_enabled(&gpio, GPIO_LD5_G, true);
+    gpio_result_t gpio_res;
+    gpio_cfg_t pin_cfg = {
+        .pin= GPIO_LD5_R, 
+        .mode= GpioModeOutPushPull
+    };
+    gpio_res = gpio_config(pin_cfg);
+    pin_cfg.pin = GPIO_LD5_B;
+    gpio_res |= gpio_config(pin_cfg);
+    pin_cfg.pin = GPIO_LD5_G;
+	gpio_res |= gpio_config(pin_cfg);
     if (gpio_res != kGpioOk) printf("Failed\n;");
-	gpio_write(&gpio, GPIO_LD5_G, false);
+
+    gpio_write(GPIO_LD5_R, false);
+    gpio_write(GPIO_LD5_B, false);
+	gpio_write(GPIO_LD5_G, false);
 
     // Setup rv_timer_0_1
     mmio_region_t timer_0_1_reg = mmio_region_from_addr(RV_TIMER_AO_START_ADDRESS);
@@ -444,30 +438,30 @@ void vToggleLED( void )
 {
   if (intr_blink == 0)
   { 
-	gpio_write(&gpio, GPIO_LD5_R, true);
-	gpio_write(&gpio, GPIO_LD5_B, false);
-	gpio_write(&gpio, GPIO_LD5_G, false);
+	gpio_write(GPIO_LD5_R, true);
+	gpio_write(GPIO_LD5_B, false);
+	gpio_write(GPIO_LD5_G, false);
 	intr_blink++;
   }
   else if (intr_blink == 1)
   { 
-	gpio_write(&gpio, GPIO_LD5_R, false);
-	gpio_write(&gpio, GPIO_LD5_B, true);
-	gpio_write(&gpio, GPIO_LD5_G, false);
+	gpio_write(GPIO_LD5_R, false);
+	gpio_write(GPIO_LD5_B, true);
+	gpio_write(GPIO_LD5_G, false);
 	intr_blink++;
   }
   else if (intr_blink == 2)
   { 
-	gpio_write(&gpio, GPIO_LD5_R, false);
-	gpio_write(&gpio, GPIO_LD5_B, false);
-	gpio_write(&gpio, GPIO_LD5_G, true);
+	gpio_write(GPIO_LD5_R, false);
+	gpio_write(GPIO_LD5_B, false);
+	gpio_write(GPIO_LD5_G, true);
 	intr_blink++;
   }
   else
   { 
-	gpio_write(&gpio, GPIO_LD5_R, false);
-	gpio_write(&gpio, GPIO_LD5_B, false);
-	gpio_write(&gpio, GPIO_LD5_G, false);
+	gpio_write(GPIO_LD5_R, false);
+	gpio_write(GPIO_LD5_B, false);
+	gpio_write(GPIO_LD5_G, false);
 	intr_blink = 0;
   }
 

sw/applications/example_gpio_cnt/main.c

@@ -14,6 +14,7 @@
 #include "pad_control.h"
 #include "pad_control_regs.h"  // Generated.
 #include "x-heep.h"
+#include "gpio_structs.h" //remove
 
 /*
 Notes:
@@ -71,16 +72,6 @@ int main(int argc, char *argv[])
     pad_control_set_mux(&pad_control, (ptrdiff_t)(PAD_CONTROL_PAD_MUX_I2C_SDA_REG_OFFSET), 1);
 #endif
 
-    gpio_params_t gpio_params;
-    gpio_t gpio;
-    gpio_result_t gpio_res;
-    gpio_params.base_addr = mmio_region_from_addr((uintptr_t)GPIO_START_ADDRESS);
-    gpio_res = gpio_init(gpio_params, &gpio);
-    if (gpio_res != kGpioOk) {
-        printf("Failed\n;");
-        return -1;
-    }
-
     plic_res = dif_plic_irq_set_priority(&rv_plic, GPIO_INTR, 1);
     if (plic_res != kDifPlicOk) {
         printf("Failed\n;");
@@ -101,29 +92,37 @@ int main(int argc, char *argv[])
     CSR_SET_BITS(CSR_REG_MIE, mask);
     external_intr_flag = 0;
 
-    gpio_res = gpio_output_set_enabled(&gpio, GPIO_TB_OUT, true);
-    if (gpio_res != kGpioOk) {
-        printf("Failed\n;");
-        return -1;
-    }
-    gpio_write(&gpio, GPIO_TB_OUT, false);
+    //gpio_reset_all();
+    gpio_result_t gpio_res;
 
-    gpio_res = gpio_input_enabled(&gpio, GPIO_TB_IN, true);
-    if (gpio_res != kGpioOk) {
+    gpio_cfg_t cfg_out = {
+        .pin = GPIO_TB_OUT,
+        .mode = GpioModeOutPushPull
+    };
+    gpio_res = gpio_config(cfg_out);
+    if (gpio_res != GpioOk) {
         printf("Failed\n;");
         return -1;
     }
-
-
-    gpio_res = gpio_irq_set_trigger(&gpio, GPIO_TB_IN, true, kGpioIrqTriggerEdgeRising);
-    if (gpio_res != kGpioOk) {
+    gpio_res = gpio_write(GPIO_TB_OUT, false);
+
+    gpio_cfg_t cfg_in = {
+        .pin = GPIO_TB_IN,
+        .mode = GpioModeIn,
+        .en_input_sampling = true,
+        .en_intr = true,
+        .intr_type = GpioIntrEdgeRising
+    };
+    gpio_res = gpio_config(cfg_in);
+    if (gpio_res != GpioOk) {
         printf("Failed\n;");
         return -1;
     }
 
     printf("Write 1 to GPIO 30 and wait for interrupt...\n");
+    external_intr_flag = 0;
     while(external_intr_flag==0) {
-        gpio_write(&gpio, GPIO_TB_OUT, true);
+        gpio_res = gpio_write(GPIO_TB_OUT, true);
         wait_for_interrupt();
     }
     printf("Success\n");

sw/applications/example_power_gating_core/main.c

@@ -22,13 +22,13 @@ static const uint64_t kTickFreqHz = 1000 * 1000; // 1 MHz
 
 static power_manager_t power_manager;
 static dif_plic_t rv_plic;
-static gpio_t gpio;
+// static gpio_t gpio;
 
 int main(int argc, char *argv[])
 {
     // Setup fast interrupt controller
-    fast_intr_ctrl_t fast_intr_ctrl;
-    fast_intr_ctrl.base_addr = mmio_region_from_addr((uintptr_t)FAST_INTR_CTRL_START_ADDRESS);
+    // fast_intr_ctrl_t fast_intr_ctrl;
+    // fast_intr_ctrl.base_addr = mmio_region_from_addr((uintptr_t)FAST_INTR_CTRL_START_ADDRESS);
 
     // Setup power_manager
     mmio_region_t power_manager_reg = mmio_region_from_addr(POWER_MANAGER_START_ADDRESS);
@@ -56,9 +56,9 @@ int main(int argc, char *argv[])
     rv_timer_init(timer_2_3_reg, (rv_timer_config_t){.hart_count = 2, .comparator_count = 1}, &timer_2_3);
 
     // Setup gpio
-    gpio_params_t gpio_params;
-    gpio_params.base_addr = mmio_region_from_addr((uintptr_t)GPIO_START_ADDRESS);
-    gpio_init(gpio_params, &gpio);
+    // gpio_params_t gpio_params;
+    // gpio_params.base_addr = mmio_region_from_addr((uintptr_t)GPIO_START_ADDRESS);
+    // gpio_init(gpio_params, &gpio);
 
     // Init cpu_subsystem's counters
     if (power_gate_counters_init(&power_manager_cpu_counters, 30, 30, 30, 30, 30, 30, 0, 0) != kPowerManagerOk_e)
@@ -127,10 +127,19 @@ int main(int argc, char *argv[])
     // Power-gate and wake-up due to plic
     dif_plic_irq_set_priority(&rv_plic, GPIO_INTR_31, 1);
     dif_plic_irq_set_enabled(&rv_plic, GPIO_INTR_31, 0, kDifPlicToggleEnabled);
-    gpio_output_set_enabled(&gpio, 30, true);
-    gpio_irq_set_trigger(&gpio, 1 << 31, kGpioIrqTriggerLevelHigh);
-    gpio_irq_set_enabled(&gpio, 31, true);
-    gpio_write(&gpio, 30, true);
+
+    // gpio_output_set_enabled(&gpio, 30, true);
+    gpio_cfg_t pin1_cfg = {.pin = 30, .mode = GpioModeOutPushPull};
+    gpio_config (pin1_cfg);
+
+    // gpio_irq_set_trigger(&gpio, 1 << 31, kGpioIrqTriggerLevelHigh);
+    // gpio_irq_set_enabled(&gpio, 31, true);
+    gpio_cfg_t pin2_cfg = {.pin = 31, .mode = GpioModeIn,.en_input_sampling = true, 
+        .en_intr = true, .intr_type = GpioIntrLevelHigh};
+    gpio_config (pin2_cfg);
+
+    // gpio_write(&gpio, 30, true);
+    gpio_write(30, true);
 
     CSR_CLEAR_BITS(CSR_REG_MSTATUS, 0x8);
     if (power_gate_core(&power_manager, kPlic_pm_e, &power_manager_cpu_counters) != kPowerManagerOk_e)

sw/applications/gpio_pmw/main.c

@@ -12,26 +12,37 @@
 
 int main(int argc, char *argv[])
 {
-    gpio_params_t gpio_params;
-    gpio_t gpio;
+    // gpio_params_t gpio_params;
+    // gpio_t gpio;
     gpio_result_t gpio_res;
-    gpio_params.base_addr = mmio_region_from_addr((uintptr_t)GPIO_AO_START_ADDRESS);
-    gpio_res = gpio_init(gpio_params, &gpio);
-    gpio_res = gpio_output_set_enabled(&gpio, GPIO_PMW, true);
+    // gpio_params.base_addr = mmio_region_from_addr((uintptr_t)GPIO_AO_START_ADDRESS);
+    // gpio_res = gpio_init(gpio_params, &gpio);
+    // gpio_res = gpio_output_set_enabled(&gpio, GPIO_PMW, true);
+    gpio_cfg_t pin_cfg = {
+        .pin = GPIO_PMW,     
+        .mode = GpioModeOutPushPull    
+    };
+    gpio_res = gpio_config (pin_cfg);
+    if (gpio_res != GpioOk)
+        printf("Gpio initialization failed!\n");
 
 #ifdef TARGET_PYNQ_Z2
 #pragma message ( "this application never ends" )
     while(1) {
-      gpio_write(&gpio, GPIO_PMW, true);
+      // gpio_write(&gpio, GPIO_PMW, true);
+        gpio_write(GPIO_PMW, true);
       for(int i=0;i<10;i++) asm volatile("nop");
-      gpio_write(&gpio, GPIO_PMW, false);
+      // gpio_write(&gpio, GPIO_PMW, false);
+        gpio_write(GPIO_PMW, false);
       for(int i=0;i<10;i++) asm volatile("nop");
     }
 #else
     for(int i=0;i<100;i++) {
-      gpio_write(&gpio, GPIO_PMW, true);
+      // gpio_write(&gpio, GPIO_PMW, true);
+        pio_write(GPIO_PMW, true);
       for(int i=0;i<10;i++) asm volatile("nop");
-      gpio_write(&gpio, GPIO_PMW, false);
+      // gpio_write(&gpio, GPIO_PMW, false);
+        gpio_write(GPIO_PMW, false);
       for(int i=0;i<10;i++) asm volatile("nop");
     }
 #endif