linguini1 opened a new pull request, #2987:
URL: https://github.com/apache/nuttx-apps/pull/2987

   ## Summary
   
   This pull request closes the issue raised 
[here](https://github.com/apache/nuttx/issues/15599).
   
   Now `DEBUG_UORB` depends on `LIBC_FLOATINGPOINT`. This is because it is not 
possible to view sensor debug output without this enabled, which is only a 
problem for systems without an FPU (systems with an FPU have this enabled by 
default).
   
   ## Impact
   
   Users now have to enable `LIBC_FLOATINGPOINT` on systems without an FPU in 
order to use `DEBUG_UORB`. The dependency marked with `depends on` means that 
the uORB debug option will not be visible until `LIBC_FLOATINGPOINT` is 
selected, but this was determined to be acceptable in the discussion under the 
linked issue.
   
   This prevents the annoyance of re-flashing the device because floating point 
printing was disabled, rendering debug output almost useless.
   
   I have also included a compile time error for the configuration mismatch.
   
   ## Testing
   
   Verified that enabling `LIBC_FLOATINGPOINT` caused `DEBUG_UORB` to appear 
and that the dependency was marked in Kconfig.


-- 
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

Reply via email to