keyboard.c: fix coding style with indent

Change-Id: Ie8efa9fb9bdc65bf8015eec197f44c432e87d907
Signed-off-by: Andrew Wu <arw@dmp.com.tw>
Reviewed-on: http://review.coreboot.org/3986
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
This commit is contained in:
Andrew Wu 2013-10-23 20:47:20 +08:00 committed by Patrick Georgi
parent a4ae3107ca
commit e33d6cac96
1 changed files with 55 additions and 45 deletions

View File

@ -19,7 +19,6 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/ */
#include <console/console.h> #include <console/console.h>
#include <pc80/keyboard.h> #include <pc80/keyboard.h>
#include <device/device.h> #include <device/device.h>
@ -32,14 +31,14 @@
#define KBD_DATA 0x60 #define KBD_DATA 0x60
#define KBD_COMMAND 0x64 #define KBD_COMMAND 0x64
#define KBD_STATUS 0x64 #define KBD_STATUS 0x64
#define KBD_IBF (1 << 1) // 1: input buffer full (data ready for ec) #define KBD_IBF (1 << 1) // 1: input buffer full (data ready for ec)
#define KBD_OBF (1 << 0) // 1: output buffer full (data ready for host) #define KBD_OBF (1 << 0) // 1: output buffer full (data ready for host)
// Keyboard Controller Commands // Keyboard Controller Commands
#define KBC_CMD_READ_COMMAND 0x20 // Read command byte #define KBC_CMD_READ_COMMAND 0x20 // Read command byte
#define KBC_CMD_WRITE_COMMAND 0x60 // Write command byte #define KBC_CMD_WRITE_COMMAND 0x60 // Write command byte
#define KBC_CMD_SELF_TEST 0xAA // Controller self-test #define KBC_CMD_SELF_TEST 0xAA // Controller self-test
#define KBC_CMD_KBD_TEST 0xAB // Keyboard Interface test #define KBC_CMD_KBD_TEST 0xAB // Keyboard Interface test
/* The Keyboard controller command byte /* The Keyboard controller command byte
* BIT | Description * BIT | Description
@ -55,14 +54,14 @@
*/ */
// Keyboard Controller Replies // Keyboard Controller Replies
#define KBC_REPLY_SELFTEST_OK 0x55 // controller self-test succeeded #define KBC_REPLY_SELFTEST_OK 0x55 // controller self-test succeeded
// //
// Keyboard Replies // Keyboard Replies
// //
#define KBD_REPLY_POR 0xAA // Power on reset #define KBD_REPLY_POR 0xAA // Power on reset
#define KBD_REPLY_ACK 0xFA // Command ACK #define KBD_REPLY_ACK 0xFA // Command ACK
#define KBD_REPLY_RESEND 0xFE // Command NACK, send command again #define KBD_REPLY_RESEND 0xFE // Command NACK, send command again
/* Wait 400ms for keyboard controller answers */ /* Wait 400ms for keyboard controller answers */
#define KBC_TIMEOUT_IN_MS 400 #define KBC_TIMEOUT_IN_MS 400
@ -70,43 +69,43 @@
static int kbc_input_buffer_empty(void) static int kbc_input_buffer_empty(void)
{ {
u32 timeout; u32 timeout;
for(timeout = KBC_TIMEOUT_IN_MS; timeout && (inb(KBD_STATUS) & KBD_IBF); timeout--) { for (timeout = KBC_TIMEOUT_IN_MS;
timeout && (inb(KBD_STATUS) & KBD_IBF); timeout--)
mdelay(1); mdelay(1);
}
if (!timeout) { if (!timeout)
printk(BIOS_WARNING, "Unexpected Keyboard controller input buffer full\n"); printk(BIOS_WARNING,
} "Unexpected Keyboard controller input buffer full\n");
return !!timeout; return !!timeout;
} }
static int kbc_output_buffer_full(void) static int kbc_output_buffer_full(void)
{ {
u32 timeout; u32 timeout;
for(timeout = KBC_TIMEOUT_IN_MS; timeout && ((inb(KBD_STATUS) & KBD_OBF) == 0); timeout--) { for (timeout = KBC_TIMEOUT_IN_MS;
timeout && ((inb(KBD_STATUS) & KBD_OBF) == 0); timeout--)
mdelay(1); mdelay(1);
}
if (!timeout) { if (!timeout)
printk(BIOS_INFO, "Keyboard controller output buffer result timeout\n"); printk(BIOS_INFO,
} "Keyboard controller output buffer result timeout\n");
return !!timeout; return !!timeout;
} }
static int kbc_cleanup_buffers(void) static int kbc_cleanup_buffers(void)
{ {
u32 timeout; u32 timeout;
for(timeout = KBC_TIMEOUT_IN_MS; timeout && (inb(KBD_STATUS) & (KBD_OBF | KBD_IBF)); timeout--) { for (timeout = KBC_TIMEOUT_IN_MS;
timeout && (inb(KBD_STATUS) & (KBD_OBF | KBD_IBF)); timeout--) {
mdelay(1); mdelay(1);
inb(KBD_DATA); inb(KBD_DATA);
} }
if (!timeout) { if (!timeout) {
printk(BIOS_ERR, "Couldn't cleanup the keyboard controller buffers\n"); printk(BIOS_ERR,
"Couldn't cleanup the keyboard controller buffers\n");
printk(BIOS_ERR, "Status (0x%x): 0x%x, Buffer (0x%x): 0x%x\n", printk(BIOS_ERR, "Status (0x%x): 0x%x, Buffer (0x%x): 0x%x\n",
KBD_STATUS, inb(KBD_STATUS), KBD_DATA, inb(KBD_DATA)); KBD_STATUS, inb(KBD_STATUS), KBD_DATA, inb(KBD_DATA));
} }
return !!timeout; return !!timeout;
@ -136,7 +135,7 @@ static int kbc_self_test(void)
if (self_test != 0x55) { if (self_test != 0x55) {
printk(BIOS_ERR, "Keyboard Controller self-test failed: 0x%x\n", printk(BIOS_ERR, "Keyboard Controller self-test failed: 0x%x\n",
self_test); self_test);
return 0; return 0;
} }
@ -156,7 +155,7 @@ static int kbc_self_test(void)
if (self_test != 0x00) { if (self_test != 0x00) {
printk(BIOS_ERR, "Keyboard Interface test failed: 0x%x\n", printk(BIOS_ERR, "Keyboard Interface test failed: 0x%x\n",
self_test); self_test);
return 0; return 0;
} }
@ -169,19 +168,21 @@ static u8 send_keyboard(u8 command)
u8 resend = 10; u8 resend = 10;
do { do {
if (!kbc_input_buffer_empty()) return 0; if (!kbc_input_buffer_empty())
return 0;
outb(command, KBD_DATA); outb(command, KBD_DATA);
/* the reset command takes much longer then normal commands and /* the reset command takes much longer then normal commands and
* even worse, some keyboards do send the ACK _after_ doing the * even worse, some keyboards do send the ACK _after_ doing the
* reset */ * reset */
if (command == 0xFF) { if (command == 0xFF) {
u8 retries; u8 retries;
for (retries = 9; retries && !kbc_output_buffer_full(); retries--) for (retries = 9; retries && !kbc_output_buffer_full();
; retries--) ;
} }
if (!kbc_output_buffer_full()) { if (!kbc_output_buffer_full()) {
printk(BIOS_ERR, "Could not send keyboard command %02x\n", printk(BIOS_ERR,
command); "Could not send keyboard command %02x\n",
command);
return 0; return 0;
} }
regval = inb(KBD_DATA); regval = inb(KBD_DATA);
@ -210,9 +211,11 @@ void pc_keyboard_init(struct pc_keyboard *keyboard)
return; return;
/* Enable keyboard interface - No IRQ */ /* Enable keyboard interface - No IRQ */
if (!kbc_input_buffer_empty()) return; if (!kbc_input_buffer_empty())
return;
outb(0x60, KBD_COMMAND); outb(0x60, KBD_COMMAND);
if (!kbc_input_buffer_empty()) return; if (!kbc_input_buffer_empty())
return;
outb(0x20, KBD_DATA); /* send cmd: enable keyboard */ outb(0x20, KBD_DATA); /* send cmd: enable keyboard */
if (!kbc_input_buffer_empty()) { if (!kbc_input_buffer_empty()) {
printk(BIOS_INFO, "Timeout while enabling keyboard\n"); printk(BIOS_INFO, "Timeout while enabling keyboard\n");
@ -220,7 +223,8 @@ void pc_keyboard_init(struct pc_keyboard *keyboard)
} }
/* clean up any junk that might have been in the keyboard */ /* clean up any junk that might have been in the keyboard */
if (!kbc_cleanup_buffers()) return; if (!kbc_cleanup_buffers())
return;
/* reset keyboard and self test (keyboard side) */ /* reset keyboard and self test (keyboard side) */
regval = send_keyboard(0xFF); regval = send_keyboard(0xFF);
@ -236,8 +240,7 @@ void pc_keyboard_init(struct pc_keyboard *keyboard)
} }
/* the reset command takes some time, so wait a little longer */ /* the reset command takes some time, so wait a little longer */
for (retries = 9; retries && !kbc_output_buffer_full(); retries--) for (retries = 9; retries && !kbc_output_buffer_full(); retries--) ;
;
if (!kbc_output_buffer_full()) { if (!kbc_output_buffer_full()) {
printk(BIOS_ERR, "Timeout waiting for keyboard after reset.\n"); printk(BIOS_ERR, "Timeout waiting for keyboard after reset.\n");
@ -246,7 +249,8 @@ void pc_keyboard_init(struct pc_keyboard *keyboard)
regval = inb(KBD_DATA); regval = inb(KBD_DATA);
if (regval != 0xAA) { if (regval != 0xAA) {
printk(BIOS_ERR, "Keyboard reset selftest failed: 0x%x\n", regval); printk(BIOS_ERR, "Keyboard reset selftest failed: 0x%x\n",
regval);
return; return;
} }
@ -265,20 +269,24 @@ void pc_keyboard_init(struct pc_keyboard *keyboard)
/* Set scancode command */ /* Set scancode command */
regval = send_keyboard(0xF0); regval = send_keyboard(0xF0);
if (regval != KBD_REPLY_ACK) { if (regval != KBD_REPLY_ACK) {
printk(BIOS_ERR, "Keyboard set scancode cmd failed ACK: 0x%x\n", regval); printk(BIOS_ERR, "Keyboard set scancode cmd failed ACK: 0x%x\n",
regval);
return; return;
} }
/* Set scancode mode 2 */ /* Set scancode mode 2 */
regval = send_keyboard(0x02); regval = send_keyboard(0x02);
if (regval != KBD_REPLY_ACK) { if (regval != KBD_REPLY_ACK) {
printk(BIOS_ERR, "Keyboard set scancode mode failed ACK: 0x%x\n", regval); printk(BIOS_ERR,
"Keyboard set scancode mode failed ACK: 0x%x\n", regval);
return; return;
} }
/* All is well - enable keyboard interface */ /* All is well - enable keyboard interface */
if (!kbc_input_buffer_empty()) return; if (!kbc_input_buffer_empty())
return;
outb(0x60, KBD_COMMAND); outb(0x60, KBD_COMMAND);
if (!kbc_input_buffer_empty()) return; if (!kbc_input_buffer_empty())
return;
outb(0x65, KBD_DATA); /* send cmd: enable keyboard and IRQ 1 */ outb(0x65, KBD_DATA); /* send cmd: enable keyboard and IRQ 1 */
if (!kbc_input_buffer_empty()) { if (!kbc_input_buffer_empty()) {
printk(BIOS_ERR, "Timeout during keyboard enable\n"); printk(BIOS_ERR, "Timeout during keyboard enable\n");
@ -305,10 +313,12 @@ void set_kbc_ps2_mode(void)
return; return;
/* Support PS/2 mode */ /* Support PS/2 mode */
if (!kbc_input_buffer_empty()) return; if (!kbc_input_buffer_empty())
return;
outb(0xcb, KBD_COMMAND); outb(0xcb, KBD_COMMAND);
if (!kbc_input_buffer_empty()) return; if (!kbc_input_buffer_empty())
return;
outb(0x01, KBD_DATA); outb(0x01, KBD_DATA);
kbc_cleanup_buffers(); kbc_cleanup_buffers();