v01d commented on pull request #3470:
URL: https://github.com/apache/incubator-nuttx/pull/3470#issuecomment-815165894


   > > I think we should also avoid explaining how
   > > to do things that are not that relevant (such as how to build openocd, 
etc).
   > 
   > I was tempted to remove this one yesterday! But I'd replace it with some 
instructions on how to use OpenOCD (Espressif's fork). From a user perspective 
it's similar to using vanilla OpenOCD just with `-c 'set ESP_RTOS NuttX'` or 
`-c 'set ESP_RTOS none'`.
   
   There's already mention of this in the doc I migrated. Still, if we already 
provide that .cfg I don't see much point in explaining how to recreate it.
   
   > Similarly for the toolchain, I wanted to simplify the instructions to just 
downloading it from 
https://dl.espressif.com/dl/xtensa-esp32-elf-gcc8_2_0-esp-2020r2-linux-amd64.tar.gz.
 BTW, it looks like you remove it all together. Do we have this information 
elsewhere? I can't find it.
   
   The instruction to download the toolchain is already there (you can actually 
already see it on the doc website). Please not that this is at the ESP32 level 
(not board level). My policy in this PR was to mostly migrate things and not 
remove big things. I would suggest to do that in a second PR as I mentioned.
   
   


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