Ouss4 commented on a change in pull request #4543:
URL: https://github.com/apache/incubator-nuttx/pull/4543#discussion_r710061800
##########
File path: arch/xtensa/src/esp32/esp32_wdt_lowerhalf.c
##########
@@ -72,14 +75,15 @@
struct esp32_wdt_lowerhalf_s
{
- FAR const struct watchdog_ops_s *ops; /* Lower half operations */
- FAR struct esp32_wdt_dev_s *wdt; /* esp32 watchdog driver */
- uint32_t timeout; /* The current timeout */
- enum wdt_peripherals peripheral; /* Indicates if it is from RTC
or Timer Module */
- uint32_t lastreset; /* The last reset time */
- bool started; /* True: Timer has been started
*/
- xcpt_t handler; /* User Handler */
- void *upper; /* Pointer to
watchdog_upperhalf_s */
+ FAR const struct watchdog_ops_s *ops; /* Lower half operations */
+ FAR struct esp32_wdt_dev_s *wdt; /* esp32 watchdog driver */
+ uint32_t timeout; /* The current timeout */
+ enum wdt_peripherals peripheral; /* Indicates if it is from RTC or
Timer Module */
+ uint32_t lastreset; /* The last reset time */
+ bool started; /* True: Timer has been started */
+ xcpt_t handler; /* User Handler */
+ void *upper; /* Pointer to watchdog_upperhalf_s */
+ spinlock_t lock; /* Device specific lock */
Review comment:
I initially haven't changed the original alignment, I just reduced the
spaces. But now I've changed it to make the members aligned as other source
codes.
--
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: commits-unsubscr...@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
