[BUG] ST库的USB鼠标小BUG发现

[复制链接]

ldskendy 发布时间：2018-2-5 10:00

文章
文章封面:	-
文章简介:	-

本帖最后由 ldskendy 于 2018-2-5 11:34 编辑

  最近用EMWIN做的一个控制检测界面，用的是STM32F746DISCO开发板，4.7寸480*272的屏，由于控扭较多，所以每个按扭做的都较小，用手指去按的话有时很难点中，所以就想加一个USB鼠标进去操作。
  没想到ST的库兼容性不好，又害我调试了一天，才发现问题，这个问题正点原子的USB鼠标有提到，但他用的库是比较早的一个版本，ST最近更新的1.9都没有去解决，下面进入正题。

  开发环境：WIN7_64Bit + MDK5.24.1
  固件库：STM32Cube_FW_F7_V1.9.0
  EMWIN库： STemWin540_CM7_Keil.lib

  usbh_hid_mouse.c文件修改如下：
  1. L92：uint32_t mouse_report_data[1];
  该变量用于保存鼠标的报告数据，但有些鼠标是报告4字节，有些为5字节，还有可能6，7，8个字节，本次我使用的鼠标就是报告5个字节，这里修改为：uint8_t mouse_report_data[8];

  2. L227对应的static USBH_StatusTypeDef USBH_HID_MouseDecode(USBH_HandleTypeDef *phost)函数在对报告数据的处理上就必须分开处理5个字节及以上的数据，这里处理方式参考正点原子的代码。修改如下：（不使用库里的HID_ReadItem函数，数据提取繁锁，浪费时间）

static USBH_StatusTypeDef USBH_HID_MouseDecode(USBH_HandleTypeDef *phost)
9 b$ U* T' v ]$ g
{
% p( D3 b. |& ~$ @' |
HID_HandleTypeDef *HID_Handle = (HID_HandleTypeDef *) phost->pActiveClass->pData;* a* M' _7 Q- @9 A' z1 h1 |7 g
2 M0 I' E3 q8 ~$ ` V I
if(HID_Handle->length == 0)! |3 p$ |& k3 Q c7 l+ T- g
{
$ w' u: } S/ V+ B3 I6 |
return USBH_FAIL;# }2 ~5 }* @1 ] ^( R$ a5 y
}! Y/ Y1 @' g1 l$ ?2 x# R
/*Fill report */ ?' ^' T5 _8 V! [8 @0 {. M
if(fifo_read(&HID_Handle->fifo, &mouse_report_data, HID_Handle->length) == HID_Handle->length)- [8 | }& f E
{- d" }& |8 G J/ w$ j& n$ k
, q& |. e+ H* {
/*Decode report */% [4 d" e" F1 M
/*LDS
- l( O7 B, Z/ ]" a
mouse_info.x = (int16_t )HID_ReadItem((HID_Report_ItemTypedef *) &prop_x, 0);7 x- F# _7 B! _" Y$ Q: v2 a
mouse_info.y = (int16_t )HID_ReadItem((HID_Report_ItemTypedef *) &prop_y, 0);; o' F, S: L6 w; q
3 R4 C6 [) z5 K' N% q7 Y, |
mouse_info.buttons[0]=(uint8_t)HID_ReadItem((HID_Report_ItemTypedef *) &prop_b1, 0);
/ M/ E) Q6 G1 s
mouse_info.buttons[1]=(uint8_t)HID_ReadItem((HID_Report_ItemTypedef *) &prop_b2, 0);
: D6 X9 Z# F1 h+ C; b3 X- z5 B* @- w2 U
mouse_info.buttons[2]=(uint8_t)HID_ReadItem((HID_Report_ItemTypedef *) &prop_b3, 0);
, H" F& u6 a* K/ m* z# w9 m5 u) W
*/" f3 z9 Y5 O6 h& g( \8 @2 x. e, Q
9 B( Q% S& t2 ^( \ q" ?
//LDS:. E) s) Y: x/ U) n, j# V
% Z7 u. ?# F! U$ D8 z, g# z
mouse_info.buttons[0]= mouse_report_data[0];5 K2 k1 h. J* _$ x4 W
if(HID_Handle->length>4) //5/6/8 ×Ö½Ú³¤¶ÈµÄ USB Êý¾Ý´¦Àí7 }5 F) K7 x. }6 |' ]7 }
{
2 A% T, N. I! X/ u4 J( R# m
mouse_info.x = mouse_report_data[1];1 [3 t; d' l3 A" M, T) D0 \
mouse_info.y = mouse_report_data[3]<<4|mouse_report_data[2]>>4;7 k) S/ y. I0 c* b, v. d
}
1 E- s( m$ q8 {4 z c2 \
else if(HID_Handle->length==4) //4×Ö½Ú³¤¶ÈµÄ USB Êý¾Ý´¦Àí9 H, x/ R @$ @3 s; q: t9 f! T! n
{2 X! n" @ f* B
mouse_info.x = mouse_report_data[1];8 P6 a# Y2 A1 `& v, W1 f
mouse_info.y = mouse_report_data[2];; a1 O( r# b. h4 g- A3 G# |* t+ Z& F% W
}- x( E' W* t3 Y6 o' L% y2 k, B
0 [" k9 w! X. g5 d
return USBH_OK; * [& j M7 k% E9 y6 |0 r! ?/ U) c
}
4 M# ?5 G5 W* O1 f0 W
return USBH_FAIL;
0 u: t5 t$ t/ `0 o+ |+ g
}