siddhitripathi25 commented on code in PR #18467:
URL: https://github.com/apache/nuttx/pull/18467#discussion_r2868712492


##########
Documentation/platforms/xtensa/esp32s3/spi.rst:
##########
@@ -0,0 +1,144 @@
+.. _esp32s3_spi:
+
+SPI Configuration for ESP32-S3
+===============================
+
+This guide explains how to set up SPI on the ESP32-S3. I wrote this after 
figuring out the configuration myself - hopefully it helps others avoid the 
same confusion.
+
+Prerequisites
+-------------
+Before starting, make sure:
+- You can already build NuttX for ESP32-S3
+- You know how to use menuconfig (basic navigation is enough)
+
+Configuration Steps
+-------------------
+Here's what I did to get SPI working:
+
+1. **Enable SPI support**
+   
+   Run `make menuconfig` and go to:
+   
+```Device Drivers -> SPI Support```
+
+
+Enable `[*] SPI`
+
+2. **Select the SPI peripheral**
+
+Go to:
+```System Type -> ESP32-S3 Peripheral Selection```
+
+Enable `[*] SPI2`
+
+3. **Enable SPI Exchange**
+
+Back in:
+```Device Drivers -> SPI Support```
+
+Enable `[*] SPI Exchange`
+
+4. **Enable Chip Select (optional)**
+
+In the same menu, enable:
+[*] SPI Chip Select
+
+You can change the CS pin from GPIO10 if needed.
+
+Pin Mapping
+-----------
+Default pins for SPI2:

Review Comment:
   @linguini1 No, these pins are specific to the ESP32-S3. Other ESP32 variants 
(like the original ESP32, ESP32-S2, etc.) use different default pin mappings.
   
   Would it help if I add a note saying something like "These are the default 
pins for ESP32-S3 only. Other ESP32 variants may have different pin 
assignments"?



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

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to