Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CHIP_FEATURE_EMB_PSRAM not set on ESP32-PICO-V3-02 (IDFGH-9938) #11233

Closed
3 tasks done
mmrein opened this issue Apr 20, 2023 · 1 comment
Closed
3 tasks done

CHIP_FEATURE_EMB_PSRAM not set on ESP32-PICO-V3-02 (IDFGH-9938) #11233

mmrein opened this issue Apr 20, 2023 · 1 comment
Assignees
Labels
Status: Reviewing Issue is being reviewed

Comments

@mmrein
Copy link

mmrein commented Apr 20, 2023

Answers checklist.

  • I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
  • I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
  • I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report

Quite simply, I would expect the esp_chip_info() to return CHIP_FEATURE_EMB_PSRAM bit set for ESP32-PICO-V3-02 (EFUSE_RD_CHIP_VER_PKG_ESP32PICOV302) as it has 2MB PSRAM included in same package. But this bit is only set for EFUSE_RD_CHIP_VER_PKG_ESP32D0WDR2V3 as we can see here:

uint32_t package = efuse_ll_get_chip_ver_pkg();
if (package == EFUSE_RD_CHIP_VER_PKG_ESP32D2WDQ5 ||
package == EFUSE_RD_CHIP_VER_PKG_ESP32PICOD2 ||
package == EFUSE_RD_CHIP_VER_PKG_ESP32PICOD4 ||
package == EFUSE_RD_CHIP_VER_PKG_ESP32PICOV302) {
out_info->features |= CHIP_FEATURE_EMB_FLASH;
}
if(package == EFUSE_RD_CHIP_VER_PKG_ESP32D0WDR2V3) {
out_info->features |= CHIP_FEATURE_EMB_PSRAM;
}

In case the Embedded PSRAM feature bit has different meaning then it would be good to clarify that in documentation.

Edit:

The esptool (IDF v4.4.4) does report Embedded PSRAM correctly, so it really seems like the issue in esp_chip_info():

esptool.py v3.3.2
Serial port /dev/ttyUSB1
Connecting....
Chip is ESP32-PICO-V3-02 (revision 3)
Features: WiFi, BT, Dual Core, 240MHz, Embedded Flash, Embedded PSRAM, VRef calibration in efuse, Coding Scheme None
Crystal is 40MHz

in which case I suppose correct if statement would be:

    if(package == EFUSE_RD_CHIP_VER_PKG_ESP32D0WDR2V3 ||
       package == EFUSE_RD_CHIP_VER_PKG_ESP32PICOV302) {
        out_info->features |= CHIP_FEATURE_EMB_PSRAM;
    }
@espressif-bot espressif-bot added the Status: Opened Issue is new label Apr 20, 2023
@github-actions github-actions bot changed the title CHIP_FEATURE_EMB_PSRAM not set on ESP32-PICO-V3-02 CHIP_FEATURE_EMB_PSRAM not set on ESP32-PICO-V3-02 (IDFGH-9938) Apr 20, 2023
@ESP-Marius
Copy link
Collaborator

@mmrein you are right, seems like we might have missed adding it to the list.

We'll fix it, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Reviewing Issue is being reviewed
Projects
None yet
Development

No branches or pull requests

3 participants