michaelh/zephyr
1
0
Fork 0
Code Releases Activity

drivers: Check and fix device const qualifier on ISR

Browse source 
Re-running the script that checks for the const qualifier missing on
struct device ISR's parameter.

The script also changes the parameter 'arg' to 'dev' when relevant.

Signed-off-by: Tomasz Bursztyka <tomasz.bursztyka@linux.intel.com>
This commit is contained in:
Tomasz Bursztyka 2022-01-06 11:27:07 +01:00 • committed by Carles Cufí
commit 9a18fdea3f
22 changed files with 77 additions and 96 deletions
Download patch file Download diff file Expand all files Collapse all files

3
drivers/adc/adc_ite_it8xxx2.c
View file

@ -371,9 +371,8 @@ static void adc_context_update_buffer_pointer(struct adc_context *ctx,
}
}
static void adc_it8xxx2_isr(const void *arg)
static void adc_it8xxx2_isr(const struct device *dev)
{
const struct device *dev = (const struct device *)arg;
struct adc_it8xxx2_data *data = dev->data;
LOG_DBG("ADC ISR triggered.");

3
drivers/adc/adc_npcx.c
View file

@ -70,9 +70,8 @@ struct adc_npcx_data {
#define HAL_INSTANCE(dev) ((struct adc_reg *)((const struct adc_npcx_config *)(dev)->config)->base)
/* ADC local functions */
static void adc_npcx_isr(void *arg)
static void adc_npcx_isr(const struct device *dev)
{
const struct device *dev = arg;
const struct adc_npcx_config *config = dev->config;
struct adc_npcx_data *const data = dev->data;
struct adc_reg *const inst = HAL_INSTANCE(dev);

7
drivers/can/can_stm32fd.c
View file

@ -163,9 +163,8 @@ int can_stm32fd_get_max_bitrate(const struct device *dev, uint32_t *max_bitrate)
return 0;
}
static void can_stm32fd_line_0_isr(void *arg)
static void can_stm32fd_line_0_isr(const struct device *dev)
{
struct device *dev = (struct device *)arg;
const struct can_stm32fd_config *cfg = dev->config;
const struct can_mcan_config *mcan_cfg = &cfg->mcan_cfg;
struct can_stm32fd_data *data = dev->data;
@ -175,11 +174,9 @@ static void can_stm32fd_line_0_isr(void *arg)
can_mcan_line_0_isr(mcan_cfg, msg_ram, mcan_data);
}
static void can_stm32fd_line_1_isr(void *arg)
static void can_stm32fd_line_1_isr(const struct device *dev)
{
struct device *dev = (struct device *)arg;
const struct can_stm32fd_config *cfg = dev->config;
struct can_stm32fd_data *data = dev->data;
const struct can_mcan_config *mcan_cfg = &cfg->mcan_cfg;
struct can_mcan_data *mcan_data = &data->mcan_data;
struct can_mcan_msg_sram *msg_ram = cfg->msg_sram;

3
drivers/dac/dac_sam.c
View file

@ -46,9 +46,8 @@ struct dac_sam_dev_data {
struct dac_channel dac_channels[DAC_CHANNEL_NO];
};
static void dac_sam_isr(void *arg)
static void dac_sam_isr(const struct device *dev)
{
const struct device *dev = (const struct device *)arg;
const struct dac_sam_dev_cfg *const dev_cfg = dev->config;
struct dac_sam_dev_data *const dev_data = dev->data;
Dacc *const dac = dev_cfg->regs;

3
drivers/dma/dma_iproc_pax_v1.c
View file

@ -551,10 +551,9 @@ static int peek_ring_cmpl(const struct device *dev,
return process_cmpl_event(dev, idx, pl_len);
}
#else
static void rm_isr(void *arg)
static void rm_isr(const struct device *dev)
{
uint32_t status, err_stat, idx;
const struct device *dev = arg;
struct dma_iproc_pax_data *pd = dev->data;
/* read and clear interrupt status */

3
drivers/dma/dma_iproc_pax_v2.c
View file

@ -637,10 +637,9 @@ static int peek_ring_cmpl(const struct device *dev,
return process_cmpl_event(dev, idx, pl_len);
}
#else
static void rm_isr(void *arg)
static void rm_isr(const struct device *dev)
{
uint32_t status, err_stat, idx;
const struct device *dev = arg;
struct dma_iproc_pax_data *pd = dev->data;
err_stat =

3
drivers/entropy/entropy_cc13xx_cc26xx.c
View file

@ -125,9 +125,8 @@ static int entropy_cc13xx_cc26xx_get_entropy(const struct device *dev,
return 0;
}
static void entropy_cc13xx_cc26xx_isr(const void *arg)
static void entropy_cc13xx_cc26xx_isr(const struct device *dev)
{
const struct device *dev = arg;
struct entropy_cc13xx_cc26xx_data *data = dev->data;
uint32_t src = 0;
uint32_t cnt;

3
drivers/espi/espi_npcx.c
View file

@ -266,9 +266,8 @@ const struct espi_bus_isr espi_bus_isr_tbl[] = {
#endif
};
static void espi_bus_generic_isr(void *arg)
static void espi_bus_generic_isr(const struct device *dev)
{
const struct device *dev = (const struct device *)arg;
struct espi_reg *const inst = HAL_INSTANCE(dev);
int i;
uint32_t mask, status;

8
drivers/espi/host_subs_npcx.c
View file

@ -223,7 +223,7 @@ static inline uint8_t host_shd_mem_wnd_size_sl(uint32_t size)
/* Host KBC sub-device local functions */
#if defined(CONFIG_ESPI_PERIPHERAL_8042_KBC)
static void host_kbc_ibf_isr(void *arg)
static void host_kbc_ibf_isr(const void *arg)
{
ARG_UNUSED(arg);
struct kbc_reg *const inst_kbc = host_sub_cfg.inst_kbc;
@ -249,7 +249,7 @@ static void host_kbc_ibf_isr(void *arg)
evt);
}
static void host_kbc_obe_isr(void *arg)
static void host_kbc_obe_isr(const void *arg)
{
ARG_UNUSED(arg);
struct kbc_reg *const inst_kbc = host_sub_cfg.inst_kbc;
@ -438,7 +438,7 @@ static void host_shared_mem_region_init(void)
#if defined(CONFIG_ESPI_PERIPHERAL_HOST_IO) || \
defined(CONFIG_ESPI_PERIPHERAL_EC_HOST_CMD)
/* Host pm (host io) sub-module isr function for all channels such as ACPI. */
static void host_pmch_ibf_isr(void *arg)
static void host_pmch_ibf_isr(const void *arg)
{
ARG_UNUSED(arg);
struct pmch_reg *const inst_acpi = host_sub_cfg.inst_pm_acpi;
@ -471,7 +471,7 @@ static void host_pmch_ibf_isr(void *arg)
/* Host port80 sub-device local functions */
#if defined(CONFIG_ESPI_PERIPHERAL_DEBUG_PORT_80)
static void host_port80_isr(void *arg)
static void host_port80_isr(const void *arg)
{
ARG_UNUSED(arg);
struct shm_reg *const inst_shm = host_sub_cfg.inst_shm;

3
drivers/gpio/gpio_rcar.c
View file

@ -73,9 +73,8 @@ static void gpio_rcar_modify_bit(const struct gpio_rcar_cfg *config,
gpio_rcar_write(config, offs, tmp);
}
static void gpio_rcar_port_isr(void *arg)
static void gpio_rcar_port_isr(const struct device *dev)
{
const struct device *dev = (const struct device *)arg;
const struct gpio_rcar_cfg *config = dev->config;
struct gpio_rcar_data *data = dev->data;
uint32_t pending, fsb, mask;

3
drivers/i2c/i2c_cc13xx_cc26xx.c
View file

@ -273,9 +273,8 @@ static int i2c_cc13xx_cc26xx_configure(const struct device *dev,
return 0;
}
static void i2c_cc13xx_cc26xx_isr(const void *arg)
static void i2c_cc13xx_cc26xx_isr(const struct device *dev)
{
const struct device *dev = arg;
const struct i2c_cc13xx_cc26xx_config *config = dev->config;
struct i2c_cc13xx_cc26xx_data *data = dev->data;
const uint32_t base = config->base;

3
drivers/i2c/i2c_dw.c
View file

@ -197,9 +197,8 @@ static inline void i2c_dw_write_byte_non_blocking(const struct device *dev, uint
static void i2c_dw_slave_read_clear_intr_bits(const struct device *dev);
#endif
static void i2c_dw_isr(void *arg)
static void i2c_dw_isr(const struct device *port)
{
const struct device *port = (const struct device *)arg;
struct i2c_dw_dev_config * const dw = port->data;
union ic_interrupt_register intr_stat;
uint32_t value;

3
drivers/i2c/i2c_ite_it8xxx2.c
View file

@ -881,9 +881,8 @@ static int i2c_it8xxx2_transfer(const struct device *dev, struct i2c_msg *msgs,
return i2c_parsing_return_value(dev);
}
static void i2c_it8xxx2_isr(void *arg)
static void i2c_it8xxx2_isr(const struct device *dev)
{
const struct device *dev = (const struct device *)arg;
struct i2c_it8xxx2_data *data = dev->data;
const struct i2c_it8xxx2_config *config = dev->config;

3
drivers/i2c/i2c_lpc11u6x.c
View file

@ -190,9 +190,8 @@ static int lpc11u6x_i2c_slave_unregister(const struct device *dev,
return 0;
}
static void lpc11u6x_i2c_isr(const void *arg)
static void lpc11u6x_i2c_isr(const struct device *dev)
{
const struct device *dev = arg;
struct lpc11u6x_i2c_data *data = dev->data;
struct lpc11u6x_i2c_regs *i2c = DEV_BASE(dev);
struct lpc11u6x_i2c_current_transfer *transfer = &data->transfer;

3
drivers/i2c/i2c_mchp_xec.c
View file

@ -723,10 +723,9 @@ static int i2c_xec_transfer(const struct device *dev, struct i2c_msg *msgs,
return 0;
}
static void i2c_xec_bus_isr(void *arg)
static void i2c_xec_bus_isr(const struct device *dev)
{
#ifdef CONFIG_I2C_SLAVE
const struct device *dev = (const struct device *)arg;
const struct i2c_xec_config *config =
(const struct i2c_xec_config *const) (dev->config);
struct i2c_xec_data *data = dev->data;

3
drivers/i2c/i2c_mchp_xec_v2.c
View file

@ -798,10 +798,9 @@ static int i2c_xec_transfer(const struct device *dev, struct i2c_msg *msgs,
return ret;
}
static void i2c_xec_bus_isr(void *arg)
static void i2c_xec_bus_isr(const struct device *dev)
{
#ifdef CONFIG_I2C_SLAVE
const struct device *dev = (const struct device *)arg;
const struct i2c_xec_config *cfg =
(const struct i2c_xec_config *const) (dev->config);
struct i2c_xec_data *data = dev->data;

3
drivers/i2s/i2s_mcux_sai.c
View file

@ -926,9 +926,8 @@ static void sai_driver_irq(const struct device *dev)
}
/* clear IRQ sources atm */
static void i2s_mcux_isr(void *arg)
static void i2s_mcux_isr(const struct device *dev)
{
const struct device *dev = (const struct device *)arg;
const struct i2s_mcux_config *dev_cfg = dev->config;
I2S_Type *base = (I2S_Type *)dev_cfg->base;

104
drivers/interrupt_controller/intc_exti_stm32.c
View file

@ -264,145 +264,145 @@ static void __stm32_exti_isr(int min, int max, const struct device *dev)
#if defined(CONFIG_SOC_SERIES_STM32F0X) || \
defined(CONFIG_SOC_SERIES_STM32L0X) || \
defined(CONFIG_SOC_SERIES_STM32G0X)
static inline void __stm32_exti_isr_0_1(const void *arg)
static inline void __stm32_exti_isr_0_1(const struct device *dev)
{
__stm32_exti_isr(0, 2, arg);
__stm32_exti_isr(0, 2, dev);
}
static inline void __stm32_exti_isr_2_3(const void *arg)
static inline void __stm32_exti_isr_2_3(const struct device *dev)
{
__stm32_exti_isr(2, 4, arg);
__stm32_exti_isr(2, 4, dev);
}
static inline void __stm32_exti_isr_4_15(const void *arg)
static inline void __stm32_exti_isr_4_15(const struct device *dev)
{
__stm32_exti_isr(4, 16, arg);
__stm32_exti_isr(4, 16, dev);
}
#else
static inline void __stm32_exti_isr_0(const void *arg)
static inline void __stm32_exti_isr_0(const struct device *dev)
{
__stm32_exti_isr(0, 1, arg);
__stm32_exti_isr(0, 1, dev);
}
static inline void __stm32_exti_isr_1(const void *arg)
static inline void __stm32_exti_isr_1(const struct device *dev)
{
__stm32_exti_isr(1, 2, arg);
__stm32_exti_isr(1, 2, dev);
}
static inline void __stm32_exti_isr_2(const void *arg)
static inline void __stm32_exti_isr_2(const struct device *dev)
{
__stm32_exti_isr(2, 3, arg);
__stm32_exti_isr(2, 3, dev);
}
static inline void __stm32_exti_isr_3(const void *arg)
static inline void __stm32_exti_isr_3(const struct device *dev)
{
__stm32_exti_isr(3, 4, arg);
__stm32_exti_isr(3, 4, dev);
}
static inline void __stm32_exti_isr_4(const void *arg)
static inline void __stm32_exti_isr_4(const struct device *dev)
{
__stm32_exti_isr(4, 5, arg);
__stm32_exti_isr(4, 5, dev);
}
#if defined(CONFIG_SOC_SERIES_STM32MP1X) || \
defined(CONFIG_SOC_SERIES_STM32L5X) || \
defined(CONFIG_SOC_SERIES_STM32U5X)
static inline void __stm32_exti_isr_5(const void *arg)
static inline void __stm32_exti_isr_5(const struct device *dev)
{
__stm32_exti_isr(5, 6, arg);
__stm32_exti_isr(5, 6, dev);
}
static inline void __stm32_exti_isr_6(const void *arg)
static inline void __stm32_exti_isr_6(const struct device *dev)
{
__stm32_exti_isr(6, 7, arg);
__stm32_exti_isr(6, 7, dev);
}
static inline void __stm32_exti_isr_7(const void *arg)
static inline void __stm32_exti_isr_7(const struct device *dev)
{
__stm32_exti_isr(7, 8, arg);
__stm32_exti_isr(7, 8, dev);
}
static inline void __stm32_exti_isr_8(const void *arg)
static inline void __stm32_exti_isr_8(const struct device *dev)
{
__stm32_exti_isr(8, 9, arg);
__stm32_exti_isr(8, 9, dev);
}
static inline void __stm32_exti_isr_9(const void *arg)
static inline void __stm32_exti_isr_9(const struct device *dev)
{
__stm32_exti_isr(9, 10, arg);
__stm32_exti_isr(9, 10, dev);
}
static inline void __stm32_exti_isr_10(const void *arg)
static inline void __stm32_exti_isr_10(const struct device *dev)
{
__stm32_exti_isr(10, 11, arg);
__stm32_exti_isr(10, 11, dev);
}
static inline void __stm32_exti_isr_11(const void *arg)
static inline void __stm32_exti_isr_11(const struct device *dev)
{
__stm32_exti_isr(11, 12, arg);
__stm32_exti_isr(11, 12, dev);
}
static inline void __stm32_exti_isr_12(const void *arg)
static inline void __stm32_exti_isr_12(const struct device *dev)
{
__stm32_exti_isr(12, 13, arg);
__stm32_exti_isr(12, 13, dev);
}
static inline void __stm32_exti_isr_13(const void *arg)
static inline void __stm32_exti_isr_13(const struct device *dev)
{
__stm32_exti_isr(13, 14, arg);
__stm32_exti_isr(13, 14, dev);
}
static inline void __stm32_exti_isr_14(const void *arg)
static inline void __stm32_exti_isr_14(const struct device *dev)
{
__stm32_exti_isr(14, 15, arg);
__stm32_exti_isr(14, 15, dev);
}
static inline void __stm32_exti_isr_15(const void *arg)
static inline void __stm32_exti_isr_15(const struct device *dev)
{
__stm32_exti_isr(15, 16, arg);
__stm32_exti_isr(15, 16, dev);
}
#endif
static inline void __stm32_exti_isr_9_5(const void *arg)
static inline void __stm32_exti_isr_9_5(const struct device *dev)
{
__stm32_exti_isr(5, 10, arg);
__stm32_exti_isr(5, 10, dev);
}
static inline void __stm32_exti_isr_15_10(const void *arg)
static inline void __stm32_exti_isr_15_10(const struct device *dev)
{
__stm32_exti_isr(10, 16, arg);
__stm32_exti_isr(10, 16, dev);
}
#if defined(CONFIG_SOC_SERIES_STM32F4X) || \
defined(CONFIG_SOC_SERIES_STM32F7X) || \
defined(CONFIG_SOC_SERIES_STM32F2X) || \
defined(CONFIG_SOC_SERIES_STM32MP1X)
static inline void __stm32_exti_isr_16(const void *arg)
static inline void __stm32_exti_isr_16(const struct device *dev)
{
__stm32_exti_isr(16, 17, arg);
__stm32_exti_isr(16, 17, dev);
}
static inline void __stm32_exti_isr_18(const void *arg)
static inline void __stm32_exti_isr_18(const struct device *dev)
{
__stm32_exti_isr(18, 19, arg);
__stm32_exti_isr(18, 19, dev);
}
static inline void __stm32_exti_isr_21(const void *arg)
static inline void __stm32_exti_isr_21(const struct device *dev)
{
__stm32_exti_isr(21, 22, arg);
__stm32_exti_isr(21, 22, dev);
}
static inline void __stm32_exti_isr_22(const void *arg)
static inline void __stm32_exti_isr_22(const struct device *dev)
{
__stm32_exti_isr(22, 23, arg);
__stm32_exti_isr(22, 23, dev);
}
#endif
#if defined(CONFIG_SOC_SERIES_STM32F7X) || \
defined(CONFIG_SOC_SERIES_STM32MP1X)
static inline void __stm32_exti_isr_23(const void *arg)
static inline void __stm32_exti_isr_23(const struct device *dev)
{
__stm32_exti_isr(23, 24, arg);
__stm32_exti_isr(23, 24, dev);
}
#endif
#endif /* CONFIG_SOC_SERIES_STM32F0X */

2
drivers/interrupt_controller/intc_gd32_exti.c
View file

@ -82,7 +82,7 @@ static const uint8_t line2irq[NUM_EXTI_LINES] = {
#endif /* CONFIG_SOC_SERIES_GD32F4XX */
};
__unused static void gd32_exti_isr(void *isr_data)
__unused static void gd32_exti_isr(const void *isr_data)
{
const struct device *dev = DEVICE_DT_INST_GET(0);
struct gd32_exti_data *data = dev->data;

2
drivers/timer/mcux_lptmr_timer.c
View file

@ -82,7 +82,7 @@ uint32_t sys_clock_cycle_get_32(void)
return LPTMR_GetCurrentTimerCount(LPTMR_BASE) + cycles;
}
static void mcux_lptmr_timer_isr(void *arg)
static void mcux_lptmr_timer_isr(const void *arg)
{
ARG_UNUSED(arg);

2
drivers/timer/mcux_os_timer.c
View file

@ -25,7 +25,7 @@ static struct k_spinlock lock;
static uint64_t last_count;
static OSTIMER_Type *base;
void mcux_lpc_ostick_isr(void *arg)
void mcux_lpc_ostick_isr(const void *arg)
{
ARG_UNUSED(arg);

3
drivers/watchdog/wdt_mcux_imx_wdog.c
View file

@ -115,9 +115,8 @@ static int mcux_wdog_feed(const struct device *dev, int channel_id)
return 0;
}
static void mcux_wdog_isr(void *arg)
static void mcux_wdog_isr(const struct device *dev)
{
const struct device *dev = (const struct device *)arg;
const struct mcux_wdog_config *config = dev->config;
struct mcux_wdog_data *data = dev->data;
WDOG_Type *base = config->base;