[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)
. m( J0 z, x" [' {
{" @/ Q8 r' |' S9 G' Z
HID_HandleTypeDef *HID_Handle = (HID_HandleTypeDef *) phost->pActiveClass->pData;
) G# Y/ G' S, ~ \4 H
( Q) Y0 x. h( }! j
if(HID_Handle->length == 0)& q8 b7 {2 H- F m- F
{# x, r& B! I4 V
return USBH_FAIL;) [0 P' F7 K/ e8 h; Q! ]) l
}
( {9 z M$ N* f( |1 [6 s* v
/*Fill report */% ?6 s" {; i2 _) e0 x
if(fifo_read(&HID_Handle->fifo, &mouse_report_data, HID_Handle->length) == HID_Handle->length)
, [9 N p+ z& m, p& G: G. k
{
6 i# F# g2 t3 K, n
; D/ [) g6 {/ B: X& I
/*Decode report */% y7 v; O0 S) C5 S, q
/*LDS
. ~: C3 F! Y. m+ {$ M( P
mouse_info.x = (int16_t )HID_ReadItem((HID_Report_ItemTypedef *) &prop_x, 0);
2 Y1 Z2 Y' M/ T( X
mouse_info.y = (int16_t )HID_ReadItem((HID_Report_ItemTypedef *) &prop_y, 0);* M( o+ M9 r: t0 K. I" r
$ R1 |, U9 i- i8 s) b/ r0 p
mouse_info.buttons[0]=(uint8_t)HID_ReadItem((HID_Report_ItemTypedef *) &prop_b1, 0);3 p3 F& |5 a5 @4 W2 a1 q
mouse_info.buttons[1]=(uint8_t)HID_ReadItem((HID_Report_ItemTypedef *) &prop_b2, 0);
, B5 Z& \( M0 l' \
mouse_info.buttons[2]=(uint8_t)HID_ReadItem((HID_Report_ItemTypedef *) &prop_b3, 0);. K5 i: X2 V' b4 J9 ^9 U& X9 g
*/
+ L8 n: |. H" U
0 g+ x/ n7 ?& M
//LDS:1 O: Q* t' s$ W* k4 D
1 j9 N6 g4 J, ]1 E0 {
mouse_info.buttons[0]= mouse_report_data[0];
3 ]9 a! p/ C5 A/ ?2 b
if(HID_Handle->length>4) //5/6/8 ×Ö½Ú³¤¶ÈµÄ USB Êý¾Ý´¦Àí
5 n# _* N: Z7 d8 r- _
{
8 Q) B( W; k! X5 z" c' N8 N
mouse_info.x = mouse_report_data[1];5 o% _4 h9 d) ?. B
mouse_info.y = mouse_report_data[3]<<4|mouse_report_data[2]>>4;' |( \3 k, k. m
}% P4 }) S! s1 o
else if(HID_Handle->length==4) //4×Ö½Ú³¤¶ÈµÄ USB Êý¾Ý´¦Àí
- z) v# \' A6 e* T) f8 d
{
$ n8 P: m* W, }# w
mouse_info.x = mouse_report_data[1];
* G' j0 l0 f; h1 ^! B6 P
mouse_info.y = mouse_report_data[2];6 L- I9 R6 n0 m ^5 ]9 T; n3 _
}
& \8 Z" P1 d7 L
( P& p5 x) A/ V1 f4 `4 c
return USBH_OK;
' v! P; F; J R, ] s
}! l/ m0 W# X) `& V: X2 \
return USBH_FAIL;
) O5 a; V9 z6 E8 H$ W
}