Nucleo-G474 and X-Nucleo-USBPDM1 sink setup for continuous changement of PDOs and APDOs

Dear @S.Cross​ 

Have you checked content of SW delivery that is available for X-CUBE-USBPDM1 package ?

(available at https://my.st.com/content/my_st_com/en/products/embedded-software/mcu-mpu-embedded-software/stm32-embedded-software/stm32cube-expansion-packages/x-cube-usbpdm1.license=1597743165145.product=X-CUBE-USBPDM1.version=2.0.2.html)

This package includes USBPD applications examples designed for STM32G071RB and STM32G474RE Nucleo boards. In both applications you could content of applicative usbpd_dpm_user and dpm_core files.

Regarding PPS aspect, please check content of the files available in the Projects\NUCLEO-G071RB\Applications\USB_PD\TCPP01_Consumer_PPS applications, as it seems to correspond to your application.

If not solving your issue, could you record a PD log on your board (using STM32CubeMonitorUCPD) ?

Regards

Guenael

Dear Guenael,

thank you for your reply. I was trying your referenced hints, but i still can't establish a stable PPS connection. But i already realized, that my 100ms request interval was wrong and now i set it up to 10s according to tPPSRequest.

But is an automatic request every 9/10 seconds generated by default from the stack, or should i do this manually? Actually there is no automatic request generated by the stack.

Thank you and best regards.

Dear @S.Cross​ 

I was just having a look at latest logs and code you attached, but lost them (maybe you remove attachements). Indeed in case of PPS (i.e. when a APDO is selected by SINK, Request messgae should be sent on a regular basis (10 sec max).

This should be handled by the USBPD Core stack.

In order to select an APDO, (example PDO5), application could use USBPD_PE_Send_Request() API. (an example is provided in X-Cube-USBPDM1 package, in USBPD_DPM_RequestMessageRequest() function in usbpd_dpm_user.c file).

An argument of the USBPD_PE_Send_Request() function is the type of power object (Fixed PDO, Variable, ...). This parameter informs the stack of the type of PDO that is selected : if APDO is selected, USBPD stack should normally manage the tPPSRequest timer.

Could you check that when you execute USBPD_PE_Send_Request() for selecting an APDO, last parameter (type USBPD_CORE_PDO_Type_TypeDef) is properly set to USBPD_CORE_PDO_TYPE_APDO (i.e. 3) ?

Please also notice that fields of RDO object are slightly different when Fixed PDO or APDO are selected :

• 0x2104B12C for a Fixed PDO indicates PDO2, Operation current 3A, Max current 3A (300 * 10 mA)
• 0x5104B12C for an APDO indicates PDO5, Operation Voltage 6V (300*20mV), Operational current 15A (300 * 50 mA)

Regards

Guenael

Dear Guenael,

thank you for your reply! I just checked my code again, and everything seems right. For example i am requesting USBPD_PE_Send_Request(0, 0x5103203C, 0x03), which means i want on port 0 pdo number 5, which is USBPD_CORE_PDO_TYPE_APDO with (8V@3A), right ?. But when i do this i can see the first request coming from the sink, but after 10 seconds i get a HardReset initiated from the source. Could it be some timing issue?

Best regards

Dear @Guenael Cadier​ 

maybe there could also be some issue with the generation of my RDOs. I put my creation code below. I tried to combine all information from the Nucleo-G071 and Nucleo-G474 example code. But maybe i produced some error with that ... The function USER_DPM_SRC_PDO_Request() is called externally with PdoPosition, RequestedVoltage and RequestedCurrent.

/**
 * @brief Build RDO for request
 * @param PortNum : the current port number
 * PdoPosition : PDO object position starting at 1
 * Voltage : requested voltage in mV
 * Current : requested current in mA
 * prdo : pointer to USBPD_SNKRDO_TypeDef
 * PtrPowerObject: pointer to USBPD_CORE_PDO_Type_TypeDef
 * @retval none
 */
void USER_DPM_SNKRDO_Builder(uint8_t PortNum, uint8_t PdoPosition, uint16_t Voltage_mV, uint16_t Current_mA, USBPD_SNKRDO_TypeDef* prdo, USBPD_CORE_PDO_Type_TypeDef* PtrPowerObject)
{
 USBPD_PDO_TypeDef pdo;
 USBPD_SNKRDO_TypeDef rdo;
 
 USBPD_HandleTypeDef *pdhandle = &DPM_Ports[PortNum];
 
 /* Initialize RDO */
 rdo.d32 = 0;
 rdo.GenericRDO.ObjectPosition = PdoPosition;
 rdo.GenericRDO.USBCommunicationsCapable = 0;
 rdo.GenericRDO.NoUSBSuspend = 1;
 
 /* Initialize PDO */
 pdo.d32 = pdhandle->DPM_ListOfRcvSRCPDO[PdoPosition - 1];
 *PtrPowerObject = pdo.GenericPDO.PowerObject;
 
 switch (pdo.GenericPDO.PowerObject)
 {
 case USBPD_CORE_PDO_TYPE_FIXED:
 case USBPD_CORE_PDO_TYPE_VARIABLE:
 { 
 rdo.FixedVariableRDO.ObjectPosition = PdoPosition;
 rdo.FixedVariableRDO.OperatingCurrentIn10mAunits = Current_mA / 10;
 rdo.FixedVariableRDO.MaxOperatingCurrent10mAunits = Current_mA / 10;
 }
 break;
 case USBPD_CORE_PDO_TYPE_APDO:
 {
 rdo.ProgRDO.ObjectPosition = PdoPosition;
 rdo.ProgRDO.OperatingCurrentIn50mAunits = Current_mA / 50;
 rdo.ProgRDO.OutputVoltageIn20mV = Voltage_mV / 20;
 }
 break;
 }
 
 pdhandle->DPM_RequestDOMsg = rdo.d32;
 pdhandle->DPM_RDOPosition = PdoPosition;
 pdhandle->DPM_RequestedCurrent = Current_mA; 
 pdhandle->DPM_RequestedVoltage = Voltage_mV;
 
 prdo->d32 = rdo.d32;
}
 
/**
 * @brief Request a PDO/APDO from given index, voltage and current
 * @param PortNum : the current port number
 * PdoPosition : PDO object position starting at 1
 * RequestedVoltage: requested voltage in mV
 * RequestedCurrent: requested current in mA
 * @retval USBPD_StatusTypeDef
 */
USBPD_StatusTypeDef USER_DPM_SRC_PDO_Request(uint8_t PortNum, uint8_t PdoPosition, uint16_t RequestedVoltage, uint16_t RequestedCurrent)
{
 USBPD_StatusTypeDef _status = USBPD_ERROR;
 
 USBPD_SNKRDO_TypeDef rdo;
 USBPD_CORE_PDO_Type_TypeDef pdo_object;
 
 /* Initialize RDO */
 rdo.d32 = 0;
 
 USER_DPM_SNKRDO_Builder(PortNum, PdoPosition, RequestedVoltage, RequestedCurrent, &rdo, &pdo_object);
 USER_DPM_Set_Request_Details(PortNum, PdoPosition, RequestedVoltage, RequestedCurrent);
 
 _status = USBPD_PE_Send_Request(PortNum, rdo.d32, pdo_object);
 
 if (_status != USBPD_OK)
 {
 DPM_USER_DEBUG_TRACE(PortNum, "USER_DPM_SRC_PDO_Request status %d", (uint8_t) _status);
 }
 return _status;
}

Best regards

Dear @S.Cross​ ,

Just a question on your last log : time spent between 2 REQUEST from Sink, in PPS mode, is expected to be around 9 sec (max is 10 sec). You mentioned it is what you observe. But looking at counter, it looks like around 18000 units occurred between 2 requests. I would have expected to see something around 9000.

Do you confirm time between Requests is around 9 seconds ?

Guenael

Dear @Guenael Cadier​ ,

the request was sent in the right time delta of 9-10 seconds. I just recognized, that an HAL_IncTick() is already executed by the interrupt handler of TIM20. So i removed my inserted HAL_IncTick() and the timebase is correct now . But error is still the same.