diff options
author | Sonic Zhang <sonic.zhang@analog.com> | 2010-01-19 06:13:12 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-03-02 17:43:17 -0500 |
commit | cdc592d5d496e5c813d9632d0d1e334658927af7 (patch) | |
tree | d00c8fbb037cb5542bb43d213b5f5583cfbd219f /drivers/serial/bfin_5xx.c | |
parent | 7ed43f6af88b042eb2ac5a3eda1e8e17efae74ee (diff) |
serial: bfin_5xx: kgdboc should accept gdb break only when it is active
If we always check for gdb breaks even when it isn't active, we get false
positives on normal code and the system panics.
URL: http://blackfin.uclinux.org/gf/tracker/5277
Signed-off-by: Sonic Zhang <sonic.zhang@analog.com>
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/serial/bfin_5xx.c')
-rw-r--r-- | drivers/serial/bfin_5xx.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/serial/bfin_5xx.c b/drivers/serial/bfin_5xx.c index b5a9b374dad5..6b8795557b38 100644 --- a/drivers/serial/bfin_5xx.c +++ b/drivers/serial/bfin_5xx.c | |||
@@ -237,7 +237,8 @@ static void bfin_serial_rx_chars(struct bfin_serial_port *uart) | |||
237 | 237 | ||
238 | #if defined(CONFIG_KGDB_SERIAL_CONSOLE) || \ | 238 | #if defined(CONFIG_KGDB_SERIAL_CONSOLE) || \ |
239 | defined(CONFIG_KGDB_SERIAL_CONSOLE_MODULE) | 239 | defined(CONFIG_KGDB_SERIAL_CONSOLE_MODULE) |
240 | if (kgdb_connected && kgdboc_port_line == uart->port.line) | 240 | if (kgdb_connected && kgdboc_port_line == uart->port.line |
241 | && kgdboc_break_enabled) | ||
241 | if (ch == 0x3) {/* Ctrl + C */ | 242 | if (ch == 0x3) {/* Ctrl + C */ |
242 | kgdb_breakpoint(); | 243 | kgdb_breakpoint(); |
243 | return; | 244 | return; |