Skip to content

Commit

Permalink
Merge pull request #300 from thnkslprpt/FIX-PSP-TYPOS
Browse files Browse the repository at this point in the history
Fix #308, Correct various typos in docs and comments

Part of nasa/cFS#352
  • Loading branch information
astrogeco committed Sep 11, 2021
2 parents 3909d4f + 95f6426 commit 1bd7bd9
Show file tree
Hide file tree
Showing 30 changed files with 45 additions and 45 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ or any other system where direct access is not possible. Renames the existing `

### Development Build: v1.5.0-rc1+dev101

- Removes unnecessary global config structure `Target_PspConfigData` and associated elements infavor of the new version API.
- Removes unnecessary global config structure `Target_PspConfigData` and associated elements in favor of the new version API.
- The mem pool stats TLM command now works on 64-bit Linux and sends out the expected telemetry packet.
Converts `CFE_PSP_MemoryTable` to internal object (instead of external) that should only be accessed via the PSP API. Replace `uint32`s with `size_t`. Use full range (SIZE_MAX) in the Linux/RTEMS implementation.
- See <https://github.com/nasa/PSP/pull/288> and <https://github.com/nasa/cFS/pull/233>
Expand Down Expand Up @@ -68,7 +68,7 @@ Converts `CFE_PSP_MemoryTable` to internal object (instead of external) that sho

### Development Build: 1.5.0-rc1+dev68

- Updates continuous integration workfow by adding static analysis with timeout and code format check. Adds status badges to ReadMe and removes old TravisCI artifacts.
- Updates continuous integration workflow by adding static analysis with timeout and code format check. Adds status badges to ReadMe and removes old TravisCI artifacts.
- Adds CodeQL analysis to continuous integration
- Apply standard formatting to psp codebase. Only changes whitespace.
- Adds missing "+dev" to development version output
Expand Down
2 changes: 1 addition & 1 deletion SECURITY.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ To report a vulnerability for the PSP subsystem please [submit an issue](https:/

For general cFS vulnerabilities please [open a cFS framework issue](https://github.com/nasa/cfs/issues/new/choose) and see our [top-level security policy](https://github.com/nasa/cFS/security/policy).

In either case please use the "Bug Report" template and provide as much information as possible. Apply appropraite labels for each report. For security related reports, tag the issue with the "security" label.
In either case please use the "Bug Report" template and provide as much information as possible. Apply appropriate labels for each report. For security related reports, tag the issue with the "security" label.

## Testing

Expand Down
4 changes: 2 additions & 2 deletions fsw/inc/cfe_psp.h
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,7 @@ extern void CFE_PSP_Main(void);
*
* Outputs an OS_time_t value indicating the time elapsed since an epoch. The
* epoch is not defined, but typically represents the system boot time. The
* value increases continously over time and cannot be reset by software.
* value increases continuously over time and cannot be reset by software.
*
* This is similar to the CFE_PSP_Get_Timebase(), but additionally it normalizes
* the output value to an OS_time_t, thereby providing consistent units to
Expand Down Expand Up @@ -221,7 +221,7 @@ extern uint32 CFE_PSP_GetProcessorId(void);

extern uint32 CFE_PSP_GetSpacecraftId(void);
/*
** CFE_PSP_GetSpacecraftId retuns the Spacecraft ID (if any )
** CFE_PSP_GetSpacecraftId returns the Spacecraft ID (if any )
*/

extern const char *CFE_PSP_GetProcessorName(void);
Expand Down
2 changes: 1 addition & 1 deletion fsw/mcp750-vxworks/inc/cfe_psp_config.h
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@
#define CFE_PSP_VX_TIMEBASE_PERIOD_DENOMINATOR 1

/*
** This define sets the number of memory ranges that are defined in the memory range defintion
** This define sets the number of memory ranges that are defined in the memory range definition
** table.
*/
#define CFE_PSP_MEM_TABLE_SIZE 10
Expand Down
4 changes: 2 additions & 2 deletions fsw/mcp750-vxworks/src/cfe_psp_exception.c
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ void CFE_PSP_ExceptionHook(TASK_ID task_id, int vector, void *vpEsf);
**
** Name: CFE_PSP_AttachExceptions
**
** Purpose: This function Initializes the task execptions and adds a hook
** Purpose: This function Initializes the task exceptions and adds a hook
** into the VxWorks exception handling. The below hook is called
** for every exception that VxWorks catches.
**
Expand Down Expand Up @@ -178,7 +178,7 @@ void CFE_PSP_SetDefaultExceptionEnvironment(void)

vxFpscrSet(vxFpscrGet() | _PPC_FPSCR_VE | /* enable exceptions for invalid operations */
_PPC_FPSCR_OE | /* enable overflow exceptions */
_PPC_FPSCR_NI | /* Non-IEEE mode for denormailized numbers */
_PPC_FPSCR_NI | /* Non-IEEE mode for denormalized numbers */
_PPC_FPSCR_ZE); /* enable divide by zero exceptions */

vxFpscrSet(vxFpscrGet() | _PPC_FPSCR_XE | /* fp inexact exc enable */
Expand Down
4 changes: 2 additions & 2 deletions fsw/mcp750-vxworks/src/cfe_psp_memory.c
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
** Purpose:
** cFE PSP Memory related functions. This is the implementation of the cFE
** memory areas that have to be preserved, and the API that is designed to allow
** acccess to them. It also contains memory related routines to return the
** access to them. It also contains memory related routines to return the
** address of the kernel code used in the cFE checksum.
**
** History:
Expand Down Expand Up @@ -457,7 +457,7 @@ int32 CFE_PSP_GetKernelTextSegmentInfo(cpuaddr *PtrToKernelSegment, uint32 *Size
/*
** Get the kernel start and end
** addresses from the BSP, because the
** symbol table does not contain the symbls we need for this
** symbol table does not contain the symbols we need for this
*/
StartAddress = (cpuaddr)GetWrsKernelTextStart();
EndAddress = (cpuaddr)GetWrsKernelTextEnd();
Expand Down
2 changes: 1 addition & 1 deletion fsw/mcp750-vxworks/src/cfe_psp_ssr.c
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
** Purpose:
** This file contains glue routines between the cFE and the OS Board Support Package ( BSP ).
** The functions here allow the cFE to interface functions that are board and OS specific
** and usually dont fit well in the OS abstraction layer.
** and usually don't fit well in the OS abstraction layer.
**
** History:
** 2005/06/05 Alan Cudmore | Initial version,
Expand Down
2 changes: 1 addition & 1 deletion fsw/mcp750-vxworks/src/cfe_psp_start.c
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ void OS_Application_Startup(void)

/*
** Set up the virtual FS mapping for the "/cf" directory
** On this platform it is will use the CF:0 physical device.
** On this platform it will use the CF:0 physical device.
*/
Status = OS_FileSysAddFixedMap(&fs_id, "CF:0", "/cf");
if (Status != OS_SUCCESS)
Expand Down
2 changes: 1 addition & 1 deletion fsw/mcp750-vxworks/src/cfe_psp_support.c
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
** Purpose:
** This file contains glue routines between the cFE and the OS Board Support Package ( BSP ).
** The functions here allow the cFE to interface functions that are board and OS specific
** and usually dont fit well in the OS abstraction layer.
** and usually don't fit well in the OS abstraction layer.
**
** History:
** 2005/06/05 Alan Cudmore | Initial version,
Expand Down
2 changes: 1 addition & 1 deletion fsw/mcp750-vxworks/src/cfe_psp_watchdog.c
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
** Purpose:
** This file contains glue routines between the cFE and the OS Board Support Package ( BSP ).
** The functions here allow the cFE to interface functions that are board and OS specific
** and usually dont fit well in the OS abstraction layer.
** and usually don't fit well in the OS abstraction layer.
**
** History:
** 2009/07/20 A. Cudmore | Initial version,
Expand Down
2 changes: 1 addition & 1 deletion fsw/modules/eeprom_direct/cfe_psp_eeprom_direct.c
Original file line number Diff line number Diff line change
Expand Up @@ -273,7 +273,7 @@ int32 CFE_PSP_EepromWrite8(cpuaddr MemoryAddress, uint8 ByteValue)
** Name: CFE_PSP_EepromWriteEnable
**
** Purpose:
** Eable the eeprom for write operation
** Enable the eeprom for write operation
**
** Assumptions and Notes:
**
Expand Down
2 changes: 1 addition & 1 deletion fsw/modules/eeprom_mmap_file/cfe_psp_eeprom_mmap_file.c
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@ int32 CFE_PSP_SetupEEPROM(uint32 EEPROMSize, cpuaddr *EEPROMAddress)

/* For read/write - As this is mmap'ed we dereference the pointer directly.
* Hopefully the caller didn't get it wrong.
* No need to anything special for 8/16/32 width access in this mode.
* No need to do anything special for 8/16/32 width access in this mode.
*/
int32 CFE_PSP_EepromWrite32(cpuaddr MemoryAddress, uint32 uint32Value)
{
Expand Down
2 changes: 1 addition & 1 deletion fsw/pc-linux/inc/cfe_psp_config.h
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@
#include <pthread.h>

/*
** This define sets the number of memory ranges that are defined in the memory range defintion
** This define sets the number of memory ranges that are defined in the memory range definition
** table.
*/
#define CFE_PSP_MEM_TABLE_SIZE 10
Expand Down
2 changes: 1 addition & 1 deletion fsw/pc-linux/src/cfe_psp_memory.c
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
** Purpose:
** cFE PSP Memory related functions. This is the implementation of the cFE
** memory areas that have to be preserved, and the API that is designed to allow
** acccess to them. It also contains memory related routines to return the
** access to them. It also contains memory related routines to return the
** address of the kernel code used in the cFE checksum.
**
** History:
Expand Down
2 changes: 1 addition & 1 deletion fsw/pc-linux/src/cfe_psp_ssr.c
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
** Purpose:
** This file contains glue routines between the cFE and the OS Board Support Package ( BSP ).
** The functions here allow the cFE to interface functions that are board and OS specific
** and usually dont fit well in the OS abstraction layer.
** and usually don't fit well in the OS abstraction layer.
**
** History:
** 2005/06/05 Alan Cudmore | Initial version,
Expand Down
2 changes: 1 addition & 1 deletion fsw/pc-linux/src/cfe_psp_support.c
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
** Purpose:
** This file contains glue routines between the cFE and the OS Board Support Package ( BSP ).
** The functions here allow the cFE to interface functions that are board and OS specific
** and usually dont fit well in the OS abstraction layer.
** and usually don't fit well in the OS abstraction layer.
**
******************************************************************************/

Expand Down
2 changes: 1 addition & 1 deletion fsw/pc-linux/src/cfe_psp_watchdog.c
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
** Purpose:
** This file contains glue routines between the cFE and the OS Board Support Package ( BSP ).
** The functions here allow the cFE to interface functions that are board and OS specific
** and usually dont fit well in the OS abstraction layer.
** and usually don't fit well in the OS abstraction layer.
**
** History:
** 2009/07/20 A. Cudmore | Initial version,
Expand Down
4 changes: 2 additions & 2 deletions fsw/pc-rtems/README.txt
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,9 @@ I. Setting up and compiling RTEMS BSP

1) install RTEMS toolchain for i386-rtems4.11 (or relevant target arch) into /opt/rtems-4.11

The RTEMS official docs have instructions for this, which should supercede anything here.
The RTEMS official docs have instructions for this, which should supersede anything here.

Basic use of the the "rtems source builder" tool which works at the time of this writing:
Basic use of the "rtems source builder" tool which works at the time of this writing:
OFFICIAL RTEMS SOURCE BUILDER CLONE URL: git://git.rtems.org/rtems-source-builder.git


Expand Down
6 changes: 3 additions & 3 deletions fsw/pc-rtems/README_RTEMS_5.txt
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,9 @@ I. Setting up and compiling RTEMS BSP

1) install RTEMS toolchain for i386-rtems5 (or relevant target arch) into ${HOME}/rtems5

The RTEMS official docs have instructions for this, which should supercede anything here.
The RTEMS official docs have instructions for this, which should supersede anything here.

Basic use of the the "rtems source builder" tool which works at the time of this writing:
Basic use of the "rtems source builder" tool which works at the time of this writing:
OFFICIAL RTEMS SOURCE BUILDER CLONE URL: git://git.rtems.org/rtems-source-builder.git


Expand All @@ -37,7 +37,7 @@ cd rtems-source-builder/rtems
Note - at the time of this writing 5.1 is the current "stable" branch and
5.1 represents the latest point release tag on that branch. We are going to
use the "5" branch, which will eventually be released as 5.2. The latest
5 branch has a bux fix for the RFS file system needed by the cFS for this target.
5 branch has a bug fix for the RFS file system needed by the cFS for this target.
If you use the 5.1 release tag (or tarball) you will most likely not be able
to run the cFS without an error.

Expand Down
2 changes: 1 addition & 1 deletion fsw/pc-rtems/inc/cfe_psp_config.h
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
#include <rtems.h>

/*
** This define sets the number of memory ranges that are defined in the memory range defintion
** This define sets the number of memory ranges that are defined in the memory range definition
** table.
*/
#define CFE_PSP_MEM_TABLE_SIZE 10
Expand Down
10 changes: 5 additions & 5 deletions fsw/pc-rtems/src/cfe_psp_memory.c
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
** Purpose:
** cFE PSP Memory related functions. This is the implementation of the cFE
** memory areas that have to be preserved, and the API that is designed to allow
** acccess to them. It also contains memory related routines to return the
** access to them. It also contains memory related routines to return the
** address of the kernel code used in the cFE checksum.
**
** History:
Expand Down Expand Up @@ -373,7 +373,7 @@ void CFE_PSP_SetupReservedMemoryMap(void)
RequiredSize += CDSSize;
RequiredSize += UserReservedSize;

OS_printf("Sizeof BSP reserved memory = %u bytes\n", (unsigned int)RequiredSize);
OS_printf("Size of BSP reserved memory = %u bytes\n", (unsigned int)RequiredSize);

PcRtems_ReservedMemBlock.BlockPtr = malloc(RequiredSize);

Expand Down Expand Up @@ -421,7 +421,7 @@ void CFE_PSP_SetupReservedMemoryMap(void)

/*
* Set up the "RAM" entry in the memory table.
* On RTEMS this is just encompasses the entire memory space, but an entry needs
* On RTEMS this just encompasses the entire memory space, but an entry needs
* to exist so that CFE_PSP_ValidateMemRange() works as intended.
*/
CFE_PSP_MemRangeSet(0, CFE_PSP_MEM_RAM, 0, SIZE_MAX, CFE_PSP_MEM_SIZE_DWORD, CFE_PSP_MEM_ATTR_READWRITE);
Expand Down Expand Up @@ -480,7 +480,7 @@ int32 CFE_PSP_GetKernelTextSegmentInfo(cpuaddr *PtrToKernelSegment, uint32 *Size
/*
** Get the kernel start and end
** addresses from the BSP, because the
** symbol table does not contain the symbls we need for this
** symbol table does not contain the symbols we need for this
*/
StartAddress = (cpuaddr)0x0;
EndAddress = (cpuaddr)0x1024;
Expand Down Expand Up @@ -522,7 +522,7 @@ int32 CFE_PSP_GetCFETextSegmentInfo(cpuaddr *PtrToCFESegment, uint32 *SizeOfCFES
/*
** Get the kernel start and end
** addresses from the BSP, because the
** symbol table does not contain the symbls we need for this
** symbol table does not contain the symbols we need for this
*/
StartAddress = (cpuaddr)0x0;
EndAddress = (cpuaddr)0x1024;
Expand Down
2 changes: 1 addition & 1 deletion fsw/pc-rtems/src/cfe_psp_ssr.c
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
** Purpose:
** This file contains glue routines between the cFE and the OS Board Support Package ( BSP ).
** The functions here allow the cFE to interface functions that are board and OS specific
** and usually dont fit well in the OS abstraction layer.
** and usually don't fit well in the OS abstraction layer.
**
** History:
** 2005/06/05 Alan Cudmore | Initial version,
Expand Down
2 changes: 1 addition & 1 deletion fsw/pc-rtems/src/cfe_psp_support.c
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
** Purpose:
** This file contains glue routines between the cFE and the OS Board Support Package ( BSP ).
** The functions here allow the cFE to interface functions that are board and OS specific
** and usually dont fit well in the OS abstraction layer.
** and usually don't fit well in the OS abstraction layer.
**
** History:
** 2005/06/05 Alan Cudmore | Initial version,
Expand Down
2 changes: 1 addition & 1 deletion fsw/pc-rtems/src/cfe_psp_watchdog.c
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
** Purpose:
** This file contains glue routines between the cFE and the OS Board Support Package ( BSP ).
** The functions here allow the cFE to interface functions that are board and OS specific
** and usually dont fit well in the OS abstraction layer.
** and usually don't fit well in the OS abstraction layer.
**
** History:
** 2009/07/20 A. Cudmore | Initial version,
Expand Down
4 changes: 2 additions & 2 deletions fsw/shared/inc/cfe_psp_module.h
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ typedef enum
} CFE_PSP_ModuleType_t;

/**
* Protoype for a PSP module initialization function
* Prototype for a PSP module initialization function
*/
typedef void (*CFE_PSP_ModuleInitFunc_t)(uint32 PspModuleId);

Expand Down Expand Up @@ -77,7 +77,7 @@ typedef const struct
*
* This function should only be called during PSP initialization before the
* system is operational. It is not intended to be called from application code after
* CFE has started. The function is not necessarily be thread-safe and should be called
* CFE has started. The function is not necessarily thread-safe and should be called
* before any child threads are created.
*
* Note that this does _not_ return any status --
Expand Down
8 changes: 4 additions & 4 deletions fsw/shared/src/cfe_psp_memrange.c
Original file line number Diff line number Diff line change
Expand Up @@ -205,7 +205,7 @@ uint32 CFE_PSP_MemRanges(void)
** Global Outputs: Changes CFE_PSP_MemoryTable
**
** Return Values:
** CFE_PSP_SUCCESS -- Memory range set successfuly.
** CFE_PSP_SUCCESS -- Memory range set successfully.
** CFE_PSP_INVALID_MEM_RANGE -- The index into the table is invalid
** CFE_PSP_INVALID_MEM_ADDR -- Starting address is not valid
** CFE_PSP_INVALID_MEM_TYPE -- Memory type associated with the range does not match the passed in type.
Expand Down Expand Up @@ -262,7 +262,7 @@ int32 CFE_PSP_MemRangeSet(uint32 RangeNum, uint32 MemoryType, cpuaddr StartAddr,
** This function retrieves one of the records in the CFE_PSP_MemoryTable.
**
** Assumptions and Notes:
** Becasue the table is fixed size, the entries are accessed by using the integer index.
** Because the table is fixed size, the entries are accessed by using the integer index.
**
** Parameters:
** RangeNum -- A 32 bit integer ( starting with 0 ) specifying the MemoryTable entry.
Expand All @@ -272,7 +272,7 @@ int32 CFE_PSP_MemRangeSet(uint32 RangeNum, uint32 MemoryType, cpuaddr StartAddr,
** is stored.
** *Size -- A pointer to the 32 bit integer where the 32 bit size of the memory range
** is stored.
** *WordSize -- A pointer to the 32 bit integer where the the minimum addressable size of the range:
** *WordSize -- A pointer to the 32 bit integer where the minimum addressable size of the range:
** ( CFE_PSP_MEM_SIZE_BYTE, CFE_PSP_MEM_SIZE_WORD, CFE_PSP_MEM_SIZE_DWORD )
** Attributes -- The attributes of the Memory Range:
** (CFE_PSP_MEM_ATTR_WRITE, CFE_PSP_MEM_ATTR_READ, CFE_PSP_MEM_ATTR_READWRITE)
Expand All @@ -282,7 +282,7 @@ int32 CFE_PSP_MemRangeSet(uint32 RangeNum, uint32 MemoryType, cpuaddr StartAddr,
** Global Outputs: Changes CFE_PSP_MemoryTable
**
** Return Values:
** CFE_PSP_SUCCESS -- Memory range returned successfuly.
** CFE_PSP_SUCCESS -- Memory range returned successfully.
** CFE_PSP_INVALID_POINTER -- Parameter error
** CFE_PSP_INVALID_MEM_RANGE -- The index into the table is invalid
*/
Expand Down
2 changes: 1 addition & 1 deletion fsw/shared/src/cfe_psp_module.c
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ static uint32 CFE_PSP_ModuleCount = 0;
/***************************************************
* Function Name: CFE_PSP_ModuleInitList
*
* Helper function to initalize a list of modules (not externally called)
* Helper function to initialize a list of modules (not externally called)
*/
void CFE_PSP_ModuleInitList(CFE_StaticModuleLoadEntry_t *ListPtr)
{
Expand Down
2 changes: 1 addition & 1 deletion unit-test-coverage/mcp750-vxworks/adaptors/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
# "Adaptors" help enable the unit test code to reach functions/objects that
# are otherwise not exposed.

# NOTE: These source files are compile with OVERRIDES on the headers just like
# NOTE: These source files are compiled with OVERRIDES on the headers just like
# the FSW code is compiled. This is how it is able to include internal headers
# which otherwise would fail. But that also means that adaptor code cannot call
# any library functions, as this would also reach a stub, not the real function.
Expand Down
2 changes: 1 addition & 1 deletion unit-test-coverage/ut-stubs/inc/PCS_vxWorks.h
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ typedef long PCS_Vx_usr_arg_t;
* without arguments, e.g. "int (*FUNCPTR)()". This is acceptable
* by some compilers but generally incompatible with the
* "-Wstrict-prototype" gcc warning option. So in this override it
* is defined as a int argument. This means that application code
* is defined as an int argument. This means that application code
* may need to cast it at the time of use (which is generally done anyway).
*/
typedef int (*PCS_FUNCPTR)(int);
Expand Down
2 changes: 1 addition & 1 deletion unit-test-coverage/ut-stubs/inc/README.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
ABOUT THE SYSTEM HEADER OVERRIDES
=================================

The "overrides" directory contain replacement versions of many system-provided
The "overrides" directory contains replacement versions of many system-provided
header files. All replacement functions and types are identified using an
`OCS_` prefix.

Expand Down

0 comments on commit 1bd7bd9

Please sign in to comment.