Problème en lecture/écriture de badge RFID mifare classic avec Raspberry et CR038

[KroMignon]

@jym92 Comme l'a déjà souligné @SGaist, l'utilisation de la variables statiques est en générale une très mauvaise idée en C++. A moins de vouloir partager une variable entre plusieurs instances de cette classe, mais c'est un cas d'usage très particulier.

Encore une fois, il n'est pas utile de créer une classe dérivée de QThread!. De plus, je fait de faire une propre implémentation du slot run() fait en sorte de ne pas créer, ni démarrer une EventQueue pour ce thread, donc le mécanisme de signals/slots de Qt ne fonctionnera pas avec ce thread!

Après, à quoi sert RFIDworker::checkRFID()? A part bloquer le thread dans une boucle infinie ;-)
Bref là également une très mauvaise idée, je veut dire par là, pas 'Qt Conforme'.
Pour que le mécanisme signals/slots fonctionne, il faut une EventQueue et ne faut pas bloquer le thread dans une boucle infinie!

Je reprends mon exemple de WorkerClass

class RfIdWorker : public QObject
{
    Q_OBJECT

public:
    explicit RfIdWorker (QObject * parent);
...

public slots:
    void start();
    void stop();

private slots:
   void checkRFID();
    void ledTimer();

signals:
    void newTag(QString tagID, int tagValue);

private:
    QSerialPort *m_serial;
    QTimer *m_timer;
    bool m_ledState;
};

Pour le C++:

RfIdWorker:RfIdWorker (QObject *parent)
    : QObject(parent)
    , m_serial(Q_NULLPTR)
    , m_timer(Q_NULLPTR)
    , m_ledState(false)
{
}

void RfIdWorker::start()
{
    if(!m_ledTimer)
    {
        m_ledTimer = new QTimer(this);
        m_ledTimer->setSingleShot(false);
        m_ledTiler->setInterval(400);
        connect(m_ledTimer, &QTimer::timeout, this, &RfIdWorker::ledTimer);
    }

    if(!m_serial)
    {
        m_serial = new QSerialPort(this);
        // serial port initialistion
        ...
        connect(m_serial, &QSerialPort::readyRead, this, &RfIdWorker::checkRFID);
    }
    m_serial>open();
    m_ledTimer->start();
}

void RfIdWorker::stop()
{
    if(m_serial && m_serial->isOpen())
    {
        m_serial->close();
    }
    if(m_ledTimer && m_ledTimer->isActive())
        m_ledTimer->stop();
}

void RfIdWorker::ledTimer()
{
    if(m_ledState)
        led_on();
    else
        led_off();
    m_ledState = !m_ledState;
}

void RfIdWorker::checkRFID()
{
    while(!m_serial->atEnd())
    {
        QByteArray rfIdRaw = m_serial->readAll();
        // process data
        ...
        if(!tag.isEmpty())
            emit newTag(tag, value);
    }
}

[jym92]

Bonjour,

Tout d'abord je vous remercie pour vos conseils et je commence à petit à petit à saisir la logique un peu tordu de Qt ;p

J'ai du coup modifier mon worker pour passer le port serie en private à la classe et j'ai mis en place le fonctionnement via le timer au lieu de la boucle infini.

Cependant le timer ne démarre pas alors qu'au vu des lignes de debug tout semble bien se lancer sauf le timer ...

voici le code du worker.h :

#ifndef RFIDworker_H
#define RFIDworker_H

#include <QObject>
#include <QtSerialPort>



class RFIDworker : public QObject
{
    Q_OBJECT

public:
    explicit RFIDworker(QObject *parent= nullptr);


public slots:
    void start();
    void stop();

private slots:
   void checkRFID();
   void ledTimer();

signals:
    void newTag(QString tagID, int tagValue);

private:
   QSerialPort *RFIDserial;
   QTimer *m_timer;
   bool m_ledState;
};

//======================================================================
//************             RFID CR038 fonctions :               ****** //
//======================================================================

//*******************************************//
//Turn on MiFare module LED subroutine
void led_on(QSerialPort *RFIDport);

//*******************************************//
//Turn on MiFare module LED subroutine
void led_off(QSerialPort *RFIDport);

et du worker.cpp :

RFIDworker::RFIDworker(QObject *parent): QObject(parent)
    , RFIDserial(Q_NULLPTR)
    , m_timer(Q_NULLPTR)
    , m_ledState(false)
{
}

void RFIDworker::start()
{
    if(!m_timer)
        {
            m_timer = new QTimer(this);
            m_timer->setSingleShot(false);
            m_timer->setInterval(400);
            connect(m_timer, &QTimer::timeout, this, &RFIDworker::ledTimer);
        }

    if(!RFIDserial)
    {
        RFIDserial = new QSerialPort(this);
        // serial port initialistion
        // ----- INIT SERIAL RFID ----//
        RFIDserial->setPortName("ttyUSB0");
        RFIDserial->setBaudRate(QSerialPort::Baud19200);
        RFIDserial->setDataBits(QSerialPort::Data8);
        RFIDserial->setParity(QSerialPort::NoParity);
        RFIDserial->setStopBits(QSerialPort::OneStop);
        RFIDserial->setFlowControl(QSerialPort::NoFlowControl);
        RFIDserial->open(QIODevice::ReadWrite);

        connect(RFIDserial, &QSerialPort::readyRead, this, &RFIDworker::checkRFID);

        if (RFIDserial->isOpen() == true)
        {
            qDebug() << "Serial ttyUSB0(USB0) is opened";
        }
        else
        {
            qDebug() << "Connexion impossible RFID ! ";
        }
        // lancement timer
        m_timer->start();


    }
}

void RFIDworker::stop()
{
    if(serialRFID && serialRFID->isOpen())
    {
        serialRFID->close();
    }
    if(m_timer && m_timer->isActive())
            m_timer->stop();
}

void RFIDworker::ledTimer()
{
    if(m_ledState)
        led_on(RFIDserial);
    else
        led_off(RFIDserial);
    m_ledState = !m_ledState;
    qDebug() << "passage dans Ledtimer";
}

void RFIDworker::checkRFID()
{
    while(!RFIDserial->atEnd())
        {
            qDebug() << "passage dans checkRFID";
            //QByteArray rfIdRaw = RFIDserial->readAll();
            // process data
            /*
            if(!tag.isEmpty())
                emit newTag(tag, value);
            */
        }
}


//======================================================================
//************             RFID CR038 fonctions :               ****** //
//======================================================================


//******************************************
//Turn on MiFare module LED subroutine
void led_on(QSerialPort *RFIDport)
{
  // on envoit tout d'un coup !
  QByteArray da(LED_ON, sizeof(LED_ON));
  RFIDport->write(da);
  //delay(200)
  QByteArray ba;
  while(!RFIDport->isOpen());
  ba = RFIDport->readAll();
  qDebug() << "reception LED_ON :" << ba;
}

//*******************************************
//Turn on MiFare module LED subroutine
void led_off(QSerialPort *RFIDport)
{

   QByteArray da(LED_OFF, sizeof(LED_OFF));
   RFIDport->write(da);
   QByteArray ba;
   //delay(200)
   while(!RFIDport->isOpen());
   ba = RFIDport->readAll();
   qDebug() << "reception LED_OFF :" << ba;
}

Coté init dans le mainwindow j'ai rien changé ....

en réponse console j'ai :

Serial ttyUSB0(USB0) is opened
 Thread RFID lancé !

donc le worker a bien été lancé ainsi que le thread RFID.

Une idée ? J'ai oublié un truc ?

J'ai essayé d'échanger Tx et Rx coté connectique mais rien ne se passe...
et je devrais quand même avoir des lignes de debug coté worker qui indique le déroulement du ledtimer...

J'essaye de coller au mieu à la "QT conforme" mais vu que mon projet est a cheval entre une IHM et de l'embarqué pur c'est pas simple...

Merci pour votre aide ;)

[KroMignon]

@jym92 said in Problème en lecture/écriture de badge RFID mifare classic avec Raspberry et CR038:

Coté init dans le mainwindow j'ai rien changé ....

Le problème est là à mon avis, est-ce ton thread est une instance de QThread?
Est-ce que tu as bien supprimé ta classe RFIDthread qui ne sert à rien?

//auto* rfThread = new RFIDthread();
auto* rfThread = new QThread();

[KroMignon]

@jym92 said in Problème en lecture/écriture de badge RFID mifare classic avec Raspberry et CR038:

'essaye de coller au mieu à la "QT conforme" mais vu que mon projet est a cheval entre une IHM et de l'embarqué pur c'est pas simple...

Le mode de fonctionnement de Qt peut sembler à première vue un peu étrange, mais c'est en fait très performant. Donc, pour de l'embarquer pas un soucis, bien au contraire.
Je te conseille vivement de prendre le temps de lire et comprendre les articles suivants:

• https://woboq.com/blog/how-qt-signals-slots-work.html
• https://woboq.com/blog/how-qt-signals-slots-work-part2-qt5.html
• https://woboq.com/blog/how-qt-signals-slots-work-part3-queuedconnection.html

Ceci va t'aider à mieux cerner la philosophie de Qt et le pourquoi comment du mécanisme signals/slots.

[jym92]

Effectivement tu avais raison, je n'avais pas supprimé mon fichier RFIDThread car je pensais qu'il été utile a lancement de la classe et l'appel au niveau du mainwindow était faux...

voila mon appel au niveau de mainwindow :

// ------------ Lancement du Thread RFID worker -----------------//
    auto* rfThread = new QThread();
    auto* rfWorker = new RFIDworker();

    // start RF ID Reader on thread start
    connect(rfThread, &QThread::started, rfWorker , &RFIDworker::start);
    // stop RF ID Reader on main application exit
    connect(qApp, &QApplication::aboutToQuit, rfWorker, &QObject::deleteLater);
    // delete thread on worker end
    connect(rfWorker, &QObject::destroyed, rfThread, [rfThread](QObject *)
    {
        if(rfThread->isRunning())
        {
             rfThread->quit();
             rfThread->wait();
        }
        rfThread->deleteLater();
    }
    , Qt::DirectConnection);  // needs "DirectConnection" to work
    // to be informed about new RF ID tag
    //connect(rfWorker,  &RFIDworker::newTag, this, &MyMainClass::newRfIdTag);
    // move worker object to worker  thread
    rfWorker->moveToThread(rfThread);
    // last but not least, start worker thread
    rfThread->start();

Du coup dans mon Thread tout fonctionne comme je le veux, à savoir j'envois des commandes via le Timer et je reçois automatiquement mes réponses dans CheckRFID ;)

Cependant un autre problème apparaît que je suspectais déjà,à savoir l'envoi et la réception de données sur mon port série USB0 brouille ma réception de donnés sur le port UART5 qui est en l'air mais normalement relié à un PIC.

Voila comment je déclare la communication série avec le PIC dans mon mainwindow :

//----------------------- Parametrage de la liaison serie -------------------------------------- //
    QList<QSerialPortInfo> listPsi; // liste des ports série existant
    listPsi = QSerialPortInfo::availablePorts(); // récupère les ports série disponibles

    for(int i=0 ; i<listPsi.size() ; i++)
     qDebug() << "Communiquer : " << listPsi.at(i).portName() <<" " << listPsi.at(i).description();

    serial = new QSerialPort(this);

    // se connecte au port série déja existant nommé "ttyAMA0"

    // remarque port serie (si uart0, uart4 et uart5 activé via config):
    // uart 0 = Ama0 pin8(tx) et 10(Rx)
    // USB0  // RFID PN532
    // uart 5 = Ama2 pin32 et 33 // PIC
    // ----- INIT SERIAL PIC ----//
    serial->setPortName("ttyAMA2");
    serial->setBaudRate(QSerialPort::Baud115200);
    serial->setDataBits(QSerialPort::Data8);
    serial->setParity(QSerialPort::NoParity);
    serial->setStopBits(QSerialPort::OneStop);
    serial->setFlowControl(QSerialPort::NoFlowControl);
    serial->open(QIODevice::ReadWrite);

    if (serial->isOpen() == true)
    {
        qDebug() << "Serial ttyAMA2(uart5) is opened";
        connect(serial, SIGNAL(readyRead()),this,SLOT(on_Received_Data()));
    }
    else
    {
        qDebug() << "Connexion impossible PIC ! ";
    }

et la reception des données je fais :

// boucle activé sur reception de données serie :
void MainWindow::on_Received_Data()
{
    // ------- choses à faire - RX side -------
    qDebug() << "reception données :";
    QByteArray ba;
    ba = serial->readAll();
    ui->label_uart_pic->setText("Données reçues : " + ba );
    qDebug() << ba;

}

Le fonctionnement est quasi le meme que pour le rfid j'ai un timer qui tourne qui vérifie l'état d'un bouton poussoir et qui suivant son état allume une led et envoie un une commande au Pic qui lui répond.

Une idée pour résoudre ce conflit au niveau des port séries ?
Y'a pas moyen de d'interdire au mainwindow l'accés au port USB0 ?

ce genre de problème est juste impensable coté C embarqué classique ;)
( on envoi/reçoit sur un port et un port en l'air reçoit des données...)

Un grand merci pour ton aide @KroMignon !

[KroMignon]

@jym92 said in Problème en lecture/écriture de badge RFID mifare classic avec Raspberry et CR038:

ce genre de problème est juste impensable coté C embarqué classique ;)
( on envoi/reçoit sur un port et un port en l'air reçoit des données...)

Ca n'a rien à voir avec le langage de programmation C ANSI ou C++, c'est une question d'organisation du code...
Tu auras le même type de problème quelque soit le langage utilisé (même C#).
Si tu autorises tout le monde à accéder à tous les ports série, et bien c'est la cacophonie assurée!

Pour ton projet, il te faut définir:

• une classe qui va s'occuper du dialogue avec ton lecture RFID
• une classe qui va s'occuper du dialogue avec ton PIC

Après tu fais le lien entre tes instances avec l'aide de signaux et slots.

En gros, dans ta classe RFID (RFIDworker) tu ajoutes un signal pour demander le contrôle de la LED (par exemple setLedState(bool state)):

class RFIDworker : public QObject
{
    Q_OBJECT

public:
    explicit RFIDworker(QObject *parent= nullptr);
...

signals:
    void setLedState(bool state);
...
};

comme ça tu peux demander d'allumer ou éteindre la LED avec emit setLedState(true);/emit setLedState(false);
Dans ta classe PIC (PicManager), tu vas créer un slot pour la commande de la LED RFID:

class PicManager: public QObject
{
    Q_OBJECT

public:
    explicit PicManager(QObject *parent= nullptr);
...

public:
    void setRfidLedState(bool state);
...
};

Chaque class va s'occuper de son port série et tout le monde sera content.

Bref, comme écrit plus haut, c'est une question d'organisation du code. Le C++ permet de simplement cloisonner tout le monde, encore faut-il le faire, ça c'est à la charge du développeur!

[jym92]

OK ok je vois du coup c'est juste un problème d'organisation du code, il faut à chaque fois définir une classe spécifique au port Série utilisé...

Et si jamais j'ai besoin j'ai besoin d'écrire ou de lancer des commandes sur l'un des ports séries depuis le mainwindows, j'appelle alors des fonctions de cette classe.

OK je mets en place une classe spécifique pour la gestion du port série de mon PIC sur le même modèle que celle utilisée par le RFID.

Je vous tiens au jus si tout ça fonctionne, merci ;)

[KroMignon]

@jym92 Exactement, et dans l'absolue tu pourrais utiliser plusieurs lecteurs RFID ou PIC, il te suffit de créer autant d'instances des classes correspondantes.
Et c'est là que le C++ (ou C#) prends largement l'avantage sur la programmation en C ANSI.

[KroMignon]

@jym92 Autre info au passage, rien ne t'oblige à faire du multi-threading avec ton application. Je pense que tu devrais largement t'en sortir uniquement avec le thread principal.
Dans ce cas, il te suffit de légèrement modifier ta séquence d'init:

    auto* rfWorker = new RFIDworker();

    // stop RF ID Reader on main application exit
    connect(qApp, &QApplication::aboutToQuit, rfWorker, &QObject::deleteLater);

#if USE_MULTITHREAD
    auto* rfThread = new QThread();
    // start RF ID Reader on thread start
    connect(rfThread, &QThread::started, rfWorker , &RFIDworker::start);
    // delete thread on worker end
    connect(rfWorker, &QObject::destroyed, rfThread, [rfThread](QObject *)
    {
        if(rfThread->isRunning())
        {
             rfThread->quit();
             rfThread->wait();
        }
        rfThread->deleteLater();
    }
    , Qt::DirectConnection);  // needs "DirectConnection" to work
    // to be informed about new RF ID tag
    //connect(rfWorker,  &RFIDworker::newTag, this, &MyMainClass::newRfIdTag);
    // move worker object to worker  thread
    rfWorker->moveToThread(rfThread);
    // last but not least, start worker thread
    rfThread->start();
#else
    rfWorker->start();
#endif

Et de même pour l'instance de ta classe de gestion du PIC.

[jym92]

Grande nouvelle ! Tout fonctionne comme je le souhaite !

j'ai même essayé de saturé le port série lié au PIC en lui envoyant des commandes depuis une Arduino vers la Raspberry en même temps que le RFID fonctionne et je n'ai vu aucun problème ;)

Je suis parti sur un Thread/Worker spécifique à chaque port série car pour le moment je n'ai qu'un PIC et un RFID mais je vais devoir ajouter encore 2 port séries pour un gprs et un monnayeur.
Je pense que dans mon cas ce cloisonnement par Thread pour chaque port série évite tout risque de débordement ou de paralysie de l'IHM.

Je vais maintenant continuer à travailler sur l’accès aux données de mon RFID(acces block).

Un grand merci pour votre aide ! ;)

[KroMignon]

@jym92 said in Problème en lecture/écriture de badge RFID mifare classic avec Raspberry et CR038:

Je pense que dans mon cas ce cloisonnement par Thread pour chaque port série évite tout risque de débordement ou de paralysie de l'IHM.

Très bonne nouvelle :)

Mais je te conseille quand même de faire un test en mono-threading, je suis persuadé que le multi-threading avec des ports séries à seulement 115200 bauds c'est donner de la confiture aux cochons.
Mais c'est à toi de voir.

Sinon, pour compléter un peu ta "culture Qt", voici un autre article pour mieux comprendre les QObject, QThread et le mécanisme signal/slots => Threads and QObjects

[SGaist]

@jym92 said in Problème en lecture/écriture de badge RFID mifare classic avec Raspberry et CR038:

Je pense que dans mon cas ce cloisonnement par Thread pour chaque port série évite tout risque de débordement ou de paralysie de l'IHM.

La première chose à faire quand on commence à penser à ajouter des threads dans une application est de bien valider la nécessiter d'utiliser des threads et de laisser passer une nuit avant de s'y attaquer.

La nature asynchrone de Qt permet souvent de s'en passer. Cela est spécialement vrai pour les ports séries qui sont généralement lents par rapport au reste du système.

En ce qui concerne le design de l'application, comme @KroMignon l'a déjà écrit, le mieux est d'encapsuler chaque élément communicant avec un port série de sorte qu'il soit bien indépendant du reste. Comme cela, si la nécessité de thread se fait réellement sentir, leur mise en application sera plus aisée.

[RAJALINGAM]

THANK YOU SIR ,I NEDD ONE MORE HELP CAN PLEASE SHARE ME RASPBERRY PI INTERFACE WITH RFID USING SPI COMMUNICATION IN QT C++.

[SGaist]

Hi,

@RAJALINGAM said in Problème en lecture/écriture de badge RFID mifare classic avec Raspberry et CR038:

THANK YOU SIR ,I NEDD ONE MORE HELP CAN PLEASE SHARE ME RASPBERRY PI INTERFACE WITH RFID USING SPI COMMUNICATION IN QT C++.

This is the French sub forum so please, write in French.
That said, do not post messages in full caps. It is considered shouting in written languages and rude.

[RAJALINGAM]

@SGaist
sorry sir, sorry for this message and i apologies