Skip to content

Swap ESP32 RX and TX pins on Metro M7 1011 #7776

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

Merged
merged 1 commit into from
Mar 23, 2023

Conversation

dhalbert
Copy link
Collaborator

The serial lines to the ESP32 AirLift processor on the Metro M7 1011 are named on the schematic from the point of view of the ESP32. But pins with the same name on other AirLift boards are named from the point of view of the main microcontroller. Flip the pins. This makes the naming congruent with what is used in the adafruit_airlift library.

@dhalbert dhalbert requested review from jepler and BlitzCityDIY March 22, 2023 18:10
@BlitzCityDIY
Copy link

BlitzCityDIY commented Mar 22, 2023

thanks @dhalbert ! just tested the BLE and now it is working with ESP().

I ran the wifi test and am now getting this error though (although maybe unrelated?):

code.py output:
ESP32 SPI webclient test
Traceback (most recent call last):
  File "code.py", line 27, in <module>
  File "adafruit_esp32spi/adafruit_esp32spi.py", line 342, in status
  File "adafruit_esp32spi/adafruit_esp32spi.py", line 331, in _send_command_get_response
  File "adafruit_esp32spi/adafruit_esp32spi.py", line 245, in _send_command
TimeoutError: ESP32 timed out on SPI select

@BlitzCityDIY
Copy link

i just tested with 8.0.4 release and i got the same TimeoutError with WiFi, so it doesn't seem to have been introduced by this fix for BLE. would you like me to file a separate issue?

@dhalbert
Copy link
Collaborator Author

i just tested with 8.0.4 release and i got the same TimeoutError with WiFi, so it doesn't seem to have been introduced by this fix for BLE. would you like me to file a separate issue?

Sure

Copy link

@BlitzCityDIY BlitzCityDIY left a comment

Choose a reason for hiding this comment

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

Fixes BLE 👍

@BlitzCityDIY
Copy link

@dhalbert that actually seemed to be a random glitch that was fixed by unplugging/replugging. i got a pyportal to test with, which worked, and then when i plugged back in the metro m7 it was also working as expected with WiFi for both 8.0.4 and this build so no worries there

@dhalbert
Copy link
Collaborator Author

@BlitzCityDIY I think we have seen this issue before, and have some issues already open. It does seem to be something about the state of the NINA-FW code or its storage.

@dhalbert dhalbert merged commit d45cde1 into adafruit:main Mar 23, 2023
@dhalbert dhalbert deleted the metro-m7-esp-tx-rx-swap branch March 23, 2023 00:09
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.

2 participants