device->user_data在rt-thread驱动层被使用，导致hardfult

[kylongmu]

在官方rt-thread源bsp\wch\risc-v\ch32v307v-r1开发板工程下测试直接死机
msh > hardfult
mepc:00013a5c
mcause:00000004
mtval:50504251
跟踪后在small_modbus_port_rtthread.c文件中

`
static int _modbus_rtdevice_open(small_modbus_t *smb)
{
small_modbus_port_rtdevice_t *smb_port_device = (small_modbus_port_rtdevice_t *)smb->port;
if (smb_port_device->device)
{
//屏蔽下面一行可以执行无硬件报错，应该是这里引起问题
//smb_port_device->device->user_data = smb_port_device;

	rt_device_set_rx_indicate(smb_port_device->device, _modbus_rtdevice_rx_indicate);
	rt_device_open(smb_port_device->device, smb_port_device->oflag);

`

---

把smb_port_device->device->user_data = smb_port_device;
放到rt_device_open后执行，能通过，但是一旦接收到数据，接收数据引用user_data就直接hardfult。看来是user_data的赋值有问题，再引用它的指针时发生了内存越界错误。

进一步检查后发现user_data在Libraries\ch32_drivers\drv_usart.c中被使用了
`
static rt_err_t ch32_configure(struct rt_serial_device *serial, struct serial_configure *cfg)
{
struct ch32_uart *uart;
GPIO_InitTypeDef GPIO_InitStructure={0};

RT_ASSERT(serial != RT_NULL);
RT_ASSERT(cfg != RT_NULL);

uart = (struct ch32_uart *) serial->parent.user_data;//此处被驱动使用

`
这个变量已经被驱动层代码占用了，应用层再用当然指针出错了，device->user_data不能用。

[kylongmu]

在其他系列MCU的drv_usart.c中查看：
`
static rt_err_t hpm_uart_configure(struct rt_serial_device *serial, struct serial_configure *cfg)
{

RT_ASSERT(serial != RT_NULL);
RT_ASSERT(cfg != RT_NULL);

uart_config_t uart_config;
struct hpm_uart *uart  = (struct hpm_uart *)serial->parent.user_data;

user_data也被使用了。 再看stm32的drv_usart.c
static rt_err_t stm32_configure(struct rt_serial_device *serial, struct serial_configure *cfg)
{
struct stm32_uart *uart;
RT_ASSERT(serial != RT_NULL);
RT_ASSERT(cfg != RT_NULL);

uart = rt_container_of(serial, struct stm32_uart, serial);

用了一个奇怪的方式，查到这个宏：
#define rt_container_of(ptr, type, member)
((type *)((char *)(ptr) - (unsigned long)(&((type *)0)->member)))
`
晦涩难懂，但是没有占用user_data
所以这个user_data被其他系列MCU的驱动使用了，只能在stm32上面正确运行。

[cazure]

好吧，在写这个代码前我还研究过这个驱动user_data到底是给驱动做用户数据还是给应用做用户数据使用。
我误认为是给应用层做数据使用的。
所以这个地方得改一下。

[kylongmu]

好吧，在写这个代码前我还研究过这个驱动user_data到底是给驱动做用户数据还是给应用做用户数据使用。 我误认为是给应用层做数据使用的。 所以这个地方得改一下。

多谢修复该问题，另外串口驱动现在有了V2版本，还请考虑一下兼容性

[cazure]

这是我以前提问的帖子 https://club.rt-thread.org/ask/question/ead12ad1be79c8f0.html
V2版本的串口驱动应该不需要改动吧，直接使用中断接收+轮询发送即可。如果是DMA之类的调整接收的帧超时或者字节超时应该也能适应的。