Mc32gest_RS232.c -> remarques / améliorations / bugs

[PBYetml]

FCT : GetMessage

• [x] => amélioration / question => define => déclaration => convention langage => où devriez-vous déclarer ceci ? https://github.com/CyrilFeliciano/TP2_PWM-RS232/blob/756bb8862e794ac2d4ada40023fd43727109f20e/firmware/src/Mc32gest_RS232.c#L34
• [x] => bugs / amélioration / spécification / question => algorithme => test/condition => lié à la variable NbrCycle => pour ne pas faire la condition, NbrCycle doit valoir 8, 9 ou 10 ? est-ce que vous respectez le nombre de cycle de la consigne ? https://github.com/CyrilFeliciano/TP2_PWM-RS232/blob/756bb8862e794ac2d4ada40023fd43727109f20e/firmware/src/Mc32gest_RS232.c#L164C1-L164C39 https://github.com/CyrilFeliciano/TP2_PWM-RS232/blob/756bb8862e794ac2d4ada40023fd43727109f20e/firmware/src/Mc32gest_RS232.c#L173

FCT : SendMessage

• [x] => question / amélioration => appel de fonction => GetWriteSpace() => valeur de retour - freeSize => estimation de la valeur max que vous pourriez recevoir https://github.com/CyrilFeliciano/TP2_PWM-RS232/blob/826fa09ddc1fa81be11922409560baff8ba975a0/firmware/src/Mc32gest_RS232.c#L225
• [x] => amélioration / question => algorithme => MAJ structure => champs : MsbCrc & LsbCrc =>autre possibilité => pourquoi ne pas utiliser l'union à disposition ? https://github.com/CyrilFeliciano/TP2_PWM-RS232/blob/826fa09ddc1fa81be11922409560baff8ba975a0/firmware/src/Mc32gest_RS232.c#L235C1-L237C42

FCT : ISR => UART

• [x] => bug / amélioration / question => type => BOOL => d'ou vient ce type => à quelle librairie faites-vous référence ? https://github.com/CyrilFeliciano/TP2_PWM-RS232/blob/826fa09ddc1fa81be11922409560baff8ba975a0/firmware/src/Mc32gest_RS232.c#L275
• [x] => bug / amélioration / question => algorithme => comment déterminez-vous si la FIFO software est pleine ou non ? https://github.com/CyrilFeliciano/TP2_PWM-RS232/blob/826fa09ddc1fa81be11922409560baff8ba975a0/firmware/src/Mc32gest_RS232.c#L312C1-L316C15
• [x] => amélioration / question => test / condition => algorithme => constante numérique - hardcodage => pourquoi la valeur de 6 ? but du test ? https://github.com/CyrilFeliciano/TP2_PWM-RS232/blob/826fa09ddc1fa81be11922409560baff8ba975a0/firmware/src/Mc32gest_RS232.c#L336C1-L336C28
• [x] => question / amélioration => variable intermédiaire => à quoi vous sert la variable i_cts ? ne pouvez-vous pas directement utiliser RS232_CTS ? d’où vient cette constante ? et que représente-t-elle ? https://github.com/CyrilFeliciano/TP2_PWM-RS232/blob/826fa09ddc1fa81be11922409560baff8ba975a0/firmware/src/Mc32gest_RS232.c#L349 https://github.com/CyrilFeliciano/TP2_PWM-RS232/blob/826fa09ddc1fa81be11922409560baff8ba975a0/firmware/src/Mc32gest_RS232.c#L365
• [ ]

[PBYetml]

L'évaluation complète sur le fichier : Mc32gest_RS232.c est fini => vous pouvez si vous les désirer faire réaliser les amélioration ou correction sur ce fichier

[CyrilFeliciano]

1. Les defines doivent être déclarer dans le fichier Mc32gest_RS232.h.
2. Pour ne pas respecter la condition la variable doit valoir 9. Cela correspond au cahier des charges
3. La valeur maximale que j'ai vais pouvoir recevoir est de 8. Cela est lié a la place maximale de la FIFO qui est de 8.
4. Très bonne question de votre part! J'aurais du clairement utiliser les champs de l'union TX.
5. Le type bool est associé à la librairie .
6. Comme discuté dans votre bureau, la FIFO software est chargé progressivement d'octet en octet. Dans mon code je ne vérifie pas si la FIFO software est pleine ou non mais vu que je retire octet par un octet et que a la fin de l'interruption je regarde si la place nécessaire dans la FIFO software est plus grande que 6, je ne vais jamais laisser la FIFO se remplir . Car dans le cas ou la place dans la FIFO est inférieur à 6, je vais stopper l'émission de message.
7. Sachant qu'un message complet prend 5 cases de la FIFO, j'ai mis la valeur de 6 pour garantir un message complet. Le but de ce test est de stopper la réception de données dans le cas ou la place dans la FIFO est inférieur à 6.
8. A connaitre l'état de la pin CTS. Il est possible d'utiliser directement "RS232_CTS", car cela correspond à la pin du microcontrôleur qui est en "PORTD". De cette sorte, je vais continuellement lire l'état de cette pin. Ce define vient directement du bsp.h Je vais changer les i_cts par RS232_CTS.