You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2022/07/21 13:52:39 UTC

[GitHub] [incubator-nuttx] Donny9 opened a new pull request, #6664: board/config: remove TESTING_SENSORTEST

Donny9 opened a new pull request, #6664:
URL: https://github.com/apache/incubator-nuttx/pull/6664

   
   
   ## Summary
   board/config: remove TESTING_SENSORTEST
   
   Signed-off-by: Jiuzhu Dong <do...@xiaomi.com>
   ## Impact
   don't using sensortest and using uorb_listener to test sensor
   ## Testing
   local test
   


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] acassis commented on pull request #6664: board/config: remove TESTING_SENSORTEST

Posted by GitBox <gi...@apache.org>.
acassis commented on PR #6664:
URL: https://github.com/apache/incubator-nuttx/pull/6664#issuecomment-1191616377

   @Donny9 @xiaoxiang781216 @pkarashchenko please see my comment on nuttx-apps, I think we should keep sensortest because it is a good option for users that don't need/want to share data with multiples applications. Also consider the overhead of uorb for people that don't need it.


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6664: board/config: remove TESTING_SENSORTEST

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on PR #6664:
URL: https://github.com/apache/incubator-nuttx/pull/6664#issuecomment-1193505956

   Ok, @Donny9 let's adjust sensortest to the new sensor driver.


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] Donny9 commented on pull request #6664: board/config: remove TESTING_SENSORTEST

Posted by GitBox <gi...@apache.org>.
Donny9 commented on PR #6664:
URL: https://github.com/apache/incubator-nuttx/pull/6664#issuecomment-1191515304

   This PR is associated with https://github.com/apache/incubator-nuttx-apps/pull/1228


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] Donny9 closed pull request #6664: board/config: remove TESTING_SENSORTEST

Posted by GitBox <gi...@apache.org>.
Donny9 closed pull request #6664: board/config: remove TESTING_SENSORTEST
URL: https://github.com/apache/incubator-nuttx/pull/6664


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org