drivers: gpio_esp32: Drop IRAM_ATTR from forward declarations

stephanosio · carlescufi · commit 11d27f12ad84 · 2022-06-20T10:28:06.000+02:00
This commit drops the `IRAM_ATTR` macro from the function declarations
because:

1. `IRAM_ATTR` macro makes use of the `__COUNTER__` preprocessor macro,
   which increments for every macro invocation and causes the section
   specified in the forward declaration to not match that of the
   function definition.

2. Section attributes need not be specified for forward declarations.

Signed-off-by: Stephanos Ioannidis &lt;root@stephanos.io&gt;
diff --git a/drivers/gpio/gpio_esp32.c b/drivers/gpio/gpio_esp32.c
@@ -415,7 +415,7 @@ static void IRAM_ATTR gpio_esp32_fire_callbacks(const struct device *dev)
 	}
 }
 
-static void IRAM_ATTR gpio_esp32_isr(void *param);
+static void gpio_esp32_isr(void *param);
 
 static int gpio_esp32_init(const struct device *dev)
 {

Original file line number	Diff line number	Diff line change
`@@ -415,7 +415,7 @@ static void IRAM_ATTR gpio_esp32_fire_callbacks(const struct device *dev)`
`415`	`415`	`}`
`416`	`416`	`}`
`417`	`417`
`418`		`-static void IRAM_ATTR gpio_esp32_isr(void *param);`
	`418`	`+static void gpio_esp32_isr(void *param);`
`419`	`419`
`420`	`420`	`static int gpio_esp32_init(const struct device *dev)`
`421`	`421`	`{`