[GitHub] [incubator-nuttx] acassis commented on pull request #1893: xtensa/esp32: Add ESP32 WiFi adapter and driver

[GitBox <gi...@apache.org>]

acassis commented on pull request #1893:
URL: https://github.com/apache/incubator-nuttx/pull/1893#issuecomment-698889964


   > I only added comments to some simple things because I cannot really review the actual contents of this PR since I'm not familiar with WiFi code in NuttX. However, I must say that the adapter file has a **lot** going on, it is a really big file with lots of auxiliary functions. Also, I feel that is quite an obscure implementation, it is very difficult to understand what is this code doing in general, I only get little pieces by looking at the small comments for each function.
   > 
   > It would be good to add some general documentation, either in the form of a README or simple as more detailed comments explaining what all this code is doing and what each part is doing.
   > I fear that besides the obvious fact that there's some closed code behind this, this code itself will be difficult to maintain/debug by someone else as is.
   > 
   > That said, I really appreciate the support for ESP32 WiFi on NuttX.
   
   hi @v01d good points. In fact in the IDF we have a document describing about the WiFi driver blocks, maybe it could come to NuttX as well. It could be added later, but now we just need guarantee that everything is integrated correctly on NuttX.


----------------------------------------------------------------
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:
users@infra.apache.org