[FL-1666] Bootloader: info screen when we going to DFU. FuriHal: port SPI to LL. Cleanup. (#634)
* FuriHal: port spi to ll. Bootloader: add spi and display. * Makefile: rollback disabled freertos introspection * FuriHal: spi lock asserts. F6: minor cleanup port sdcard shenanigans to furi_hal_gpio. * SdCard: port missing bits to furi-hal-gpio * FuriHal: fix broken RX in SPI, update SPI API usage. RFAL: more asserts in SPI platform code. * GUI: clear canvas on start. FuriHal: no pullup on radio spi bus. * FuriHal: use check instead of assert in spi lock routines * FuriHal: remove timeouts * SdHal: add guard time to SDCARD CS PIN control. * FuriHal: proper name for SPI device reconfigure routine. SdHal: one more enterprise delay and better documentation. * Bootloader: update DFU text and add image. * FuriHal: drop unused ST HAL modules. * SdHal: fixed swapped hal_gpio_init_ex arguments * SpiHal: fixed swapped hal_gpio_init_ex arguments * IrdaHal: use hal_gpio_init instead of hal_gpio_init_ex * RfidHal: fixed swapped hal_gpio_init_ex arguments Co-authored-by: DrZlo13 <who.just.the.doctor@gmail.com>
This commit is contained in:
@@ -1,26 +1,18 @@
|
||||
#include "main.h"
|
||||
#include "furi-hal-spi.h"
|
||||
#include <furi-hal.h>
|
||||
#include <furi.h>
|
||||
|
||||
#define SD_DUMMY_BYTE 0xFF
|
||||
|
||||
const uint32_t SpiTimeout = 1000;
|
||||
uint8_t SD_IO_WriteByte(uint8_t Data);
|
||||
static const FuriHalSpiDevice* sd_spi_dev = &furi_hal_spi_devices[FuriHalSpiDeviceIdSdCardFast];
|
||||
|
||||
static const FuriHalSpiDevice* sd_spi_dev = &furi_hal_spi_devices[FuriHalSpiDeviceIdSdCardFast];
|
||||
|
||||
/******************************************************************************
|
||||
BUS OPERATIONS
|
||||
*******************************************************************************/
|
||||
|
||||
/**
|
||||
* @brief SPI error treatment function
|
||||
* @retval None
|
||||
*/
|
||||
static void SPIx_Error(void) {
|
||||
/* Re-Initiaize the SPI communication BUS */
|
||||
furi_hal_spi_bus_reset(sd_spi_dev->bus);
|
||||
}
|
||||
|
||||
/**
|
||||
* @brief SPI Write byte(s) to device
|
||||
* @param DataIn: Pointer to data buffer to write
|
||||
@@ -29,13 +21,7 @@ static void SPIx_Error(void) {
|
||||
* @retval None
|
||||
*/
|
||||
static void SPIx_WriteReadData(const uint8_t* DataIn, uint8_t* DataOut, uint16_t DataLength) {
|
||||
bool status = furi_hal_spi_bus_trx(sd_spi_dev->bus, (uint8_t*)DataIn, DataOut, DataLength, SpiTimeout);
|
||||
|
||||
/* Check the communication status */
|
||||
if(!status) {
|
||||
/* Execute user timeout callback */
|
||||
SPIx_Error();
|
||||
}
|
||||
furi_check(furi_hal_spi_bus_trx(sd_spi_dev->bus, (uint8_t*)DataIn, DataOut, DataLength, SpiTimeout));
|
||||
}
|
||||
|
||||
/**
|
||||
@@ -44,15 +30,7 @@ static void SPIx_WriteReadData(const uint8_t* DataIn, uint8_t* DataOut, uint16_t
|
||||
* @retval None
|
||||
*/
|
||||
__attribute__((unused)) static void SPIx_Write(uint8_t Value) {
|
||||
uint8_t data;
|
||||
|
||||
bool status = furi_hal_spi_bus_trx(sd_spi_dev->bus, (uint8_t*)&Value, &data, 1, SpiTimeout);
|
||||
|
||||
/* Check the communication status */
|
||||
if(!status) {
|
||||
/* Execute user timeout callback */
|
||||
SPIx_Error();
|
||||
}
|
||||
furi_check(furi_hal_spi_bus_tx(sd_spi_dev->bus, (uint8_t*)&Value, 1, SpiTimeout));
|
||||
}
|
||||
|
||||
/******************************************************************************
|
||||
@@ -70,6 +48,7 @@ void SD_IO_Init(void) {
|
||||
|
||||
/* SD chip select high */
|
||||
hal_gpio_write(sd_spi_dev->chip_select, true);
|
||||
delay_us(10);
|
||||
|
||||
/* Send dummy byte 0xFF, 10 times with CS high */
|
||||
/* Rise CS and MOSI for 80 clocks cycles */
|
||||
@@ -85,10 +64,13 @@ void SD_IO_Init(void) {
|
||||
* @retval None
|
||||
*/
|
||||
void SD_IO_CSState(uint8_t val) {
|
||||
/* Some SD Cards are prone to fail if CLK-ed too soon after CS transition. Worst case found: 8us */
|
||||
if(val == 1) {
|
||||
delay_us(10); // Exit guard time for some SD cards
|
||||
hal_gpio_write(sd_spi_dev->chip_select, true);
|
||||
} else {
|
||||
hal_gpio_write(sd_spi_dev->chip_select, false);
|
||||
delay_us(10); // Entry guard time for some SD cards
|
||||
}
|
||||
}
|
||||
|
||||
|
@@ -91,8 +91,9 @@
|
||||
#include "stdlib.h"
|
||||
#include "string.h"
|
||||
#include "stdio.h"
|
||||
#include "spi.h"
|
||||
#include <furi-hal-spi.h>
|
||||
#include <furi-hal-gpio.h>
|
||||
#include <furi-hal-resources.h>
|
||||
#include <furi-hal-power.h>
|
||||
#include <furi-hal-delay.h>
|
||||
#include <furi-hal-sd.h>
|
||||
@@ -282,6 +283,25 @@ static uint8_t SD_ReadData(void);
|
||||
|
||||
/* Private functions ---------------------------------------------------------*/
|
||||
|
||||
void SD_SPI_Bus_To_Down_State(){
|
||||
hal_gpio_init_ex(&gpio_spi_d_miso, GpioModeOutputPushPull, GpioPullNo, GpioSpeedVeryHigh, GpioAltFnUnused);
|
||||
hal_gpio_init_ex(&gpio_spi_d_mosi, GpioModeOutputPushPull, GpioPullNo, GpioSpeedVeryHigh, GpioAltFnUnused);
|
||||
hal_gpio_init_ex(&gpio_spi_d_sck, GpioModeOutputPushPull, GpioPullNo, GpioSpeedVeryHigh, GpioAltFnUnused);
|
||||
|
||||
hal_gpio_write(&gpio_sdcard_cs, false);
|
||||
hal_gpio_write(&gpio_spi_d_miso, false);
|
||||
hal_gpio_write(&gpio_spi_d_mosi, false);
|
||||
hal_gpio_write(&gpio_spi_d_sck, false);
|
||||
}
|
||||
|
||||
void SD_SPI_Bus_To_Normal_State(){
|
||||
hal_gpio_write(&gpio_sdcard_cs, true);
|
||||
|
||||
hal_gpio_init_ex(&gpio_spi_d_miso, GpioModeAltFunctionPushPull, GpioPullUp, GpioSpeedVeryHigh, GpioAltFn5SPI2);
|
||||
hal_gpio_init_ex(&gpio_spi_d_mosi, GpioModeAltFunctionPushPull, GpioPullUp, GpioSpeedVeryHigh, GpioAltFn5SPI2);
|
||||
hal_gpio_init_ex(&gpio_spi_d_sck, GpioModeAltFunctionPushPull, GpioPullUp, GpioSpeedVeryHigh, GpioAltFn5SPI2);
|
||||
}
|
||||
|
||||
/** @defgroup STM32_ADAFRUIT_SD_Private_Functions
|
||||
* @{
|
||||
*/
|
||||
@@ -295,9 +315,7 @@ static uint8_t SD_ReadData(void);
|
||||
*/
|
||||
uint8_t BSP_SD_Init(bool reset_card) {
|
||||
/* Slow speed init */
|
||||
const FuriHalSpiDevice* sd_spi_slow_dev = &furi_hal_spi_devices[FuriHalSpiDeviceIdSdCardSlow];
|
||||
furi_hal_spi_bus_lock(sd_spi_slow_dev->bus);
|
||||
furi_hal_spi_bus_configure(sd_spi_slow_dev->bus, sd_spi_slow_dev->config);
|
||||
const FuriHalSpiDevice* sd_spi_slow_dev = furi_hal_spi_device_get(FuriHalSpiDeviceIdSdCardSlow);
|
||||
|
||||
/* We must reset card in spi_lock context */
|
||||
if(reset_card) {
|
||||
@@ -326,7 +344,7 @@ uint8_t BSP_SD_Init(bool reset_card) {
|
||||
if(res == BSP_SD_OK) break;
|
||||
}
|
||||
|
||||
furi_hal_spi_bus_unlock(sd_spi_slow_dev->bus);
|
||||
furi_hal_spi_device_return(sd_spi_slow_dev);
|
||||
|
||||
/* SD initialized and set to SPI mode properly */
|
||||
return res;
|
||||
|
@@ -35,7 +35,6 @@
|
||||
|
||||
/* Includes ------------------------------------------------------------------*/
|
||||
#include "user_diskio.h"
|
||||
#include "spi.h"
|
||||
#include "furi-hal-spi.h"
|
||||
/* Private typedef -----------------------------------------------------------*/
|
||||
/* Private define ------------------------------------------------------------*/
|
||||
@@ -53,7 +52,6 @@ static DSTATUS User_CheckStatus(BYTE lun) {
|
||||
return Stat;
|
||||
}
|
||||
|
||||
static const FuriHalSpiDevice* sd_spi_fast_dev = &furi_hal_spi_devices[FuriHalSpiDeviceIdSdCardFast];
|
||||
/* USER CODE END DECL */
|
||||
|
||||
/* Private function prototypes -----------------------------------------------*/
|
||||
@@ -89,12 +87,11 @@ Diskio_drvTypeDef USER_Driver = {
|
||||
DSTATUS USER_initialize(BYTE pdrv) {
|
||||
/* USER CODE BEGIN INIT */
|
||||
|
||||
furi_hal_spi_bus_lock(sd_spi_fast_dev->bus);
|
||||
furi_hal_spi_bus_configure(sd_spi_fast_dev->bus, sd_spi_fast_dev->config);
|
||||
const FuriHalSpiDevice* sd_spi_fast_dev = furi_hal_spi_device_get(FuriHalSpiDeviceIdSdCardFast);
|
||||
|
||||
DSTATUS status = User_CheckStatus(pdrv);
|
||||
|
||||
furi_hal_spi_bus_unlock(sd_spi_fast_dev->bus);
|
||||
furi_hal_spi_device_return(sd_spi_fast_dev);
|
||||
|
||||
return status;
|
||||
/* USER CODE END INIT */
|
||||
@@ -123,8 +120,7 @@ DRESULT USER_read(BYTE pdrv, BYTE* buff, DWORD sector, UINT count) {
|
||||
/* USER CODE BEGIN READ */
|
||||
DRESULT res = RES_ERROR;
|
||||
|
||||
furi_hal_spi_bus_lock(sd_spi_fast_dev->bus);
|
||||
furi_hal_spi_bus_configure(sd_spi_fast_dev->bus, sd_spi_fast_dev->config);
|
||||
const FuriHalSpiDevice* sd_spi_fast_dev = furi_hal_spi_device_get(FuriHalSpiDeviceIdSdCardFast);
|
||||
|
||||
if(BSP_SD_ReadBlocks((uint32_t*)buff, (uint32_t)(sector), count, SD_DATATIMEOUT) == MSD_OK) {
|
||||
/* wait until the read operation is finished */
|
||||
@@ -133,7 +129,7 @@ DRESULT USER_read(BYTE pdrv, BYTE* buff, DWORD sector, UINT count) {
|
||||
res = RES_OK;
|
||||
}
|
||||
|
||||
furi_hal_spi_bus_unlock(sd_spi_fast_dev->bus);
|
||||
furi_hal_spi_device_return(sd_spi_fast_dev);
|
||||
|
||||
return res;
|
||||
/* USER CODE END READ */
|
||||
@@ -153,8 +149,7 @@ DRESULT USER_write(BYTE pdrv, const BYTE* buff, DWORD sector, UINT count) {
|
||||
/* USER CODE HERE */
|
||||
DRESULT res = RES_ERROR;
|
||||
|
||||
furi_hal_spi_bus_lock(sd_spi_fast_dev->bus);
|
||||
furi_hal_spi_bus_configure(sd_spi_fast_dev->bus, sd_spi_fast_dev->config);
|
||||
const FuriHalSpiDevice* sd_spi_fast_dev = furi_hal_spi_device_get(FuriHalSpiDeviceIdSdCardFast);
|
||||
|
||||
if(BSP_SD_WriteBlocks((uint32_t*)buff, (uint32_t)(sector), count, SD_DATATIMEOUT) == MSD_OK) {
|
||||
/* wait until the Write operation is finished */
|
||||
@@ -163,7 +158,7 @@ DRESULT USER_write(BYTE pdrv, const BYTE* buff, DWORD sector, UINT count) {
|
||||
res = RES_OK;
|
||||
}
|
||||
|
||||
furi_hal_spi_bus_unlock(sd_spi_fast_dev->bus);
|
||||
furi_hal_spi_device_return(sd_spi_fast_dev);
|
||||
|
||||
return res;
|
||||
/* USER CODE END WRITE */
|
||||
@@ -185,8 +180,7 @@ DRESULT USER_ioctl(BYTE pdrv, BYTE cmd, void* buff) {
|
||||
|
||||
if(Stat & STA_NOINIT) return RES_NOTRDY;
|
||||
|
||||
furi_hal_spi_bus_lock(sd_spi_fast_dev->bus);
|
||||
furi_hal_spi_bus_configure(sd_spi_fast_dev->bus, sd_spi_fast_dev->config);
|
||||
const FuriHalSpiDevice* sd_spi_fast_dev = furi_hal_spi_device_get(FuriHalSpiDeviceIdSdCardFast);
|
||||
|
||||
switch(cmd) {
|
||||
/* Make sure that no pending write process */
|
||||
@@ -219,7 +213,7 @@ DRESULT USER_ioctl(BYTE pdrv, BYTE cmd, void* buff) {
|
||||
res = RES_PARERR;
|
||||
}
|
||||
|
||||
furi_hal_spi_bus_unlock(sd_spi_fast_dev->bus);
|
||||
furi_hal_spi_device_return(sd_spi_fast_dev);
|
||||
|
||||
return res;
|
||||
/* USER CODE END IOCTL */
|
||||
|
Reference in New Issue
Block a user