-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 reading firmware version from WS90 #2517
Conversation
Added support to read firmware version from the WS90.
src/devices/fineoffset_ws90.c
Outdated
@@ -32,7 +32,7 @@ Preamble is aaaa aaaa aaaa, sync word is 2dd4. | |||
|
|||
Packet layout: | |||
|
|||
0 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 | |||
0 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 ZZ 30 31 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be one line below, right? "29" and below "ZZ"
src/devices/fineoffset_ws90.c
Outdated
@@ -131,6 +133,7 @@ static int fineoffset_ws90_decode(r_device *decoder, bitbuffer_t *bitbuffer) | |||
"unknown", "Unknown", DATA_COND, unknown != 0x3fff, DATA_INT, unknown, | |||
"rain_mm", "Total Rain", DATA_FORMAT, "%.1f mm", DATA_DOUBLE, rain_raw * 0.1f, | |||
"supercap_V", "Supercap Voltage", DATA_COND, supercap_V != 0xff, DATA_FORMAT, "%.1f V", DATA_DOUBLE, supercap_V * 0.1f, | |||
"firmware", "Firmware Version", DATA_FORMAT, "%d", DATA_INT, firmware, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Plain "%d" needs no format. Just ...Version", DATA_INT, ...
Fixed typos.
Fixed firmware type/format.
Looks great! Thanks! |
Some day I will get it correct the first time! ;-) |
Not needed ;) Nobody so far found that version field -- and you got that right for the first time :) |
This got garbled in the commit which added FW version discovery. Fixes: c94ee77 Change Fineoffset-WS90 to add firmware version (merbanan#2517)
Added support to read firmware version from the WS90.