Changeset 87ab18e in avrstuff for kbd/xtk/code/main.c

Timestamp:

Jul 31, 2014, 11:00:37 PM (10 years ago)

Author:

Adrien Destugues <pulkomandy@…>

Branches:

main

Children:

0010058

Parents:

ea32107

Message:

Drop hacked "shift" handling from libps2.

• It's up to clients to do that if they wish so (useful if you're converting

to ASCII, for example, but not for a regular keyboard adapter)

Improve callback system to handle release events.

• The callback would only be called for pressed keys, not for releases, leading

to weird results. Things work much better now in XTK.

• Also ignore the E0 byte for extended keys in XTK. This avoids apps going

crazy as the Turbo XT BIOS doesn't handle it right.

git-svn-id: ​svn://pulkomandy.tk/avrstuff@114 c6672c3c-f6b6-47f9-9001-1fd6b12fecbe

File:

• kbd/xtk/code/main.c (modified) (3 diffs)

kbd/xtk/code/main.c

@@ -24 +24 @@
 void callback()
 {
-        uint8_t key_code = 0;
-        key_code = read_char(); // TODO this function is blocking. Can it disturb main?
-
-        key = pgm_read_byte(&(at2xt[key_code]));
-        if(release)
-                key |= 0x80;
+        // Ignore all keys above 0x80, and importantly ignore 0xE0 from extended keys.
+        if (kbd_data < 0x80) {
+                key = pgm_read_byte(&(at2xt[kbd_data]));
+                if(release)
+                        key |= 0x80;
+        }
 }
 
@@ -52 +52 @@
 #endif
 
-        static const int delay = 25;
+        static const int delay = 12;
 
         // XT init - configure pins directions
@@ -58 +58 @@
         DDRXT &= ~(PCLK | PDAT); // both pins as inputs (floating)
 
-        DDRB |= (1<<PB2); // LED
-
         uint8_t k;
         while(1) {
-                PORTB ^= (1<<PB2); // LED
-
                 while ((PINXT & (PDAT|PCLK)) != (PDAT|PCLK))
                         ; // Wait for PC to be ready to receive data