search

adafruit / Adafruit_CircuitPython_ESP_ATcontrol

Use the ESP AT command sent to communicate with the Interwebs
MIT License
20 stars 17 forks source link

implement wifmanager and update examples #28

Closed jerryneedell closed 5 years ago

jerryneedell commented 5 years ago

added support for wifimanager updated examples

jerryneedell commented 5 years ago

tested with Feather_m4_express and feather huzzah32

jerryneedell commented 5 years ago

also tested most functions on particle argon -- not sure if we want to add particle argon examples but I thought it best to get this PR in and released before adding argon specific examples.

makermelissa commented 5 years ago

Is this supposed to work with the ESP8266? I haven't tried it, but I'm trying to figure out which combination to try wiring up.

jerryneedell commented 5 years ago

It should also work with ESP8266, but there will likely be more communication errors and retries. I will do some testing

makermelissa commented 5 years ago

Is there firmware for the ESP32? Also, what is the wiring for say a Metro M4 and ESP32? I see the WiFi Co-Proc learning guide, but it has other hardware and I'm not quite sure what the equivalent to the GPIO0 is.

jerryneedell commented 5 years ago

posted AT firmware file to discord

flash with 
/usr/local/bin/esptool.py --chip esp32 --port /dev/ttyUSB0 --baud 115200 --before default_reset --after hard_reset write_flash -z --flash_mode dio --flash_freq 40m --flash_size detect 0 factory_WROOM-32.bin 
pins - 
VUSB - VUSB
GND - GND
RX - TX
TX -RX
D5 - ESP32 RESET
D6 - ESP32 GPIO15

GPIO is not needed
makermelissa commented 5 years ago

@jerryneedell, what is your confidence level on this? Should somebody else review it too or do you feel good about it?

jerryneedell commented 5 years ago

I'm confident that it it works OK and have no concerns with merging or releasing it. That said, I'm also happy to give more time for other reviews/ comments. There is no rush.

makermelissa commented 5 years ago

Sounds good to me. I'd say let's merge it so people can start using it if they want.