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

Don't change default baud rate for MKR Vidor 4000 #110

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sameer
Copy link

@sameer sameer commented Mar 19, 2020

The esptool expects an initial detection phase where baud rate is 115200. Setting it to 119400 is causing wacky warnings like detecting an incorrect crystal frequency:

WARNING: Detected crystal freq 38.94MHz is quite different to normalized freq 40MHz. Unsupported crystal in use?
Crystal is 40MHz

It still works fine with 119400 rate, but it creates a warning and there's no need to distinguish Vidor from the other MKR boards here.

The esptool expects an initial detection phase where baud rate is 115200. Setting it to 119400 is causing wacky errors like detecting an incorrect crystal frequency:
```
WARNING: Detected crystal freq 38.94MHz is quite different to normalized freq 40MHz. Unsupported crystal in use?
Crystal is 40MHz
```
It still works fine with 119400 rate, but it creates a warning and there's no need to distinguish Vidor from the other MKR boards here.
@sameer
Copy link
Author

sameer commented Mar 19, 2020

See https://github.com/espressif/esptool#baud-rate for why it has to be 115200.

@sameer sameer changed the title Don't change baud rate for MKR Vidor 4000 Don't change default baud rate for MKR Vidor 4000 Mar 20, 2020
@gagarcr
Copy link

gagarcr commented Apr 6, 2020

Well, that’s the point. Currency It IS 119400, the PR is reverting it back to the default 115200

@sameer
Copy link
Author

sameer commented Apr 6, 2020

Well, that’s the point. Currency It IS 119400, the PR is reverting it back to the default 115200

I couldn't find any info on why it's changed to 119400 and not kept at 115200 which esptool expects during the initial detection phase.

Is there something I am missing here?

@sameer sameer closed this Apr 6, 2020
@sameer sameer reopened this Apr 6, 2020
@CLAassistant
Copy link

CLAassistant commented Apr 9, 2021

CLA assistant check
All committers have signed the CLA.

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.

3 participants