-
-
Notifications
You must be signed in to change notification settings - Fork 37.7k
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
[Keyboard] Add/Update Pimoroni keybow 2040 #23986
base: master
Are you sure you want to change the base?
Conversation
e7d50a0
to
1396152
Compare
Hello @zvecr |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missed a few things on the last review.
7e13f35
to
f322d39
Compare
Rename info.json to keyboard.json Update keyboards/pimoroni/keybow2040/readme.md Update keyboards/pimoroni/keybow2040/readme.md Update keyboards/pimoroni/keybow2040/mcuconf.h Update keyboards/pimoroni/keybow2040/info.json Update keyboards/pimoroni/keybow2040/config.h Delete rules.mk Update keyboards/pimoroni/keybow2040/keybow2040.c Update keyboards/pimoroni/keybow2040/keyboard.json Update keyboards/pimoroni/keybow2040/config.h Co-Authored-By: Joel Challis <[email protected]>
No problem |
#define I2C_DRIVER I2CD0 | ||
#define I2C1_SDA_PIN GP4 | ||
#define I2C1_SCL_PIN GP5 | ||
#define I2C1_CLOCK_SPEED 100000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only 100kHz? I know the 3731 can handle 400kHz.
This is a PR to try to continue the work done on #21481
I am not the original author of the firmware.
Description
This is a continuation, I tries to update files according to the last unresolved request changes.
Types of Changes
Checklist