Skip to content

Conversation

@Volkalex28
Copy link
Contributor

Rx and Tx slots can have different configurations for the same I2S
So far I have added this only for tdm mode, let me know if this needs to be done for std as well

@Volkalex28 Volkalex28 force-pushed the i2s_split_tdm_slots_cfg branch from 269e4f8 to 9c36c45 Compare June 16, 2024 01:00
@ivmarkov
Copy link
Collaborator

@Volkalex28 Would you mind if this waits until after we do the upcoming release?
These changes are a tad more obtrusive, so I feel we could do them post-release..

@Volkalex28
Copy link
Contributor Author

Volkalex28 commented Jun 16, 2024

@ivmarkov Do you mean releasing patch .1 in a few days or delaying it until the next release?

@ivmarkov
Copy link
Collaborator

@ivmarkov Do you mean releasing patch .1 in a few days or delaying it until the next release?

What do you mean by .1? I mean merging this patch in-here after we do a new release. Since releases are branched off master we should not merge this PR right now. Unless you have very good reasons to, and you are in a hurry.

@Volkalex28
Copy link
Contributor Author

@ivmarkov I got you. If it doesn't take more than two weeks, that's fine for me. Thank you

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