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

Typo in aws_iot_config.h #1869

Closed
wants to merge 1 commit into from
Closed

Typo in aws_iot_config.h #1869

wants to merge 1 commit into from

Conversation

ksecord
Copy link
Contributor

@ksecord ksecord commented Apr 18, 2018

Underscore has been omitted, which breaks compilation when 'Override Shadow RX buffer size' is enabled via menuconfig. Reproducible with the thing_shadow example.

-breaks compilation when 'Override Shadow RX buffer size' is enabled via
menuconfig
@CLAassistant
Copy link

CLAassistant commented Apr 18, 2018

CLA assistant check
All committers have signed the CLA.

@projectgus
Copy link
Contributor

Thanks for submitting this fix @ksecord . It should be merged shortly.

@projectgus projectgus added the Status: Pending blocked by some other factor label Apr 23, 2018
igrr pushed a commit that referenced this pull request Apr 23, 2018
-breaks compilation when 'Override Shadow RX buffer size' is enabled via
menuconfig

Merges #1869
@projectgus
Copy link
Contributor

Cherry-picked in 6807d80, thanks!

@ksecord ksecord closed this Apr 24, 2018
@ksecord ksecord deleted the patch-1 branch April 24, 2018 14:38
@igrr igrr removed the Status: Pending blocked by some other factor label Aug 9, 2018
0xFEEDC0DE64 pushed a commit to 0xFEEDC0DE64/esp-idf that referenced this pull request May 5, 2021
espressif#1869 exposed a resource exhaustion issue. The current HAL layer for I2C support is designed to use a shared interrupt, But, during debugging to solve the interrupt overloading condition identified in espressif#1588, and the generation of pr espressif#1717, the interrupt allocation parameters were changed.  This change was unnecessary, the code will work successfully with shared interrupts.  So, there is no need to assign a private interrupt for each I2C peripheral.
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.

4 participants