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

[PBYetml]

ISR_TIMER1 ou Callback1

• [x] => amélioration / question => constante numérique => test / condition => comment pourriez-vous remplacer cette cst numérique ? et que représente-t-elle ?
• [x] => bug / amélioration / question => test / condition => algorithme => combien de tic comptez-vous avec votre test ? https://github.com/CyrilFeliciano/MINF_TP1/blob/06dd25acbe9af0bdabc6bd883d72be4ed668b872/TP1_PWM%202/firmware/src/system_config/default/system_interrupt.c#L101
• [x] => bug / amélioration / question => algorithme => test/condition => appel de fonction => est-ce que l'état APP_STATE_INIT doit remis à jour toutes les 20ms ? https://github.com/CyrilFeliciano/MINF_TP1/blob/dc2b0a808b3d872861c7904583380733fc1f13a3/TP1_PWM%202/firmware/src/system_config/default/system_interrupt.c#L101C1-L105C6
• [x] => bug / amélioration / question => algorithme => respect du CDC => voir PDF - TP1_TimerPwm_v1_31 - p5 => est-ce que vous devez appeler des fonctions d'affichage ? https://github.com/CyrilFeliciano/MINF_TP1/blob/dc2b0a808b3d872861c7904583380733fc1f13a3/TP1_PWM%202/firmware/src/system_config/default/system_interrupt.c#L111C1-L116C10
• [x] => bug / amélioration / question => algorithme => appel de fonction => à quoi vous sert la fct callback_timer1 https://github.com/CyrilFeliciano/MINF_TP1/blob/dc2b0a808b3d872861c7904583380733fc1f13a3/TP1_PWM%202/firmware/src/system_config/default/system_interrupt.c#L127C9-L127C24

ISR_TIMER4 ou Callback4

AUTRES

• [x] => amélioration / question => déclaration de fonction => différence entre callBaclk(){}

_ISR(Timer1_...) {}

https://github.com/CyrilFeliciano/MINF_TP1/blob/06dd25acbe9af0bdabc6bd883d72be4ed668b872/TP1_PWM%202/firmware/src/system_config/default/system_interrupt.c#L95 https://github.com/CyrilFeliciano/MINF_TP1/blob/06dd25acbe9af0bdabc6bd883d72be4ed668b872/TP1_PWM%202/firmware/src/app.c#L113

• [ ]

[CyrilFeliciano]

1. On peut la remplacer par un define. Elle représente le temps d'initialisation dans notre programme.
2. Il faut mettre "<=" à TEMPSINIT pour faire 150 pas.
3. Non l'état d'initialisation ne doit pas être mit à jour toute les 20 ms.
4. Dans le cahier des charges, il n'est pas spécifiée que nous ne pouvons pas mettre des fonctions d'affichages dans l'interruption. Cependant, il est mis que la fonction "GPWM_DispSettings()" gère l'affichage. Nous avons donc mis ces lignes dans cette fonction.
5. Met à jour l'état de notre machine d'état (APP_STATE_SERVICE_TASKS).
6. L'ISR se produit tous les X temps précis en fonction d'un timer. Le callbakc se trouve dans l'isr et ne se produit pas tout les x temps précisement.

Les modifications ont été apportées dans le fichier concerné.