yuping

一般人手按键动作，从按下到稳定，大概在1mS-10mS之间（根据按键类型不同而不同），稳定期大概在50mS-200mS，当然不算长按。

单片机上一般延时10mS足够去抖（前提是无硬件防抖）。

CE上因为中断响应慢（典型10mS），估计等中断函数开始响应时，抖动已经过去了，所以根据实际情况测试，这部分延时可以考虑很短甚至不用。

针对你的情况，建议你试试这样：
用示波器抓波形，看看一般你快按快放，到底多长时间，多测几次，有个基本值。

在中断响应函数里，什么都不做，即进去就return，这个时间该算是最短的。如果这样还无法满足先降后升的时间，那就不需要考虑延时的问题，而是要考虑目前中断处理里面到底有什么导致了处理慢了。

摸清了这两点，才可有根据的进行下一步。

tjj104923185

谢谢shuiyan前辈。

我把IST改成什么都不做，果然，就可以双边触发了。无论我怎么按都会产生两次中断。

1. BOOL
   
2. KeybdIstLoop(
   
3.     PKEYBD_IST pKeybdIst
   
4.     )
   
5. {
   
6.     SETFNAME(_T("KeybdIstLoop"));
   
7. 
   
8.     UINT32  rguiScanCode[16];
   
9.     BOOL    rgfKeyUp[16];
   
10.     UINT    cEvents;
    
11.         // add by wogo at 2009-03-12
    
12.         RETAILMSG(1, (TEXT("In KeybdIstLoop Funtion:\r\n")));
    
13.     DEBUGCHK(pKeybdIst->hevInterrupt != NULL);
    
14.     DEBUGCHK(pKeybdIst->pfnGetKeybdEvent != NULL);
    
15.     DEBUGCHK(pKeybdIst->pfnKeybdEvent != NULL);
    
16. 
    
17.     SetThreadPriority(GetCurrentThread(), THREAD_PRIORITY_HIGHEST);
    
18. 
    
19. wait_for_keybd_interrupt:
    
20.     if (WaitForSingleObject(pKeybdIst->hevInterrupt, INFINITE) == WAIT_OBJECT_0)
    
21.     {
    
22.                 // add by wogo at 2009-03-12
    
23.                 //RETAILMSG(1, (TEXT("Deal with hevInterrupt@@@!!! :\r\n")));
    
24.         #if 0
    
25.         cEvents = (*pKeybdIst->pfnGetKeybdEvent)
    
26.             (pKeybdIst->uiPddId, rguiScanCode, rgfKeyUp);
    
27.             
    
28.         for (UINT iEvent = 0; iEvent < cEvents; ++iEvent) {
    
29.             (*pKeybdIst->pfnKeybdEvent)(pKeybdIst->uiPddId,
    
30.                 rguiScanCode[iEvent], rgfKeyUp[iEvent]);
    
31.         }
    
32.         // cEvents could be 0 if this was a partial scan code, like 0xE0
    
33.      #endif
    
34. 
    
35.         InterruptDone(pKeybdIst->dwSysIntr_Keybd);
    
36.                 // add by wogo at 2009-03-12
    
37.                 RETAILMSG(1, (TEXT("InterruptDone(pKeybdIst->dwSysIntr_Keybd)@@@!!! :\r\n")));
    
38.     }
    
39. 
    
40.     goto wait_for_keybd_interrupt;
    
41. 
    
42.     ERRORMSG(1, (TEXT("KeybdIstLoop: Keyboard driver thread terminating.\r\n")));
    
43.         RETAILMSG(1, (TEXT("KeybdIstLoop: Keyboard driver thread terminating!!! :\r\n")));
    
44.     return TRUE;
    
45. }

复制代码

不过被屏蔽部分正好是扫描键值的函数，我想想怎么把这个函数尽量减小。

从这个可以看出，这个双边触发的响应速度没有配置成单边触发的快。

anylzer

不行啊，我已经把驱动的很多延时都删除了，还是不能响应弹起中断。

咋整的啊。

真是知道原因就是搞不定。

mjdhj_520

1.     #if 0
   
2.         cEvents = (*pKeybdIst->pfnGetKeybdEvent)(pKeybdIst->uiPddId, rguiScanCode, rgfKeyUp);
   
3.             
   
4.         for (UINT iEvent = 0; iEvent < cEvents; ++iEvent) {
   
5.             (*pKeybdIst->pfnKeybdEvent)(pKeybdIst->uiPddId,rguiScanCode[iEvent], rgfKeyUp[iEvent]);
   
6.         }
   
7.         // cEvents could be 0 if this was a partial scan code, like 0xE0
   
8.      #endif

复制代码

怎么这么复杂？键盘芯片不是直接通过I2C读寄存器吗？怎么还需要循环cEvents次？cEvents一般是多少？
for里面的(*pKeybdIst->pfnKeybdEvent)到底执行什么功能？


btw:你怎么用goto来实现WaitForSingleObject的死循环呢？极其少见呀。

kly

1. 回shuiyan前辈，这个上面的代码不是我写的啊，是微软自带的键盘IST，我只是移植到BSP下编译，这样可以增加打印信息。
   
2. 
   
3. 这样写没有问题吧。
   
4. 
   
5. 
   
6. (*pKeybdIst->pfnKeybdEvent)是BSP下的代码转换过来的。
   
7. 这里有
   
8. 
   
9. KEYBD_IST keybdIst;
   
10.         keybdIst.hevInterrupt = m_hevInterrupt;
    
11.         keybdIst.dwSysIntr_Keybd = dwSysIntr_Keybd;
    
12.         keybdIst.uiPddId = v_uiPddId;
    
13.         keybdIst.pfnGetKeybdEvent = KeybdPdd_GetEventEx2;
    
14.         keybdIst.pfnKeybdEvent = v_pfnKeybdEvent;
    
15.                
    
16.         KeybdIstLoop(&keybdIst);
    
17. 
    
18. KeybdPdd_GetEventEx2是扫面键盘的函数，里面调用了7290驱动，我也纳闷啊，代码没有改，在4.2BSP行，在5.0就不行了。
    
19. 
    
20. //主要的扫描码处理函数
    
21. static UINT KeybdPdd_GetEventEx2(UINT uiPddId, UINT32 rguiScanCode[16], BOOL rgfKeyUp[16])
    
22. {
    
23.     SETFNAME(_T("KeybdPdd_GetEventEx2"));
    
24. 
    
25.     UINT32   scInProgress = 0;
    
26.     static UINT32   scPrevious;
    
27. 
    
28.     BOOL            fKeyUp;
    
29.     UINT8           ui8ScanCode=0;
    
30.     UINT            cEvents = 0;
    
31. //        int                                i;
    
32. 
    
33.         unsigned char   key[4];
    
34. 
    
35.     //DEBUGCHK(rguiScanCode != NULL);
    
36.     //DEBUGCHK(rgfKeyUp != NULL);
    
37. 
    
38.         //for(i=0;i<4;i++)
    
39.         //        key[i] = 0;
    
40.         //getsFromKBCTL(&ui8ScanCode, 1);
    
41.         ZLG7290_GetKey(key);
    
42.        
    
43. //        RETAILMSG(1, (_T("zlg7290 get keys: %x %x %x %x \r\n"),key[0],key[1],key[2],key[3]));
    
44. //        OEMWriteDebugString((TEXT("zlg7290 get keys!!!!\r\n")));
    
45.        
    
46. //        RETAILMSG(1, (_T("v_pIICregs->rIICCON : %x\r\n"),v_pIICregs->rIICCON));
    
47. //        RETAILMSG(1, (_T("v_pIOPregs->rGPECON : %x\r\n"),v_pIOPregs->rGPECON));
    
48. 
    
49.         if(key[0]&0x01) //key down
    
50.         {               
    
51.                 fKeyUp = FALSE;
    
52.                 // enable rising int trigger
    
53.                 //v_pIOPregs->rEXTINT1 &= ~(0x7 << 12);   // Clear EINT11
    
54.             //v_pIOPregs->rEXTINT1 |= (0x2 << 8);             // fallig edge triggered for EINT11
    
55.                 //v_pIOPregs->rEXTINT1 |= (0x6 << 12);                // both edge
    
56.                 //v_pIOPregs->rEXTINT1 |= (0x1 << 14);        // enable rising int trigger
    
57.         }
    
58.         else//key up
    
59.         {               
    
60.                 fKeyUp = TRUE;
    
61.                 // enable falling int trigger
    
62.                 //v_pIOPregs->rEXTINT1 &= ~(0x7 << 12);   // Low level  triggered
    
63.             //v_pIOPregs->rEXTINT1 |= (0x2 << 8);             // fallig edge triggered for EINT11
    
64.                 //v_pIOPregs->rEXTINT1 |= (0x6 << 12);                // both edge
    
65.                 //v_pIOPregs->rEXTINT1 |= (0x1 << 13);        //  enable falling int trigger
    
66.         }
    
67. 
    
68. /*        if( (key[3] & 0x01) == 0)//F1
    
69.         {
    
70.                 ui8ScanCode = 0x01;                       
    
71.         }
    
72.         else if( (key[3] & 0x02) == 0)//F2
    
73.         {
    
74.                 ui8ScanCode        = 0x02;               
    
75.         }
    
76.         else if( (key[3] & 0x04) == 0)//F3
    
77.         {
    
78.                 ui8ScanCode        = 0x03;       
    
79.         }
    
80.         else if( (key[3] & 0x08) == 0)//F4
    
81.         {
    
82.                 ui8ScanCode = 0x04;                       
    
83.         }
    
84.         else */
    
85.         if(key[1] != 0x0)//other KEYS
    
86.         {                       
    
87.                 ui8ScanCode = key[1];       
    
88.         }
    
89.         else
    
90.         {
    
91.                 ui8ScanCode = 0;
    
92.         }
    
93.         //sent bl key press event to backlight driver
    
94.         /*if(ui8ScanCode == 19) //bl key scancode = 19
    
95.         {
    
96.                 //only get key down
    
97.                 if(fKeyUp == FALSE)
    
98.                         SetEvent(evtBLKeyPress);
    
99.                 return cEvents;
    
100.         }*/ //wsf 20080924
     
101.         //v_pIOPregs->rGPFCON |= 0x01;//GPF0 OUTPUT
     
102. 
     
103.    // if (fKeyUp==FALSE)                        
     
104.    //      v_pIOPregs->rGPFDAT &= ~(1 << 1); /* LED 0 On                 */
     
105.    // else
     
106.    //     v_pIOPregs->rGPFDAT |=  (1 << 1); /* LED 0 Off                */
     
107.        
     
108. //    DEBUGMSG(ZONE_SCANCODES,
     
109. //        (_T("%s: scan code 0x%08x, code in progress 0x%08x, previous 0x%08x\r\n"),
     
110. //       pszFname, ui8ScanCode, scInProgress, scPrevious));
     
111. 
     
112.         scInProgress = ui8ScanCode;
     
113.         //按键有效，而且键值相同，auto-repeat
     
114.     //if ( (scInProgress == scPrevious) && (fKeyUp == FALSE) )
     
115.         //{
     
116.         //        mdd handles auto-repeat so ignore auto-repeats from keybd
     
117.     //}
     
118.         //else
     
119.         //{
     
120.         // Not a repeated key.  This is the real thing.               
     
121.                 scPrevious = scInProgress;
     
122.                                                                                                       
     
123.         rguiScanCode[cEvents] = scInProgress;
     
124.                 rgfKeyUp[cEvents] = fKeyUp;
     
125.         ++cEvents;
     
126.     //}
     
127.         //send a user input event to backlight driver!
     
128.         SetEvent(evtUserInput);
     
129. 
     
130.     return cEvents;
     
131. }

复制代码

xsslf

按键按下只触发一个中断脚，只要通过ZLG7290_GetKey(key);读取寄存器值就行的。
那for (UINT iEvent = 0; iEvent < cEvents; ++iEvent)，循环这么多次干吗？

lgspace

1. 
   
2. static UINT KeybdPdd_GetEventEx2(UINT uiPddId, UINT32 rguiScanCode[16], BOOL rgfKeyUp[16])
   
3. {
   
4.     SETFNAME(_T("KeybdPdd_GetEventEx2"));
   
5. 
   
6.     UINT32   scInProgress = 0;
   
7.     static UINT32   scPrevious;
   
8. 
   
9.     BOOL            fKeyUp;
   
10.     UINT8           ui8ScanCode;
    
11.     UINT            cEvents = 0;
    
12. 
    
13.     DEBUGCHK(rguiScanCode != NULL);
    
14.     DEBUGCHK(rgfKeyUp != NULL);
    
15. 
    
16.         getsFromKBCTL(&ui8ScanCode, 1);
    
17. 
    
18.     DEBUGMSG(ZONE_SCANCODES,
    
19.         (_T("%s: scan code 0x%08x, code in progress 0x%08x, previous 0x%08x\r\n"),
    
20.         pszFname, ui8ScanCode, scInProgress, scPrevious));
    
21. 
    
22.     scInProgress = ui8ScanCode;
    
23.     if (scInProgress == scPrevious) {
    
24.         //        mdd handles auto-repeat so ignore auto-repeats from keybd
    
25.     } else {
    
26.         // Not a repeated key.  This is the real thing.
    
27.         scPrevious = scInProgress;
    
28.         
    
29.         if (ui8ScanCode & scKeyUpMask) {
    
30.             fKeyUp = TRUE;
    
31.             scInProgress &= ~scKeyUpMask;
    
32.         } else {
    
33.             fKeyUp = FALSE;
    
34.         }
    
35.         
    
36.         rguiScanCode[cEvents] = scInProgress;
    
37.                 rgfKeyUp[cEvents] = fKeyUp;
    
38.         ++cEvents;
    
39.     }
    
40. 
    
41.     return cEvents;
    
42. }

复制代码

——这是没有修改之前的三星自带的键盘扫描函数。cEvents是按下的次数吧。
感觉这样写怪怪的。但是微软也是这么做。

jeans703

原来的是PS2键盘，使用的中断是下降沿触发的，并没有使用什么双边沿触发，就实现了键盘的功能。估计是弹起的时候也会产生下降沿的。
这个ZLG7290也应该同一个道理吧，想想使用什么办法可以实现。

panama

RETAILMSG(1, (TEXT("cEvents=%d\r\n"), cEvents));
我在里面加入如下打印信息，这个cEvents是等于1的，所以这个不是问题。
真的难以想象，这东西没有多少东西啊。

ping1125

键盘功能通过偏方实现了功能，但是不能扑捉到键盘弹起中断的问题依然没有解决。

我在按下线程里面加了一个键盘弹起的标志，告诉系统，就这样，这个键盘就搞定了。哈哈
哎，就是加了两行东西。
wait_for_keybd_interrupt:
    if (WaitForSingleObject(pKeybdIst->hevInterrupt, INFINITE) == WAIT_OBJECT_0)
    {
                // add by wogo at 2009-03-12
                //RETAILMSG(1, (TEXT("Deal with hevInterrupt@@@!!! :\r\n")));
        //#if 0
        cEvents = (*pKeybdIst->pfnGetKeybdEvent)
            (pKeybdIst->uiPddId, rguiScanCode, rgfKeyUp);
        //
        //RETAILMSG(1, (TEXT("cEvents=%d\r\n"), cEvents));
        for (UINT iEvent = 0; iEvent < cEvents; ++iEvent) {
        //UINT iEvent = 0
        (*pKeybdIst->pfnKeybdEvent)(pKeybdIst->uiPddId, rguiScanCode[iEvent], rgfKeyUp[iEvent]);
        rgfKeyUp[iEvent]=TRUE;——我原来太执着于本身了，不过键盘功能是实现了，但是本质没有解决。
        (*pKeybdIst->pfnKeybdEvent)(pKeybdIst->uiPddId, rguiScanCode[iEvent], rgfKeyUp[iEvent]);               
        }
        // cEvents could be 0 if this was a partial scan code, like 0xE0
     //#endif

        InterruptDone(pKeybdIst->dwSysIntr_Keybd);
                // add by wogo at 2009-03-12
                //RETAILMSG(1, (TEXT("InterruptDone.Keybd)\r\n")));
    }

感谢各位关照，尤其是shuiyan前辈，他自己忙着加班，还要回我的问题。

bblfeng

那就把这个GetEvent里面用二分法来找最耗时的那个语句（估计是ZLG7290_GetKey(key);），找到后再决定怎么处理。

MAX7349只支持按下触发下降沿中断，释放不触发。MAX7359支持按下、释放都触发下降沿中断。

所以，这个功能是由扫描芯片决定的。

zhangle8311

引用 30 楼 shuiyan 的回复:
那就把这个GetEvent里面用二分法来找最耗时的那个语句（估计是ZLG7290_GetKey(key);），找到后再决定怎么处理。

MAX7349只支持按下触发下降沿中断，释放不触发。MAX7359支持按下、释放都触发下降沿中断。

所以，这个功能是由扫描芯片决定的。

以后用MAX7359了，比较安全一点。现在我先不管那么多了，功能实现了，还有别的驱动，音频要搞吧，、
貌似串口还有BUG，搞完这几个估计要搞应用了，现在感觉很汗。什么都不懂。对操作系统了解少之又少。惭愧。

sxfzdw

那个贵，一般别指望用。国产也有很多的，或者做maxim兼容的也应该有。

这种解决方法倒特别。以后再解决也可以，只要记得这里有个问题。

meiwenbin

我觉得有点奇怪的是在4.2BSP可行，5.0BSP就不行，难道 5.0BSP的效率还要低于4.2？

在ISR中，5.0BSP使用了一些概括性的写法。使用if else
4.2BSP 使用switch，我觉得这个东西效率应该差不多吧。

至今没有搞明白是怎么回事。

lhgxcl

随便说一下。你的系统是不是很忙？把IST的线程优先级调高点看看：
SetThreadPriority(GetCurrentThread(), 100);

wangzicc

引用 34 楼 singlerace 的回复:
随便说一下。你的系统是不是很忙？把IST的线程优先级调高点看看：
SetThreadPriority(GetCurrentThread(), 100);

回singlerace 前辈，没有啊，运行，反应都很快。
我觉得也有可能，有些驱动我没有用到，但是我并没有删除。
我已经把优先级跳到100了，还是不行。

我只能像我上面那样做了。

很多什么PCCARD等驱动我没有去掉，以后去掉看看会怎么样。

2315862

IIC时钟是多少？你看看WaitForSingleObject到InterruptDone花了多长时间？

lawly

还有，可以用示波器量波形。看中断产生到IIC上有数据延迟是多少，IIC读数据花多长时间，IIC波形好不好，等等。

hjj000

引用 36 楼 singlerace 的回复:
IIC时钟是多少？你看看WaitForSingleObject到InterruptDone花了多长时间？

我把键盘扫描函数屏蔽了，就能产生两次中断了，这个明显是没有扑捉到中断，

现在我这样直接处理效果反而好，不用再次处理中断了。

IIC那个时钟，我现在记不得，明天到公司看看代码才知道。不过我本来在4.2下运行正常的，在IIC相关函数有很多for延时，我现在把它删除了，还是无济于事。
时钟都没有变，把一些延时不要了，还是会这样，所以我只能像上面那样修改IST来巧妙处理。

mu03029827

UP UP 这方面不是很会 写的好！！！！！