Merge branch 'DarkFlippers:dev' into 420darkupds

CHANGELOG.md

@@ -1,37 +1,7 @@
 ### New changes
-* Issues with hopping in subghz are fixed in this release
+* SubGHz: Fixed bugs with Decode option for RAW signals (blank screen, broken files)
-* OFW PR: Desktop: Set external apps as favorites (OFW PR 1816 by @djsime1) (and fixed forgotten furi_record_close)
+* SubGHz: Allowed usage of hopper with detect raw feature (with attempt to fix) - WARNING -> This feature is very unstable, may cause crashes(with lost of captured signal), use at your own risk, it was enabled due to many requests from users
-* PR -> Plugins: Add CAME 12bit 303MHz to SubGHz Bruteforcer (by @derskythe | PR #87)
+* PR -> Docs, fix description - Slight update for newest dev method (by @UberGuidoZ | PR #93)
-* PR -> BadUSB: Added Norwegian keyboard layout for BadUSB (by @jd-raymaker | PR #88)
-* PR -> Plugins: Feature: allow to set ball speed in Arkanoid (by @an4tur0r | PR #92)
-* Add USB Keyboard (& Mouse) plugin, replacing default USB Mouse demo included (plugin by @huuck) [Link to original](https://github.com/huuck/FlipperZeroUSBKeyboard)
-* Fix USB Keyboard plugin wrong icon in mouse screen, rewrite view models to new type
-* Updated universal remote assets (by @Amec0e)
-* Plugins: SubGHz Bruteforcer - Fix wrong max value in BF existing dump
-* API 3.0 -> 3.2 (all previous compiled apps still compatible)
-* Add 312.2 MHz to subghz user config
-* SubGHz: Fix double click after delete scene, fix rename bug
-* SubGHz: proper free of rainbow tables
-* SubGHz: Fixed stability issues with Came atomo, Nice Flor S, limited max history items to 60 (was 65)
-* SubGHz: Fix Read screen GUI (still bugged in OFW)
-* Adapted all plugins and other code to new FuriString, fixed all new issues with new string type
-* Adapted all plugins to new printf format
-* Adapted all plugins to new view model format (aka Removing lambdas)
-* Adapted all plugins to new furi_stream
-* OFW: Elf loader: do not load .ARM.* sections
-* OFW: Removing lambdas
-* OFW: BadUSB: add SYSRQ keys
-* OFW: Gui: fix memory leak in file browser module 
-* OFW: music_player: Return to browser instead of exiting on back button
-* OFW: More correct elf loader
-* OFW: Furi stream buffer
-* OFW: Printf function attributes
-* OFW: App name in CLI loader command, RFID data edit fix 
-* OFW: Show in-app icons & names in archive browser
-* OFW: M*LIB: non-inlined strings, FuriString primitive
-* OFW: Remove string_push_uint64
-* OFW: Mifare Classic read improvements
-* OFW PR: updated icon names (OFW PR 1829 by nminaylov)
 
 #### [🎲 Download extra apps pack](https://download-directory.github.io/?url=https://github.com/UberGuidoZ/Flipper/tree/main/Applications/Unleashed)
 

applications/debug/uart_echo/application.fam

@@ -7,6 +7,6 @@ App(
     requires=["gui"],
     stack_size=2 * 1024,
     order=70,
-    fap_icon="../../../assets/icons/Archive/keyboard_10px.png",
+    fap_icon="uart_10px.png",
     fap_category="Misc",
 )

applications/main/subghz/scenes/subghz_scene_decode_raw.c

@@ -22,15 +22,6 @@
 // [X] Find good value for SAMPLES_TO_READ_PER_TICK
 // [X] Fix: read errors (slow flash) after aborting decode read
 
-typedef enum {
-    SubGhzDecodeRawStateStart,
-    SubGhzDecodeRawStateLoading,
-    SubGhzDecodeRawStateLoaded,
-} SubGhzDecodeRawState;
-
-SubGhzDecodeRawState decode_raw_state = SubGhzDecodeRawStateStart;
-SubGhzFileEncoderWorker* file_worker_encoder;
-
 static void subghz_scene_receiver_update_statusbar(void* context) {
     SubGhz* subghz = context;
     FuriString* history_stat_str;
@@ -114,8 +105,9 @@ bool subghz_scene_decode_raw_start(SubGhz* subghz) {
     if(success) {
         //FURI_LOG_I(TAG, "Listening at \033[0;33m%s\033[0m.", furi_string_get_cstr(file_name));
 
-        file_worker_encoder = subghz_file_encoder_worker_alloc();
+        subghz->decode_raw_file_worker_encoder = subghz_file_encoder_worker_alloc();
-        if(subghz_file_encoder_worker_start(file_worker_encoder, furi_string_get_cstr(file_name))) {
+        if(subghz_file_encoder_worker_start(
+               subghz->decode_raw_file_worker_encoder, furi_string_get_cstr(file_name))) {
             //the worker needs a file in order to open and read part of the file
             furi_delay_ms(100);
         } else {
@@ -123,7 +115,7 @@ bool subghz_scene_decode_raw_start(SubGhz* subghz) {
         }
 
         if(!success) {
-            subghz_file_encoder_worker_free(file_worker_encoder);
+            subghz_file_encoder_worker_free(subghz->decode_raw_file_worker_encoder);
         }
     }
 
@@ -135,13 +127,14 @@ bool subghz_scene_decode_raw_next(SubGhz* subghz) {
     LevelDuration level_duration;
 
     for(uint32_t read = SAMPLES_TO_READ_PER_TICK; read > 0; --read) {
-        level_duration = subghz_file_encoder_worker_get_level_duration(file_worker_encoder);
+        level_duration =
+            subghz_file_encoder_worker_get_level_duration(subghz->decode_raw_file_worker_encoder);
         if(!level_duration_is_reset(level_duration)) {
             bool level = level_duration_get_level(level_duration);
             uint32_t duration = level_duration_get_duration(level_duration);
             subghz_receiver_decode(subghz->txrx->receiver, level, duration);
         } else {
-            decode_raw_state = SubGhzDecodeRawStateLoaded;
+            subghz->decode_raw_state = SubGhzDecodeRawStateLoaded;
             subghz->state_notifications = SubGhzNotificationStateIDLE;
 
             subghz_view_receiver_add_data_progress(subghz->subghz_receiver, "100%");
@@ -152,7 +145,8 @@ bool subghz_scene_decode_raw_next(SubGhz* subghz) {
     // Update progress info
     FuriString* progress_str;
     progress_str = furi_string_alloc();
-    subghz_file_encoder_worker_get_text_progress(file_worker_encoder, progress_str);
+    subghz_file_encoder_worker_get_text_progress(
+        subghz->decode_raw_file_worker_encoder, progress_str);
 
     subghz_view_receiver_add_data_progress(
         subghz->subghz_receiver, furi_string_get_cstr(progress_str));
@@ -183,11 +177,11 @@ void subghz_scene_decode_raw_on_enter(void* context) {
         false);
     subghz_receiver_set_filter(subghz->txrx->receiver, SubGhzProtocolFlag_Decodable);
 
-    if(decode_raw_state == SubGhzDecodeRawStateStart) {
+    if(subghz->decode_raw_state == SubGhzDecodeRawStateStart) {
         //Decode RAW to history
         subghz_history_reset(subghz->txrx->history);
         if(subghz_scene_decode_raw_start(subghz)) {
-            decode_raw_state = SubGhzDecodeRawStateLoading;
+            subghz->decode_raw_state = SubGhzDecodeRawStateLoading;
             subghz->state_notifications = SubGhzNotificationStateRx;
         }
     } else {
@@ -216,14 +210,14 @@ bool subghz_scene_decode_raw_on_event(void* context, SceneManagerEvent event) {
     if(event.type == SceneManagerEventTypeCustom) {
         switch(event.event) {
         case SubGhzCustomEventViewReceiverBack:
-            decode_raw_state = SubGhzDecodeRawStateStart;
+            subghz->decode_raw_state = SubGhzDecodeRawStateStart;
             subghz->txrx->idx_menu_chosen = 0;
             subghz_receiver_set_rx_callback(subghz->txrx->receiver, NULL, subghz);
 
-            if(subghz_file_encoder_worker_is_running(file_worker_encoder)) {
+            if(subghz_file_encoder_worker_is_running(subghz->decode_raw_file_worker_encoder)) {
-                subghz_file_encoder_worker_stop(file_worker_encoder);
+                subghz_file_encoder_worker_stop(subghz->decode_raw_file_worker_encoder);
             }
-            subghz_file_encoder_worker_free(file_worker_encoder);
+            subghz_file_encoder_worker_free(subghz->decode_raw_file_worker_encoder);
 
             subghz->state_notifications = SubGhzNotificationStateIDLE;
             scene_manager_set_scene_state(
@@ -268,7 +262,7 @@ bool subghz_scene_decode_raw_on_event(void* context, SceneManagerEvent event) {
             break;
         }
 
-        switch(decode_raw_state) {
+        switch(subghz->decode_raw_state) {
         case SubGhzDecodeRawStateLoading:
             subghz_scene_decode_raw_next(subghz);
             break;

applications/main/subghz/scenes/subghz_scene_receiver.c

@@ -167,6 +167,7 @@ bool subghz_scene_receiver_on_event(void* context, SceneManagerEvent event) {
                 subghz_sleep(subghz);
             }
             subghz->txrx->hopper_state = SubGhzHopperStateOFF;
+            subghz_history_set_hopper_state(subghz->txrx->history, false);
             subghz->txrx->idx_menu_chosen = 0;
             subghz_receiver_set_rx_callback(subghz->txrx->receiver, NULL, subghz);
 

applications/main/subghz/scenes/subghz_scene_receiver_config.c

@@ -189,7 +189,7 @@ static void subghz_scene_receiver_config_set_detect_raw(VariableItem* item) {
     SubGhz* subghz = variable_item_get_context(item);
     uint8_t index = variable_item_get_current_value_index(item);
 
-    if(subghz->txrx->hopper_state == 0) {
+    //if(subghz->txrx->hopper_state == 0) {
     variable_item_set_current_value_text(item, detect_raw_text[index]);
 #ifdef SUBGHZ_SAVE_DETECT_RAW_SETTING
     subghz->last_settings->detect_raw = index;
@@ -203,16 +203,16 @@ static void subghz_scene_receiver_config_set_detect_raw(VariableItem* item) {
             subghz->txrx->receiver, SUBGHZ_PROTOCOL_RAW_NAME),
         (index == 1));
 #endif
-    } else {
+    /*} else {
         variable_item_set_current_value_index(item, 0);
-    }
+    }*/
 }
 
 static void subghz_scene_receiver_config_set_hopping_running(VariableItem* item) {
     SubGhz* subghz = variable_item_get_context(item);
     uint8_t index = variable_item_get_current_value_index(item);
 
-    if(subghz_receiver_get_filter(subghz->txrx->receiver) == SubGhzProtocolFlag_Decodable) {
+    //if(subghz_receiver_get_filter(subghz->txrx->receiver) == SubGhzProtocolFlag_Decodable) {
     variable_item_set_current_value_text(item, hopping_text[index]);
     if(hopping_value[index] == SubGhzHopperStateOFF) {
         char text_buf[10] = {0};
@@ -226,8 +226,7 @@ static void subghz_scene_receiver_config_set_hopping_running(VariableItem* item)
             (VariableItem*)scene_manager_get_scene_state(
                 subghz->scene_manager, SubGhzSceneReceiverConfig),
             text_buf);
-            subghz->txrx->preset->frequency =
+        subghz->txrx->preset->frequency = subghz_setting_get_default_frequency(subghz->setting);
-                subghz_setting_get_default_frequency(subghz->setting);
         variable_item_set_current_value_index(
             (VariableItem*)scene_manager_get_scene_state(
                 subghz->scene_manager, SubGhzSceneReceiverConfig),
@@ -244,9 +243,10 @@ static void subghz_scene_receiver_config_set_hopping_running(VariableItem* item)
     }
 
     subghz->txrx->hopper_state = hopping_value[index];
-    } else {
+    subghz_history_set_hopper_state(subghz->txrx->history, (index == 1));
+    /*} else {
         variable_item_set_current_value_index(item, 0);
-    }
+    }*/
 }
 
 static void subghz_scene_receiver_config_var_list_enter_callback(void* context, uint32_t index) {

applications/main/subghz/scenes/subghz_scene_receiver_info.c

@@ -186,6 +186,10 @@ bool subghz_scene_receiver_info_on_event(void* context, SceneManagerEvent event)
             if((subghz->txrx->decoder_result->protocol->flag & SubGhzProtocolFlag_Save) ==
                SubGhzProtocolFlag_Save) {
                 subghz_file_name_clear(subghz);
+
+                if(subghz->in_decoder_scene) {
+                    subghz->in_decoder_scene_skip = true;
+                }
                 scene_manager_next_scene(subghz->scene_manager, SubGhzSceneSaveName);
             }
             return true;
@@ -214,7 +218,7 @@ bool subghz_scene_receiver_info_on_event(void* context, SceneManagerEvent event)
 
 void subghz_scene_receiver_info_on_exit(void* context) {
     SubGhz* subghz = context;
-    if(subghz->in_decoder_scene) {
+    if(subghz->in_decoder_scene && !subghz->in_decoder_scene_skip) {
         subghz->in_decoder_scene = false;
     }
     widget_reset(subghz->widget);

applications/main/subghz/scenes/subghz_scene_save_success.c

@@ -27,6 +27,7 @@ bool subghz_scene_save_success_on_event(void* context, SceneManagerEvent event)
     SubGhz* subghz = context;
     if(event.type == SceneManagerEventTypeCustom) {
         if(event.event == SubGhzCustomEventSceneSaveSuccess) {
+            if(!subghz->in_decoder_scene) {
                 if(!scene_manager_search_and_switch_to_previous_scene(
                        subghz->scene_manager, SubGhzSceneReceiver)) {
                     subghz->txrx->rx_key_state = SubGhzRxKeyStateRAWSave;
@@ -39,6 +40,24 @@ bool subghz_scene_save_success_on_event(void* context, SceneManagerEvent event)
                         }
                     }
                 }
+            } else {
+                subghz->decode_raw_state = SubGhzDecodeRawStateStart;
+                subghz->txrx->idx_menu_chosen = 0;
+                subghz_receiver_set_rx_callback(subghz->txrx->receiver, NULL, subghz);
+
+                if(subghz_file_encoder_worker_is_running(subghz->decode_raw_file_worker_encoder)) {
+                    subghz_file_encoder_worker_stop(subghz->decode_raw_file_worker_encoder);
+                }
+                subghz_file_encoder_worker_free(subghz->decode_raw_file_worker_encoder);
+
+                subghz->state_notifications = SubGhzNotificationStateIDLE;
+                scene_manager_set_scene_state(
+                    subghz->scene_manager, SubGhzSceneReadRAW, SubGhzCustomEventManagerNoSet);
+                if(!scene_manager_search_and_switch_to_previous_scene(
+                       subghz->scene_manager, SubGhzSceneSaved)) {
+                    scene_manager_next_scene(subghz->scene_manager, SubGhzSceneSaved);
+                }
+            }
             return true;
         }
     }
@@ -48,6 +67,11 @@ bool subghz_scene_save_success_on_event(void* context, SceneManagerEvent event)
 void subghz_scene_save_success_on_exit(void* context) {
     SubGhz* subghz = context;
 
+    if(subghz->in_decoder_scene) {
+        subghz->in_decoder_scene = false;
+        subghz->in_decoder_scene_skip = false;
+    }
+
     // Clear view
     Popup* popup = subghz->popup;
     popup_set_header(popup, NULL, 0, 0, AlignCenter, AlignBottom);

applications/main/subghz/subghz.c

@@ -70,6 +70,7 @@ SubGhz* subghz_alloc(bool alloc_for_tx_only) {
     subghz->gui = furi_record_open(RECORD_GUI);
 
     subghz->in_decoder_scene = false;
+    subghz->in_decoder_scene_skip = false;
 
     // View Dispatcher
     subghz->view_dispatcher = view_dispatcher_alloc();

applications/main/subghz/subghz_history.c

@@ -41,6 +41,7 @@ struct SubGhzHistory {
     uint8_t code_last_hash_data;
     FuriString* tmp_string;
     bool write_tmp_files;
+    bool is_hopper_running;
     Storage* storage;
     SubGhzHistoryStruct* history;
 };
@@ -134,6 +135,8 @@ SubGhzHistory* subghz_history_alloc(void) {
     instance->storage = furi_record_open(RECORD_STORAGE);
     instance->write_tmp_files = subghz_history_check_sdcard(instance);
 
+    instance->is_hopper_running = false;
+
     if(!instance->write_tmp_files) {
         FURI_LOG_E(TAG, "Unstable work! Cannot use SD Card!");
     }
@@ -209,6 +212,12 @@ void subghz_history_reset(SubGhzHistory* instance) {
     instance->code_last_hash_data = 0;
 }
 
+void subghz_history_set_hopper_state(SubGhzHistory* instance, bool hopper_state) {
+    furi_assert(instance);
+
+    instance->is_hopper_running = hopper_state;
+}
+
 uint16_t subghz_history_get_item(SubGhzHistory* instance) {
     furi_assert(instance);
     return instance->last_index_write;
@@ -354,16 +363,23 @@ bool subghz_history_add_to_history(
                 item->protocol_name, "%s", furi_string_get_cstr(instance->tmp_string));
         }
         if(!strcmp(furi_string_get_cstr(instance->tmp_string), "RAW")) {
+            // Check if hopper enabled we need to add little delay
+            if(instance->is_hopper_running) {
+                furi_delay_ms(40);
+            }
+            // Enable writing temp files to micro sd
+            tmp_file_for_raw = true;
+            // Write display name
             furi_string_printf(
                 item->item_str,
                 "RAW %03ld.%02ld",
                 preset->frequency / 1000000 % 1000,
                 preset->frequency / 10000 % 100);
-
+            // Rewind
             if(!flipper_format_rewind(item->flipper_string)) {
                 FURI_LOG_E(TAG, "Rewind error");
             }
-            tmp_file_for_raw = true;
+
             break;
         } else if(!strcmp(furi_string_get_cstr(instance->tmp_string), "KeeLoq")) {
             furi_string_set(instance->tmp_string, "KL ");

applications/main/subghz/subghz_history.h

@@ -103,3 +103,10 @@ bool subghz_history_add_to_history(
  * @return SubGhzProtocolCommonLoad*
  */
 FlipperFormat* subghz_history_get_raw_data(SubGhzHistory* instance, uint16_t idx);
+
+/** Set hopper state for internal usage in history
+ * 
+ * @param instance  - SubGhzHistory instance
+ * @param hopper_state - bool is hopper running?
+ */
+void subghz_history_set_hopper_state(SubGhzHistory* instance, bool hopper_state);

applications/main/subghz/subghz_i.h

@@ -29,6 +29,8 @@
 
 #include <lib/subghz/subghz_worker.h>
 
+#include <lib/subghz/subghz_file_encoder_worker.h>
+
 #include <lib/subghz/receiver.h>
 #include <lib/subghz/transmitter.h>
 
@@ -49,6 +51,12 @@ typedef struct {
     uint8_t seed[4];
 } SecureData;
 
+typedef enum {
+    SubGhzDecodeRawStateStart,
+    SubGhzDecodeRawStateLoading,
+    SubGhzDecodeRawStateLoaded,
+} SubGhzDecodeRawState;
+
 struct SubGhzTxRx {
     SubGhzWorker* worker;
 
@@ -109,6 +117,11 @@ struct SubGhz {
     SubGhzLock lock;
 
     bool in_decoder_scene;
+    bool in_decoder_scene_skip;
+
+    SubGhzDecodeRawState decode_raw_state;
+    SubGhzFileEncoderWorker* decode_raw_file_worker_encoder;
+
     void* rpc_ctx;
 };
 

documentation/AppsOnSDCard.md

@@ -13,7 +13,7 @@ FAPs are created and developed the same way as internal applications that are pa
 
 To build your application as a FAP, just create a folder with your app's source code in `applications_user`, then write its code the way you'd do when creating a regular built-in application. Then configure its `application.fam` manifest — and set its *apptype* to FlipperAppType.EXTERNAL. See [Application Manifests](./AppManifests.md#application-definition) for more details. 
 
- * To build your application, run `./fbt firmware_{APPID}`, where APPID is your application's ID in its manifest.
+ * To build your application, run `./fbt fap_{APPID}`, where APPID is your application's ID in its manifest.
  * To build your app, then upload it over USB & run it on Flipper, use `./fbt launch_app APPSRC=applications/path/to/app`. This command is configured in default [VSCode profile](../.vscode/ReadMe.md) as "Launch App on Flipper" build action (Ctrl+Shift+B menu).
  * To build all FAPs, run `./fbt plugin_dist`.