8 years, 4 months ago.

The first thing you did wrong was skip the <<code>> tags. Please see the editing tips for how to format code.

#include "mbed.h"
#include "file1.h"

DigitalOut myled(LED1);
DigitalOut TM_STB(PA_0);
DigitalOut TM_CLK(PA_1);
DigitalOut TM_DIO(PA_4);
int main()
{
    while(1) {
        myled = 1;
        TM_STB = 1;
        TM_CLK= 1;
        TM_DIO = 1;
        wait(0.02);
        myled = 0;
        TM_STB = 0;
        TM_CLK= 0;
        TM_DIO = 0;
        wait(0.020);
    }
}

#include "mbed.h"
void Liga() { TM_STB = 1; }

void Liga() ;

On to the actual problem...

There are some clear bugs with your code, I would have expected them to generate a different sort of error (the one you are getting is normally a missing } or ; in your header file but I can't see one missing) but they need to be fixed so do that and then see if the error goes away. If not please post the exact code used in a way that's readable..

Firstly you need to include file1.h in file1.c, just because they have the same name doesn't mean the compiler knows they are related.

Secondly you need to protect file1.h from getting included twice in the same .c file. With such a simple situation that isn't an issue, this generally is caused by another .h file that is used also including it. However it's good practice to add this to all headers so that it doesn't come back and bite you later.

Finally you need to declare the variable you want to share between the c files in the header. You can't actually define the pin in the header, if you did that it would end up being defined twice, once in each .c file. What you do is use the extern directive to tell the compiler that some .c file somewhere will define a digitalOut called TM_STB and so assume that it exists. Since DigitalOut is in the mbed library we also need to include the mbed library in the header or we could end up with code that is dependent upon the order of the #includes in a .c file. That's not a situation you want to be in.

#include "mbed.h"
#include "file1.h" // <- need to include this header here too.
void Liga() { TM_STB = 1; }

// standard method to prevent header from getting included twice
#ifndef __file1_h__
#define __file1_h__

#include "mbed.h"

extern DigitalOut TM_STB; // tell anything including this that TM_STB exists but is defined elsewhere

void Liga() ;

#endif

UPDATE

Fixed typo on file1.h pre-compiler directives.

Also as noted below file1 should be .cpp not .c otherwise including the mbed library (which is c++) creates compiler errors.

#include "mbed.h"
#include "file1.h"

DigitalOut myled(LED1);
DigitalOut TMSTB(PA_0);
DigitalOut TMCLK(PA_1);
DigitalOut TMDIO(PA_4);


int main() {
    while(1) {
        myled = 1; // LED is ON
        TMSTB = 1;
        TMCLK= 1;
        TMDIO = 1;

        wait(0.02); // 200 ms
        myled = 0; // LED is OFF
        TMSTB = 0;
        TMCLK= 0;
        TMDIO = 0;
        wait(0.020); // 1 sec
    }
}

#include "mbed.h"
#include "file1.h"

void Liga()    
{
 TMSTB = 1;
}

#ifndef __file1_h__
#define __file1_h__

extern DigitalOut TMSTB; 
 
void Liga() ;
 #endif