-
-
Notifications
You must be signed in to change notification settings - Fork 605
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
Three project examples does not work with ESP32 WROVER KIT #121
Comments
This is platform.ini content
|
For the first one the defined LED_BUILTIN number is not correct, I it should be 0 (red led of the RGB in the Wrover Kit v4 and v3) instead of the default 13 See here the documentation for the Wrover Kit v3 : https://docs.espressif.com/projects/esp-idf/en/latest/get-started/get-started-wrover-kit.html#related-documents /*
* Blink
* Turns on an LED on for one second,
* then off for one second, repeatedly.
*/
#include <Arduino.h>
#ifndef LED_BUILTIN
// Set LED_BUILTIN if it is not defined by Arduino framework
#define LED_BUILTIN 13 //HERE it should be 0
#endif
void setup()
{
// initialize LED digital pin as an output.
pinMode(LED_BUILTIN, OUTPUT);
}
void loop()
{
// turn the LED on (HIGH is the voltage level)
digitalWrite(LED_BUILTIN, HIGH);
// wait for a second
delay(1000);
// turn the LED off by making the voltage LOW
digitalWrite(LED_BUILTIN, LOW);
// wait for a second
delay(1000);
} |
To solve the first bug a config file can be added with all espressif 32 boards and their led builtin pin out. main.c #include"arduino.h"
#include"ESP_Config.h"
... ESP_Config //Choose your board
#define LED_BUILTIN ESP_WROVER_KIT_LED
#define ESP_WROVER_KIT_LED 0
#define ESP32_FEATHER_LED 13
etc... The second and third bird has been referenced here and the solution is to remove the spaces. |
Please run "Verbose Upload" form PlatformIO IDE for VSCode. You will see detailed upload command and arguments. Maybe, the issue with spaces in environment name. |
@ivankravets no need we've already found the bug. Yes the second and third issue come from the spaces in environnement names. This is because the platform.ini is configurated like that in the default example codes for ESP-IDF Look at here: https://github.com/platformio/platform-espressif32/blob/master/examples/espidf-hello-world/platformio.ini#L16 |
Hm... All tests are passed. Could you provide an output from |
Please look at here: https://github.com/platformio/platform-espressif32/blob/master/examples/espidf-hello-world/platformio.ini#L16 I don't have my board with me at the moment, but no need, because we already now (and I've tested it behind you in the issue #1883 today), that the bug come from the spaces in environement name |
I can do this only tomorrow if you need |
For the build it is normal that it is passing, it is the upload which causes trouble, when esptool is called !
|
@ivankravets look at this phrase in particular
Esptool doesn't find "esp" file or directory but the full name is not "esp" but "esp wrover kit" the spaces have made esptool crash |
For the first bug The problem is that the led builtin pin number vary in function of each board: And here is the list of all board with their led builtin number: And for ESP-WROVER-KIT it's "LED_BUILTIN 0" (for GPIO 0, the red led of the RGB) It is related with the issue here: espressif/arduino-esp32#74 |
@ivankravets look by yourself
I'm not able to upload all the rest of the verbose output because of github limitations. |
@ivankravets can you check the updates on this topics please |
I'm having the same issue, here.
My solution was to manually edit - env.Append(UPLOADERFLAGS=[ image[0], "%s" % image[1]])
+ env.Append(UPLOADERFLAGS=[ image[0], "\"%s\"" % image[1]]) After re-building and uploading:
Now whitespaces are between double quotes, and esptool works just fine.
Hope this helps. |
* develop: Bump version to 1.4.0 Add default "LED_BUILTIN" for ESP Wrover Kit // Resolve #121 Add "Blink" example for ESP-IDF // Resolve #122 Fix define value in sdkconfig file Update esp-idf package version Update sdkconfig file in esp-idf examples Update esp-idf build script // Issue #91, #125 Escape space symbol in the image path // Issue #118, #121 # Conflicts: # platform.json
Duplicate of #118 |
excellent |
First one:
I've imported the arduino blinky, and uploaded to ESP-32-WROVER however no led blinks at the result.
Second one (and third one):
I tried to do the same with the hello world example project with ESP-IDF however it fails into uploading, it gave me this (same remark for other ESP-IDF example like ble-adv example):
The text was updated successfully, but these errors were encountered: