os/board/rtl8730e: add driver layer implemenation for i2s tdm #6541
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
lhenry-realtek
force-pushed
the
i2s_tdm_impl
branch
from
8188501 to
1cf7b9d
Compare
Add initial porting/os layer commit
- TDM parameters fixed on I2S2 to generate required signal for AIS25BA
- DMA is not tested and likely requires further implementation
Config:
- RTL8730E Peripheral Support:
[ ] AMEBASMART_I2C
[*] Amebasmart I2S
[*] I2S_2
[*] Enable I2S RX
[*] Enable I2S TDM Support
- Device Drivers:
[ ] I2C Driver Support
[*] I2S Driver Support
- Application Configuration > Examples:
[*] I2S character driver test
[*] Use I2S receiver
[ ] Use I2S transmitter
/dev/i2schar4 I2S character device path
lhenry-realtek
force-pushed
the
i2s_tdm_impl
branch
from
1cf7b9d to
058db6b
Compare
Fix clock selection parameters to get targeted signal value: MCLK = 12.288MHz (OK) BCLK = 2.048MHz (OK) WCLK (need to setup Rx api in order to trigger WCLK activity)
|
MCLK/BCLK signal observed with LA and verified to be within parameters |
ziyik
reviewed
Nov 27, 2024
- WCLK was not started because there was no I2S and DMA transaction being started, added calls to relevant APIs - Currently DMA for I2S TDM is not properly initialized and will hardfault when rx page, currently under investigation - User is required to call I2S_RESUME and I2S_PAUSE macros when attempting a TDM operation (e.g READ), as these macros will start WCLK (for MASTER mode) - Add some debug logging and revert changes to irrelevant files
BCLK(green) is 2.048MHz, WCLK(blue) is properly generated at 16kHz when I2S_RESUME is called before an I2S_READ. The break at the end is due to hardfault, which is currently being investigated |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add initial porting/os layer commit
Config: