ctinyosnescradio-transmission

Compile error when creating packet


I'm studying THIS tutorial for tinyos and I wanted to try it out. I try to create the packet but it gives me the following error. I don't know what's wrong. It is probably something simple but I can't figure out what it is.

#include "TestMsg.h"
    ...
        event void AMControl.startDone(error_t error) {
            if (error == SUCCESS) {
                call Leds.led0On();

                //create packet
                TestMsg_t* msg = call Packet.getPayload(&packet, sizeof(TestMsg_t));
                msg->NodeID = TOS_NODE_ID;
    //          
    //          //TODO in the meantime this can change
    //          button_state_t val = call Get.get();
    //          msg->Data = ( val == BUTTON_PRESSED ? 1 : 0 );
    //          
    //          //send packet
    //          if (call AMSend.send(AM_BROADCAST_ADDR, &packet, sizeof(TestMsg_t)) == SUCCESS) {
    //              radioBusy = TRUE;
    //          }
            } else {
                call AMControl.start();
            }
        }
    ...

Here is TestMsg.h

#ifndef TEST_MSG_H
#define TEST_MSG_H

typedef nx_struct _TestMsg {
    nx_uint16_t NodeID;
    nx_uint8_t Data;
} TestMsg_t;

enum {
    AM_RADIO = 6
};

#endif /* TEST_MSG_H */

Here is the part where it is declared in the video

The error I get it this:

In file included from /home/advanticsys/ws/TestRadio/src/TestRadioAppC.nc:5:
In component `TestRadioC':
/home/advanticsys/ws/TestRadio/src/TestRadioC.nc: In function `AMControl.startDone':
/home/advanticsys/ws/TestRadio/src/TestRadioC.nc:43: syntax error before `*'
/home/advanticsys/ws/TestRadio/src/TestRadioC.nc:44: `msg' undeclared (first use in this function)
/home/advanticsys/ws/TestRadio/src/TestRadioC.nc:44: (Each undeclared identifier is reported only once
/home/advanticsys/ws/TestRadio/src/TestRadioC.nc:44: for each function it appears in.)

Update

Something is wrong with structs and headers.

#include "Szar.h"
#include "BarType.h"

module SzarP {
    uses interface Boot;
    uses interface Leds;
}

implementation {

    event void Boot.booted() {
        // TODO Auto-generated method stub
        call Leds.led0On();

        Szar_t foo;
        Szar_t *szar = &foo;

        BarType_t barVar;
        barVar.data = 0;
        BarType_t *pBarVar = &barVar;
        pBarVar->data = 1;

    }
}

Here are the 2 header files.

#ifndef SZAR_H
#define SZAR_H

typedef nx_struct _Szar {
    nx_uint8_t szar1;
    nx_uint16_t szar2;
} Szar_t;

#endif /* SZAR_H */


#ifndef BAR_TYPE_H
#define BAR_TYPE_H

typedef struct _BarType {
    uint8_t id;
    uint32_t data;
} BarType_t;

#endif /* BAR_TYPE_H */

And the errors:

In file included from /home/advanticsys/ws/Szar/src/SzarAppC.nc:6:
In component `SzarP':
/home/advanticsys/ws/Szar/src/SzarP.nc: In function `Boot.booted':
/home/advanticsys/ws/Szar/src/SzarP.nc:15: syntax error before `foo'
/home/advanticsys/ws/Szar/src/SzarP.nc:19: `barVar' undeclared (first use in this function)
/home/advanticsys/ws/Szar/src/SzarP.nc:19: (Each undeclared identifier is reported only once
/home/advanticsys/ws/Szar/src/SzarP.nc:19: for each function it appears in.)
/home/advanticsys/ws/Szar/src/SzarP.nc:20: syntax error before `*'
/home/advanticsys/ws/Szar/src/SzarP.nc:21: `pBarVar' undeclared (first use in this function)

Solution

  • For some strange reason I have to declare EVERY variable outside the function, and then it works. Example:

    bool radioBusy = FALSE;
    message_t packet;
    TestMsg_t *messageToSend;
    button_state_t buttonState;
    
    event void AMControl.startDone(error_t error) {
        if (error == SUCCESS) {
            call Leds.led0On();
    
            messageToSend = call Packet.getPayload(&packet, sizeof(TestMsg_t));
            messageToSend->NodeID = TOS_NODE_ID;
    
            //TODO in the meantime this can change
            buttonState = call Get.get();
            messageToSend->Data = ( buttonState == BUTTON_PRESSED ? 1 : 0 );
    
            //send packet
            if (call AMSend.send(AM_BROADCAST_ADDR, &packet, sizeof(TestMsg_t)) == SUCCESS) {
                radioBusy = TRUE;
            }
        } else {
            call AMControl.start();
        }
    }
    

    It also works if I declare my variables at the beginning of the functions/events/commands without any code before them.