RT_NAME_MAX引发的惨案

发布于 2021-01-13 11:20:49 浏览：2163 订阅该版

背景：
**ART-Pi** 板载了两颗 `FLASH`. (SPI FLASH 16M)(QSPI FLASH 8M) 于是就尝试了 `SFUD` 和 `FAL` 一起使用，在使用过程中发现了`RT_NAME_MAX` 导致的 `FLASH` 容量识别错误的问题。

现象：
先看测试代码
```c
    extern rt_spi_flash_device_t rt_sfud_flash_probe(const char *spi_flash_dev_name, const char *spi_dev_name);
    extern int fal_init(void);
    const struct fal_flash_dev *flash_dev = NULL;
    
    rt_hw_spi_device_attach("spi1", "spi10", GPIOA, GPIO_PIN_4);
    
    /* initialize SPI Flash device */
    if (RT_NULL == rt_sfud_flash_probe("norflash0", "spi10"))
    {
        rt_kprintf("sfud error!!\r\n");
        return -RT_ERROR;
    };    
    
    
    stm32_qspi_bus_attach_device("qspi1", "qspi10", RT_NULL, 4, w25qxx_enter_qspi_mode, RT_NULL);
    if (RT_NULL == rt_sfud_flash_probe("norflash1", "qspi10"))
    {
        return -RT_ERROR;
    }

fal_init();
    
    flash_dev = fal_flash_device_find("norflash0");
    
    rt_kprintf("spi_flash_dev %x \r\n", flash_dev->len);
    
    flash_dev = fal_flash_device_find("norflash1");
    rt_kprintf("qspi_flash_dev %x \r\n", flash_dev->len);
```
输出结果：
```
spi_flash_dev 1000000 
qspi_flash_dev 1000000

```
问题：
容量不一致的 `FLASH` 结果显示了一样的容量。

修改代码：
```c
    stm32_qspi_bus_attach_device("qspi1", "qspi10", RT_NULL, 4, w25qxx_enter_qspi_mode, RT_NULL);
    if (RT_NULL == rt_sfud_flash_probe("norflash1", "qspi10"))
    {
        return -RT_ERROR;
    }   
    
    rt_hw_spi_device_attach("spi1", "spi10", GPIOA, GPIO_PIN_4);
    
    /* initialize SPI Flash device */
    if (RT_NULL == rt_sfud_flash_probe("norflash0", "spi10"))
    {
        rt_kprintf("sfud error!!\r\n");
        return -RT_ERROR;
    };  
    
fal_init();  
```
调整初始化顺序
输出结果：
```
spi_flash_dev 800000 
qspi_flash_dev 800000 
```

可得结论：
谁先初始化探测到的容量就是谁的。

解决办法：
```c
#define RT_NAME_MAX 9
```

问题原因：
1. `FAL` 软件包默认设置的设备名字是 `norflash0`，长度为 9
2. `SFUD` 中回调用`rt_device_register` 注册设备，这里会注册两次设备，因为`RT_NAME_MAX = 8` 会导致第二个设备无法正确注册,会返回错误。但是依然会`return rtt_dev;`,更安全的做法时对 `rt_device_register`的返回值做出判断。
3. 在`fal_flash_sfud_port.c`中使用到了  `sfud_dev = rt_sfud_flash_find_by_dev_name(FAL_USING_NOR_FLASH_DEV_NAME);` 为了省事，对 `QSPI FLASH`的操作直接拷贝了这个文件的代码进行修改。
`rt_sfud_flash_find_by_dev_name` 中使用到了 `rt_device_find`，那么`RT_NAME_MAX = 8` 所以只能查找到 `norflash` 字符串被截断了。