| Subject | Re: [Linux-stm32] [PATCH v10 16/33] counter: stm32-lptimer-cnt: Add const qualifier for actions_list array | From | Fabrice Gasnier <> | Date | Mon, 22 Mar 2021 17:23:44 +0100 |
| |
On 3/19/21 12:00 PM, William Breathitt Gray wrote: > The struct counter_synapse actions_list member expects a const enum > counter_synapse_action array. This patch adds the const qualifier to the > stm32_lptim_cnt_synapse_actions to match actions_list. > > Cc: Fabrice Gasnier <fabrice.gasnier@st.com> > Signed-off-by: William Breathitt Gray <vilhelm.gray@gmail.com> > --- > drivers/counter/stm32-lptimer-cnt.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)
Hi William,
You can add my: Reviewed-by: Fabrice Gasnier <fabrice.gasnier@foss.st.com>
Thanks, Fabrice
> > diff --git a/drivers/counter/stm32-lptimer-cnt.c b/drivers/counter/stm32-lptimer-cnt.c > index 0f7d3f1ec1b6..c19d998df5ba 100644 > --- a/drivers/counter/stm32-lptimer-cnt.c > +++ b/drivers/counter/stm32-lptimer-cnt.c > @@ -146,7 +146,7 @@ enum stm32_lptim_synapse_action { > STM32_LPTIM_SYNAPSE_ACTION_NONE, > }; > > -static enum counter_synapse_action stm32_lptim_cnt_synapse_actions[] = { > +static const enum counter_synapse_action stm32_lptim_cnt_synapse_actions[] = { > /* Index must match with stm32_lptim_cnt_polarity[] (priv->polarity) */ > [STM32_LPTIM_SYNAPSE_ACTION_RISING_EDGE] = COUNTER_SYNAPSE_ACTION_RISING_EDGE, > [STM32_LPTIM_SYNAPSE_ACTION_FALLING_EDGE] = COUNTER_SYNAPSE_ACTION_FALLING_EDGE, >
|