fdcavalcanti opened a new pull request, #18611:
URL: https://github.com/apache/nuttx/pull/18611

   ## Summary
   
   <!-- This field should contain a summary of the changes. It will be 
pre-filled with the commit's message and descriptions. Adjust it accordingly -->
   
   * arch/risc-v/espressif: select EFUSE when ESPRESSIF_EFUSE is enabled
   
   Add 'select EFUSE' entry to enable NuttX EFUSE support.
   
   
   ## Impact
   <!-- Please fill the following sections with YES/NO and provide a brief 
explanation -->
   
   Impact on user: No.
   <!-- Does it impact user's applications? How? -->
   
   Impact on build: Yes, fixes build error when 'ESPRESSIF_EFUSE' is selected 
but 'EFUSE' is not.
   <!-- Does it impact on building NuttX? How? (please describe the required 
changes on the build system) -->
   
   Impact on hardware: Affects ESP32-C3|C6|H2|P4.
   <!-- Does it impact a specific hardware supported by NuttX? -->
   
   Impact on documentation: No.
   <!-- Does it impact the existing documentation? Please provide additional 
documentation to reflect that -->
   
   Impact on security: No.
   <!-- Does it impact NuttX's security? -->
   
   Impact on compatibility: No.
   <!-- Does it impact compatibility between previous and current versions? Is 
this a breaking change? -->
   
   ## Testing
   <!-- Please provide all the testing procedure. Consider that upstream 
reviewers should be able to reproduce the same testing performed internally -->
   
   ### Building
   <!-- Provide how to build the test for each SoC being tested -->
   Build any defconfig, then enable `ESPRESSIF_EFUSE`. It does not select 
`EFUSE` by default so compilation fails since the upper-half part requires 
`CONFIG_EFUSE` to compile.
   
   ### Results
   <!-- Provide tests' results and runtime logs -->
   Before change:
   ```
   [177/1312] Building C object 
arch/CMakeFiles/arch.dir/risc-v/src/common/espressif/esp_efuse.c.o
   FAILED: arch/CMakeFiles/arch.dir/risc-v/src/common/espressif/esp_efuse.c.o 
   ...
   -jump-tables -fno-tree-switch-conversion -Wno-deprecated-declarations 
@arch/CMakeFiles/arch.dir/risc-v/src/common/espressif/esp_efuse.c.o.rsp -MD -MT 
arch/CMakeFiles/arch.dir/risc-v/src/common/espressif/esp_efuse.c.o -MF 
arch/CMakeFiles/arch.dir/risc-v/src/common/espressif/esp_efuse.c.o.d -o 
arch/CMakeFiles/arch.dir/risc-v/src/common/espressif/esp_efuse.c.o -c 
/home/fdcavalcanti/nuttxspace5/nuttx/arch/risc-v/src/common/espressif/esp_efuse.c
   
/home/fdcavalcanti/nuttxspace5/nuttx/arch/risc-v/src/common/espressif/esp_efuse.c:66:43:
 error: unknown type name 'efuse_desc_t'
      66 |                                     const efuse_desc_t *field[],
         |                                           ^~~~~~~~~~~~
   ```
   
   After changes:
   
   Build succeeds.


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