nuttxs commented on PR #15434:
URL: https://github.com/apache/nuttx/pull/15434#issuecomment-2586789227

   > I tested but couldn't reproduce it properly. I used `esp32s3-devkit:timer` 
configuration with enabling `ESP32S3_MWDT0` and `ESP32S3_MWDT1` and then run 
`timer -d /dev/timer1` for multiple times but seems it is working fine. Also 
you are trying to enable `TIMER_GROUP1` peripheral which should be enabled for 
`CONFIG_ESP32S3_TIMER2` and `CONFIG_ESP32S3_TIMER3` according to [technical 
reference 
manual](https://www.espressif.com/sites/default/files/documentation/esp32-s3_technical_reference_manual_en.pdf#page=654)
   > 
   > Last but not least, seems the code you are doing was already implemented 
[there](https://github.com/apache/nuttx/blob/master/arch/xtensa/src/esp32s3/esp32s3_tim.c#L975-L982).
 The code in `esp32s3_tim.c` is enabling `TIMER_GROUP1` in `TIMER_GROUP0` 
peripheral timer which should not make sense in action (in theory).
   
   Hi @eren-terzioglu 
   Okay, my local environment is using an older version (12.5.x), and I 
apologize for not noticing the update of periph_module_enable(xxx) before. This 
part has already been implemented in the current PR. I will close the current 
PR. Thank you very much!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to