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 M5 Stamp S3 support #393

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

jpbelo
Copy link

@jpbelo jpbelo commented Nov 18, 2023

The stamp s3 works fine with the multi s3 build, but the built in LED is always white full brightness.
This basically duplicates the multi s3 configuration adding the neo pixel on pin 21.
Maybe some configurations are not needed, but I tried to make as little change as needed compared to the multi s3.

Tested on a M5 Stamp S3 mounted on a flipper zero and works fine, and the LED displays the correct colors for the modes.

Still curious why the BT is disabled on the S3, but when I enable it the build breaks.

@jpbelo jpbelo changed the title Add M5 Stamp S3 support (#1) Add M5 Stamp S3 support Nov 18, 2023
@@ -4562,4 +4562,4 @@ void WiFiScan::main(uint32_t currentTime)
packets_sent = 0;
}
}
}
}
Copy link
Author

Choose a reason for hiding this comment

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

yeah, I'm not sure what happened here

@@ -373,4 +373,4 @@ class WiFiScan
static void wifiSnifferCallback(void* buf, wifi_promiscuous_pkt_type_t type);
static void addPacket(wifi_promiscuous_pkt_t *snifferPacket, int len);
};
#endif
#endif
Copy link
Author

Choose a reason for hiding this comment

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

here as well...

@kokein
Copy link

kokein commented Jan 7, 2024

Hi ! Did You manage to M5Stamp full functionality?

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

2 participants