address review feedback

This commit is contained in:
Thorsten Laux 2019-01-12 12:45:36 +01:00 committed by mikeller
parent 006749a66c
commit d90ef12248
3 changed files with 2 additions and 9 deletions

View File

@ -423,11 +423,6 @@ static const char * const lookupTableTpaMode[] = {
};
#endif
static const char* const lookupTableSchedulerPolicy[] = {
"PERIOD", "RATE"
};
#define LOOKUP_TABLE_ENTRY(name) { name, ARRAYLEN(name) }
const lookupTableEntry_t lookupTables[] = {
@ -536,7 +531,6 @@ const lookupTableEntry_t lookupTables[] = {
#ifdef USE_TPA_MODE
LOOKUP_TABLE_ENTRY(lookupTableTpaMode),
#endif
LOOKUP_TABLE_ENTRY(lookupTableSchedulerPolicy)
};
#undef LOOKUP_TABLE_ENTRY
@ -1181,7 +1175,7 @@ const clivalue_t valueTable[] = {
{ "cpu_overclock", VAR_UINT8 | MASTER_VALUE | MODE_LOOKUP, .config.lookup = { TABLE_OVERCLOCK }, PG_SYSTEM_CONFIG, offsetof(systemConfig_t, cpu_overclock) },
#endif
{ "pwr_on_arm_grace", VAR_UINT8 | MASTER_VALUE, .config.minmax = { 0, 30 }, PG_SYSTEM_CONFIG, offsetof(systemConfig_t, powerOnArmingGraceTime) },
{ "scheduler_policy", VAR_UINT8 | MASTER_VALUE | MODE_LOOKUP, .config.lookup = { TABLE_SCHEDULER_POLICY }, PG_SYSTEM_CONFIG, offsetof(systemConfig_t, schedulerPolicy) },
{ "scheduler_optimize_rate", VAR_UINT8 | MASTER_VALUE | MODE_LOOKUP, .config.lookup = { TABLE_OFF_ON }, PG_SYSTEM_CONFIG, offsetof(systemConfig_t, schedulerPolicy) },
// PG_VTX_CONFIG
#ifdef USE_VTX_COMMON

View File

@ -131,7 +131,6 @@ typedef enum {
#ifdef USE_TPA_MODE
TABLE_TPA_MODE,
#endif
TABLE_SCHEDULER_POLICY,
LOOKUP_TABLE_COUNT
} lookupTableIndex_e;

View File

@ -347,7 +347,7 @@ FAST_CODE void scheduler(void)
selectedTask->taskLatestDeltaTime = currentTimeUs - selectedTask->lastExecutedAt;
float period = currentTimeUs - selectedTask->lastExecutedAt;
selectedTask->lastExecutedAt = currentTimeUs;
selectedTask->lastDesiredAt += (cmpTimeUs(currentTimeUs,selectedTask->lastDesiredAt) / selectedTask->desiredPeriod) * selectedTask->desiredPeriod;
selectedTask->lastDesiredAt += (cmpTimeUs(currentTimeUs, selectedTask->lastDesiredAt) / selectedTask->desiredPeriod) * selectedTask->desiredPeriod;
selectedTask->dynamicPriority = 0;
// Execute task