STM32H7 SpiComIF and first device handler #9

muellerr · 2021-06-11T10:40:43+02:00

muellerr commented

2021-06-11 10:40:43 +02:00

This was a lot more work than expected..

Added SpiComIF for the STM32H7. Only tested for the H743ZI board
Added L3GD20 device handler. Device handlers are generic, so we can add them here as well
Tested all 3 transfer modes: Polling, IRQ and DMA. All STM32H7 peripherals offer the same modes, so this ComIF will be an ideal base to implement other communication interfaces.
Only tested with one device
No GpioIF* which would allow custom callbacks. Not worth the effort as long as noone requires it explicitely.. Simple prototyping usually does not require complex decoding logic, SPI chip select are simply driven by dedicated GPIO pins
Bugfixes for the L3GD20H device, conversion was not exactly correct
No ugly macro magic necessary for complete MSP configuration, replaced by proper structs at the price of code size. MSP config struct is deleted automatically by the SpiComIF. Here is the example testing code:

    spi::MspCfgBase* mspCfg = nullptr;
    spi::TransferModes transferMode = spi::TransferModes::DMA;
    if(transferMode == spi::TransferModes::POLLING) {
        auto typedCfg = new spi::MspPollingConfigStruct();
        spi::h743zi::standardPollingCfg(*typedCfg);
        mspCfg = typedCfg;
    }
    else if(transferMode == spi::TransferModes::INTERRUPT) {
        auto typedCfg = new spi::MspIrqConfigStruct();
        spi::h743zi::standardInterruptCfg(*typedCfg, IrqPriorities::HIGHEST_FREERTOS);
        mspCfg = typedCfg;
    }
    else if(transferMode == spi::TransferModes::DMA) {
        auto typedCfg = new spi::MspDmaConfigStruct();
        txHandle = new DMA_HandleTypeDef();
        rxHandle = new DMA_HandleTypeDef();
        spi::setDmaHandles(txHandle, rxHandle);
        spi::h743zi::standardDmaCfg(*typedCfg, IrqPriorities::HIGHEST_FREERTOS,
                IrqPriorities::HIGHEST_FREERTOS, IrqPriorities::HIGHEST_FREERTOS);
        mspCfg = typedCfg;
    }

    new SpiComIF(objects::SPI_COM_IF);
    auto spiCookie = new SpiCookie(0, spi::SpiBus::SPI_1, transferMode, mspCfg, 3900000,
            spi::SpiModes::MODE_3, GPIO_PIN_14, GPIOD, 32);
    auto gyroDevice = new GyroHandlerL3GD20H(objects::SPI_DEVICE_TEST, objects::SPI_COM_IF,
            spiCookie);
    gyroDevice->setStartUpImmediately();
    gyroDevice->setGoNormalModeAtStartup();

This was a lot more work than expected.. - Added `SpiComIF` for the STM32H7. Only tested for the H743ZI board - Added L3GD20 device handler. Device handlers are generic, so we can add them here as well - Tested all 3 transfer modes: Polling, IRQ and DMA. All STM32H7 peripherals offer the same modes, so this ComIF will be an ideal base to implement other communication interfaces. - Only tested with one device - No GpioIF* which would allow custom callbacks. Not worth the effort as long as noone requires it explicitely.. Simple prototyping usually does not require complex decoding logic, SPI chip select are simply driven by dedicated GPIO pins - Bugfixes for the L3GD20H device, conversion was not exactly correct - No ugly macro magic necessary for complete MSP configuration, replaced by proper structs at the price of code size. MSP config struct is deleted automatically by the `SpiComIF`. Here is the example testing code: ```cpp spi::MspCfgBase* mspCfg = nullptr; spi::TransferModes transferMode = spi::TransferModes::DMA; if(transferMode == spi::TransferModes::POLLING) { auto typedCfg = new spi::MspPollingConfigStruct(); spi::h743zi::standardPollingCfg(*typedCfg); mspCfg = typedCfg; } else if(transferMode == spi::TransferModes::INTERRUPT) { auto typedCfg = new spi::MspIrqConfigStruct(); spi::h743zi::standardInterruptCfg(*typedCfg, IrqPriorities::HIGHEST_FREERTOS); mspCfg = typedCfg; } else if(transferMode == spi::TransferModes::DMA) { auto typedCfg = new spi::MspDmaConfigStruct(); txHandle = new DMA_HandleTypeDef(); rxHandle = new DMA_HandleTypeDef(); spi::setDmaHandles(txHandle, rxHandle); spi::h743zi::standardDmaCfg(*typedCfg, IrqPriorities::HIGHEST_FREERTOS, IrqPriorities::HIGHEST_FREERTOS, IrqPriorities::HIGHEST_FREERTOS); mspCfg = typedCfg; } new SpiComIF(objects::SPI_COM_IF); auto spiCookie = new SpiCookie(0, spi::SpiBus::SPI_1, transferMode, mspCfg, 3900000, spi::SpiModes::MODE_3, GPIO_PIN_14, GPIOD, 32); auto gyroDevice = new GyroHandlerL3GD20H(objects::SPI_DEVICE_TEST, objects::SPI_COM_IF, spiCookie); gyroDevice->setStartUpImmediately(); gyroDevice->setGoNormalModeAtStartup(); ```

muellerr added 36 commits 2021-06-11 10:40:43 +02:00

added gyrol3gd20h stm32 test file 4c546820fd

continued stm32 spi code 22384aec6a

added spi hal functions dc6327b909

extended spi core 21414c3594

added IRS abstraction 78a66e1b67

refactored irq code abe62d807a

stll some issues a9390b145b

added msp init functions 863dfa68a0

added polling msp init 66a7a4dbbe

added spi transfer modes ff7421e1dd

polling transfer worked 20815c1d6b