[Patch] Removing legacy method from arm csb337

Andreas Heinig andreas.heinig at cs.tu-dortmund.de
Tue Jul 2 07:18:34 UTC 2013


On Tuesday 02 July 2013 14:52:52 Vipul Nayyar wrote:
> Hello Andreas,
> 
> Thanks Andreas for pointing out that silly mistake. I didn't focus on other
> variables being used. I've updated the patch, and wanted to ask, what
> should be done when status code returned is not successful ?
> 
> commit 9b21c1effe34cb716c00ea1c11777ef16ce9c7cc
> Author: Vipul Nayyar <nayyar_vipul at yahoo.com>
> Date:   Tue Jul 2 12:20:49 2013 +0530
> 
>     Updated Legacy code in arm csb337
> 
> diff --git a/c/src/lib/libbsp/arm/csb337/network/network.c
> b/c/src/lib/libbsp/arm/csb337/network/network.c index e31bcc0..53d60d2
> 100644
> --- a/c/src/lib/libbsp/arm/csb337/network/network.c
> +++ b/c/src/lib/libbsp/arm/csb337/network/network.c
> @@ -358,6 +358,7 @@ void at91rm9200_emac_init(void *arg)
>  {
>      at91rm9200_emac_softc_t     *sc = arg;
>      struct ifnet *ifp = &sc->arpcom.ac_if;
> +    rtems_status_code stc = RTEMS_SUCCESSFUL;

I think you should name the rtems_status_code either 'sc' or 'status'. These 
names are used by default in most of the RTEMS modules. 

In your case I suggest to rename the at91rm9200_emac_softc_t variable, because 
sc is confusing here anyway.

Cheers
Andreas



More information about the devel mailing list