Skip to content

Fix: Lillygo Tbeam 1W Fan Control and LNA Sequencing improvements - #2418

Closed
Robowarrior834 wants to merge 76 commits into
meshcore-dev:devfrom
Robowarrior834:Tbeam1W-Fix
Closed

Fix: Lillygo Tbeam 1W Fan Control and LNA Sequencing improvements #2418
Robowarrior834 wants to merge 76 commits into
meshcore-dev:devfrom
Robowarrior834:Tbeam1W-Fix

Conversation

@Robowarrior834

Copy link
Copy Markdown

Ensures proper sequencing of the LNA and amp for the TBEAM 1W. Also adds fan control to keep the fan on for 10 seconds.

@Robowarrior834 Robowarrior834 changed the title Tbeam1W Fix Fix: Lillygo Tbeam 1W Fan Control and LNA Sequencing improvements Apr 27, 2026
@Robowarrior834

Copy link
Copy Markdown
Author

Had to rebuild the repository as I accidently added other local changes I made on my machine to the old pull request

#ifdef DISPLAY_CLASS
ui_task.loop();
#endif
rtc_clock.tick();

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is probably time to introduce a new virtual MainBoard::loop(), and a no-op as default.
And have the TBeam1WBoard class override, and put the updateFan() logic there

setFanEnabled(true);
_fan_off_millis = millis() + 10000;
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ie. here make this the board's loop() method

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

So I did this. Would I also need to add a loop function to every variant?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I answered my own question. Added a new virtual loop to the mainboard class so other variants compile and adjusted the tbeam 1w to override.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Robowarrior834 and others added 4 commits April 28, 2026 08:42
@r0ck40k

r0ck40k commented Jun 6, 2026

Copy link
Copy Markdown

I basically did the same. Which seems to improve reliability on this board a lot. While changing this i wanted to ask if it makes sense to allow adjusting the PA ramp time dynamically?
I read the Lilygo documentation and for this board they prefer >= 800us for PA ramp time.
image

Maybe it makes sense to add something like:

  1. The build flag (definition) — variants/lilygo_tbeam_1w/platformio.ini
    -D SX126X_PA_RAMP=0x05

  2. Initial application at radio init — variants/lilygo_tbeam_1w/target.cpp
    radio.setTxParams(LORA_TX_POWER, SX126X_PA_RAMP);

  3. Re-application after every TX power change — src/helpers/radiolib/CustomSX1262Wrapper
    void setTxPower(int8_t dbm) override {
    ((CustomSX1262 *)_radio)->setOutputPower(dbm);
    #ifdef SX126X_PA_RAMP
    ((CustomSX1262 *)_radio)->setPaRampTime(SX126X_PA_RAMP);
    #endif
    }

This is at least my understanding. Please correct me if i am wrong.

Dom4n and others added 26 commits July 17, 2026 15:21
…tion is now performed right after radio module start.
…rse-error

fix: JSON parsing regression in build.sh
…esh Solar and Heltec Tracker.

Not supported: Mesh Adventurer (unable to off GPS power) and ThinkNode M1 (manual GPS switch).
CAD on a still-running SX126x duty-cycle sequencer can lose CAD-done IRQ when the RTC knocks the chip back to standby, leaving scanChannel() waiting forever.
…p-privacy

Clarify hashtag privacy and group sender identity
Added RX duty-cycle power saving support
@Robowarrior834

Copy link
Copy Markdown
Author

to out of date to try and merge

@Robowarrior834
Robowarrior834 deleted the Tbeam1W-Fix branch August 5, 2026 15:06
@Robowarrior834
Robowarrior834 restored the Tbeam1W-Fix branch August 5, 2026 15:07
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.