[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)
* `0 {- n2 y4 c& L+ E: I; a) V. ]* a' U
{3 R6 d1 l) }$ B y
HID_HandleTypeDef *HID_Handle = (HID_HandleTypeDef *) phost->pActiveClass->pData;& z, X( g0 |8 J+ o
7 ?# E E; D! ~ |
if(HID_Handle->length == 0)6 N( N, V, o8 E$ @0 O P
{
2 u) c2 G, e1 O$ p7 }% l3 l
return USBH_FAIL;
( k; R9 B# i' K1 J" q' U" h9 i( h
}
( t# a2 v$ |8 h+ _/ N
/*Fill report */1 j/ {2 e8 s) o3 S& ^ P
if(fifo_read(&HID_Handle->fifo, &mouse_report_data, HID_Handle->length) == HID_Handle->length)% u. n( V; J1 u$ r% h& k+ d1 p
{
4 \! m/ z/ X( ?5 ]; G
) _2 `8 ^0 I7 ~ D( X( z
/*Decode report */8 Q6 \7 \0 ^1 ^- Q* D
/*LDS
- [. G" A# c& i9 S5 b5 M9 O
mouse_info.x = (int16_t )HID_ReadItem((HID_Report_ItemTypedef *) &prop_x, 0);
7 Y: X+ o5 R$ C# _9 {
mouse_info.y = (int16_t )HID_ReadItem((HID_Report_ItemTypedef *) &prop_y, 0);
, O( ~* e5 ~+ N& ?8 |
+ |; ^/ M% V0 {1 U/ v4 M
mouse_info.buttons[0]=(uint8_t)HID_ReadItem((HID_Report_ItemTypedef *) &prop_b1, 0);
) x: U, {$ H. l# {5 b7 f! \
mouse_info.buttons[1]=(uint8_t)HID_ReadItem((HID_Report_ItemTypedef *) &prop_b2, 0);: D$ [( R8 q9 g/ t. ^7 [: |; V; ^
mouse_info.buttons[2]=(uint8_t)HID_ReadItem((HID_Report_ItemTypedef *) &prop_b3, 0);
) ?1 P. L6 X! A9 n# V3 t. w
*/
0 G, a7 u6 r* f# a$ ~2 }( C9 g
( d6 c c% Q+ |- S* T
//LDS:
- m% v- w9 {& u. y2 h
7 _0 @3 a* ^' H& C
mouse_info.buttons[0]= mouse_report_data[0];
/ H1 I* t7 e8 e" T$ k3 r( C# ?4 m
if(HID_Handle->length>4) //5/6/8 ×Ö½Ú³¤¶ÈµÄ USB Êý¾Ý´¦Àí }# G! j$ P X8 q" f% e
{
$ [- X5 \4 m# ] k5 ?8 w
mouse_info.x = mouse_report_data[1];
0 j+ R9 h# R4 x, q
mouse_info.y = mouse_report_data[3]<<4|mouse_report_data[2]>>4;
0 T5 U7 n( M/ L6 _, g# ~; r
}5 z5 ?* r1 A+ E" L
else if(HID_Handle->length==4) //4×Ö½Ú³¤¶ÈµÄ USB Êý¾Ý´¦Àí
0 c0 b3 c+ M) U
{; F% `# O& _8 ]
mouse_info.x = mouse_report_data[1];
6 n& Z- ^: K" M, b* r* v1 z
mouse_info.y = mouse_report_data[2];
, G: ^8 t9 T. _0 m
}
4 {/ Z7 q+ F- o
4 y1 B2 ~( u, ]& U0 B* X
return USBH_OK; ! d4 ]! l# J( D$ z4 e( G* L' d( [
}
( {- q9 x6 i- a
return USBH_FAIL; N& s' \+ P8 S* p# o- \: k
}