diff --git a/CHANGELOG.md b/CHANGELOG.md index 62f5e25e9..520c26d5f 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -11,7 +11,7 @@ - Games: Umpire Indicator (by @RocketGod-git) - Sub-GHz: - Show satellites count with an icon (#215 by @m7i-org) - - Add Bresser 3CH weather station protocol (#217 by @m7i-org) + - Add Bresser 3CH weather station protocol (#217 #245 by @m7i-org) - UL: Add Marantec24 protocol (static 24 bit) with add manually (by @xMasterX) - UL: Add GangQi protocol (static 34 bit) with button parsing and add manually (by @xMasterX & @Skorpionm) - UL: Add Hollarm protocol (static 42 bit) with button parsing and add manually (by @xMasterX & @Skorpionm) diff --git a/applications/debug/unit_tests/resources/unit_tests/subghz/bresser_3ch_v0.sub b/applications/debug/unit_tests/resources/unit_tests/subghz/bresser_3ch_v0.sub new file mode 100644 index 000000000..b381d6b28 --- /dev/null +++ b/applications/debug/unit_tests/resources/unit_tests/subghz/bresser_3ch_v0.sub @@ -0,0 +1,16 @@ +Filetype: Flipper SubGhz Key File +Version: 1 +Frequency: 433920000 +Preset: FuriHalSubGhzPresetOok270Async +Latitute: nan +Longitude: nan +Protocol: Bresser-3CH +Id: 117 +Bit: 36 +Data: 00 00 00 07 58 05 9F 46 +Batt: 0 +Hum: 70 +Ts: 1728202096 +Ch: 1 +Btn: 0 +Temp: 8.900000 diff --git a/applications/debug/unit_tests/resources/unit_tests/subghz/bresser_3ch_v0_raw.sub b/applications/debug/unit_tests/resources/unit_tests/subghz/bresser_3ch_v0_raw.sub new file mode 100644 index 000000000..5fc18e094 --- /dev/null +++ b/applications/debug/unit_tests/resources/unit_tests/subghz/bresser_3ch_v0_raw.sub @@ -0,0 +1,6 @@ +Filetype: Flipper SubGhz RAW File +Version: 1 +Frequency: 433920000 +Preset: FuriHalSubGhzPresetOok270Async +Protocol: RAW +RAW_Data: 520 -1000 470 -998 502 -1020 516 -992 482 -2002 482 -992 514 -1962 482 -1990 504 -984 488 -1020 478 -1988 536 -1982 478 -1968 488 -1982 500 -1976 478 -1014 488 -2010 448 -1018 484 -1022 512 -1004 476 -2002 482 -1980 488 -1014 530 -1008 480 -1014 464 -1000 512 -1012 476 -998 502 -1000 502 -986 504 -1012 512 -1002 470 -1030 472 -1020 478 -1014 478 -1000 504 -1016 482 -988 480 -1042 480 -4016 476 -1010 488 -1984 478 -1976 512 -1976 472 -1010 496 -1994 470 -1008 508 -1968 534 -1952 510 -1012 474 -1000 506 -984 518 -992 480 -1010 488 -1020 480 -1016 514 -1006 482 -1982 472 -1012 512 -1970 482 -1978 488 -1018 490 -1010 478 -1980 544 -1950 500 -1974 512 -1970 482 -1978 524 -982 490 -1966 516 -1004 490 -1008 510 -1012 490 -1968 480 -1998 482 -1030 486 -1012 480 -1016 472 -1030 482 -996 484 -1008 484 -1022 482 -1020 482 -1006 506 -984 480 -1016 474 -1004 504 -986 518 -986 476 -1000 480 -1026 480 -1026 482 -3986 530 -1018 476 -1994 480 -1982 456 -2008 478 -1002 500 -1988 476 -1010 476 -1998 514 -1964 484 -1002 482 -1014 474 -998 502 -996 500 -986 510 -982 510 -1006 504 -1016 468 -1972 510 -1010 458 -2006 476 -1966 484 -1020 476 -1006 482 -1990 502 -1976 494 -1996 476 -1964 518 -1948 520 -984 488 -1996 476 -1006 488 -1002 502 -1002 480 -1986 494 -1998 488 -1010 486 -1010 478 -1012 500 -1008 462 -1010 476 -1010 500 -1002 478 -1014 476 -1030 506 -1008 458 -1006 508 -976 508 -1002 468 -1014 482 -1020 478 -1004 500 -1028 470 -3998 554 -984 496 -1966 470 -2002 480 -1982 472 -1006 506 -1958 518 -986 476 -2000 514 -1970 484 -994 502 -1008 470 -1004 508 -1006 468 -996 502 -1014 470 -1002 478 -1012 502 -1974 476 -1004 504 -1976 486 -1964 486 -1002 514 -974 482 -2008 522 -1964 502 -1954 528 -2002 460 -1978 508 -1010 458 -2010 476 -1008 456 -1048 510 -984 476 -1990 482 -1984 468 -1028 500 -1026 468 -1018 466 -1008 504 -998 468 -1020 482 -1018 476 -1010 488 -1020 510 -986 476 -1002 480 -1026 468 -1014 502 -1006 472 -998 504 -982 484 -1016 512 -3980 492 -1008 474 -1994 482 -1982 502 -1970 478 -1016 488 -1966 480 -1028 482 -1986 486 -1988 490 -1004 474 -1012 520 -986 488 -1000 472 -1014 502 -982 508 -1012 506 -1006 480 -1980 504 -978 508 -1966 484 -1972 500 -994 512 -970 512 -1962 508 -2002 480 -1972 508 -1966 500 -1976 470 -998 502 -1960 508 -1010 472 -996 504 -1010 502 -1982 500 -1964 470 -1040 498 -984 510 -1010 486 -984 492 -1004 478 -1014 498 -984 504 -976 506 -998 512 -996 496 -1006 502 -974 506 -996 470 -1028 472 -1008 490 -1006 478 -2372 752 -720 734 -710 742 -752 710 -742 236 -470 diff --git a/applications/debug/unit_tests/tests/subghz/subghz_test.c b/applications/debug/unit_tests/tests/subghz/subghz_test.c index 752fcec2e..8f1471089 100644 --- a/applications/debug/unit_tests/tests/subghz/subghz_test.c +++ b/applications/debug/unit_tests/tests/subghz/subghz_test.c @@ -677,11 +677,18 @@ MU_TEST(subghz_decoder_solight_te44_test) { "Test decoder " WS_PROTOCOL_SOLIGHT_TE44_NAME " error\r\n"); } -MU_TEST(subghz_decoder_bresser_3ch_test) { +MU_TEST(subghz_decoder_bresser_3ch_v1_test) { mu_assert( subghz_decoder_test( EXT_PATH("unit_tests/subghz/bresser_3ch_raw.sub"), WS_PROTOCOL_BRESSER_3CH_NAME), - "Test decoder " WS_PROTOCOL_BRESSER_3CH_NAME " error\r\n"); + "Test decoder " WS_PROTOCOL_BRESSER_3CH_NAME " v1 error\r\n"); +} + +MU_TEST(subghz_decoder_bresser_3ch_v0_test) { + mu_assert( + subghz_decoder_test( + EXT_PATH("unit_tests/subghz/bresser_3ch_v0_raw.sub"), WS_PROTOCOL_BRESSER_3CH_NAME), + "Test decoder " WS_PROTOCOL_BRESSER_3CH_NAME " v0 error\r\n"); } //test encoders @@ -907,6 +914,8 @@ MU_TEST_SUITE(subghz) { MU_RUN_TEST(subghz_decoder_mastercode_test); MU_RUN_TEST(subghz_decoder_dickert_test); MU_RUN_TEST(subghz_decoder_solight_te44_test); + MU_RUN_TEST(subghz_decoder_bresser_3ch_v1_test); + MU_RUN_TEST(subghz_decoder_bresser_3ch_v0_test); MU_RUN_TEST(subghz_encoder_princeton_test); MU_RUN_TEST(subghz_encoder_came_test); diff --git a/lib/subghz/protocols/bresser_3ch.c b/lib/subghz/protocols/bresser_3ch.c index 70957081c..8fbc81501 100644 --- a/lib/subghz/protocols/bresser_3ch.c +++ b/lib/subghz/protocols/bresser_3ch.c @@ -3,11 +3,15 @@ #define TAG "WSProtocolBresser3ch" /* + * Bresser sensor protocol in versions V1 and V0 + * + * ---------------------------------------------------------------------------- + * Bresser-3CH V1 format + * ---------------------------------------------------------------------------- + * * Help: * https://github.com/merbanan/rtl_433/blob/master/src/devices/bresser_3ch.c * - * Bresser sensor protocol. - * * The protocol is for the wireless Temperature/Humidity sensor * - Bresser Thermo-/Hygro-Sensor 3CH * - also works for Renkforce DM-7511 @@ -30,15 +34,49 @@ * - humi is 8 bit relative humidity percentage * - chk is the sum of the four data bytes * + * ---------------------------------------------------------------------------- + * Bresser-3CH V0 format + * ---------------------------------------------------------------------------- + * + * Typically gets sent before V1: same weather station parameters, but in another format. + * Probably for older weather stations, as it seems simpler and without a CRC check. No rtl_433 driver yet. + * + * Encoding, PPM modulated with 0 and 1 swapped: + * - Data sync begins with 475 -3975 + * - A zero is encoded 475 -990 + * - A one is encoded 475 -1980 + * + * Very much deduced data structure: + * + * - 8 repetitions of the same 52 bit payload, a sync between each repetition + * - 52 bit payload format: IIII IIII CCBB TTTT TTTT TTTT 1111 HHHH HHHH 0000 0000 0000 0000 + * ^ ^ ^ ^ ^ ^ + * | | | | | | + * 8 bit sensor ID -/ | | | | | + * 2 bit channel, bat and btn / | | | | + * 12 bit signed int temp(C/10) ---/ / | | + * separator ------------------------------------/ / / + * 8 bit relative humidity, percentage ---------------/ / + * 16 bit zero tail to end (omitted in the saved data) --------/ + * * @m7i-org - because there's more stuff screaming in the ether than you might think * */ -static const SubGhzBlockConst ws_protocol_bresser_3ch_const = { +#define BRESSER_V0_DATA 36 +#define BRESSER_V0_DATA_AND_TAIL 52 +#define BRESSER_V1_DATA 40 + +static const SubGhzBlockConst ws_protocol_bresser_3ch_v0_const = { + .te_short = 475, + .te_long = 3900, + .te_delta = 150, +}; + +static const SubGhzBlockConst ws_protocol_bresser_3ch_v1_const = { .te_short = 250, .te_long = 500, .te_delta = 150, - .min_count_bit_for_found = 40, }; struct WSProtocolDecoderBresser3ch { @@ -92,11 +130,14 @@ const SubGhzProtocol ws_protocol_bresser_3ch = { typedef enum { Bresser3chDecoderStepReset = 0, - Bresser3chDecoderStepPreambleDn, - Bresser3chDecoderStepPreambleUp, - Bresser3chDecoderStepSaveDuration, - Bresser3chDecoderStepCheckDuration, -} Bresser3chDecoderStep; + Bresser3chDecoderStepV0SaveDuration, + Bresser3chDecoderStepV0CheckDuration, + Bresser3chDecoderStepV0TailCheckDuration, + Bresser3chDecoderStepV1PreambleDn, + Bresser3chDecoderStepV1PreambleUp, + Bresser3chDecoderStepV1SaveDuration, + Bresser3chDecoderStepV1CheckDuration, +} Bresser3chDecoderStepV1; void* ws_protocol_decoder_bresser_3ch_alloc(SubGhzEnvironment* environment) { UNUSED(environment); @@ -118,7 +159,27 @@ void ws_protocol_decoder_bresser_3ch_reset(void* context) { instance->decoder.parser_step = Bresser3chDecoderStepReset; } -static bool ws_protocol_bresser_3ch_check(WSProtocolDecoderBresser3ch* instance) { +static bool ws_protocol_bresser_3ch_check_v0(WSProtocolDecoderBresser3ch* instance) { + if(!instance->decoder.decode_data) return false; + + // No CRC, so better sanity checks here + + if(((instance->decoder.decode_data >> 8) & 0x0f) != 0x0f) return false; // separator not 0xf + if(((instance->decoder.decode_data >> 28) & 0xff) == 0xff) return false; // ID only ones? + if(((instance->decoder.decode_data >> 28) & 0xff) == 0x00) return false; // ID only zeroes? + if(((instance->decoder.decode_data >> 25) & 0x0f) == 0x0f) return false; // flags only ones? + if(((instance->decoder.decode_data >> 25) & 0x0f) == 0x00) return false; // flags only zeroes? + if(((instance->decoder.decode_data >> 12) & 0x0fff) == 0x0fff) + return false; // temperature maxed out? + if((instance->decoder.decode_data & 0xff) < 20) + return false; // humidity percentage less than 20? + if((instance->decoder.decode_data & 0xff) > 95) + return false; // humidity percentage more than 95? + + return true; +} + +static bool ws_protocol_bresser_3ch_check_v1(WSProtocolDecoderBresser3ch* instance) { if(!instance->decoder.decode_data) return false; uint8_t sum = (((instance->decoder.decode_data >> 32) & 0xff) + @@ -131,10 +192,30 @@ static bool ws_protocol_bresser_3ch_check(WSProtocolDecoderBresser3ch* instance) } /** - * Analysis of received data + * Analysis of received data for V0 * @param instance Pointer to a WSBlockGeneric* instance */ -static void ws_protocol_bresser_3ch_extract_data(WSBlockGeneric* instance) { +static void ws_protocol_bresser_3ch_extract_data_v0(WSBlockGeneric* instance) { + instance->id = (instance->data >> 28) & 0xff; + instance->channel = ((instance->data >> 27) & 0x01) | (((instance->data >> 26) & 0x01) << 1); + instance->btn = ((instance->data >> 25) & 0x1); + instance->battery_low = ((instance->data >> 24) & 0x1); + + int16_t temp = (instance->data >> 12) & 0x0fff; + /* Handle signed data */ + if(temp & 0x0800) { + temp |= 0xf000; + } + instance->temp = (float)temp / 10.0; + + instance->humidity = instance->data & 0xff; +} + +/** + * Analysis of received data for V1 + * @param instance Pointer to a WSBlockGeneric* instance + */ +static void ws_protocol_bresser_3ch_extract_data_v1(WSBlockGeneric* instance) { instance->id = (instance->data >> 32) & 0xff; instance->battery_low = ((instance->data >> 31) & 0x1); instance->btn = (instance->data >> 30) & 0x1; @@ -152,48 +233,117 @@ void ws_protocol_decoder_bresser_3ch_feed(void* context, bool level, uint32_t du switch(instance->decoder.parser_step) { case Bresser3chDecoderStepReset: - if(level && DURATION_DIFF(duration, ws_protocol_bresser_3ch_const.te_short * 3) < - ws_protocol_bresser_3ch_const.te_delta) { + if(level && DURATION_DIFF(duration, ws_protocol_bresser_3ch_v1_const.te_short * 3) < + ws_protocol_bresser_3ch_v1_const.te_delta) { instance->decoder.te_last = duration; - instance->decoder.parser_step = Bresser3chDecoderStepPreambleDn; + instance->decoder.parser_step = Bresser3chDecoderStepV1PreambleDn; + instance->decoder.decode_data = 0; + instance->decoder.decode_count_bit = 0; + } else if((!level) && duration >= ws_protocol_bresser_3ch_v0_const.te_long) { + instance->decoder.parser_step = Bresser3chDecoderStepV0SaveDuration; instance->decoder.decode_data = 0; instance->decoder.decode_count_bit = 0; } break; - case Bresser3chDecoderStepPreambleDn: - if((!level) && DURATION_DIFF(duration, ws_protocol_bresser_3ch_const.te_short * 3) < - ws_protocol_bresser_3ch_const.te_delta) { - if(DURATION_DIFF( - instance->decoder.te_last, ws_protocol_bresser_3ch_const.te_short * 12) < - ws_protocol_bresser_3ch_const.te_delta * 2) { - // End of sync after 4*750 (12*250) high values, start reading the message - instance->decoder.parser_step = Bresser3chDecoderStepSaveDuration; + case Bresser3chDecoderStepV0SaveDuration: + if(level) { + instance->decoder.te_last = duration; + if(instance->decoder.decode_count_bit < BRESSER_V0_DATA) { + instance->decoder.parser_step = Bresser3chDecoderStepV0CheckDuration; } else { - instance->decoder.parser_step = Bresser3chDecoderStepPreambleUp; + instance->decoder.parser_step = Bresser3chDecoderStepV0TailCheckDuration; } } else { instance->decoder.parser_step = Bresser3chDecoderStepReset; } break; - case Bresser3chDecoderStepPreambleUp: - if(level && DURATION_DIFF(duration, ws_protocol_bresser_3ch_const.te_short * 3) < - ws_protocol_bresser_3ch_const.te_delta) { - instance->decoder.te_last = instance->decoder.te_last + duration; - instance->decoder.parser_step = Bresser3chDecoderStepPreambleDn; + case Bresser3chDecoderStepV0CheckDuration: + if(!level) { + if(DURATION_DIFF(instance->decoder.te_last, ws_protocol_bresser_3ch_v0_const.te_short) < + ws_protocol_bresser_3ch_v0_const.te_delta) { + if(DURATION_DIFF(duration, ws_protocol_bresser_3ch_v0_const.te_short * 2) < + ws_protocol_bresser_3ch_v0_const.te_delta) { + subghz_protocol_blocks_add_bit(&instance->decoder, 0); + instance->decoder.parser_step = Bresser3chDecoderStepV0SaveDuration; + } else if( + DURATION_DIFF(duration, ws_protocol_bresser_3ch_v0_const.te_short * 4) < + ws_protocol_bresser_3ch_v0_const.te_delta) { + subghz_protocol_blocks_add_bit(&instance->decoder, 1); + instance->decoder.parser_step = Bresser3chDecoderStepV0SaveDuration; + } else + instance->decoder.parser_step = Bresser3chDecoderStepReset; + } else + instance->decoder.parser_step = Bresser3chDecoderStepReset; + } else + instance->decoder.parser_step = Bresser3chDecoderStepReset; + break; + + case Bresser3chDecoderStepV0TailCheckDuration: + if(!level) { + if(duration >= ws_protocol_bresser_3ch_v0_const.te_long) { + if(instance->decoder.decode_count_bit == BRESSER_V0_DATA_AND_TAIL && + ws_protocol_bresser_3ch_check_v0(instance)) { + instance->generic.data = instance->decoder.decode_data; + instance->decoder.decode_count_bit = BRESSER_V0_DATA; + instance->generic.data_count_bit = instance->decoder.decode_count_bit; + ws_protocol_bresser_3ch_extract_data_v0(&instance->generic); + + if(instance->base.callback) { + instance->base.callback(&instance->base, instance->base.context); + } + } + instance->decoder.decode_data = 0; + instance->decoder.decode_count_bit = 0; + instance->decoder.parser_step = Bresser3chDecoderStepReset; + } else if( + instance->decoder.decode_count_bit < BRESSER_V0_DATA_AND_TAIL && + DURATION_DIFF( + instance->decoder.te_last, ws_protocol_bresser_3ch_v0_const.te_short) < + ws_protocol_bresser_3ch_v0_const.te_delta && + DURATION_DIFF(duration, ws_protocol_bresser_3ch_v0_const.te_short * 2) < + ws_protocol_bresser_3ch_v0_const.te_delta) { + instance->decoder.decode_count_bit++; + instance->decoder.parser_step = Bresser3chDecoderStepV0SaveDuration; + } else + instance->decoder.parser_step = Bresser3chDecoderStepReset; + } else + instance->decoder.parser_step = Bresser3chDecoderStepReset; + break; + + case Bresser3chDecoderStepV1PreambleDn: + if((!level) && DURATION_DIFF(duration, ws_protocol_bresser_3ch_v1_const.te_short * 3) < + ws_protocol_bresser_3ch_v1_const.te_delta) { + if(DURATION_DIFF( + instance->decoder.te_last, ws_protocol_bresser_3ch_v1_const.te_short * 12) < + ws_protocol_bresser_3ch_v1_const.te_delta * 2) { + // End of sync after 4*750 (12*250) high values, start reading the message + instance->decoder.parser_step = Bresser3chDecoderStepV1SaveDuration; + } else { + instance->decoder.parser_step = Bresser3chDecoderStepV1PreambleUp; + } } else { instance->decoder.parser_step = Bresser3chDecoderStepReset; } break; - case Bresser3chDecoderStepSaveDuration: - if(instance->decoder.decode_count_bit == - ws_protocol_bresser_3ch_const.min_count_bit_for_found) { - if(ws_protocol_bresser_3ch_check(instance)) { + case Bresser3chDecoderStepV1PreambleUp: + if(level && DURATION_DIFF(duration, ws_protocol_bresser_3ch_v1_const.te_short * 3) < + ws_protocol_bresser_3ch_v1_const.te_delta) { + instance->decoder.te_last = instance->decoder.te_last + duration; + instance->decoder.parser_step = Bresser3chDecoderStepV1PreambleDn; + } else { + instance->decoder.parser_step = Bresser3chDecoderStepReset; + } + break; + + case Bresser3chDecoderStepV1SaveDuration: + if(instance->decoder.decode_count_bit == BRESSER_V1_DATA) { + if(ws_protocol_bresser_3ch_check_v1(instance)) { instance->generic.data = instance->decoder.decode_data; instance->generic.data_count_bit = instance->decoder.decode_count_bit; - ws_protocol_bresser_3ch_extract_data(&instance->generic); + ws_protocol_bresser_3ch_extract_data_v1(&instance->generic); if(instance->base.callback) { instance->base.callback(&instance->base, instance->base.context); @@ -204,27 +354,27 @@ void ws_protocol_decoder_bresser_3ch_feed(void* context, bool level, uint32_t du instance->decoder.parser_step = Bresser3chDecoderStepReset; } else if(level) { instance->decoder.te_last = duration; - instance->decoder.parser_step = Bresser3chDecoderStepCheckDuration; + instance->decoder.parser_step = Bresser3chDecoderStepV1CheckDuration; } else { instance->decoder.parser_step = Bresser3chDecoderStepReset; } break; - case Bresser3chDecoderStepCheckDuration: + case Bresser3chDecoderStepV1CheckDuration: if(!level) { - if(DURATION_DIFF(instance->decoder.te_last, ws_protocol_bresser_3ch_const.te_short) < - ws_protocol_bresser_3ch_const.te_delta && - DURATION_DIFF(duration, ws_protocol_bresser_3ch_const.te_long) < - ws_protocol_bresser_3ch_const.te_delta) { + if(DURATION_DIFF(instance->decoder.te_last, ws_protocol_bresser_3ch_v1_const.te_short) < + ws_protocol_bresser_3ch_v1_const.te_delta && + DURATION_DIFF(duration, ws_protocol_bresser_3ch_v1_const.te_long) < + ws_protocol_bresser_3ch_v1_const.te_delta) { subghz_protocol_blocks_add_bit(&instance->decoder, 0); - instance->decoder.parser_step = Bresser3chDecoderStepSaveDuration; + instance->decoder.parser_step = Bresser3chDecoderStepV1SaveDuration; } else if( - DURATION_DIFF(instance->decoder.te_last, ws_protocol_bresser_3ch_const.te_long) < - ws_protocol_bresser_3ch_const.te_delta && - DURATION_DIFF(duration, ws_protocol_bresser_3ch_const.te_short) < - ws_protocol_bresser_3ch_const.te_delta) { + DURATION_DIFF(instance->decoder.te_last, ws_protocol_bresser_3ch_v1_const.te_long) < + ws_protocol_bresser_3ch_v1_const.te_delta && + DURATION_DIFF(duration, ws_protocol_bresser_3ch_v1_const.te_short) < + ws_protocol_bresser_3ch_v1_const.te_delta) { subghz_protocol_blocks_add_bit(&instance->decoder, 1); - instance->decoder.parser_step = Bresser3chDecoderStepSaveDuration; + instance->decoder.parser_step = Bresser3chDecoderStepV1SaveDuration; } else instance->decoder.parser_step = Bresser3chDecoderStepReset; } else @@ -253,8 +403,18 @@ SubGhzProtocolStatus ws_protocol_decoder_bresser_3ch_deserialize(void* context, FlipperFormat* flipper_format) { furi_assert(context); WSProtocolDecoderBresser3ch* instance = context; - return ws_block_generic_deserialize_check_count_bit( - &instance->generic, flipper_format, ws_protocol_bresser_3ch_const.min_count_bit_for_found); + + SubGhzProtocolStatus ret = ws_block_generic_deserialize(&instance->generic, flipper_format); + + if(ret == SubGhzProtocolStatusOk) { + if(instance->generic.data_count_bit != BRESSER_V0_DATA && + instance->generic.data_count_bit != BRESSER_V1_DATA) { + FURI_LOG_D(TAG, "Wrong number of bits in key for Bresser-3CH V0 or V1 packet format"); + ret = SubGhzProtocolStatusErrorValueBitCount; + } + } + + return ret; } void ws_protocol_decoder_bresser_3ch_get_string(void* context, FuriString* output) {