Skip to content

separate the codec driver - #38

Merged
Wolfvak merged 5 commits into
masterfrom
ctr-codec-v0
May 13, 2021
Merged

Wolfvak merged 5 commits into
masterfrom
ctr-codec-v0

Conversation

@Wolfvak

@Wolfvak Wolfvak commented Apr 22, 2021

Copy link
Copy Markdown
Member

this separates the codec driver that directly accessed the spi bus into a "master" codec device that exposes a regmap, so that two separate drivers ("tsc" and eventually "snd") can access it safely

ideally we would also split off the virtual kbd stuff but I'm not sure drivers can access the internal input state

@Wolfvak Wolfvak linked an issue Apr 22, 2021 that may be closed by this pull request
@Wolfvak Wolfvak self-assigned this Apr 22, 2021
@Wolfvak
Wolfvak marked this pull request as ready for review May 2, 2021 14:10
@Wolfvak

Wolfvak commented May 2, 2021

Copy link
Copy Markdown
Member Author

ok, so it's taking longer than expected to deal with CODEC since it requires three pieces: codec support (modified chip from TI, no datasheets or anything), I2S support (simple enough) and the proper PCM engine driver (not that simple), plus all the interconnections between them

I think it's acceptable to merge the tsc driver for now, since I won't be able to work on this for a while

@Wolfvak
Wolfvak requested review from nickdesaulniers and xerpi May 2, 2021 14:10
@Wolfvak Wolfvak linked an issue May 2, 2021 that may be closed by this pull request
@Wolfvak Wolfvak removed a link to an issue May 2, 2021
Comment thread drivers/platform/nintendo3ds/ctr_tsc.c Resolved
Comment thread drivers/platform/nintendo3ds/ctr_tsc.c Resolved
Comment thread drivers/platform/nintendo3ds/ctr_tsc.c Resolved
@Wolfvak
Wolfvak requested review from nickdesaulniers and removed request for xerpi May 12, 2021 20:38
@Wolfvak

Wolfvak commented May 12, 2021

Copy link
Copy Markdown
Member Author

alright, pushed another commit that should fix those issues

@Wolfvak
Wolfvak merged commit e893b8e into master May 13, 2021
@Wolfvak
Wolfvak deleted the ctr-codec-v0 branch May 13, 2021 21:06
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.

keyboard single quotes is actually comma

2 participants