[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) |' V- `* q1 N6 o* [6 f7 |
{
% `+ `# P9 Y' ?3 n; Z
HID_HandleTypeDef *HID_Handle = (HID_HandleTypeDef *) phost->pActiveClass->pData;
) g% }- O9 B; B7 {: U
' o( C# H9 J' w; ^! i2 n
if(HID_Handle->length == 0)& x# k' z7 \9 Y& B3 @
{& h0 q% A3 Q# g! T% [: j- ]% J) {9 s
return USBH_FAIL;
9 W; x( v% a2 J' Z( w& t9 l) ^. y
}9 b9 a' D3 C4 T% m3 O+ p% ]6 l, J
/*Fill report */
2 W1 y% Z. m% ]* N0 n: s v
if(fifo_read(&HID_Handle->fifo, &mouse_report_data, HID_Handle->length) == HID_Handle->length)8 T/ w- i% i; T' H& W
{, Z1 k, y. k# E* ~) J+ F
7 b/ l9 N& e1 {& d) ~- f# x1 J
/*Decode report */+ B% h( z; c, |3 I
/*LDS
: a$ Z5 B0 Z! {0 Q: t! J: p0 G
mouse_info.x = (int16_t )HID_ReadItem((HID_Report_ItemTypedef *) &prop_x, 0);$ A, V, G w% A P
mouse_info.y = (int16_t )HID_ReadItem((HID_Report_ItemTypedef *) &prop_y, 0);
: k# B; J1 p+ C4 n
7 r3 z' M& S. W* i+ m: }
mouse_info.buttons[0]=(uint8_t)HID_ReadItem((HID_Report_ItemTypedef *) &prop_b1, 0);, U r; O0 t& ?% T- q- M; H
mouse_info.buttons[1]=(uint8_t)HID_ReadItem((HID_Report_ItemTypedef *) &prop_b2, 0);' P! m: @3 E" C* O. q. |
mouse_info.buttons[2]=(uint8_t)HID_ReadItem((HID_Report_ItemTypedef *) &prop_b3, 0);
6 f) y0 m5 t& v8 b- S
*/9 ^4 V# [- q" Z: ]. b. z4 G; z
4 F5 i+ g' ?7 F5 T- {9 X+ M
//LDS:4 i6 |# N9 a( T$ v( d
: P1 }0 a: d: B* E9 i8 Z3 s4 I
mouse_info.buttons[0]= mouse_report_data[0];& P. L' a; d! L% I& s: ^5 l9 U/ U
if(HID_Handle->length>4) //5/6/8 ×Ö½Ú³¤¶ÈµÄ USB Êý¾Ý´¦Àí. N: Q, d+ O8 O ]* l8 D9 B
{6 V. y3 x/ O+ |* l+ B
mouse_info.x = mouse_report_data[1];
* s+ Z% f! F {: a
mouse_info.y = mouse_report_data[3]<<4|mouse_report_data[2]>>4;" a' {+ A y& o
}
+ @* P' p+ T: M4 R; q. S, R
else if(HID_Handle->length==4) //4×Ö½Ú³¤¶ÈµÄ USB Êý¾Ý´¦Àí |! P! u( H. @: ~: Y3 T
{- E; ]& P9 F: w" A3 A
mouse_info.x = mouse_report_data[1];9 B6 ]# Q. F8 F& }% w& G" ?
mouse_info.y = mouse_report_data[2];
# n. N0 H" K7 I* c
}
& S' p8 W" A" r, N
0 q. ~+ P/ @" g l. U
return USBH_OK;
, p) g2 b* L9 n2 h+ F4 C- e: E: L+ D& n
}
( x1 U: |. g+ K, [# Q5 R* K; x: ^
return USBH_FAIL;2 @, E/ D7 V" C( S
}