Skip to content

CFE_PSP_TIMER_LOW32_ROLLOVER defined but not used in vxworks timebase implementation  #348

@skliper

Description

@skliper

Is your feature request related to a problem? Please describe.
Defined here:

/**
* \brief The number that the least significant 32 bits of the 64 bit
* time stamp returned by CFE_PSP_Get_Timebase rolls over. If the lower 32
* bits rolls at 1 second, then the CFE_PSP_TIMER_LOW32_ROLLOVER will be 1000000.
* if the lower 32 bits rolls at its maximum value (2^32) then
* CFE_PSP_TIMER_LOW32_ROLLOVER will be 0.
*/
#define CFE_PSP_TIMER_LOW32_ROLLOVER 0

Expected use here:

/******************************************************************************
** Function: CFE_PSP_GetTimerLow32Rollover()
**
** Purpose:
** Provides the number that the least significant 32 bits of the 64 bit
** time stamp returned by CFE_PSP_Get_Timebase rolls over. If the lower 32
** bits rolls at 1 second, then the CFE_PSP_TIMER_LOW32_ROLLOVER will be 1000000.
** if the lower 32 bits rolls at its maximum value (2^32) then
** CFE_PSP_TIMER_LOW32_ROLLOVER will be 0.
**
** Arguments:
**
** Return:
** The number that the least significant 32 bits of the 64 bit time stamp
** returned by CFE_PSP_Get_Timebase rolls over.
*/
uint32 CFE_PSP_GetTimerLow32Rollover(void)
{
return 0;
}

Describe the solution you'd like
Use the define or delete it. Note it's defined as 0 and the function returns 0 so behavior wouldn't change.

Describe alternatives you've considered
None.

Additional context
None

Requester Info
Jacob Hageman - NASA/GSFC

Ping @superolmo

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions