-
Notifications
You must be signed in to change notification settings - Fork 231
DS3231 Alarm features #805
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
base: dev
Are you sure you want to change the base?
Conversation
|
Hello @gKits thanks for picking up on this work. Perhaps you might want to replace the Please see https://github.com/tinygo-org/drivers/blob/dev/internal/regmap/i2c-spi.go#L67 What do you think? |
make main function more concise to avoid llvm error for pico
|
@deadprogram Sure can do. I used the Maybe the |
It seems the original PR #758 was abandoned by @mfulz.
I tried to contact them and contribute the missing Alarm2 features to their PR but sadly I got no response.
Therefore I forked their fork and added the proposed changes in #758.
This PR adds: