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

[ESP32] Where are Discriminator and Setup-pin-code in menuconfig? #21391

Closed
mmt-10 opened this issue Jul 29, 2022 · 4 comments
Closed

[ESP32] Where are Discriminator and Setup-pin-code in menuconfig? #21391

mmt-10 opened this issue Jul 29, 2022 · 4 comments
Labels

Comments

@mmt-10
Copy link

mmt-10 commented Jul 29, 2022

Problem

I cannot find discriminator and setup pin code setting in menuconfig.
Where is it?
How do I change it?
https://github.com/project-chip/connectedhomeip/tree/master/examples/all-clusters-app/esp32#commissioning-over-ble

I tried grep "f00" sdkconfig, grep "3840" sdkconfig and grep "20202021" sdkconfig, but not found.

connectedhomeip : commit ae2ca6d (HEAD -> master, origin/master, origin/HEAD)
ESP-IDF : v4.4.1
device : ESP32-C3-DevkitM-1

@bzbarsky-apple
Copy link
Contributor

@shubhamdp @dhrishi

@dhrishi
Copy link
Contributor

dhrishi commented Jul 29, 2022

@tatsutaigu In the end products, these parameters won't be configurable through menuconfig but would be fixed and pre-configured in the device. As we are getting closer to production, the support for this was removed on purpose. You can check the sub-section "Using ESP32 Factory Data Provider (Optional)" here to configure the values of your choice. If the values are not set using the factory data provider, the default ones are used (discriminator 3840 and setup pincode 20202021) from the common SDK code.

And, the documentation you pointed out seems to be stale and we will update it right away. Thanks for reporting!

@shubhamdp
Copy link
Contributor

I have removed the those stale pointers in #21340

@mmt-10
Copy link
Author

mmt-10 commented Jul 29, 2022

@dhrishi and @shubhamdp

Thank you for your prompt reply and action.
I understand it.

@mmt-10 mmt-10 closed this as completed Jul 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants