Merge branch 'dev' into filebrowser

applications/main/subghz/views/subghz_test_carrier.c

@@ -120,7 +120,10 @@ bool subghz_test_carrier_input(InputEvent* event, void* context) {
                 furi_hal_subghz_rx();
             } else {
                 furi_hal_gpio_init(
-                    furi_hal_subghz.cc1101_g0_pin, GpioModeOutputPushPull, GpioPullNo, GpioSpeedLow);
+                    furi_hal_subghz.cc1101_g0_pin,
+                    GpioModeOutputPushPull,
+                    GpioPullNo,
+                    GpioSpeedLow);
                 furi_hal_gpio_write(furi_hal_subghz.cc1101_g0_pin, true);
                 if(!furi_hal_subghz_tx()) {
                     furi_hal_gpio_init(

firmware/targets/f7/api_symbols.csv

@@ -1146,13 +1146,23 @@ Function,+,furi_hal_i2c_tx,_Bool,"FuriHalI2cBusHandle*, const uint8_t, const uin
 Function,+,furi_hal_i2c_write_mem,_Bool,"FuriHalI2cBusHandle*, uint8_t, uint8_t, uint8_t*, uint8_t, uint32_t"
 Function,+,furi_hal_i2c_write_reg_16,_Bool,"FuriHalI2cBusHandle*, uint8_t, uint8_t, uint16_t, uint32_t"
 Function,+,furi_hal_i2c_write_reg_8,_Bool,"FuriHalI2cBusHandle*, uint8_t, uint8_t, uint8_t, uint32_t"
+Function,+,furi_hal_ibutton_add_interrupt,void,"GpioExtiCallback, void*"
 Function,+,furi_hal_ibutton_emulate_set_next,void,uint32_t
 Function,+,furi_hal_ibutton_emulate_start,void,"uint32_t, FuriHalIbuttonEmulateCallback, void*"
 Function,+,furi_hal_ibutton_emulate_stop,void,
 Function,-,furi_hal_ibutton_init,void,
 Function,+,furi_hal_ibutton_pin_configure,void,
+Function,+,furi_hal_ibutton_pin_get_level,_Bool,
+Function,+,furi_hal_ibutton_pin_high,void,
+Function,+,furi_hal_ibutton_pin_low,void,
 Function,+,furi_hal_ibutton_pin_reset,void,
 Function,+,furi_hal_ibutton_pin_write,void,const _Bool
+Function,+,furi_hal_ibutton_remove_interrupt,void,
+Function,+,furi_hal_ibutton_start_drive,void,
+Function,+,furi_hal_ibutton_start_drive_in_isr,void,
+Function,+,furi_hal_ibutton_start_interrupt,void,
+Function,+,furi_hal_ibutton_start_interrupt_in_isr,void,
+Function,+,furi_hal_ibutton_stop,void,
 Function,+,furi_hal_info_get,void,"PropertyValueCallback, char, void*"
 Function,+,furi_hal_infrared_async_rx_set_capture_isr_callback,void,"FuriHalInfraredRxCaptureCallback, void*"
 Function,+,furi_hal_infrared_async_rx_set_timeout,void,uint32_t

firmware/targets/f7/furi_hal/furi_hal_ibutton.c

@@ -89,6 +89,51 @@ void furi_hal_ibutton_emulate_stop() {
     }
 }
 
+void furi_hal_ibutton_start_drive() {
+    furi_hal_ibutton_pin_high();
+    furi_hal_gpio_init(&ibutton_gpio, GpioModeOutputOpenDrain, GpioPullNo, GpioSpeedLow);
+}
+
+void furi_hal_ibutton_start_drive_in_isr() {
+    furi_hal_gpio_init(&ibutton_gpio, GpioModeOutputOpenDrain, GpioPullNo, GpioSpeedLow);
+    LL_EXTI_ClearFlag_0_31(ibutton_gpio.pin);
+}
+
+void furi_hal_ibutton_start_interrupt() {
+    furi_hal_ibutton_pin_high();
+    furi_hal_gpio_init(&ibutton_gpio, GpioModeInterruptRiseFall, GpioPullNo, GpioSpeedLow);
+}
+
+void furi_hal_ibutton_start_interrupt_in_isr() {
+    furi_hal_gpio_init(&ibutton_gpio, GpioModeInterruptRiseFall, GpioPullNo, GpioSpeedLow);
+    LL_EXTI_ClearFlag_0_31(ibutton_gpio.pin);
+}
+
+void furi_hal_ibutton_stop() {
+    furi_hal_ibutton_pin_high();
+    furi_hal_gpio_init(&ibutton_gpio, GpioModeAnalog, GpioPullNo, GpioSpeedLow);
+}
+
+void furi_hal_ibutton_add_interrupt(GpioExtiCallback cb, void* context) {
+    furi_hal_gpio_add_int_callback(&ibutton_gpio, cb, context);
+}
+
+void furi_hal_ibutton_remove_interrupt() {
+    furi_hal_gpio_remove_int_callback(&ibutton_gpio);
+}
+
+void furi_hal_ibutton_pin_low() {
+    furi_hal_gpio_write(&ibutton_gpio, false);
+}
+
+void furi_hal_ibutton_pin_high() {
+    furi_hal_gpio_write(&ibutton_gpio, true);
+}
+
+bool furi_hal_ibutton_pin_get_level() {
+    return furi_hal_gpio_read(&ibutton_gpio);
+}
+
 void furi_hal_ibutton_pin_configure() {
     furi_hal_gpio_write(&ibutton_gpio, true);
     furi_hal_gpio_init(&ibutton_gpio, GpioModeOutputOpenDrain, GpioPullNo, GpioSpeedLow);
@@ -101,4 +146,4 @@ void furi_hal_ibutton_pin_reset() {
 
 void furi_hal_ibutton_pin_write(const bool state) {
     furi_hal_gpio_write(&ibutton_gpio, state);
-}
+}

firmware/targets/f7/furi_hal/furi_hal_ibutton.h

@@ -7,6 +7,7 @@
 
 #include <stdbool.h>
 #include <stdint.h>
+#include <furi_hal_gpio.h>
 
 #ifdef __cplusplus
 extern "C" {
@@ -17,27 +18,70 @@ typedef void (*FuriHalIbuttonEmulateCallback)(void* context);
 /** Initialize */
 void furi_hal_ibutton_init();
 
-/**
- * Start emulation timer
- * @param period timer period
- * @param callback timer callback
- * @param context callback context
- */
 void furi_hal_ibutton_emulate_start(
     uint32_t period,
     FuriHalIbuttonEmulateCallback callback,
     void* context);
 
+void furi_hal_ibutton_emulate_set_next(uint32_t period);
+
+void furi_hal_ibutton_emulate_stop();
+
 /**
- * Update emulation timer period
- * @param period new timer period
+ * Sets the pin to normal mode (open collector), and sets it to float
  */
-void furi_hal_ibutton_emulate_set_next(uint32_t period);
+void furi_hal_ibutton_start_drive();
 
 /**
- * Stop emulation timer
+ * Sets the pin to normal mode (open collector), and clears pin EXTI interrupt.
+ * Used in EXTI interrupt context.
  */
-void furi_hal_ibutton_emulate_stop();
+void furi_hal_ibutton_start_drive_in_isr();
+
+/**
+ * Sets the pin to interrupt mode (EXTI interrupt on rise or fall), and sets it to float
+ */
+void furi_hal_ibutton_start_interrupt();
+
+/**
+ * Sets the pin to interrupt mode (EXTI interrupt on rise or fall), and clears pin EXTI interrupt.
+ * Used in EXTI interrupt context.
+ */
+void furi_hal_ibutton_start_interrupt_in_isr();
+
+/**
+ * Sets the pin to analog mode, and sets it to float
+ */
+void furi_hal_ibutton_stop();
+
+/**
+ * Attach interrupt callback to iButton pin
+ * @param cb callback
+ * @param context context
+ */
+void furi_hal_ibutton_add_interrupt(GpioExtiCallback cb, void* context);
+
+/**
+ * Remove interrupt callback from iButton pin
+ */
+void furi_hal_ibutton_remove_interrupt();
+
+/**
+ * Sets the pin to low
+ */
+void furi_hal_ibutton_pin_low();
+
+/**
+ * Sets the pin to high (float in iButton pin modes)
+ */
+void furi_hal_ibutton_pin_high();
+
+/**
+ * Get pin level
+ * @return true if level is high
+ * @return false if level is low
+ */
+bool furi_hal_ibutton_pin_get_level();
 
 /**
  * Set the pin to normal mode (open collector), and sets it to float

firmware/targets/f7/furi_hal/furi_hal_rfid.c

@@ -77,7 +77,7 @@ void furi_hal_rfid_init() {
 
 void furi_hal_rfid_pins_reset() {
     // ibutton bus disable
-    furi_hal_ibutton_pin_reset();
+    furi_hal_ibutton_stop();
 
     // pulldown rfid antenna
     furi_hal_gpio_init(&gpio_rfid_carrier_out, GpioModeOutputPushPull, GpioPullNo, GpioSpeedLow);
@@ -94,8 +94,8 @@ void furi_hal_rfid_pins_reset() {
 
 void furi_hal_rfid_pins_emulate() {
     // ibutton low
-    furi_hal_ibutton_pin_configure();
-    furi_hal_ibutton_pin_write(false);
+    furi_hal_ibutton_start_drive();
+    furi_hal_ibutton_pin_low();
 
     // pull pin to timer out
     furi_hal_gpio_init_ex(
@@ -115,8 +115,8 @@ void furi_hal_rfid_pins_emulate() {
 
 void furi_hal_rfid_pins_read() {
     // ibutton low
-    furi_hal_ibutton_pin_configure();
-    furi_hal_ibutton_pin_write(false);
+    furi_hal_ibutton_start_drive();
+    furi_hal_ibutton_pin_low();
 
     // dont pull rfid antenna
     furi_hal_gpio_init(&gpio_nfc_irq_rfid_pull, GpioModeOutputPushPull, GpioPullNo, GpioSpeedLow);

lib/one_wire/ibutton/ibutton_worker_modes.c

@@ -234,13 +234,16 @@ void ibutton_worker_emulate_timer_cb(void* context) {
     furi_assert(context);
     iButtonWorker* worker = context;
 
-    const LevelDuration level_duration =
+    LevelDuration level =
         protocol_dict_encoder_yield(worker->protocols, worker->protocol_to_encode);
 
-    const bool level = level_duration_get_level(level_duration);
+    furi_hal_ibutton_emulate_set_next(level_duration_get_duration(level));
 
-    furi_hal_ibutton_emulate_set_next(level);
-    furi_hal_ibutton_pin_write(level);
+    if(level_duration_get_level(level)) {
+        furi_hal_ibutton_pin_high();
+    } else {
+        furi_hal_ibutton_pin_low();
+    }
 }
 
 void ibutton_worker_emulate_timer_start(iButtonWorker* worker) {
@@ -263,7 +266,7 @@ void ibutton_worker_emulate_timer_start(iButtonWorker* worker) {
     protocol_dict_set_data(worker->protocols, worker->protocol_to_encode, key_id, key_size);
     protocol_dict_encoder_start(worker->protocols, worker->protocol_to_encode);
 
-    furi_hal_ibutton_pin_configure();
+    furi_hal_ibutton_start_drive();
     furi_hal_ibutton_emulate_start(0, ibutton_worker_emulate_timer_cb, worker);
 }
 

lib/one_wire/one_wire_host.c

@@ -1,5 +1,5 @@
 #include <furi.h>
-
+#include <furi_hal.h>
 #include "one_wire_host.h"
 #include "one_wire_host_timing.h"
 
@@ -24,47 +24,49 @@ void onewire_host_free(OneWireHost* host) {
 }
 
 bool onewire_host_reset(OneWireHost* host) {
+    UNUSED(host);
     uint8_t r;
     uint8_t retries = 125;
 
     // wait until the gpio is high
-    furi_hal_gpio_write(host->gpio_pin, true);
+    furi_hal_ibutton_pin_high();
     do {
         if(--retries == 0) return 0;
         furi_delay_us(2);
-    } while(!furi_hal_gpio_read(host->gpio_pin));
+    } while(!furi_hal_ibutton_pin_get_level());
 
     // pre delay
     furi_delay_us(OWH_RESET_DELAY_PRE);
 
     // drive low
-    furi_hal_gpio_write(host->gpio_pin, false);
+    furi_hal_ibutton_pin_low();
     furi_delay_us(OWH_RESET_DRIVE);
 
     // release
-    furi_hal_gpio_write(host->gpio_pin, true);
+    furi_hal_ibutton_pin_high();
     furi_delay_us(OWH_RESET_RELEASE);
 
     // read and post delay
-    r = !furi_hal_gpio_read(host->gpio_pin);
+    r = !furi_hal_ibutton_pin_get_level();
     furi_delay_us(OWH_RESET_DELAY_POST);
 
     return r;
 }
 
 bool onewire_host_read_bit(OneWireHost* host) {
+    UNUSED(host);
     bool result;
 
     // drive low
-    furi_hal_gpio_write(host->gpio_pin, false);
+    furi_hal_ibutton_pin_low();
     furi_delay_us(OWH_READ_DRIVE);
 
     // release
-    furi_hal_gpio_write(host->gpio_pin, true);
+    furi_hal_ibutton_pin_high();
     furi_delay_us(OWH_READ_RELEASE);
 
     // read and post delay
-    result = furi_hal_gpio_read(host->gpio_pin);
+    result = furi_hal_ibutton_pin_get_level();
     furi_delay_us(OWH_READ_DELAY_POST);
 
     return result;
@@ -89,21 +91,22 @@ void onewire_host_read_bytes(OneWireHost* host, uint8_t* buffer, uint16_t count)
 }
 
 void onewire_host_write_bit(OneWireHost* host, bool value) {
+    UNUSED(host);
     if(value) {
         // drive low
-        furi_hal_gpio_write(host->gpio_pin, false);
+        furi_hal_ibutton_pin_low();
         furi_delay_us(OWH_WRITE_1_DRIVE);
 
         // release
-        furi_hal_gpio_write(host->gpio_pin, true);
+        furi_hal_ibutton_pin_high();
         furi_delay_us(OWH_WRITE_1_RELEASE);
     } else {
         // drive low
-        furi_hal_gpio_write(host->gpio_pin, false);
+        furi_hal_ibutton_pin_low();
         furi_delay_us(OWH_WRITE_0_DRIVE);
 
         // release
-        furi_hal_gpio_write(host->gpio_pin, true);
+        furi_hal_ibutton_pin_high();
         furi_delay_us(OWH_WRITE_0_RELEASE);
     }
 }
@@ -121,13 +124,13 @@ void onewire_host_skip(OneWireHost* host) {
 }
 
 void onewire_host_start(OneWireHost* host) {
-    furi_hal_gpio_write(host->gpio_pin, true);
-    furi_hal_gpio_init(host->gpio_pin, GpioModeOutputOpenDrain, GpioPullNo, GpioSpeedLow);
+    UNUSED(host);
+    furi_hal_ibutton_start_drive();
 }
 
 void onewire_host_stop(OneWireHost* host) {
-    furi_hal_gpio_write(host->gpio_pin, true);
-    furi_hal_gpio_init(host->gpio_pin, GpioModeAnalog, GpioPullNo, GpioSpeedLow);
+    UNUSED(host);
+    furi_hal_ibutton_stop();
 }
 
 void onewire_host_reset_search(OneWireHost* host) {
@@ -148,7 +151,7 @@ void onewire_host_target_search(OneWireHost* host, uint8_t family_code) {
     host->last_device_flag = false;
 }
 
-uint8_t onewire_host_search(OneWireHost* host, uint8_t* new_addr, OneWireHostSearchMode mode) {
+uint8_t onewire_host_search(OneWireHost* host, uint8_t* newAddr, OneWireHostSearchMode mode) {
     uint8_t id_bit_number;
     uint8_t last_zero, rom_byte_number, search_result;
     uint8_t id_bit, cmp_id_bit;
@@ -257,7 +260,7 @@ uint8_t onewire_host_search(OneWireHost* host, uint8_t* new_addr, OneWireHostSea
         host->last_family_discrepancy = 0;
         search_result = false;
     } else {
-        for(int i = 0; i < 8; i++) new_addr[i] = host->saved_rom[i];
+        for(int i = 0; i < 8; i++) newAddr[i] = host->saved_rom[i];
     }
 
     return search_result;

lib/one_wire/one_wire_host.h

@@ -22,7 +22,7 @@ typedef struct OneWireHost OneWireHost;
 
 /**
  * Allocate onewire host bus
- * @param pin
+ * @param gpio 
  * @return OneWireHost* 
  */
 OneWireHost* onewire_host_alloc(const GpioPin* gpio_pin);
@@ -114,7 +114,7 @@ void onewire_host_target_search(OneWireHost* host, uint8_t family_code);
  * @param mode 
  * @return uint8_t 
  */
-uint8_t onewire_host_search(OneWireHost* host, uint8_t* new_addr, OneWireHostSearchMode mode);
+uint8_t onewire_host_search(OneWireHost* host, uint8_t* newAddr, OneWireHostSearchMode mode);
 
 #ifdef __cplusplus
 }