Unable to send and receive images between 2 Raspberry connected to Zigbee via UART.

[Rika]

@mrjj
144 sizeeeee:
160 sizeeeee:
176 sizeeeee:
192 sizeeeee:
208 sizeeeee:
224 sizeeeee:
240 sizeeeee:
241 sizeeeee:

[jsulm]

@Rika As I already said: you have to accumulate all these data in ba and when you got everything then you store it in the file.
Again the code:

void serial::on_readdata()
{
    ba.apend(serialPort->readAll());
    qDebug()<<ba.size()<<"sizeeeee:";
    if (ba.size() == imageSize) {
        QFile newDoc("/home/pi/Desktop/imagexx.jpg");
        if(newDoc.open(QIODevice::WriteOnly)){
            newDoc.write(ba);
        }
        newDoc.close();
    }
}

[mrjj]

@Rika
But it stops at 241 every time?

Maybe its due to data.
Could you try in your sent function

void serial::on_pushButton_2_clicked()
{


        QByteArray ba;
        ba.fill('A', 1000);
        if(serialPort->isOpen()==true){
            serialPort->write(ba);
            qDebug()<<ba.size()<<"size_send:";
        }
}

and see if that changes how you the reading goes.
This just sends 1000 A.

Also i wonder if write will copy the data and then send / if write sent it all there as else we have the issue of the sent buffer running out of scope.

[Rika]

@jsulm I have made your way, the above is the result from qDebug (). It only received 241 bytes so file imagexxx.jpg was not created. (my imageSize= 31021).

[J.Hilk]

@Rika said in Unable to send and receive images between 2 Raspberry connected to Zigbee via UART.:

@jsulm I have made your way, the above is the result from qDebug (). It only received 241 bytes so file imagexxx.jpg was not created. (my imageSize= 31021).

at this point, you're not getting away with a simple I've tried that, it does not work.

@jsulm is absolutely correct. Show us your actual code

[Rika]

@mrjj I tried with 1000 A, received only 190

[mrjj]

@J-Hilk
Do you know if write blocks ?
QByteArray ba; // local
ba.fill('A', 1000);
if(serialPort->isOpen()==true){
serialPort->write(ba); // blocks here ? as ba is by reference

[Rika]

@J-Hilk

Send:

#include "ui_serial.h"
#include <QDebug>
#include <QBuffer>
#include <QPixmap>
#include <QFile>


serial::serial(QWidget *parent) :
    QMainWindow(parent),
    ui(new Ui::serial)
{
    ui->setupUi(this);

    serialPort = new QSerialPort(this);
    serialPort->setPortName("ttyAMA0");
    serialPort->setBaudRate(QSerialPort::Baud115200);
    serialPort->setDataBits(QSerialPort::Data8);
    serialPort->setStopBits(QSerialPort::OneStop);
    serialPort->setFlowControl(QSerialPort::NoFlowControl);
    serialPort->open(QIODevice::ReadWrite);

    if (serialPort->isOpen() == true){
        qDebug() <<"Port Opened....";
        connect(serialPort, SIGNAL(readyRead()), this, SLOT(on_readdata()));
    }
}
void serial::on_pushButton_2_clicked()
{
    QFile* imageFile = new QFile("/home/pi/Desktop/image.jpg");
        imageFile->open(QIODevice::ReadOnly);
        QByteArray ba = imageFile->readAll();
        imageFile->close();
        delete imageFile;
        if(serialPort->isOpen()==true){
            serialPort->write(ba);
            qDebug()<<ba.size()<<"size_send:";
        }
}

Rec:

#include "serial.h"
#include "ui_serial.h"
#include <QtSerialPort/QSerialPort>
#include <QDebug>
#include <QBuffer>
#include <QPixmap>
#include <QFile>
#include <QMessageBox>

serial::serial(QWidget *parent) :
   QMainWindow(parent),
   ui(new Ui::serial)
{
   ui->setupUi(this);

   serialPort = new QSerialPort(this);
   serialPort->setPortName("ttyAMA0");
   serialPort->setBaudRate(QSerialPort::Baud115200);
   serialPort->setDataBits(QSerialPort::Data8);
   serialPort->setStopBits(QSerialPort::OneStop);
   serialPort->setFlowControl(QSerialPort::NoFlowControl);
   serialPort->open(QIODevice::ReadWrite);

   if (serialPort->isOpen() == true){
       qDebug() <<"Port Opened....";
       connect(serialPort, SIGNAL(readyRead()), this, SLOT(on_readdata()));
   }

}

serial::~serial()
{
   delete ui;
   serialPort->close();
}


void serial::on_readdata()
{
   ba.append( serialPort->readAll());
   qDebug()<<ba.size()<<"sizeeeee:";
   if (ba.size() == 31021) {
       QFile newDoc("/home/pi/Desktop/imagexx.jpg");
           if(newDoc.open(QIODevice::WriteOnly)){
               newDoc.write(ba);
           }
           newDoc.close();
   }
//    ui->label->setText(ba);

//      QPixmap b;
//      if(b.loadFromData(ba,"JPG")){
//          ui->label->setPixmap(b);
//      }

}

[J.Hilk]

@mrjj IIRC, it's passed on to the OS during the write call and that copies the data. But I'm not entirely sure 😬

[mrjj]

@J-Hilk
Yeah i was wondering since its a board if that still happens as it would
make sense we get cut if not.

@Rika
Can you try this

void serial::on_pushButton_2_clicked()
{
        static QByteArray ba;
        ba.fill('A', 1000);
        if(serialPort->isOpen()==true){
            serialPort->write(ba);
            qDebug()<<ba.size()<<"size_send:";
        }
}

and tell me if we still only get up to 190 bytes with the sizeeeee message.

[Rika]

@mrjj I tried, still only get 190.
16 sizeeeee:
32 sizeeeee:
64 sizeeeee:
64 sizeeeee:
14 sizeeeee:

[mrjj]

@Rika
Good testing.
So even it says
qDebug()<<ba.size()<<"size_send:";
1000 size_send
you only get
5 lines of
16 sizeeeee:
32 sizeeeee:
64 sizeeeee:
64 sizeeeee:
14 sizeeeee:
?

[Rika]

@mrjj Sometimes 4 lines, sometimes 7 lines, but the total is 190 bytes. With 500-A, received only 230

[mrjj]

@Rika
Ok
if we do
ba.fill('A', 10);
do you then get all ?

[Rika]

@mrjj yes.

[mrjj]

@Rika
So it seems data get cut over some size.

What Os is running on the boards ?

Also maybe Zigbee has some sort of max transfer size ?

[Rika]

@mrjj I use raspbian. I tried using a PC connected to zigbee and sent photos to my Pi4 + Zigbee with Realterm, my Pi4 received the photos. I think the error is in the program sends the data.

[J.Hilk]

well, lets narrow it down,
write returns a quint64 with the actual number of bytes written to the serialport, what does that return ?

void serial::on_pushButton_2_clicked()
{
        static QByteArray ba;
        ba.fill('A', 1000);
        if(serialPort->isOpen()==true){
            quint64 send = serialPort->write(ba);
            qDebug()<<send << " of" << ba.size()<<"size_send:";
        }
}

[Rika]

@J-Hilk 1000 of 1000 size.

[J.Hilk]

@Rika well,
to make sure your 2nd application is not simply very late in its creation/startup process, try clicking the button a 2nd time, see if something changes :D

After that, time for a serial port listing device, to check what is actually transferred