patrick/zephyr
1
0
Fork 0
Code Issues Pull requests Actions 8 Packages Projects Releases Wiki Activity

driver: pwm: Error handling issues in pwm_stm32_get_cycles_per_sec

Browse source 
This patch tests the return code when calling clock_control_get_rate
and completes the issue  #20503 seen on watchdog
[Coverity CID :205655]

Fixes #20503

Signed-off-by: Francois Ramu <francois.ramu@st.com>
This commit is contained in:
Francois Ramu 2019-11-13 09:02:51 +01:00 committed by Maureen Helm
parent a0bd434fa5
commit 458a8a0a41
1 changed files with 8 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
drivers/pwm/pwm_stm32.c
View file

@ -16,6 +16,9 @@
#include "pwm_stm32.h"
#include <logging/log.h>
LOG_MODULE_REGISTER(pwm_stm32);
/* convenience defines */
#define DEV_CFG(dev) \
((const struct pwm_stm32_config * const)(dev)->config->config_info)
@ -160,8 +163,11 @@ static int pwm_stm32_get_cycles_per_sec(struct device *dev, u32_t pwm,
}
/* Timer clock depends on APB prescaler */
clock_control_get_rate(data->clock,
(clock_control_subsys_t *)&cfg->pclken, &bus_clk);
if (clock_control_get_rate(data->clock,
(clock_control_subsys_t *)&cfg->pclken, &bus_clk) < 0) {
LOG_ERR("Failed call clock_control_get_rate");
return -EIO;
}
tim_clk = __get_tim_clk(bus_clk,
(clock_control_subsys_t *)&cfg->pclken);