TP1 - gestPWM.c -> bug / amélioration

[PBYetml]

• => question => variable => local VS globale => visibilité => comme fonctionne la visibilité d'une variable globale -> fonction -> fichier -> plusieurs fichiers ?
  https://github.com/NicolasBessson/TP1_TimerPWM/blob/f652cdede10ab833ab73bbc1231be5fe7df48c46/TP1_TimerPWM/firmware/src/gestPWM.c#L22C1-L23C50
• => amélioration / question => variable => globale => PWMData => à quoi elle vous sert ?
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/gestPWM.c

  Line 22 in f652cde

  S_pwmSettings PWMData; // pour les settings

=> FCT : GPWM_Initialize <=

• => bug => structure => initialisation des champs => pourquoi initialisez-vous que 3 champs sur 4 =>
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/gestPWM.h

  Line 54 in f652cde

  int8_t AngleSetting; // consigne angle -90 à +90

  
  https://github.com/NicolasBessson/TP1_TimerPWM/blob/f652cdede10ab833ab73bbc1231be5fe7df48c46/TP1_TimerPWM/firmware/src/gestPWM.c#L41C1-L43C52

=> FCT : GPWM_GetSettings <=

• => bug => librairie => types => quelle librairie est associée aux types suivants : uint16_t, uint8_t ?
  https://github.com/NicolasBessson/TP1_TimerPWM/blob/f652cdede10ab833ab73bbc1231be5fe7df48c46/TP1_TimerPWM/firmware/src/gestPWM.c#L78C1-L85C92
• => amélioration / question => type => uint16_t => plage de valeurs ?
• => amélioration / question => variable => indexMoyenneGlissante => valeurs max ?
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/gestPWM.c

  Line 80 in f652cde

  static uint16_t indexMoyenneGlissante = 0; // Indice pour suivre la position dans les tableaux de moyenne glissante

• => bug => librairie => appel de fonction => BSP_ReadAllADC => quelle librairie est associé à cet appel de fonction ?
• => amélioration / question => variable => appData => comment est perçu cette variable dans la fct ? est-ce une bonne pratique ?
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/gestPWM.c

  Line 88 in f652cde

  appData.AdcRes = BSP_ReadAllADC();

  
  => bug => librairie => appel de fonction => abs => quelle librairie est associé à cet appel de fonction ?
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/gestPWM.c

  Line 118 in f652cde

  pData->absSpeed = abs(pData->SpeedSetting);

• => question=> structure => champs => SpeedSetting => quelle est la plage de valeur que vous vous attendez pour cette variable ?
  https://github.com/NicolasBessson/TP1_TimerPWM/blob/f652cdede10ab833ab73bbc1231be5fe7df48c46/TP1_TimerPWM/firmware/src/gestPWM.c#L115C8-L115C21
• => question=> structure => champs => absAngle => quelle est la plage de valeur que vous vous attendez pour cette variable ?
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/gestPWM.c

  Line 121 in f652cde

  pData->absAngle = ((ADC1_Angle_M * moyenne2) / ADC_RES);

• => question=> structure => champs => AngleSetting => quelle est la plage de valeur que vous vous attendez pour cette variable ?
  https://github.com/NicolasBessson/TP1_TimerPWM/blob/f652cdede10ab833ab73bbc1231be5fe7df48c46/TP1_TimerPWM/firmware/src/gestPWM.c#L124C8-L124C20

=> FCT : GPWM_DispSettings <=

• => question / amélioration => gestion LCD => appel de fonction => printf_lcd() => une fois cet appel de fct exécuté, comment se comporte l'affichage ? est-ce que l'information est gradée en mémoire si vous écrivez sur une autre ligne ?
• => question / amélioration => gestion LCD => appel de fonction => printf_lcd() => est-ce que certains appel de fct peuvent-il être appelé qu'une seule fois ?
  https://github.com/NicolasBessson/TP1_TimerPWM/blob/f652cdede10ab833ab73bbc1231be5fe7df48c46/TP1_TimerPWM/firmware/src/gestPWM.c#L145C1-L146C91
• => question => gestion LCD => appel de fct => printf_lcd() => opérateur de conversion => %xd => que représente le x ? que se passe-t-il si la valeur à affiché est plus grande que x et même question si plus petite ?
• => question / amélioration => gestion LCD => appel de fct => printf_lcd() => opérateur de conversion => %xd => comment pourriez-vous gérer les signes : + et - ?
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/gestPWM.c

  Line 153 in f652cde

  printf_lcd("%3d",pData->SpeedSetting); // Affiche la vitesse négative

  
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/gestPWM.c

  Line 158 in f652cde

  printf_lcd("%2d",pData->SpeedSetting); // Affiche la vitesse positive

  
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/gestPWM.c

  Line 163 in f652cde

  printf_lcd("%2d",pData->absSpeed); // Affiche la vitesse absolue

  
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/gestPWM.c

  Line 168 in f652cde

  printf_lcd("%4d",pData->AngleSetting); // Affiche la valeur de l'angle avec un format de 4 chiffres

=> FCT : GPWM_ExecPWM <=

• => amélioration / question => type => int => plage de valeurs ? Est-ce que le type est utilisée est judicieux ?
  https://github.com/NicolasBessson/TP1_TimerPWM/blob/f652cdede10ab833ab73bbc1231be5fe7df48c46/TP1_TimerPWM/firmware/src/gestPWM.c#L187C1-L188C32
• => amélioration / question => test / condition => algorithme => vous mettez 3 conditions de test (OK), mais qu'elle serait la 4ème (else) ?
  https://github.com/NicolasBessson/TP1_TimerPWM/blob/f652cdede10ab833ab73bbc1231be5fe7df48c46/TP1_TimerPWM/firmware/src/gestPWM.c#L192C1-L211C6
• => documentation / amélioration => variable => PulseWidthAngle ; PulseWidthVitesse => plage de valeur ?
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/gestPWM.c

  Line 215 in f652cde

  PulseWidthAngle = ((pData->absAngle * PLAGE_OC3 ) / PLAGE_ANGLE_MAX ) + OFFSET_OC3 ;

  
  https://github.com/NicolasBessson/TP1_TimerPWM/blob/f652cdede10ab833ab73bbc1231be5fe7df48c46/TP1_TimerPWM/firmware/src/gestPWM.c#L219C5-L219C23
• => amélioration / question => appel de fct => DRV_TMR1_PeriodValueGet => paramètre de sortie => quelle valeur vous ressort cette fct ?
• => amélioration / question => constante numérique => hardcodage => que représente ce 99 ?
  https://github.com/NicolasBessson/TP1_TimerPWM/blob/f652cdede10ab833ab73bbc1231be5fe7df48c46/TP1_TimerPWM/firmware/src/gestPWM.c#L219C5-L219C23

=> FCT : GPWM_ExecPWMSoft <=

• => bug => librairie => types => quelle librairie est associée aux types suivants : uint8_t ?
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/gestPWM.c

  Line 226 in f652cde

  static uint8_t compteur = 0;

• [ ]

[MathieuBucher]

1. la variable globale est visible uniquement dans le fichier où elle est déclarée. Si on souhaite la rendre visible dans d'autre fichier il faut la mettre en extern dans un .h et include ce .h dans les fichiers en questions.
2. C'est un moyen de regrouper les réglages du PWM pour que toutes les fonctions chargées d’interagir avec le PWM puissent y accéder directement.
   => FCT : GPWM_Initialize <=
3. C'est un oublie.

=> FCT : GPWM_GetSettings <=

1. de la librairie "stdint.h".
2. 16 bits non signé de 0 à 65535.
3. La résolution max de l'adc de 0 à 0 à 4095.
4. de la librairie bsp.h (bsp_sys_init.c).
   5.1 La fonction accède directement à la variable globale. Non n’importe quelle fonction du programme peut modifier appData. Risques de confusion et de bugs si l’on ne sait pas exactement qui lit ou écrit dedans.
   5.2 Dans "stdlib.h".
5. Plage de -99 à 99.
6. Plage de 0 à 180.
7. Plage de -90 à +90.

[MathieuBucher]

=> FCT : GPWM_DispSettings <=

1. Oui l'affichage reste en mémoire
2. Oui on pourrait afficher une seule fois "TP1 PWM 2024-25" vue qu'il ne change jamais.
3. Le chiffre x est la largeur minimale de l’affichage. Si la valeur est plus petite, on complète avec des espaces. Si elle est plus grande, le texte sort du cadre minimal.
4. En rajoutant le "+" si cla valeur est positive. Exemple : printf_lcd("%+2d",pData->SpeedSetting);

=> FCT : GPWM_ExecPWM <=

1. Sur une plage de 0 à 65535. On pourrait utiliser un uint16_t.
2. Le "else if(pData->SpeedSetting == 0) peut se transformer en else
3. Plage entre 2999(si pData->absAngle = 0) à 11999(si pData->absAngle = 180) pour l'angle et 0 à 99 pour la vitesse.
4. Elle renvoie le nombre de pas qui détermine la période du timer 1 (Prescaler = 24′999) qui permet de recalculer la largeur d'impulsion du PWM
5. La vitesse maximale que peut atteindre le système

=> FCT : GPWM_ExecPWMSoft <=

1. de la librairie "stdint.h".