nrf5/modules/timer: Optimizing timer object structure and updating the module to use new hal_timer_init structures and parameters.

This commit is contained in:
Glenn Ruben Bakke 2017-04-30 20:15:03 +02:00
parent 0437fafaac
commit a0638880b3
3 changed files with 16 additions and 31 deletions

View File

@ -29,7 +29,7 @@
#ifdef HAL_TIMER_MODULE_ENABLED
void hal_timer_init(NRF_TIMER_Type * p_instance, hal_timer_init_t const * p_timer_init) {
void hal_timer_init(hal_timer_conf_t const * p_timer_conf) {
}

View File

@ -58,18 +58,10 @@
* @brief Timer Configuration Structure definition
*/
typedef struct {
} hal_timer_init_t;
uint8_t id;
uint8_t irq_priority;
} hal_timer_conf_t;
/**
* @brief Timer handle Structure definition
*/
typedef struct __Timer_HandleTypeDef
{
NRF_TIMER_Type *instance; /* Timer registers base address */
hal_timer_init_t init; /* Timer initialization parameters */
uint8_t id; /* Timer instance id */
} Timer_HandleTypeDef;
void hal_timer_init(NRF_TIMER_Type * p_instance, hal_timer_init_t const * p_timer_init);
void hal_timer_init(hal_timer_conf_t const * p_timer_config);
#endif // HAL_TIMER_H__

View File

@ -37,34 +37,27 @@
typedef struct _machine_timer_obj_t {
mp_obj_base_t base;
Timer_HandleTypeDef *timer;
hal_timer_conf_t * p_config;
} machine_timer_obj_t;
Timer_HandleTypeDef TimerHandle0 = {.instance = NULL, .id = 0};
Timer_HandleTypeDef TimerHandle1 = {.instance = NULL, .id = 1};
Timer_HandleTypeDef TimerHandle2 = {.instance = NULL, .id = 2};
static hal_timer_conf_t timer_config0 = {.id = 0};
static hal_timer_conf_t timer_config1 = {.id = 1};
static hal_timer_conf_t timer_config2 = {.id = 2};
STATIC const machine_timer_obj_t machine_timer_obj[] = {
{{&machine_timer_type}, &TimerHandle0},
{{&machine_timer_type}, &TimerHandle1},
{{&machine_timer_type}, &TimerHandle2},
{{&machine_timer_type}, &timer_config0},
{{&machine_timer_type}, &timer_config1},
{{&machine_timer_type}, &timer_config2},
};
void timer_init0(void) {
// reset the Timer handles
memset(&TimerHandle0, 0, sizeof(Timer_HandleTypeDef));
TimerHandle0.instance = TIMER_BASE(0);
memset(&TimerHandle1, 0, sizeof(Timer_HandleTypeDef));
TimerHandle0.instance = TIMER_BASE(1);
memset(&TimerHandle2, 0, sizeof(Timer_HandleTypeDef));
TimerHandle0.instance = TIMER_BASE(2);
}
STATIC int timer_find(mp_obj_t id) {
// given an integer id
int timer_id = mp_obj_get_int(id);
if (timer_id >= 0 && timer_id <= MP_ARRAY_SIZE(machine_timer_obj)
&& machine_timer_obj[timer_id].timer != NULL) {
&& machine_timer_obj[timer_id].p_config != NULL) {
return timer_id;
}
nlr_raise(mp_obj_new_exception_msg_varg(&mp_type_ValueError,
@ -73,7 +66,7 @@ STATIC int timer_find(mp_obj_t id) {
STATIC void timer_print(const mp_print_t *print, mp_obj_t o, mp_print_kind_t kind) {
machine_timer_obj_t *self = o;
mp_printf(print, "Timer(%u)", self->timer->id);
mp_printf(print, "Timer(%u)", self->p_config->id);
}
/******************************************************************************/
@ -103,7 +96,7 @@ STATIC mp_obj_t machine_timer_make_new(const mp_obj_type_t *type, size_t n_args,
int timer_id = timer_find(args[ARG_NEW_id].u_obj);
const machine_timer_obj_t *self = &machine_timer_obj[timer_id];
hal_timer_init(self->timer->instance, &self->timer->init);
hal_timer_init(self->p_config);
return MP_OBJ_FROM_PTR(self);
}