diff options
author | Joel Sherrill <joel.sherrill@OARcorp.com> | 2008-12-19 16:32:33 +0000 |
---|---|---|
committer | Joel Sherrill <joel.sherrill@OARcorp.com> | 2008-12-19 16:32:33 +0000 |
commit | 1de98d97138c073e8f274c978d894f9368f78115 (patch) | |
tree | 786436c2d325e53661f871b9eeade8dffa37aea3 /cpukit | |
parent | 2008-12-19 Sebastian Huber <sebastian.huber@embedded-brains.de> (diff) | |
download | rtems-1de98d97138c073e8f274c978d894f9368f78115.tar.bz2 |
2008-12-19 Joel Sherrill <joel.sherrill@oarcorp.com>
* rtems/src/dpmemcreate.c: Spacing.
* rtems/src/timerserver.c: Move error check to RTEMS_DEBUG. Cannot
happen under normal circumstances.
Diffstat (limited to 'cpukit')
-rw-r--r-- | cpukit/ChangeLog | 6 | ||||
-rw-r--r-- | cpukit/rtems/src/dpmemcreate.c | 2 | ||||
-rw-r--r-- | cpukit/rtems/src/timerserver.c | 9 |
3 files changed, 13 insertions, 4 deletions
diff --git a/cpukit/ChangeLog b/cpukit/ChangeLog index 1dcd875875..af6cb5a5d7 100644 --- a/cpukit/ChangeLog +++ b/cpukit/ChangeLog @@ -1,3 +1,9 @@ +2008-12-19 Joel Sherrill <joel.sherrill@oarcorp.com> + + * rtems/src/dpmemcreate.c: Spacing. + * rtems/src/timerserver.c: Move error check to RTEMS_DEBUG. Cannot + happen under normal circumstances. + 2008-12-19 Sebastian Huber <sebastian.huber@embedded-brains.de> * libblock/src/ide_part_table.c: Fixed NULL pointer access. diff --git a/cpukit/rtems/src/dpmemcreate.c b/cpukit/rtems/src/dpmemcreate.c index 37afc59e87..789f075a6b 100644 --- a/cpukit/rtems/src/dpmemcreate.c +++ b/cpukit/rtems/src/dpmemcreate.c @@ -53,7 +53,7 @@ rtems_status_code rtems_port_create( { register Dual_ported_memory_Control *the_port; - if ( !rtems_is_name_valid( name) ) + if ( !rtems_is_name_valid( name ) ) return RTEMS_INVALID_NAME; if ( !id ) diff --git a/cpukit/rtems/src/timerserver.c b/cpukit/rtems/src/timerserver.c index 5f4b318e35..7e48a3d57a 100644 --- a/cpukit/rtems/src/timerserver.c +++ b/cpukit/rtems/src/timerserver.c @@ -465,7 +465,8 @@ rtems_status_code rtems_timer_initiate_server( (rtems_task_entry) _Timer_Server_body, /* the timer server entry point */ 0 /* there is no argument */ ); - if (status) { + + #if defined(RTEMS_DEBUG) /* * One would expect a call to rtems_task_delete() here to clean up * but there is actually no way (in normal circumstances) that the @@ -473,8 +474,10 @@ rtems_status_code rtems_timer_initiate_server( * be good. If this service fails, something is weirdly wrong on the * target such as a stray write in an ISR or incorrect memory layout. */ - initialized = false; - } + if (status) { + initialized = false; + } + #endif return status; } |