[SOLVED] Exception with advance_pointer

[203Null]

I'm currently using TinyUSB in my CPP project targeted STM32F103RET6. However, I'm tried to port example code to my project, the USB can't be registered in WIndows and an exception is thrown at acvance_pointer(). I have no clue what could cause this but compile the example code works flawlessly for me, so I must screw something at my end.

[203Null]

Code (Modified for readability):

Device.cpp (Device Initialization code)

namespace Device
{
    void Device_Init()
    {
        HAL_Init();
        SystemClock_Config();

        GPIO_Init();
    }

    void Bootloader()
    {
        
    }

    void Reboot()
    {
        // NVIC_SystemReset();
    }

    void Delay(uint32_t interval)
    {
        HAL_Delay(interval);
    }

    uint32_t Millis()
    {
        return HAL_GetTick();
    }

    namespace
    {
        /**
         * @brief  System Clock Configuration
         *         The system Clock is configured as follow :
         *            System Clock source            = PLL (HSE)
         *            SYSCLK(Hz)                     = 72000000
         *            HCLK(Hz)                       = 72000000
         *            AHB Prescaler                  = 1
         *            APB1 Prescaler                 = 2
         *            APB2 Prescaler                 = 1
         *            HSE Frequency(Hz)              = 8000000
         *            HSE PREDIV1                    = 1
         *            PLLMUL                         = 9
         *            Flash Latency(WS)              = 2
         * @param  None
         * @retval None
         */
        void SystemClock_Config(void)
        {
            RCC_OscInitTypeDef RCC_OscInitStruct = {0};
            RCC_ClkInitTypeDef RCC_ClkInitStruct = {0};
            RCC_PeriphCLKInitTypeDef PeriphClkInit = {0};

            /** Initializes the RCC Oscillators according to the specified parameters
             * in the RCC_OscInitTypeDef structure.
             */
            RCC_OscInitStruct.OscillatorType = RCC_OSCILLATORTYPE_HSE;
            RCC_OscInitStruct.HSEState = RCC_HSE_ON;
            RCC_OscInitStruct.HSEPredivValue = RCC_HSE_PREDIV_DIV1;
            RCC_OscInitStruct.HSIState = RCC_HSI_ON;
            RCC_OscInitStruct.PLL.PLLState = RCC_PLL_ON;
            RCC_OscInitStruct.PLL.PLLSource = RCC_PLLSOURCE_HSE;
            RCC_OscInitStruct.PLL.PLLMUL = RCC_PLL_MUL9;
            if (HAL_RCC_OscConfig(&RCC_OscInitStruct) != HAL_OK)
            {
                Error_Handler();
            }
            /** Initializes the CPU, AHB and APB buses clocks
             */
            RCC_ClkInitStruct.ClockType = RCC_CLOCKTYPE_HCLK|RCC_CLOCKTYPE_SYSCLK
                                        |RCC_CLOCKTYPE_PCLK1|RCC_CLOCKTYPE_PCLK2;
            RCC_ClkInitStruct.SYSCLKSource = RCC_SYSCLKSOURCE_PLLCLK;
            RCC_ClkInitStruct.AHBCLKDivider = RCC_SYSCLK_DIV1;
            RCC_ClkInitStruct.APB1CLKDivider = RCC_HCLK_DIV2;
            RCC_ClkInitStruct.APB2CLKDivider = RCC_HCLK_DIV1;

            if (HAL_RCC_ClockConfig(&RCC_ClkInitStruct, FLASH_LATENCY_2) != HAL_OK)
            {
                Error_Handler();
            }
            PeriphClkInit.PeriphClockSelection = RCC_PERIPHCLK_USB;
            PeriphClkInit.UsbClockSelection = RCC_USBCLKSOURCE_PLL_DIV1_5;
            if (HAL_RCCEx_PeriphCLKConfig(&PeriphClkInit) != HAL_OK)
            {
                Error_Handler();
            }
        }

        void GPIO_Init()
        {
            GPIO_InitTypeDef GPIO_InitStruct = {0};

            /* Configure USB FS GPIOs */
            __HAL_RCC_GPIOA_CLK_ENABLE();

            /* Configure USB D+ D- Pins */
            GPIO_InitStruct.Pin = GPIO_PIN_11 | GPIO_PIN_12;
            GPIO_InitStruct.Mode = GPIO_MODE_INPUT;
            GPIO_InitStruct.Pull = GPIO_NOPULL;
            GPIO_InitStruct.Speed = GPIO_SPEED_FREQ_HIGH;
            HAL_GPIO_Init(GPIOA, &GPIO_InitStruct);

            // USB Clock enable
            __HAL_RCC_USB_CLK_ENABLE();
        }
    }

    void Error_Handler()
    {
        
    }
}

extern "C" {
    void USB_HP_IRQHandler(void)
    {
    tud_int_handler(0);
    }

    void USB_LP_IRQHandler(void)
    {
    tud_int_handler(0);
    }

    void USBWakeUp_IRQHandler(void)
    {
    tud_int_handler(0);
    }

    void HardFault_Handler (void)
    {
        while(true){

        }
    }

    extern "C" void _init(void) {;}
}

main.cpp

void midi_task();

int main()
{
    Device::Device_Init();

    bool init = tusb_init();

    while(true)
    {
        tud_task();
        midi_task();
    }

    return 0;             
}

//--------------------------------------------------------------------+
// MIDI Task
//--------------------------------------------------------------------+

// Variable that holds the current position in the sequence.
uint32_t note_pos = 0;

// Store example melody as an array of note values
uint8_t note_sequence[] =
{
  74,78,81,86,90,93,98,102,57,61,66,69,73,78,81,85,88,92,97,100,97,92,88,85,81,78,
  74,69,66,62,57,62,66,69,74,78,81,86,90,93,97,102,97,93,90,85,81,78,73,68,64,61,
  56,61,64,68,74,78,81,86,90,93,98,102
};

void midi_task(void)
{
  static uint32_t start_ms = 0;

  uint8_t const cable_num = 0; // MIDI jack associated with USB endpoint
  uint8_t const channel   = 0; // 0 for channel 1

  // The MIDI interface always creates input and output port/jack descriptors
  // regardless of these being used or not. Therefore incoming traffic should be read
  // (possibly just discarded) to avoid the sender blocking in IO
  uint8_t packet[4];
  while ( tud_midi_available() ) tud_midi_packet_read(packet);

  // send note periodically
  if (MatrixOS::SYS::Millis() - start_ms < 286) return; // not enough time
  start_ms += 286;

  // Previous positions in the note sequence.
  int previous = note_pos - 1;

  // If we currently are at position 0, set the
  // previous position to the last note in the sequence.
  if (previous < 0) previous = sizeof(note_sequence) - 1;

  // Send Note On for current position at full velocity (127) on channel 1.
  uint8_t note_on[3] = { 0x90 | channel, note_sequence[note_pos], 127 };
  tud_midi_stream_write(cable_num, note_on, 3);

  // Send Note Off for previous note.
  uint8_t note_off[3] = { 0x80 | channel, note_sequence[previous], 0};
  tud_midi_stream_write(cable_num, note_off, 3);

  // Increment position
  note_pos++;

  // If we are at the end of the sequence, start over.
  if (note_pos >= sizeof(note_sequence)) note_pos = 0;
}

Compiler Flags

CFLAGS  -DSTM32F103xE -flto -mthumb -mabi=aapcs -mcpu=cortex-m3 -mfloat-abi=soft -DCFG_TUSB_MCU=OPT_MCU_STM32F1 -nostdlib -nostartfiles -Og -g -Idevices/MatrixBlock5/FounderEdition -Ilib/CMSIS_5/CMSIS/Core/Include -Icore/stm32f1/cmsis_device_f1/Include -Icore/stm32f1/stm32f1xx_hal_driver/Inc -Idevices/MatrixBlock5 -Isrc/ -Idevices/ -Isrc/usb/ -Ilib/tinyusb/src/  

CPPFLAGS  -std=gnu++17  -DSTM32F103xE -flto -mthumb -mabi=aapcs -mcpu=cortex-m3 -mfloat-abi=soft -DCFG_TUSB_MCU=OPT_MCU_STM32F1 -nostdlib -nostartfiles -Og -g -Idevices/MatrixBlock5/FounderEdition -Ilib/CMSIS_5/CMSIS/Core/Include -Icore/stm32f1/cmsis_device_f1/Include -Icore/stm32f1/stm32f1xx_hal_driver/Inc -Idevices/MatrixBlock5 -Isrc/ -Idevices/ -Isrc/usb/ -Ilib/tinyusb/src/

LDFLAGS -DSTM32F103xE -flto -mthumb -mabi=aapcs -mcpu=cortex-m3 -mfloat-abi=soft -DCFG_TUSB_MCU=OPT_MCU_STM32F1 -nostdlib -nostartfiles -Og -g -Idevices/MatrixBlock5/FounderEdition -Ilib/CMSIS_5/CMSIS/Core/Include -Icore/stm32f1/cmsis_device_f1/Include -Icore/stm32f1/stm32f1xx_hal_driver/Inc -Idevices/MatrixBlock5 -Isrc/ -Idevices/ -Isrc/usb/ -Ilib/tinyusb/src/ -Wl,-T,devices/MatrixBlock5/FounderEdition/STM32F103RETx_FLASH.ld -Wl,-Map=.map -Wl,-cref -Wl,-gc-sections -specs=nosys.specs -specs=nano.specs

ASFLAGS -DSTM32F103xE -flto -mthumb -mabi=aapcs -mcpu=cortex-m3 -mfloat-abi=soft -DCFG_TUSB_MCU=OPT_MCU_STM32F1 -nostdlib -nostartfiles -Og -g -Idevices/MatrixBlock5/FounderEdition -Ilib/CMSIS_5/CMSIS/Core/Include -Icore/stm32f1/cmsis_device_f1/Include -Icore/stm32f1/stm32f1xx_hal_driver/Inc -Idevices/MatrixBlock5 -Isrc/ -Idevices/ -Isrc/usb/ -Ilib/tinyusb/src/  

[HiFiPhile]

Hi,

What's in your tusb_config.h ? In your trace I think buffer address is wrong.

[203Null]

It would be the same as the midi_test example.
tusb_config.h

#ifndef _TUSB_CONFIG_H_
#define _TUSB_CONFIG_H_

#ifdef __cplusplus
 extern "C" {
#endif

//--------------------------------------------------------------------
// COMMON CONFIGURATION
//--------------------------------------------------------------------

// defined by board.mk
#ifndef CFG_TUSB_MCU
  #error CFG_TUSB_MCU must be defined
#endif

// RHPort number used for device can be defined by board.mk, default to port 0
#ifndef BOARD_DEVICE_RHPORT_NUM
  #define BOARD_DEVICE_RHPORT_NUM     0
#endif

// RHPort max operational speed can defined by board.mk
// Default to Highspeed for MCU with internal HighSpeed PHY (can be port specific), otherwise FullSpeed
#ifndef BOARD_DEVICE_RHPORT_SPEED
  #if (CFG_TUSB_MCU == OPT_MCU_LPC18XX || CFG_TUSB_MCU == OPT_MCU_LPC43XX || CFG_TUSB_MCU == OPT_MCU_MIMXRT10XX || \
       CFG_TUSB_MCU == OPT_MCU_NUC505  || CFG_TUSB_MCU == OPT_MCU_CXD56)
    #define BOARD_DEVICE_RHPORT_SPEED   OPT_MODE_HIGH_SPEED
  #else
    #define BOARD_DEVICE_RHPORT_SPEED   OPT_MODE_FULL_SPEED
  #endif
#endif

// Device mode with rhport and speed defined by board.mk
#if   BOARD_DEVICE_RHPORT_NUM == 0
  #define CFG_TUSB_RHPORT0_MODE     (OPT_MODE_DEVICE | BOARD_DEVICE_RHPORT_SPEED)
#elif BOARD_DEVICE_RHPORT_NUM == 1
  #define CFG_TUSB_RHPORT1_MODE     (OPT_MODE_DEVICE | BOARD_DEVICE_RHPORT_SPEED)
#else
  #error "Incorrect RHPort configuration"
#endif

#ifndef CFG_TUSB_OS
#define CFG_TUSB_OS               OPT_OS_NONE
#endif

// CFG_TUSB_DEBUG is defined by compiler in DEBUG build
// #define CFG_TUSB_DEBUG           0

/* USB DMA on some MCUs can only access a specific SRAM region with restriction on alignment.
 * Tinyusb use follows macros to declare transferring memory so that they can be put
 * into those specific section.
 * e.g
 * - CFG_TUSB_MEM SECTION : __attribute__ (( section(".usb_ram") ))
 * - CFG_TUSB_MEM_ALIGN   : __attribute__ ((aligned(4)))
 */
#ifndef CFG_TUSB_MEM_SECTION
#define CFG_TUSB_MEM_SECTION
#endif

#ifndef CFG_TUSB_MEM_ALIGN
#define CFG_TUSB_MEM_ALIGN          __attribute__ ((aligned(4)))
#endif

//--------------------------------------------------------------------
// DEVICE CONFIGURATION
//--------------------------------------------------------------------

#ifndef CFG_TUD_ENDPOINT0_SIZE
#define CFG_TUD_ENDPOINT0_SIZE    64
#endif

//------------- CLASS -------------//
#define CFG_TUD_CDC               0
#define CFG_TUD_MSC               0
#define CFG_TUD_HID               0
#define CFG_TUD_MIDI              1
#define CFG_TUD_VENDOR            0

// MIDI FIFO size of TX and RX
#define CFG_TUD_MIDI_RX_BUFSIZE   (TUD_OPT_HIGH_SPEED ? 512 : 64)
#define CFG_TUD_MIDI_TX_BUFSIZE   (TUD_OPT_HIGH_SPEED ? 512 : 64)

#ifdef __cplusplus
 }
#endif

#endif /* _TUSB_CONFIG_H_ */

usb_descriptors.c

#include "tusb.h"

#ifdef __cplusplus
 extern "C" {
#endif


/* A combination of interfaces must have a unique product id, since PC will save device driver after the first plug.
 * Same VID/PID with different interface e.g MSC (first), then CDC (later) will possibly cause system error on PC.
 *
 * Auto ProductID layout's Bitmap:
 *   [MSB]       MIDI | HID | MSC | CDC          [LSB]
 */
#define _PID_MAP(itf, n)  ( (CFG_TUD_##itf) << (n) )
#define USB_PID           (0x4000 | _PID_MAP(CDC, 0) | _PID_MAP(MSC, 1) | _PID_MAP(HID, 2) | \
                           _PID_MAP(MIDI, 3) | _PID_MAP(VENDOR, 4) )

//--------------------------------------------------------------------+
// Device Descriptors
//--------------------------------------------------------------------+
tusb_desc_device_t const desc_device =
{
    .bLength            = sizeof(tusb_desc_device_t),
    .bDescriptorType    = TUSB_DESC_DEVICE,
    .bcdUSB             = 0x0200,
    .bDeviceClass       = 0x00,
    .bDeviceSubClass    = 0x00,
    .bDeviceProtocol    = 0x00,
    .bMaxPacketSize0    = CFG_TUD_ENDPOINT0_SIZE,

    .idVendor           = 0xCafe,
    .idProduct          = USB_PID,
    .bcdDevice          = 0x0100,

    .iManufacturer      = 0x01,
    .iProduct           = 0x02,
    .iSerialNumber      = 0x03,

    .bNumConfigurations = 0x01
};

// Invoked when received GET DEVICE DESCRIPTOR
// Application return pointer to descriptor
uint8_t const * tud_descriptor_device_cb(void)
{
  return (uint8_t const *) &desc_device;
}


//--------------------------------------------------------------------+
// Configuration Descriptor
//--------------------------------------------------------------------+

enum
{
  ITF_NUM_MIDI = 0,
  ITF_NUM_MIDI_STREAMING,
  ITF_NUM_TOTAL
};

#define CONFIG_TOTAL_LEN  (TUD_CONFIG_DESC_LEN + TUD_MIDI_DESC_LEN)

#if CFG_TUSB_MCU == OPT_MCU_LPC175X_6X || CFG_TUSB_MCU == OPT_MCU_LPC177X_8X || CFG_TUSB_MCU == OPT_MCU_LPC40XX
  // LPC 17xx and 40xx endpoint type (bulk/interrupt/iso) are fixed by its number
  // 0 control, 1 In, 2 Bulk, 3 Iso, 4 In etc ...
  #define EPNUM_MIDI   0x02
#else
  #define EPNUM_MIDI   0x01
#endif

uint8_t const desc_fs_configuration[] =
{
  // Config number, interface count, string index, total length, attribute, power in mA
  TUD_CONFIG_DESCRIPTOR(1, ITF_NUM_TOTAL, 0, CONFIG_TOTAL_LEN, TUSB_DESC_CONFIG_ATT_REMOTE_WAKEUP, 100),

  // Interface number, string index, EP Out & EP In address, EP size
  TUD_MIDI_DESCRIPTOR(ITF_NUM_MIDI, 0, EPNUM_MIDI, 0x80 | EPNUM_MIDI, 64)
};

#if TUD_OPT_HIGH_SPEED
uint8_t const desc_hs_configuration[] =
{
  // Config number, interface count, string index, total length, attribute, power in mA
  TUD_CONFIG_DESCRIPTOR(1, ITF_NUM_TOTAL, 0, CONFIG_TOTAL_LEN, TUSB_DESC_CONFIG_ATT_REMOTE_WAKEUP, 100),

  // Interface number, string index, EP Out & EP In address, EP size
  TUD_MIDI_DESCRIPTOR(ITF_NUM_MIDI, 0, EPNUM_MIDI, 0x80 | EPNUM_MIDI, 512)
};
#endif

// Invoked when received GET CONFIGURATION DESCRIPTOR
// Application return pointer to descriptor
// Descriptor contents must exist long enough for transfer to complete
uint8_t const * tud_descriptor_configuration_cb(uint8_t index)
{
  (void) index; // for multiple configurations

#if TUD_OPT_HIGH_SPEED
  // Although we are highspeed, host may be fullspeed.
  return (tud_speed_get() == TUSB_SPEED_HIGH) ?  desc_hs_configuration : desc_fs_configuration;
#else
  return desc_fs_configuration;
#endif
}

//--------------------------------------------------------------------+
// String Descriptors
//--------------------------------------------------------------------+

// array of pointer to string descriptors
char const* string_desc_arr [] =
{
  (const char[]) { 0x09, 0x04 }, // 0: is supported language is English (0x0409)
  "TinyUSB",                     // 1: Manufacturer
  "TinyUSB Device",              // 2: Product
  "123456",                      // 3: Serials, should use chip ID
};

static uint16_t _desc_str[32];

// Invoked when received GET STRING DESCRIPTOR request
// Application return pointer to descriptor, whose contents must exist long enough for transfer to complete
uint16_t const* tud_descriptor_string_cb(uint8_t index, uint16_t langid)
{
  (void) langid;

  uint8_t chr_count;

  if ( index == 0)
  {
    memcpy(&_desc_str[1], string_desc_arr[0], 2);
    chr_count = 1;
  }else
  {
    // Note: the 0xEE index string is a Microsoft OS 1.0 Descriptors.
    // https://docs.microsoft.com/en-us/windows-hardware/drivers/usbcon/microsoft-defined-usb-descriptors

    if ( !(index < sizeof(string_desc_arr)/sizeof(string_desc_arr[0])) ) return NULL;

    const char* str = string_desc_arr[index];

    // Cap at max char
    chr_count = strlen(str);
    if ( chr_count > 31 ) chr_count = 31;

    // Convert ASCII string into UTF-16
    for(uint8_t i=0; i<chr_count; i++)
    {
      _desc_str[1+i] = str[i];
    }
  }

  // first byte is length (including header), second byte is string type
  _desc_str[0] = (TUSB_DESC_STRING << 8 ) | (2*chr_count + 2);

  return _desc_str;

#ifdef __cplusplus
 }
#endif
}

[203Null]

Also, the exception does jump around a few times, sometimes in tusb_init, sometimes in midi_task (tud_midi_available()?) but they all end with WWDG_IRQHandler. I'm new to debugging with VSCode so I might did something wrong.

[HiFiPhile]

I suggest you to read tutorials about how to debug a HardFault, for example:
https://community.silabs.com/s/article/debug-a-hardfault?language=en_US

Then you can find out where exactly the fault occurred and the reason why it's occurred.

[203Null]

Solved. I tried to port to STM32HAL but I forgot to port

  void SysTick_Handler(void)
  {
      HAL_IncTick();
  }