diff options
author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2012-12-14 12:07:28 +0100 |
---|---|---|
committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2012-12-18 11:22:18 +0100 |
commit | 573822b1e94a996aff60875a68c203b9f5690c9f (patch) | |
tree | 717d4b726260fd3ecaf1412f9284ba0bbcfbed5b /c | |
parent | 5970dcc9e2d5e824b3d47e73043f51316f8fcd30 (diff) |
bsps/arm: Use assert() instead of status macros
Diffstat (limited to 'c')
-rw-r--r-- | c/src/lib/libbsp/arm/shared/lpc/network/lpc-ethernet.c | 42 |
1 files changed, 9 insertions, 33 deletions
diff --git a/c/src/lib/libbsp/arm/shared/lpc/network/lpc-ethernet.c b/c/src/lib/libbsp/arm/shared/lpc/network/lpc-ethernet.c index 8dc40328ac..33d41998d9 100644 --- a/c/src/lib/libbsp/arm/shared/lpc/network/lpc-ethernet.c +++ b/c/src/lib/libbsp/arm/shared/lpc/network/lpc-ethernet.c @@ -28,6 +28,7 @@ #include <stdlib.h> #include <stdio.h> #include <string.h> +#include <assert.h> #include <rtems.h> #include <rtems/rtems_bsdnet.h> @@ -50,8 +51,6 @@ #include <bsp/lpc-ethernet-config.h> #include <bsp/utility.h> -#include <rtems/status-checks.h> - #if MCLBYTES > (2 * 1024) #error "MCLBYTES to large" #endif @@ -541,7 +540,7 @@ static void lpc_eth_receive_task(void *arg) RTEMS_NO_TIMEOUT, &events ); - RTEMS_CLEANUP_SC(sc, cleanup, "wait for events"); + assert(sc == RTEMS_SUCCESSFUL); LPC_ETH_PRINTF("rx: wake up: 0x%08" PRIx32 "\n", events); @@ -683,17 +682,6 @@ static void lpc_eth_receive_task(void *arg) } } } - -cleanup: - - /* Clear task ID */ - e->receive_task = RTEMS_ID_NONE; - - /* Release network semaphore */ - rtems_bsdnet_semaphore_release(); - - /* Terminate self */ - (void) rtems_task_delete(RTEMS_SELF); } static struct mbuf *lpc_eth_next_fragment( @@ -790,7 +778,7 @@ static void lpc_eth_transmit_task(void *arg) RTEMS_NO_TIMEOUT, &events ); - RTEMS_CLEANUP_SC(sc, cleanup, "wait for events"); + assert(sc == RTEMS_SUCCESSFUL); LPC_ETH_PRINTF("tx: wake up: 0x%08" PRIx32 "\n", events); @@ -1038,17 +1026,6 @@ static void lpc_eth_transmit_task(void *arg) lpc_eth_enable_transmit_interrupts(); } } - -cleanup: - - /* Clear task ID */ - e->transmit_task = RTEMS_ID_NONE; - - /* Release network semaphore */ - rtems_bsdnet_semaphore_release(); - - /* Terminate self */ - (void) rtems_task_delete(RTEMS_SELF); } static void lpc_eth_mdio_wait_for_not_busy(void) @@ -1363,7 +1340,7 @@ static void lpc_eth_interface_start(struct ifnet *ifp) ifp->if_flags |= IFF_OACTIVE; sc = rtems_bsdnet_event_send(e->transmit_task, LPC_ETH_EVENT_START); - RTEMS_SYSLOG_ERROR_SC(sc, "send transmit start event"); + assert(sc == RTEMS_SUCCESSFUL); } static void lpc_eth_interface_watchdog(struct ifnet *ifp) @@ -1429,17 +1406,16 @@ static int lpc_eth_attach(struct rtems_bsdnet_ifconfig *config) /* Check parameter */ if (unit_index < 0) { - RTEMS_SYSLOG_ERROR("parse error for interface name\n"); return 0; } if (unit_index != 0) { - RTEMS_DO_CLEANUP(cleanup, "unexpected unit number"); + goto cleanup; } if (config->hardware_address == NULL) { - RTEMS_DO_CLEANUP(cleanup, "MAC address missing"); + goto cleanup; } - if (e->state != LPC_ETH_NOT_INITIALIZED) { - RTEMS_DO_CLEANUP(cleanup, "already attached"); + if (e->state != LPC_ETH_STATE_NOT_INITIALIZED) { + goto cleanup; } /* MDIO */ @@ -1498,7 +1474,7 @@ static int lpc_eth_attach(struct rtems_bsdnet_ifconfig *config) + LPC_ETH_CONFIG_TX_BUF_SIZE); table_area = lpc_eth_config_alloc_table_area(table_area_size); if (table_area == NULL) { - RTEMS_DO_CLEANUP(cleanup, "no memory for table area"); + goto cleanup; } memset(table_area, 0, table_area_size); |