Ouss4 commented on a change in pull request #3657:
URL: https://github.com/apache/incubator-nuttx/pull/3657#discussion_r626393716



##########
File path: arch/risc-v/src/esp32c3/hardware/esp32c3_gpio.h
##########
@@ -25,7 +25,7 @@
  * Included Files
  ****************************************************************************/
 
-#include "hardware/esp32c3_soc.h"
+#include "esp32c3_soc.h"
 

Review comment:
       > A safer solution would be to remove arch/xxx/src/xxx/hardware from the 
include search path and always preface the inclusion with hardware/.
   
   Is arch/xxx/src/xxx/hardware in the include search path?  I can't see it.
   As Gustavo mentioned this relies on the fact that `" "` includes will look 
first in the current directory.  I only changed files from that same directory.




-- 
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:
[email protected]


Reply via email to