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

Added support for heltec_wifi_lora_32_V3 and heltec_wifi_kit_32_V3 #7410

Merged
merged 14 commits into from Dec 21, 2022

Conversation

Baptou88
Copy link
Contributor

Changes to be committed:
new file: variants/heltec_wifi_kit_32_v3/pins_arduino.h
new file: variants/heltec_wifi_lora_32_V3/pins_arduino.h
Source: https://github.com/Heltec-Aaron-Lee/WiFi_Kit_series/tree/master/esp32/variants

Changes to be committed:
	new file:   variants/heltec_wifi_kit_32_v3/pins_arduino.h
	new file:   variants/heltec_wifi_lora_32_V3/pins_arduino.h
@CLAassistant
Copy link

CLAassistant commented Oct 30, 2022

CLA assistant check
All committers have signed the CLA.

@P-R-O-C-H-Y
Copy link
Member

P-R-O-C-H-Y commented Nov 2, 2022

@Baptou88 The new boards are missing in our boards.txt file and has to be added if needed. So adding this is useless without that. If you want these boards in Arduino-esp32 feel free to add them to the boards.txt. After that I will review the changes.

Changes to be committed:
	new file:   variants/heltec_wifi_kit_32_v3/pins_arduino.h
	new file:   variants/heltec_wifi_lora_32_V3/pins_arduino.h
	modified:   boards.txt
@Baptou88
Copy link
Contributor Author

Baptou88 commented Nov 2, 2022

thanks @P-R-O-C-H-Y for your reply! I've modified board.txt, I hope it will be good

@P-R-O-C-H-Y
Copy link
Member

@Baptou88 Overall looks good, but as the previous HELTEC boards already existing in our boards.txt can you please add the "heltec" at the start of the name? Will make more sense better :)

@Baptou88
Copy link
Contributor Author

Baptou88 commented Nov 3, 2022

@P-R-O-C-H-Y I just fixed boards.txt file

Copy link
Member

@P-R-O-C-H-Y P-R-O-C-H-Y left a comment

Choose a reason for hiding this comment

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

@Baptou88 PTAL on my comments. After changes please try to compile with the boards to be sure its working 👍

boards.txt Outdated Show resolved Hide resolved
boards.txt Show resolved Hide resolved
boards.txt Show resolved Hide resolved
boards.txt Outdated
heltec_wifi_lora_V3.upload.tool.default=esptool_py
heltec_wifi_lora_V3.upload.tool.network=esp_ota

heltec_wifi_lora_V3.upload.maximum_size=1310720
Copy link
Member

Choose a reason for hiding this comment

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

Same here. When partition is fixed to default_8MB, I can't see that upload.maximus_size corresponds to the partition default_8MB.

esp32s3.menu.PartitionScheme.default_8MB.upload.maximum_size=3342336

boards.txt Show resolved Hide resolved
static const uint8_t SCL_OLED = 18;
static const uint8_t SDA_OLED = 17;

#endif /* Pins_Arduino_h */
Copy link
Member

Choose a reason for hiding this comment

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

Please add new line at the end of the file to keep standart formatting.

static const uint8_t RST_OLED = 21;
static const uint8_t SCL_OLED = 18;
static const uint8_t SDA_OLED = 17;
#endif /* Pins_Arduino_h */
Copy link
Member

Choose a reason for hiding this comment

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

Please add new line at the end of the file to keep standart formatting.

Copy link
Member

Choose a reason for hiding this comment

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

Add line after #endif /* Pins_Arduino_h */. So there will be empty line number 83.

boards.txt Outdated Show resolved Hide resolved
boards.txt Outdated Show resolved Hide resolved
@Baptou88
Copy link
Contributor Author

Baptou88 commented Nov 4, 2022

I'm testing the changes I made: here's what I did:

  1. I created a platformio project configured this way:
[env:heltec_wifi_kit_32_V3]
platform = C:\Users\Baptou88\Documents\GitHub\platform-espressif32
board = heltec_wifi_kit_32_V3
framework = arduino
platform_packages = framework-arduinoespressif32 @ C:/Users/Baptou88/Documents/GitHub/arduino-esp32

[env:heltec_wifi_lora_32_V3]
platform = C:\Users\Baptou88\Documents\GitHub\platform-espressif32
board = heltec_wifi_lora_32_V3
framework = arduino
platform_packages = framework-arduinoespressif32 @ symlink://C:/Users/Baptou88/Documents/GitHub/arduino-esp32
  1. I've use the command 'Build ALL' with this result:
Successfully created esp32s3 image.
====================================================================== [SUCCESS] Took 10.41 seconds ======================================================================

Environment             Status    Duration
----------------------  --------  ------------
heltec_wifi_kit_32_V3   SUCCESS   00:00:12.342
heltec_wifi_lora_32_V3  SUCCESS   00:00:10.411
====================================================================== 2 succeeded in 00:00:22.754 ====================================================================== 
  1. I've upload a simple sketch =>it works
  2. I've created a data folder in my project with a file in it, then i've use the command Upload FileSystem image : Works
  3. I've used the commad "Erase Flash": Works

@P-R-O-C-H-Y
Copy link
Member

I'm testing the changes I made: here's what I did:

  1. I created a platformio project configured this way:
[env:heltec_wifi_kit_32_V3]
platform = C:\Users\Baptou88\Documents\GitHub\platform-espressif32
board = heltec_wifi_kit_32_V3
framework = arduino
platform_packages = framework-arduinoespressif32 @ C:/Users/Baptou88/Documents/GitHub/arduino-esp32

[env:heltec_wifi_lora_32_V3]
platform = C:\Users\Baptou88\Documents\GitHub\platform-espressif32
board = heltec_wifi_lora_32_V3
framework = arduino
platform_packages = framework-arduinoespressif32 @ symlink://C:/Users/Baptou88/Documents/GitHub/arduino-esp32
  1. I've use the command 'Build ALL' with this result:
Successfully created esp32s3 image.
====================================================================== [SUCCESS] Took 10.41 seconds ======================================================================

Environment             Status    Duration
----------------------  --------  ------------
heltec_wifi_kit_32_V3   SUCCESS   00:00:12.342
heltec_wifi_lora_32_V3  SUCCESS   00:00:10.411
====================================================================== 2 succeeded in 00:00:22.754 ====================================================================== 
  1. I've upload a simple sketch =>it works
  2. I've created a data folder in my project with a file in it, then i've use the command Upload FileSystem image : Works
  3. I've used the commad "Erase Flash": Works

@Baptou88 Can you commit the changes so I can review again and do test?

@P-R-O-C-H-Y P-R-O-C-H-Y added the Resolution: Awaiting response Waiting for response of author label Nov 9, 2022
static const uint8_t RST_OLED = 21;
static const uint8_t SCL_OLED = 18;
static const uint8_t SDA_OLED = 17;
#endif /* Pins_Arduino_h */
Copy link
Member

Choose a reason for hiding this comment

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

Add line after #endif /* Pins_Arduino_h */. So there will be empty line number 83.

boards.txt Show resolved Hide resolved
@P-R-O-C-H-Y
Copy link
Member

@Baptou88 Any updates?

@Baptou88
Copy link
Contributor Author

Baptou88 commented Dec 8, 2022

I did not update this PR. I don't have the skills to do it right.
to make this PR I based myself on the manufacturer's deposit (the directory is not updated and the data does not seem reliable)

I don't understand what modification I have to make for it to be valid

@P-R-O-C-H-Y P-R-O-C-H-Y merged commit e91efd4 into espressif:master Dec 21, 2022
@Baptou88
Copy link
Contributor Author

thank you for helping me @P-R-O-C-H-Y !
when this pr was approved, an update of this repository Heltec / wifi kit series was noticed after a very long absence resulting in some modifications, I will have to take the time to analyze the modifications to correct

@P-R-O-C-H-Y
Copy link
Member

No problem @Baptou88 :) feel free to open new PR with the modifications.

@dberlin
Copy link
Contributor

dberlin commented Jan 12, 2023

There are only two changes since the merge -
wireless stick v3 support was added (I'll try to get to this ni a bit)
The I2c pins on wifi_lora_32_v3 were changed.

I will create a PR for the latter right now since it's easy.

@dberlin
Copy link
Contributor

dberlin commented Jan 12, 2023

The I2C pin change is now #7709

I'll get to the wireless stick add in a day or two.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants