next topic »

[on:]

I have a Due that is using a gps and I am trying to build a library to simplify its use. I actually am going to reuse the code on a couple of Dues so I want to be able to pass a specific serial port to use for communication so I can change it based on what pins I use. For now I have a simple class set up that accepts a HardwareSerial pointer in the constructor and a function to print a test message. The code compiles fine and on a Uno it actually works as expected. When I compile it for the Due I get the following warning.

sketch_jan14a.cpp.o: In function `SerialTest::begin(long)':
C:\Users\pepi0003\Downloads\arduino-1.5.1r2/sketch_jan14a.ino:29: warning: undefined reference to `HardwareSerial::begin(unsigned long)'
Binary sketch size: 9,720 bytes (of a 524,288 byte maximum)

The code still compiles and can be uploaded to the due but doesn't actually print anything on the serial port. Here is the code I am using. Any help would be appreciated.

Code:

class SerialTest{
  public:
    SerialTest(HardwareSerial *ser);
    void begin(long baud);
    void printHelloWorld();
     
  private:
    HardwareSerial *hwSerial;   
};


SerialTest test(&Serial);

void setup(){
 
  test.begin(9600);
}

void loop(){
  test.printHelloWorld();
  delay(1000);
}

SerialTest::SerialTest(HardwareSerial *ser){
  hwSerial = ser;
}

void SerialTest::begin(long baud){
  hwSerial->begin(baud);
}

void SerialTest::printHelloWorld(){
  hwSerial->println("Hello world!");
}

[Reply #1 on:]

Not sure if this helps, but here is how I have done it in my endeavours:

I originally tried to use HardwareSerial Class to pass, but seems it wasn't the right object.  In the compiler error output it complained about trying to convert UARTClass to HardwareSerial.  Once I dug around a few, I found the actual two classes I needed to be able to pass a HardwareSerial object.

[Updated]

Code:

UARTClass *ucTx;
ucTx = &Serial;

[/Updated]

Code:

USARTClass *ucTx;
ucTx = &Serial1; // Can be Serial 1-3 [s]Could just use Serial for HardwareSerial0[/s]

-Dave

Edit: USARTClass is for HardwareSerial1-3(Serial1-Serial3).
HardwareSerial0(Serial) is of class UARTClass (virtual COM port via ATmega16U2 to PC).

main.cpp includes Arduino.h
Arduino.h includes HardwareSerial.h and variant.h
variant.h includes both UARTClass.h and USARTClass.h

[Reply #2 on:]

Code:

class SerialDevice
{
    HardwareSerial&     _serial;

public:
    SerialDevice(HardwareSerial& serial)
        : _serial(serial)
    {}

    void begin(long baud)   { _serial.begin(baud);             }
    void printHelloWorld()  { _serial.println("Hello world!"); }
};


SerialDevice device(Serial);

void loop()
{
    device.printHelloWorld();
    delay(1000UL);
}

void setup()
{
    device.begin(9600);
}

[Reply #3 on:]

Where does your class learn what the HardwareSerial name refers to? You appear to missing some necessary include statements.

[Reply #4 on:]

I tried your code Dave and it is working perfectly now. I had been poking around in the old libraries and didn't realized that the ones for the due where stored in a separate folder. Is there any drawback to passing a reference to the class like in lloydeans code as apposed to using a pointer.

[Reply #5 on:]

I believe in the context above pointer/reference would equal to the same thing.

The goal I was trying to achieve passing by pointer or reference was to eliminate passing by value, ie. sending the whole USARTClass object as a function param.  Save more memory just passing reference or actual memory location to USARTClass versus a (basically) literal copy of it.

Dave

[Reply #6 on:]

If by reference as above the compiler will never allow you to compile without passing the Serial device.

Code:

SerialDevice device;

Will generate a compile time error/warning