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

   ## Summary
   Small fix for my previous sloppy work on amg88xx example.
   I forgot to add correct dependency for kconfig options.
   Also, I've followed DTH driver as example, you can see that in the kconfig 
stack size option :D
   
   ## Impact
   In non-interactive build, some defines previously polluted the header file 
with no impact on build. Now is fixed.
   In interactive build (running make menuconfig ~or equivalent~) the amg88xx 
example was visible without enabling the needed driver
   
   ## Testing
   Manually.
   Checked that the define is no longer included in the output header.
   Check that the amg88xx example is not longer present in the menuconfig if 
the amg88xx driver is disabled.
   


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