Skip to content
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

Add support for M5Stack Core2 v1.1 #1324

Closed

Conversation

meganetaaan
Copy link
Contributor

This is a pull request to support the minor update version v1.1 of the M5Stack Core2.
In v1.1, the power management IC has been changed from AXP192 to AXP2101 (which is also used in M5Stack CoreS3).

  • In the setup-target.js of m5stack_core2, the power management IC's id is read and automatically identified. This maintains compatibility with previous versions of the M5Stack Core2.
  • Added some functions that are present in AXP192 but not in AXP2101.
  • Some methods, such as setChargeEnable, reported by another user as not functioning, have been temporarily set to a not implemented status. We consider this to be of low urgency and would like to address it in a separate PR.

This change has been tested by writing the piu/balls demo to M5Stack v1.0, v1.1, and M5Stack CoreS3, and confirming its operation.

This PR was created with the cooperation of a new contributor, @RChikamura . He has been instructed to submit a CLA.

@@ -70,7 +72,14 @@ export default function (done) {
};

// power
globalThis.power = new Power();
debugger;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please remove the debugger statement.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

(Oops) fixed!

@phoddie
Copy link
Collaborator

phoddie commented Mar 21, 2024

Welcome to @RChikamura! Thank you both for the PR. After we receive the CLA, we will look at merging this.

@phoddie
Copy link
Collaborator

phoddie commented Mar 26, 2024

This has been merged. Thank you!

@phoddie phoddie closed this Mar 26, 2024
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.

None yet

3 participants