Skip to content

Conversation

@jfurcean
Copy link

@jfurcean jfurcean commented Oct 6, 2021

TODO

@jfurcean jfurcean force-pushed the add_macropad_rp2040 branch from 4c9b05f to 4ba63f5 Compare October 7, 2021 13:26
@jfurcean jfurcean marked this pull request as ready for review October 7, 2021 13:26
@jfurcean
Copy link
Author

@execuc and @caternuson can you review?

@caternuson
Copy link
Contributor

@jfurcean Were you able to build this locally and test it on a MacroPad?

@jfurcean
Copy link
Author

Yes, it required the other two PRs above to be installed locally as well. The display doesn't work due to an existing issue in Adafruit_Blinka_Displayio adafruit/Adafruit_Blinka_Displayio#54.

I tested with the below code.

import board
import time
import neopixel
import keypad

pixels = neopixel.NeoPixel(board.NEOPIXEL, 12)

for i in range(12):
	pixels[i] = (0,255,0)
	time.sleep(.5)

for i in range(12):
	pixels[i] = (0,0,0)
	time.sleep(.5)

key_pins = (board.KEY1, board.KEY2, board.KEY3, board.KEY4, board.KEY5, board.KEY6,
            board.KEY7, board.KEY8, board.KEY9, board.KEY10, board.KEY11, board.KEY12)
keys = keypad.Keys(key_pins, value_when_pressed=False, pull=True)

while True:

    event = keys.events.get()
    if event:
        print(event)

I also tested i2c with the Stemma QT rotary encoder example - https://github.com/adafruit/Adafruit_CircuitPython_seesaw/blob/main/examples/seesaw_rotary_simpletest.py

@caternuson
Copy link
Contributor

Cool. I tested this real quick locally and it all seems to work. LGTM..

@freemansoft
Copy link

Do we think this will get merged?

#define I2C0_ENABLED 1
#define U2IF_I2C0_SDA 20
#define U2IF_I2C0_SCL 21
// SPI1

Choose a reason for hiding this comment

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

Why is this SPI1 instead of SPI2?

#define U2IF_SPI1_CK 26
#define U2IF_SPI1_MOSI 27
#define U2IF_SPI1_MISO 28
// UART1

Choose a reason for hiding this comment

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

Why is this UART1 instead of UART0?

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