Skip to content

Added MAC address display to splash screen#434

Open
Yukti007 wants to merge 1 commit intomainfrom
414-firmware-factorytest_wmenu-report-mac-address
Open

Added MAC address display to splash screen#434
Yukti007 wants to merge 1 commit intomainfrom
414-firmware-factorytest_wmenu-report-mac-address

Conversation

@Yukti007
Copy link
Copy Markdown
Collaborator

Links

What & Why

  • WiFi.macAddress() was returning 00:00:00:00:00:00 in the splash because the WiFi driver wasn't initialized yet
  • Added WiFi.mode(WIFI_STA) + delay(100) in setup() before the splash prints, then immediately set back to WIFI_OFF
  • Added [boot] status lines for LittleFS and WiFi init so operator knows board is alive during startup
  • Added MAC address to splashserial() for device traceability

Validation / How to Verify

  1. Flash v0.4.5.2 and open serial monitor at 115200 baud
  2. Confirm [boot] LittleFS... OK and [boot] WiFi init... OK appear on startup
  3. Confirm MAC (STA): in the splash shows a real MAC address (not 00:00:00:00:00:00)

Artifacts (attach if relevant)

  • Screenshots / PDFs / STLs
  • Logs

Checklist

  • Only related changes FactoryTest_wMenu.ino
  • Folder structure respected, work directory Firmware/factoryTest/FactoryTest_wMenu/FactoryTest_wMenu.ino
  • Validation steps written

@Yukti007 Yukti007 linked an issue Mar 28, 2026 that may be closed by this pull request
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 504cf430f5

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".


WiFi.mode(WIFI_STA);
delay(100);
Serial.println(F("[boot] WiFi init... OK"));
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Gate WiFi init success log on actual init result

The new boot status line always prints "[boot] WiFi init... OK" even if WiFi.mode(WIFI_STA) fails, which can mask real radio/init failures during factory bring-up and make later MAC/Wi-Fi test failures harder to diagnose. In this setup path, the success message should be conditioned on the WiFi.mode(...) return value (or equivalent error check) so the log reflects the true boot state.

Useful? React with 👍 / 👎.

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.

Firmware, FactoryTest_wMenu, report MAC address

1 participant