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

update clk_flags requirement for i2c docs (IDFGH-5949) #7637

Closed
wants to merge 1 commit into from

Conversation

waheedi
Copy link
Contributor

@waheedi waheedi commented Oct 3, 2021

As it seems, the latest version is requiring the clk_flags to have a value for the i2c driver to initialize, not sure if its intended to be like this or not honestly, but the docs are not inclined with code apparently, either the docs or the code needs to be updated, this is the shortest path :)

As it seems, the latest version is requiring the clk_flags to have a value for the i2c driver to initialize, not sure if its intended to be like this or not honestly, but the docs are not inclined with code apparently, either the docs or the code needs to be updated, this is the shortest path :)
@CLAassistant
Copy link

CLAassistant commented Oct 3, 2021

CLA assistant check
All committers have signed the CLA.

@espressif-bot espressif-bot added the Status: Opened Issue is new label Oct 3, 2021
@github-actions github-actions bot changed the title update clk_flags requirement for i2c docs update clk_flags requirement for i2c docs (IDFGH-5949) Oct 3, 2021
@Alvin1Zhang
Copy link
Collaborator

Thanks for your contribution.

@espressif-bot espressif-bot added Status: In Progress Work is in progress and removed Status: Opened Issue is new labels Oct 10, 2021
@suda-morris suda-morris added the PR-Sync-Merge Pull request sync as merge commit label Jan 4, 2022
@suda-morris
Copy link
Collaborator

sha=06a03d1dcbbd8ab334d5374670f2073d92f9d70c

@zikalino zikalino self-assigned this Feb 15, 2022
@espressif-bot espressif-bot added Status: Reviewing Issue is being reviewed Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally Resolution: Done Issue is done internally and removed Status: In Progress Work is in progress Status: Reviewing Issue is being reviewed Resolution: NA Issue resolution is unavailable labels Jul 27, 2022
espressif-bot pushed a commit that referenced this pull request Jul 29, 2022
@mahavirj
Copy link
Member

mahavirj commented Aug 8, 2022

Merged with d5bfbfc

@mahavirj mahavirj closed this Aug 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR-Sync-Merge Pull request sync as merge commit Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants