i2s: adds i2s_rxtxdrive_begin(enableRx, enableTx, driveRxClocks, driveTxClocks) by d-a-v · Pull Request #7748 · esp8266/Arduino (original) (raw)

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 andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation4 Commits4 Checks0 Files changed

Conversation

This file contains hidden or 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

[ Show hidden characters]({{ revealButtonHref }})

d-a-v

A dac rx/rxDrive/tx/txDrive parameters are added to allow using only i2s in/out data pin without clock pins.
This is necessary for example in no-DAC mode when i2so-bck and i2so-ws are repurposed,
Found when using SDFS because SPI SS GPIO was overlapped.

edit: Related: earlephilhower/ESP8266Audio#344

@d-a-v

A dac parameter is added to allow using only i2s-out-data pin. This is necessary in no-DAC mode when i2so-bck and i2so-ws are repurposed especially because they overlap with SPI GPIO.

@d-a-v d-a-v changed the titlei2s: adds i2s_rxtxdac_begin(rx,tx,dac): i2s: adds i2s_rxtxdac_begin(rx,tx,dac)

Dec 9, 2020

@d-a-v d-a-v mentioned this pull request

Dec 9, 2020

earlephilhower

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thx for fixing this!

@d-a-v

@d-a-v

@d-a-v d-a-v changed the titlei2s: adds i2s_rxtxdac_begin(rx,tx,dac) i2s_rxtxdrive_begin(enableRx, enableTx, driveRxClocks, driveTxClocks)

Dec 11, 2020

@d-a-v d-a-v changed the titlei2s_rxtxdrive_begin(enableRx, enableTx, driveRxClocks, driveTxClocks) i2s: adds i2s_rxtxdrive_begin(enableRx, enableTx, driveRxClocks, driveTxClocks)

Dec 11, 2020

earlephilhower

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@earlephilhower

davisonja added a commit to davisonja/Arduino that referenced this pull request

Dec 28, 2020

@davisonja

…lash

2 participants

@d-a-v @earlephilhower