Switch to finaliser for PWMOut and audiopwmio

Fixes #8726 which was caused by a mix of bulk reset and finaliser
use.

Work towards #8960.

Author: tannewt

ports/atmel-samd/Makefile

@@ -309,12 +309,6 @@ $(BUILD)/lib/tlsf/tlsf.o: CFLAGS += -Wno-cast-align
 
 $(BUILD)/lib/tinyusb/src/portable/microchip/samd/dcd_samd.o: CFLAGS += -Wno-missing-prototypes
 
-# This is an OR because it filters to any 1s and then checks to see if it is not
-# empty.
-ifneq (,$(filter 1,$(CIRCUITPY_PWMIO) $(CIRCUITPY_AUDIOIO) $(CIRCUITPY_RGBMATRIX)))
-SRC_C += shared_timers.c
-endif
-
 ifeq ($(CIRCUITPY_SAMD),1)
 SRC_C += bindings/samd/Clock.c bindings/samd/__init__.c
 endif

ports/atmel-samd/common-hal/pulseio/PulseIn.c

@@ -140,15 +140,6 @@ void pulsein_interrupt_handler(uint8_t channel) {
     common_hal_mcu_enable_interrupts();
 }
 
-void pulsein_reset() {
-    #ifdef SAMD21
-    rtc_end_pulse();
-    #endif
-    refcount = 0;
-    pulsein_tc_index = 0xff;
-    overflow_count = 0;
-}
-
 void common_hal_pulseio_pulsein_construct(pulseio_pulsein_obj_t *self,
     const mcu_pin_obj_t *pin, uint16_t maxlen, bool idle_state) {
     if (!pin->has_extint) {
@@ -243,7 +234,6 @@ void common_hal_pulseio_pulsein_construct(pulseio_pulsein_obj_t *self,
     #ifdef SAMD21
     rtc_start_pulse();
     #endif
-
 }
 
 bool common_hal_pulseio_pulsein_deinited(pulseio_pulsein_obj_t *self) {

ports/atmel-samd/common-hal/pulseio/PulseIn.h

@@ -46,8 +46,6 @@ typedef struct {
     volatile bool errored_too_fast;
 } pulseio_pulsein_obj_t;
 
-void pulsein_reset(void);
-
 void pulsein_interrupt_handler(uint8_t channel);
 void pulsein_timer_interrupt_handler(uint8_t index);
 #ifdef SAMD21

ports/atmel-samd/common-hal/pulseio/PulseOut.c

@@ -92,15 +92,6 @@ void pulseout_interrupt_handler(uint8_t index) {
     tc->COUNT16.INTFLAG.reg = TC_INTFLAG_MC0;
 }
 
-void pulseout_reset() {
-    refcount = 0;
-    pulseout_tc_index = 0xff;
-    active_pincfg = NULL;
-    #ifdef SAMD21
-    rtc_end_pulse();
-    #endif
-}
-
 void common_hal_pulseio_pulseout_construct(pulseio_pulseout_obj_t *self,
     const mcu_pin_obj_t *pin,
     uint32_t frequency,

ports/atmel-samd/common-hal/pwmio/PWMOut.c

@@ -31,7 +31,6 @@
 #include "common-hal/pwmio/PWMOut.h"
 #include "shared-bindings/pwmio/PWMOut.h"
 #include "shared-bindings/microcontroller/Processor.h"
-#include "shared_timers.h"
 #include "timer_handler.h"
 
 #include "atmel_start_pins.h"
@@ -61,25 +60,9 @@ uint8_t tcc_channels[5];   // Set by pwmout_reset() to {0xc0, 0xf0, 0xf8, 0xfc,
 
 
 void common_hal_pwmio_pwmout_never_reset(pwmio_pwmout_obj_t *self) {
-    timer_never_reset(self->timer->index, self->timer->is_tc);
-
     never_reset_pin_number(self->pin->number);
 }
 
-void pwmout_reset(void) {
-    // Reset all timers
-    for (int i = 0; i < TCC_INST_NUM; i++) {
-        target_tcc_frequencies[i] = 0;
-        tcc_refcount[i] = 0;
-    }
-    for (int i = 0; i < TCC_INST_NUM; i++) {
-        if (!timer_ok_to_reset(i, false)) {
-            continue;
-        }
-        tcc_channels[i] = 0xff << tcc_cc_num[i];
-    }
-}
-
 static uint8_t tcc_channel(const pin_timer_t *t) {
     // For the SAMD51 this hardcodes the use of OTMX == 0x0, the output matrix mapping, which uses
     // SAMD21-style modulo mapping.
@@ -97,7 +80,6 @@ pwmout_result_t common_hal_pwmio_pwmout_construct(pwmio_pwmout_obj_t *self,
     uint16_t duty,
     uint32_t frequency,
     bool variable_frequency) {
-    self->pin = pin;
     self->variable_frequency = variable_frequency;
     self->duty_cycle = duty;
 
@@ -242,6 +224,7 @@ pwmout_result_t common_hal_pwmio_pwmout_construct(pwmio_pwmout_obj_t *self,
     }
 
     self->timer = timer;
+    self->pin = pin;
 
     gpio_set_pin_function(pin->number, GPIO_PIN_FUNCTION_E + mux_position);
 
@@ -257,7 +240,6 @@ void common_hal_pwmio_pwmout_deinit(pwmio_pwmout_obj_t *self) {
     if (common_hal_pwmio_pwmout_deinited(self)) {
         return;
     }
-    timer_reset_ok(self->timer->index, self->timer->is_tc);
     const pin_timer_t *t = self->timer;
     if (t->is_tc) {
         Tc *tc = tc_insts[t->index];

ports/atmel-samd/common-hal/pwmio/PWMOut.h

@@ -24,8 +24,7 @@
  * THE SOFTWARE.
  */
 
-#ifndef MICROPY_INCLUDED_ATMEL_SAMD_COMMON_HAL_PWMIO_PWMOUT_H
-#define MICROPY_INCLUDED_ATMEL_SAMD_COMMON_HAL_PWMIO_PWMOUT_H
+#pragma once
 
 #include "common-hal/microcontroller/Pin.h"
 
@@ -38,7 +37,3 @@ typedef struct {
     bool variable_frequency;
     uint16_t duty_cycle;
 } pwmio_pwmout_obj_t;
-
-void pwmout_reset(void);
-
-#endif // MICROPY_INCLUDED_ATMEL_SAMD_COMMON_HAL_PWMIO_PWMOUT_H

ports/atmel-samd/common-hal/rgbmatrix/RGBMatrix.c

@@ -29,15 +29,13 @@
 #include "common-hal/rgbmatrix/RGBMatrix.h"
 
 #include "samd/timers.h"
-#include "shared_timers.h"
 #include "timer_handler.h"
 
 void *common_hal_rgbmatrix_timer_allocate(rgbmatrix_rgbmatrix_obj_t *self) {
     uint8_t timer_index = find_free_timer();
     if (timer_index == 0xff) {
         return NULL;
     }
-    timer_never_reset(timer_index, true);
     return tc_insts[timer_index];
 }
 
@@ -75,5 +73,4 @@ void common_hal_rgbmatrix_timer_free(void *ptr) {
     }
     tc_set_enable(ptr, false);
     tc_reset(ptr);
-    timer_reset_ok(timer_index, true);
 }

ports/atmel-samd/shared_timers.c

@@ -77,15 +77,6 @@
 
 #include "common-hal/microcontroller/Pin.h"
 
-#if CIRCUITPY_PULSEIO
-#include "common-hal/pulseio/PulseIn.h"
-#include "common-hal/pulseio/PulseOut.h"
-#endif
-
-#if CIRCUITPY_PWMIO
-#include "common-hal/pwmio/PWMOut.h"
-#endif
-
 #if CIRCUITPY_PS2IO
 #include "common-hal/ps2io/Ps2.h"
 #endif
@@ -111,9 +102,7 @@
 #include "samd/dma.h"
 #include "shared-bindings/microcontroller/__init__.h"
 #include "shared-bindings/rtc/__init__.h"
-#include "shared_timers.h"
 #include "reset.h"
-#include "common-hal/pulseio/PulseIn.h"
 
 #include "supervisor/background_callback.h"
 #include "supervisor/shared/safe_mode.h"
@@ -412,19 +401,6 @@ void reset_port(void) {
 
     eic_reset();
 
-    #if CIRCUITPY_PULSEIO
-    pulsein_reset();
-    pulseout_reset();
-    #endif
-
-    #if CIRCUITPY_PWMIO
-    pwmout_reset();
-    #endif
-
-    #if CIRCUITPY_PWMIO || CIRCUITPY_AUDIOIO || CIRCUITPY_FREQUENCYIO
-    reset_timers();
-    #endif
-
     #if CIRCUITPY_ANALOGIO
     analogin_reset();
     analogout_reset();