ghn-certi commented on a change in pull request #2215: URL: https://github.com/apache/incubator-nuttx/pull/2215#discussion_r518232810
########## File path: include/nuttx/sensors/sensor.h ########## @@ -495,6 +495,58 @@ struct sensor_ops_s CODE int (*batch)(FAR struct sensor_lowerhalf_s *lower, FAR unsigned int *latency_us); + + /************************************************************************** + * Name: fetch + * + * We can fetch sensor register data by this function. It will use buffer Review comment: ```suggestion * Fetch sensor register data by this function. It will use buffer ``` I think it is best if the documentation is not written in the first person. Some references: https://developers.google.com/style/person https://docs.microsoft.com/en-us/style-guide/grammar/person ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org