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

[PBYetml]

ISR_TIMER1 ou Callback1

• [x] => amélioration / question => constante numérique => hardcodage => test / condition => comment pouvez-vous remplacer une constante numérique pour une meilleure maintenabilité de votre soft ? https://github.com/Melissaperret/TP1_MINF/blob/0b2cc65bf86d3abcae1f4a8e7c6eada937870d37/Programmation/TP1_TimerPWM/firmware/src/system_config/default/system_interrupt.c#L86
• [x] => amélioration / question / documentation => variable => déclaration => nommage => est-ce que i veut dire quelques chose pour vous ? https://github.com/Melissaperret/TP1_MINF/blob/0b2cc65bf86d3abcae1f4a8e7c6eada937870d37/Programmation/TP1_TimerPWM/firmware/src/system_config/default/system_interrupt.c#L83
• [ ]

ISR_TIMER4 ou Callback4

• [x] => amélioration / question / documentation => variable => compteur => utilité ? => algorithme ? comment vous l'utiliser ? mis à part dans l'ISR ? auriez-vous une meilleure approche ? https://github.com/Melissaperret/TP1_MINF/blob/0b2cc65bf86d3abcae1f4a8e7c6eada937870d37/Programmation/TP1_TimerPWM/firmware/src/system_config/default/system_interrupt.c#L118C1-L124C6
• [x] => amélioration / question => constante numérique => hardcodage => test / condition => comment pouvez-vous remplacer une constante numérique pour une meilleure maintenabilité de votre soft ? https://github.com/Melissaperret/TP1_MINF/blob/0b2cc65bf86d3abcae1f4a8e7c6eada937870d37/Programmation/TP1_TimerPWM/firmware/src/system_config/default/system_interrupt.c#L121
• [ ]

AUTRES

• [x] => bug / amélioration / question => variable => globale VS local => compteur => pourquoi avoir besoin d'une variable globale pour la mise à jour d'un compteur ? est-ce que cela correspond à la demande du CDC - PDF - TP1_TimerPwm_v1_31 - p5 https://github.com/Melissaperret/TP1_MINF/blob/0b2cc65bf86d3abcae1f4a8e7c6eada937870d37/Programmation/TP1_TimerPWM/firmware/src/system_config/default/system_interrupt.c#L77C9-L77C18
• [ ] => => bug / amélioration / question => variable => relation ou différence entre pData -> déclaré dans le fichier source : system_interrupt.c et PWMData-> déclarer dans le headerfile : gestPWM.h ? quelle utilité pour ces 2 variables ? https://github.com/Melissaperret/TP1_MINF/blob/0b2cc65bf86d3abcae1f4a8e7c6eada937870d37/Programmation/TP1_TimerPWM/firmware/src/gestPWM.h#L32C22-L32C29 https://github.com/Melissaperret/TP1_MINF/blob/0b2cc65bf86d3abcae1f4a8e7c6eada937870d37/Programmation/TP1_TimerPWM/firmware/src/system_config/default/system_interrupt.c#L76C15-L76C20
• [ ]

[Melissaperret]

ISR_TIMER1 ou Callback1 : 1) Avec un DEFINE :) 2) Cela signifie généralement le mot " index "

ISR_TIMER4 ou Callback4 : 1) on l'utilise également pour générer le pwmsoft dans le fichier gestPWM.c :

void GPWM_ExecPWMSoft(S_pwmSettings *pData) { if(compteur < pData->absSpeed) { LED2_W = 1; } else { LED2_W = 0; } }

Pour une meilleure approche on aurait pû faire une remise à 0 en dehors de l'ISR

2) Avec un DEFINE

AUTRES :

1) On aurait pu passer la variable " compteur " en paramètre au lieu d'en faire une globale, non ça correspond pas aux CDC.

2) Pour "pdata" c'est la variable de sauvegarde de la vitesse et de l'angle et "PWMdata" sert seulement pour l'initialisation du pont en H.