app.c -> bugs / amélioration / remarque

ETAT INIT

[x] => amélioration / question / bug => librairie => appel de fonction => à quelle "librairie - headerfile" l'appel de fonction GPWM_Initialize est liée ? qu'est-ce qui manque dans votre code ?
[x] => question / amélioration / bug => algorithme => est-ce que cela a du sens de faire démarrer les timers avant toutes les autres init (affichage, ADC, etc) ? pourquoi votre approche serait bonne ? et pourquoi cela pourrait poser problème ? https://github.com/Melissaperret/TP1_MINF/blob/0b2cc65bf86d3abcae1f4a8e7c6eada937870d37/Programmation/TP1_TimerPWM/firmware/src/app.c#L146
[x] => amélioration / question / bug => librairie => appel de fonction => à quelle "librairie - headerfile" l'appel de fonction BSP_InitADC10 est liée ? qu'est-ce qui manque dans votre code ? https://github.com/Melissaperret/TP1_MINF/blob/0b2cc65bf86d3abcae1f4a8e7c6eada937870d37/Programmation/TP1_TimerPWM/firmware/src/app.c#L157
[ ]

ETAT WAIT

ETAT EXECUTE

AUTRES

[x] => amélioration / question / bug => librairie => type => structure => ou se trouve la déclaration de la structure S_pwmSettings ? qu'est-ce qui manque dans votre code ? https://github.com/Melissaperret/TP1_MINF/blob/0b2cc65bf86d3abcae1f4a8e7c6eada937870d37/Programmation/TP1_TimerPWM/firmware/src/app.c#L82C1-L82C15
[x] => amélioration / question => algorithme => gestion leds => si 100 leds à allumer ou éteindre, codage de 100 appel de fonction ? voir issue 3 our issue 3bis https://github.com/Melissaperret/TP1_MINF/blob/0b2cc65bf86d3abcae1f4a8e7c6eada937870d37/Programmation/TP1_TimerPWM/firmware/src/app.c#L200C1-L210C2
[ ]

ETAT_INIT

1) La bibliothèque pour "GPWM_Initialize" est définie dans "gestPWM.h". Cependant, nous l'avons incluse dans "app.h" et avons ensuite inclus "app.h" dans notre fichier source "app.c". Cependant, il aurait été préférable d'inclure directement la bibliothèque dans "app.c".

2) Notre méthode présente l'avantage de nous offrir un temps d'initialisation précis. Cependant, elle est sujette à des interruptions pendant cette phase, ce que nous souhaitons éviter. Finalement, on aurait du le mettre à la fin du code avant la ligne suivante : APP_UpdateState(APP_STATE_WAIT);

Cela nous aurait donner la solution suivante :

case APP_STATE_INIT: {
//Initialiser le lcd lcd_init(); lcd_bl_on(); printf_lcd("TP1 Pwm 2023-2024"); lcd_gotoxy(1,2); printf_lcd("Melissa Perret"); lcd_gotoxy(1,3); printf_lcd("Jeremy Affolter");

        BSP_InitADC10(); //Initialisation de l'ADC 

        LED_Off(); //Eteindre toutes les leds

        GPWM_Initialize(&PWMData); //Initialisation du pont en H 

        APP_UpdateState(APP_STATE_WAIT); //Fait passer à l'état WAIT
        break;

3) La bibliothèque pour "BSP_InitADC10" est définie dans ""Mc32DriverAdc.h". Cependant, nous l'avons incluse dans "app.h" et avons ensuite inclus "app.h" dans notre fichier source "app.c". Cependant, il aurait été préférable d'inclure directement la bibliothèque dans "app.c".

AUTRES 1) La bibliothèque pour "S_pwmSettings" est définie dans "gestPWM.h". Cependant, nous l'avons incluse dans "app.h" et avons ensuite inclus "app.h" dans notre fichier source "app.c". Cependant, il aurait été préférable d'inclure directement la bibliothèque dans "app.c".

2) J'aurai pu faire une boucle for en utilisant soit une constante pour le nombre de led soit en utilisant inférieur à 7

void LED_Off(void) // fonction allumage des LEDS { for(int i = 0; i < NOMBRE_LED; i++ ) // NOMBRE_LED = constante { BSP_LED_Off(LED[i]); } }

ou comme ceci

void LED_Off(void) // fonction allumage des LEDS { for(int i = 0; i < 7; i++ ) // NOMBRE_LED = constante { BSP_LED_Off(LED[i]); } }

J'espère que les corrections sont suffisantes :)

Melissaperret / TP1_MINF

app.c -> bugs / amélioration / remarque #2