cnlohr / esp82xx

Useful ESP8266 C Environment
Other
288 stars 107 forks source link

[WIP] Update ESP Non-OS SDK to 3.0 #80

Closed AEFeinstein closed 5 years ago

AEFeinstein commented 5 years ago

Also some cleanup

cnlohr commented 5 years ago

Woof. Should I commit?

cnlohr commented 5 years ago

Err actually... I kind of want @con-f-use to chime in... But Just for background for him, we are using this on our swadges at the moment and it is working.

AEFeinstein commented 5 years ago

No, don't accept this PR yet. At a minimum you want me to put the partition stuff in the user example code. I'd be up for a full code review if you want as well.

con-f-use commented 5 years ago

Sorry, absolutely swarmed at work, the soonest I'll be able to look at it is a week from now.

cnlohr commented 5 years ago

A-ok, @con-f-use . we can work on this on our end.

AEFeinstein commented 5 years ago

uart_recvTaskPrio is the priority for the uart_recvTask(). uart0_rx_intr_handler() used to process UART bytes immediately, but the new UART file uses a task queue to handle them synchronously. As long as it doesn't collide with procTaskPrio, everything is fine.