HRTIM fault clearing

[OJ]

Hi,

I'm thinking what is the right way to clear HRTIM interrupt status register (HRTIM_ISR) bits?

In the description of the HRTIM_ISR register STM32G4 reference manual (RM0440 Rev 8 page 1041) says: This bit is set by hardware when fault 1 event occurs. It is cleared by software writing it at 1.

This gives the impression that you can clear HRTIM_ISR bits by writing to this register (HRTIM_ISR)

Then again bits of this register are read only ('r')?

Then we also have HRTIM interrupt clear register (HRTIM_ICR) (RM0440 Rev 8 page 1042) where we have separate bits to clear HRTIM_ISR bits.

So, which is the right way to clear HRTIM_ISR bits? Or can you clear these bits both ways?

[Pierre_Paris · Accepted solution]

Hello @OJ,

Thanks for your question and feedback !

"This gives the impression that you can clear HRTIM_ISR bits by writing to this register (HRTIM_ISR)" -> Indeed, you are right, the sentence is ambiguous. We will specifies the register HRTIM_ICR for next rev. I confirm that if you want to clear the flag in HRTIM_ISR you need to write at 1 the corresponding bit in HRTIM_ICR.

I advise you to use this macro define in STM32G4xx_HAL_Driver :

/** @brief  Clears the specified HRTIM common pending flag.
  * @param  __HANDLE__ specifies the HRTIM Handle.
  * @param  __INTERRUPT__ specifies the interrupt pending bit to clear.
  *        This parameter can be one of the following values:
  *            @arg HRTIM_IT_FLT1: Fault 1 interrupt clear flag
  *            @arg HRTIM_IT_FLT2: Fault 2 interrupt clear flag
  *            @arg HRTIM_IT_FLT3: Fault 3 clear flag
  *            @arg HRTIM_IT_FLT4: Fault 4 clear flag
  *            @arg HRTIM_IT_FLT5: Fault 5 clear flag
  *            @arg HRTIM_IT_FLT6: Fault 6 clear flag
  *            @arg HRTIM_IT_SYSFLT: System Fault interrupt clear flag
  *            @arg HRTIM_IT_DLLRDY: DLL ready interrupt clear flag
  *            @arg HRTIM_IT_BMPER: Burst mode period interrupt clear flag
  * @retval None
  */
#define __HAL_HRTIM_CLEAR_IT(__HANDLE__, __INTERRUPT__)   ((__HANDLE__)->Instance->sCommonRegs.ICR = (__INTERRUPT__))

Kind Regards,

Pierre

[Sarra.S]

Hello @OJ,

Thank you for your post 

>>Then again bits of this register are read-only ('r')?

Indeed, the description in the RM0440 of the bits FLT1 to FLT6 is not correct, I reported this typo internally (Ticket 176481). 

To clear the interrupt service register, use the HRTIM interrupt clear register (HRTIM_ICR). 

[OJ]

Hi,

Thank you for your answer and advice!

Now everything is clear.