Skip to content

Arduino Nano 33 IoT support #2638

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 5 commits into from
Feb 22, 2020
Merged

Arduino Nano 33 IoT support #2638

merged 5 commits into from
Feb 22, 2020

Conversation

fgallaire
Copy link

No description provided.

@ladyada ladyada requested a review from dhalbert February 19, 2020 16:12
@ladyada
Copy link
Member

ladyada commented Feb 19, 2020

lgtm - please be aware that there's not enough memory in circuitpy to talk to the wifi module and do internetty things :)

ladyada
ladyada previously approved these changes Feb 19, 2020
Copy link
Collaborator

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

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

Looks good, but it would be good to add wifi module pins.

ladyada
ladyada previously approved these changes Feb 21, 2020
Copy link
Member

@ladyada ladyada left a comment

Choose a reason for hiding this comment

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

Copy link
Collaborator

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

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

Thanks for the changes. There are still some missing ESP32 pins, still, mainly having to do with the separate TX/RX lines for updating, but it's not vital they're added. GPIO33 is called "...BUSY" by us, but '...ACK" on the Arduino schematic. I'm not sure it's worth changing these now or not. We could just get the board in. @ladyada do you have an opinion?

@ladyada
Copy link
Member

ladyada commented Feb 21, 2020

oh shoot i missed the serial nina pins - lets merge so this is in before the next realaseand @fgallaire please submit a pr for those two pins :)

@dhalbert
Copy link
Collaborator

Schematics:

Typical Adafruit board:
Selection_136

Arduino Nano 33 IoT:
Selection_135

@ladyada
Copy link
Member

ladyada commented Feb 22, 2020

yay - will merge once passing

@fgallaire
Copy link
Author

fgallaire commented Feb 22, 2020

yay - will merge once passing

a noob question before:
I named the NINA RESET_N ESP_RESET after other boards naming, but RESET_N is linked to +3V3 while typical Adafruit board have ESP_RESET linked to GND, so should I rename my ESP_RESET in ESP_RESET_N ?

@ladyada
Copy link
Member

ladyada commented Feb 22, 2020

nope - the pin itself is the same :)

@ladyada ladyada requested a review from dhalbert February 22, 2020 04:44
Copy link
Collaborator

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

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

Thank you!

@dhalbert dhalbert merged commit dcc8abf into adafruit:master Feb 22, 2020
@fgallaire
Copy link
Author

lgtm - please be aware that there's not enough memory in circuitpy to talk to the wifi module and do internetty things :)

Hello, nevertheless I would like to test it, so I need to upgrade the NINA firmware. So I need a Serial Passthroug firmware for the Nano IoT but I don't find the repo to fork/how to do it.

@dhalbert
Copy link
Collaborator

Check out this thread: https://forum.arduino.cc/index.php?topic=654833.0
The Arduino forums and website are probably best for inquiring about this.

@fgallaire
Copy link
Author

Thanks @dhalbert I'm so focused on CircuitPython that I forgot that it's an Arduino board ;-)
Flashing the firmware with the Arduino IDE works fine:
https://www.arduino.cc/en/Tutorial/WiFiNINA-FirmwareUpdater

@fgallaire
Copy link
Author

I have done my best:

  • adafruit_esp32spi as frozen module
  • Adafruit NINA firmware 1.6 flashed via Arduino IDE
  • MOSI pin changed to PB09 to match the pin change adafruit/nina-fw@9e76479

But always the same error:

RuntimeError: Timed out waiting for SPI char

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.

3 participants