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

ESP32C3: DR_REG_I2S_BASE should be DR_REG_I2S0_BASE? (IDFGH-11518) #12643

Closed
3 tasks done
moefear85 opened this issue Nov 21, 2023 · 3 comments
Closed
3 tasks done

ESP32C3: DR_REG_I2S_BASE should be DR_REG_I2S0_BASE? (IDFGH-11518) #12643

moefear85 opened this issue Nov 21, 2023 · 3 comments
Assignees
Labels
Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally

Comments

@moefear85
Copy link

moefear85 commented Nov 21, 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

In soc/esp32c3/include/soc/soc.h:23

#define REG_I2S_BASE(i)                         (DR_REG_I2S_BASE + (i) * 0x1E000)

but I'm getting compile errors, because DR_REG_I2S_BASE isn't defined anywhere for the C3, unlike 32/S2/S3/C6. What does exist in soc/esp32c3/include/soc/reg_base.h:44:

#define DR_REG_I2S0_BASE                        0x6002D000

If I replace DR_REG_I2S_BASE with DR_REG_I2S0_BASE in soc.h, it compiles fine. Is this an oversight, or am I missing something here?

@espressif-bot espressif-bot added the Status: Opened Issue is new label Nov 21, 2023
@github-actions github-actions bot changed the title ESP32C3: DR_REG_SPI_BASE should be DR_REG_I2S0_BASE? ESP32C3: DR_REG_SPI_BASE should be DR_REG_I2S0_BASE? (IDFGH-11518) Nov 21, 2023
@moefear85 moefear85 changed the title ESP32C3: DR_REG_SPI_BASE should be DR_REG_I2S0_BASE? (IDFGH-11518) ESP32C3: DR_REG_I2S_BASE should be DR_REG_I2S0_BASE? (IDFGH-11518) Nov 21, 2023
@AxelLin
Copy link
Contributor

AxelLin commented Jan 11, 2024

@L-KAYA Any update?

@L-KAYA
Copy link
Collaborator

L-KAYA commented Jan 11, 2024

Fixing

@espressif-bot espressif-bot added Status: Reviewing Issue is being reviewed Status: Done Issue is done internally Resolution: NA Issue resolution is unavailable and removed Status: Opened Issue is new Status: Reviewing Issue is being reviewed labels Jan 11, 2024
espressif-bot pushed a commit that referenced this issue Feb 2, 2024
espressif-bot pushed a commit that referenced this issue Mar 1, 2024
espressif-bot pushed a commit that referenced this issue Mar 1, 2024
hathach pushed a commit to adafruit/esp-idf that referenced this issue Mar 27, 2024
@tobozo
Copy link

tobozo commented May 28, 2024

hi @L-KAYA same problem occurs with esp32H2 and DR_REG_SPI_BASE / DR_REG_SPI0_BASE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally
Projects
None yet
Development

No branches or pull requests

5 participants