[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)% C; Q7 V! m+ O7 b8 M
{
& o w5 e) | l
HID_HandleTypeDef *HID_Handle = (HID_HandleTypeDef *) phost->pActiveClass->pData;* t4 z. R2 S4 a* v: G" E
/ q4 H3 [. P) N/ L g. u( J7 K
if(HID_Handle->length == 0); A z, ^( a O# f% ^& _
{$ h0 X: `. v8 C
return USBH_FAIL;: s' q& {; w5 I* E
}
, r' Q' ]: t2 ^& k
/*Fill report */ ?' ?3 d. N! g7 B8 I
if(fifo_read(&HID_Handle->fifo, &mouse_report_data, HID_Handle->length) == HID_Handle->length)2 o0 G- A l5 K- u" l
{
( Y# f: o* Q# M8 j1 v
* K- r1 X6 ]+ y" }' F( T
/*Decode report */) ]$ w6 r, d2 a1 A) v
/*LDS% P( V* ~, b9 j0 L
mouse_info.x = (int16_t )HID_ReadItem((HID_Report_ItemTypedef *) &prop_x, 0);
0 n( o; Z# i1 Z8 i& g! g8 y( c' I
mouse_info.y = (int16_t )HID_ReadItem((HID_Report_ItemTypedef *) &prop_y, 0);# @7 S. R$ J' C, C! V A) |
2 ~7 K+ ^4 M7 |; f' r* t
mouse_info.buttons[0]=(uint8_t)HID_ReadItem((HID_Report_ItemTypedef *) &prop_b1, 0);$ p" B) e2 Z& @# A C8 h2 e4 g
mouse_info.buttons[1]=(uint8_t)HID_ReadItem((HID_Report_ItemTypedef *) &prop_b2, 0);& y6 v0 h" ^1 I$ r7 n* Q
mouse_info.buttons[2]=(uint8_t)HID_ReadItem((HID_Report_ItemTypedef *) &prop_b3, 0);0 F8 q% C3 l, X# u' @" ?- k$ \
*/
- t4 B% K8 J; ?
! g3 l9 K( x5 k$ [9 n
//LDS:
, N" f' V: z( X
" K2 o! V, ? S J0 j9 B
mouse_info.buttons[0]= mouse_report_data[0];) q/ Y% F4 T9 `1 @+ c& s( H
if(HID_Handle->length>4) //5/6/8 ×Ö½Ú³¤¶ÈµÄ USB Êý¾Ý´¦Àí
9 Z' l2 s: l' O8 k. n" B
{9 f' [1 q" U3 h4 Q. ?
mouse_info.x = mouse_report_data[1];
K$ t" w- B8 N$ b
mouse_info.y = mouse_report_data[3]<<4|mouse_report_data[2]>>4;- d# N! K' ]0 T; Z6 m5 p
}
/ o! ^4 A/ c# B' ~2 v0 v! O8 |
else if(HID_Handle->length==4) //4×Ö½Ú³¤¶ÈµÄ USB Êý¾Ý´¦Àí
' c4 U' g6 W$ C2 ?3 |$ S
{$ a3 z, g6 [1 E8 {; n% ~! v2 h
mouse_info.x = mouse_report_data[1];* B+ x0 s/ j6 w: T. F( ]4 v R
mouse_info.y = mouse_report_data[2];) |4 {6 Q" ]/ \, h
}
$ ^* o C) K: s4 L/ c' b
/ v/ l' P5 c( Z1 I/ t/ o& q
return USBH_OK;
5 Q" F( e: A# r% Z
}
# Q: P- I3 _/ C- ?
return USBH_FAIL;/ }$ s' t5 C4 x: l) r4 u* e
}