Skip to content

Document TFA Marbella pool thermometer - #1566

Merged
arendst merged 2 commits into
tasmota:masterfrom
schroeer-haren:tfa-marbella
Aug 31, 2026
Merged

arendst merged 2 commits into
tasmota:masterfrom
schroeer-haren:tfa-marbella

Conversation

@pschroee

@pschroee pschroee commented Aug 14, 2026 •

Copy link
Copy Markdown
Contributor

Documentation for the TFA Marbella driver proposed in arendst/Tasmota#24959.

Adds:

  • docs/TFA-Marbella.md - wiring, configuration, commands, tuning, sensor output
  • an entry in docs/Supported-Peripherals.md

The wiring table deliberately separates the shared SPI bus lines from the two roles that belong to the module (CC1101 CS, CC1101 GDO0), since that is the part that is easy to get wrong. It also mentions the antenna: the wire supplied with common CC1101 breakout boards is cut for 433 MHz, not 868 MHz.

The sensor output section documents LQI with an explicit warning that it runs the other way round than the LQI of Zigbee transceivers - 0 to 127 with low being good. It also states why no RSSI is reported: the value the chip appends to the packet is not reachable through RadioLib, and the register that is holds the noise floor by the time the frame is complete, so publishing it would show a plausible number that means nothing.

Updated after review of the driver PR. SetOption166 is gone; readings are published as they arrive and SetOption147 suppresses that, shared with the other receiving drivers. The SetOption166 row in docs/Commands.md is removed again.

Also documents the new Marbella <MHz> tuning command and adds a section explaining what it is for. A CC1101 derives its frequency from a 26 MHz crystal, and a cheap one lands the receiver beside the frequency it was told - measured on one module, 868.000 put it on 867.973. That still fits inside the 135 kHz receive filter but spends the margin the signal needs, and the symptom is losing frames while the sensor is demonstrably transmitting.

The TFA Dostmann Marbella transmits the water temperature on 868 MHz once
a minute; Tasmota receives it with a CC1101 and reports temperature, sensor
id, rolling counter, battery state and link quality.

The LQI needs the note that it runs the other way round than the one Zigbee
transceivers report - low is good here. The RSSI is deliberately absent: the
value the chip appends to the packet is not reachable through RadioLib, and
the register that is holds the noise floor once the frame is complete, which
would look like a reading and be none.
@pschroee
pschroee marked this pull request as draft August 14, 2026 22:25
@pschroee pschroee changed the title Document TFA Marbella pool thermometer and SetOption166 WIP: Document TFA Marbella pool thermometer and SetOption166 Aug 14, 2026
@pschroee pschroee changed the title WIP: Document TFA Marbella pool thermometer and SetOption166 Document TFA Marbella pool thermometer and SetOption166 Aug 15, 2026
@pschroee
pschroee marked this pull request as ready for review August 15, 2026 08:31
@pschroee
pschroee marked this pull request as draft August 18, 2026 12:10
Follows the driver change: SetOption166 is gone, readings are published as
they arrive and SetOption147 suppresses that, shared with the other
receiving drivers.

Adds the "Marbella <MHz>" tuning command and a section explaining what it is
for. A CC1101 derives its frequency from a 26 MHz crystal, and a cheap one
lands the receiver beside the frequency it was told - measured on one module,
868.000 put it on 867.973. That still fits inside the receive filter but
spends the margin the signal needs, and the symptom is losing frames while
the sensor is demonstrably transmitting.
@pschroee pschroee changed the title Document TFA Marbella pool thermometer and SetOption166 Document TFA Marbella pool thermometer Aug 24, 2026
@pschroee
pschroee marked this pull request as ready for review August 24, 2026 17:13
@arendst
arendst merged commit 44b071b into tasmota:master Aug 31, 2026
1 check passed
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