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

Feature/support and hint for ttgo v2 board #944

Conversation

sauttefk
Copy link
Contributor

No description provided.

@@ -126,6 +127,11 @@ upload_protocol = custom

[env:usb]
upload_protocol = esptool
upload_speed = 921600
Copy link
Owner

Choose a reason for hiding this comment

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

This setting is global, for all type boards. Thus, we can't have this here, because some older boards do not support high upload speed. Please remove.

@@ -126,6 +127,11 @@ upload_protocol = custom

[env:usb]
upload_protocol = esptool
upload_speed = 921600
monitor_speed = 115200
Copy link
Owner

Choose a reason for hiding this comment

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

why did you move this from section common to here?

@@ -375,7 +375,7 @@ void dp_refresh(bool nextPage) {
dp_clear();
break;
#else // skip this page
DisplayPage++;
DisplayPage = 0;
Copy link
Owner

Choose a reason for hiding this comment

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

This seems like a hack aorund a bug to me. I think the bug is in display.h:
#define DISPLAY_PAGES (7) // number of paxcounter display pages Please try with (6). That should work.

@@ -1,6 +1,7 @@
; ---> SELECT ONE TARGET BOARD FROM FOLLOWING ROWS <---
;
; Note: For "LILYGO TTGO ESP32-Paxcounter LoRa32 V2.1 1.6.1" select board ttgov21new.h
; Note: For "LILYGO TTGO ESP32-Paxcounter LoRa32 V2.0 1.6.0" select board ttgov2.h and add bodge wire from LORA_IO1 to GPIO33
Copy link
Owner

Choose a reason for hiding this comment

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

Please move this hint to ttgov2.h, since it's board specific.

@cyberman54 cyberman54 merged commit c0082eb into cyberman54:master Apr 28, 2023
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.

2 participants