Foc 3 Shunt Current Sensing Bug Report

[LahmYang]

MCSDK Version: 6.4.1 Build: 250722.224358

uint16_t R3_2_SetADCSampPointSectX_OVM(PWMC_Handle_t *pHdl)

Line 1246.

Should there be Tcase2? 

Tcase3 here is wrong, Tcase2 with no caller.

[GMA · Accepted solution]

Hello @LahmYang,

Thank you for your report. The corrected firmware is as follows:

uint16_t R3_2_SetADCSampPointSectX_OVM(PWMC_Handle_t *pHdl)
{
  uint16_t retVal;
... 
      /* In every sector there is always one phase with maximum complementary duty, one with minimum 
       * complementary duty and one with variable complementary duty. In this case, phases with variable
       * complementary duty and with maximum duty are converted and the first will be always the phase
       * with variable complementary duty cycle */
  
      if ( DeltaDuty >= pHandle->pParams_str->Tcase3 )
      {
        SamplingPoint = pHdl->lowDuty - pHandle->pParams_str->Tbefore;
      }
      else
      {
        /* Case 2 (cf user manual) */
        if ( ( pHandle->Half_PWMPeriod - pHdl->lowDuty ) > pHandle->pParams_str->Tcase2 )
        {
          /* ADC trigger edge must be changed from positive to negative */
          pHandle->ADCTriggerEdge = LL_ADC_INJ_TRIG_EXT_FALLING;
          SamplingPoint = pHdl->lowDuty + pHandle->pParams_str->Tbefore;
        }

This issue will be reported to the dedicated team to address and correct it for future deliveries.