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

Added multi-pin TDM support. #470

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ccrome
Copy link
Contributor

@ccrome ccrome commented Jan 18, 2024

This allows you use use more than one TX or RX pin in TDM mode, allowing for up to 64 channels of record or playback data

This fix goes hand-in-hand with the multi-channel USB support. PaulStoffregen/cores#732

input_tdm: every odd channel had every other sample swapped In every odd channel in TDM input (1, 3, 5, 7, 9, 11, 13, 15), every other word was swapped due to an incorrect copy from 32-bits to
16-bits. This fix corrects the odd channels. The Shift-by zeros and the
extraneous logical ands are there for clarity, and I verified they
don't end up affecting final code optimization as long as optimization
is turned on.

This allows you use use more than one TX or RX pin in TDM mode,
allowing for up to 64 channels of record or playback data

This fix goes hand-in-hand with the multi-channel USB support.
PaulStoffregen/cores#732

input_tdm: every odd channel had every other sample swapped
In every odd channel in TDM input (1, 3, 5, 7, 9, 11, 13, 15), every
other word was swapped due to an incorrect copy from 32-bits to
16-bits. This fix corrects the odd channels.   The Shift-by zeros  and the
extraneous logical ands are there for clarity, and I verified they
don't end up affecting final code optimization as long as optimization
is turned on.
@ccrome ccrome force-pushed the ccrome/tdm-multipin branch from 8a494ed to 28d3ab4 Compare January 18, 2024 23:26
@ccrome ccrome marked this pull request as ready for review January 18, 2024 23:26
@h4yn0nnym0u5e
Copy link
Contributor

This is non-functional, as the changes to TDM output are not complete

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants