X-Git-Url: https://codewiz.org/gitweb?a=blobdiff_plain;f=drv%2Fkdebug.c;h=03a86430787ad181e27902deecde4337b79800a0;hb=5b25ae54d86d894e5d01adeef10938df1bb10e16;hp=471e6f4ea41d52841dd897e51059d7a0aadb4fbb;hpb=0e46cbbbd56d4a79f41cec484f19048370e6df80;p=bertos.git diff --git a/drv/kdebug.c b/drv/kdebug.c index 471e6f4e..03a86430 100755 --- a/drv/kdebug.c +++ b/drv/kdebug.c @@ -1,21 +1,35 @@ /*! * \file * * - * \brief General pourpose debug functions. + * \brief General pourpose debug support for embedded systems (implementation). * * \version $Id$ - * * \author Bernardo Innocenti * \author Stefano Fedrigo */ /*#* *#* $Log$ + *#* Revision 1.18 2004/12/08 08:52:00 bernie + *#* Save some more RAM on AVR. + *#* + *#* Revision 1.17 2004/10/03 18:41:28 bernie + *#* Restore \version header killed by mistake in previous commit. + *#* + *#* Revision 1.16 2004/10/03 18:40:50 bernie + *#* Use new CPU macros. + *#* + *#* Revision 1.15 2004/09/14 21:03:46 bernie + *#* Use debug.h instead of kdebug.h. + *#* + *#* Revision 1.14 2004/09/06 21:39:56 bernie + *#* Allow partial redefinition of BUS macros. + *#* *#* Revision 1.13 2004/08/29 22:04:26 bernie *#* Convert 485 macros to generic BUS macros; *#* kputchar(): New public function. @@ -58,7 +72,8 @@ *#* *#*/ -#include "kdebug.h" +#include +#include #include "hw.h" #include "config.h" @@ -93,28 +108,30 @@ * Support for special bus policies or external transceivers * on UART0 (to be overridden in "hw.h"). */ - #if !defined(KDBG_UART0_BUS_INIT) - #if defined(KDBG_UART0_BUS_RX) || defined(KDBG_UART0_BUS_TX) - #error KDBG_UART0_BUS_INIT, KDBG_UART0_BUS_RX and KDBG_UART0_BUS_TX must be defined together - #endif - #define KDBG_UART0_BUS_INIT do {} while (0) - #define KDBG_UART0_BUS_TX do {} while (0) - #define KDBG_UART0_BUS_RX do {} while (0) - #elif !defined(KDBG_UART0_BUS_RX) || !defined(KDBG_UART0_BUS_TX) - #error KDBG_UART0_BUS_INIT, KDBG_UART0_BUS_RX and KDBG_UART0_BUS_TX must be defined together + #ifndef KDBG_UART0_BUS_INIT + #define KDBG_UART0_BUS_INIT do {} while (0) + #endif + #ifndef KDBG_UART0_BUS_RX + #define KDBG_UART0_BUS_RX do {} while (0) + #endif + #ifndef KDBG_UART0_BUS_TX + #define KDBG_UART0_BUS_TX do {} while (0) #endif - #if defined(__AVR_ATmega64__) + #if CPU_AVR_ATMEGA64 #define UCR UCSR0B #define UDR UDR0 #define USR UCSR0A - #elif defined(__AVR_ATmega8__) + #elif CPU_AVR_ATMEGA8 #define UCR UCSRB #define USR UCSRA + #else + #error Unknown CPU #endif #define KDBG_WAIT_READY() do { loop_until_bit_is_set(USR, UDRE); } while(0) #define KDBG_WAIT_TXDONE() do { loop_until_bit_is_set(USR, TXC); } while(0) + /* * We must clear the TXC flag before sending a new character to allow * KDBG_WAIT_TXDONE() to work properly. @@ -127,10 +144,10 @@ #define KDBG_WRITE_CHAR(c) do { USR |= BV(TXC); UDR = (c); } while(0) #define KDBG_MASK_IRQ(old) do { \ - KDBG_UART0_BUS_TX; \ (old) = UCR; \ UCR |= BV(TXEN); \ UCR &= ~(BV(TXCIE) | BV(UDRIE)); \ + KDBG_UART0_BUS_TX; \ } while(0) #define KDBG_RESTORE_IRQ(old) do { \ @@ -148,14 +165,13 @@ * on UART1 (to be overridden in "hw.h"). */ #ifndef KDBG_UART1_BUS_INIT - #if defined(KDBG_UART1_BUS_RX) || defined(KDBG_UART1_BUS_TX) - #error KDBG_UART1_BUS_INIT, KDBG_UART1_BUS_RX and KDBG_UART1_BUS_TX must be defined together - #endif - #define KDBG_UART1_BUS_INIT do {} while (0) - #define KDBG_UART1_BUS_TX do {} while (0) - #define KDBG_UART1_BUS_RX do {} while (0) - #elif !defined(KDBG_UART1_BUS_RX) || !defined(KDBG_UART1_BUS_TX) - #error KDBG_UART1_BUS_INIT, KDBG_UART1_BUS_RX and KDBG_UART1_BUS_TX must be defined together + #define KDBG_UART1_BUS_INIT do {} while (0) + #endif + #ifndef KDBG_UART1_BUS_RX + #define KDBG_UART1_BUS_RX do {} while (0) + #endif + #ifndef KDBG_UART1_BUS_TX + #define KDBG_UART1_BUS_TX do {} while (0) #endif #define KDBG_WAIT_READY() do { loop_until_bit_is_set(UCSR1A, UDRE); } while(0) @@ -163,10 +179,10 @@ #define KDBG_WRITE_CHAR(c) do { UCSR1A |= BV(TXC); UDR1 = (c); } while(0) #define KDBG_MASK_IRQ(old) do { \ - KDBG_UART1_BUS_TX; \ (old) = UCSR1B; \ UCSR1B |= BV(TXEN); \ UCSR1B &= ~(BV(TXCIE) | BV(UDRIE)); \ + KDBG_UART1_BUS_TX; \ } while(0) #define KDBG_RESTORE_IRQ(old) do { \ @@ -210,7 +226,7 @@ void kdbg_init(void) /* Compute the baud rate */ uint16_t period = (((CLOCK_FREQ / 16UL) + (CONFIG_KDEBUG_BAUDRATE / 2)) / CONFIG_KDEBUG_BAUDRATE) - 1; - #if defined(__AVR_ATmega64__) + #if CPU_AVR_ATMEGA64 #if CONFIG_KDEBUG_PORT == 0 UBRR0H = (uint8_t)(period>>8); UBRR0L = (uint8_t)period; @@ -222,14 +238,14 @@ void kdbg_init(void) #else #error CONFIG_KDEBUG_PORT must be either 0 or 1 #endif - #elif defined(__AVR_ATmega8__) + #elif CPU_AVR_ATMEGA8 UBRRH = (uint8_t)(period>>8); UBRRL = (uint8_t)period; - #elif defined(__AVR_ATmega103__) + #elif CPU_AVR_ATMEGA103 UBRR = (uint8_t)period; KDBG_UART0_BUS_INIT; #else - #error Unknown arch + #error Unknown CPU #endif #endif /* !CPU_I196 && !CPU_AVR */ @@ -272,6 +288,8 @@ void kputchar(char c) void PGM_FUNC(kprintf)(const char * PGM_ATTR fmt, ...) { + +#if CONFIG_PRINTF va_list ap; /* Mask serial TX intr */ @@ -284,6 +302,10 @@ void PGM_FUNC(kprintf)(const char * PGM_ATTR fmt, ...) /* Restore serial TX intr */ KDBG_RESTORE_IRQ(irqsave); +#else + /* A better than nothing printf() surrogate. */ + PGM_FUNC(kputs)(fmt); +#endif /* CONFIG_PRINTF */ } @@ -302,22 +324,61 @@ void PGM_FUNC(kputs)(const char * PGM_ATTR str) } -int PGM_FUNC(__assert)(const char * PGM_ATTR cond, const char * PGM_ATTR file, int line) +/*! + * Cheap function to print small integers without using printf(). + */ +static int kputnum(int num) +{ + int output_len = 0; + int divisor = 10000; + int digit; + + do + { + digit = num / divisor; + num %= divisor; + + if (digit || output_len || divisor == 1) + { + kputchar(digit + '0'); + ++output_len; + } + } + while (divisor /= 10); + + return output_len; +} + + +static void klocation(const char * PGM_ATTR file, int line) { PGM_FUNC(kputs)(file); - PGM_FUNC(kprintf)(PSTR(":%d: Assertion failed: "), line); + kputchar(':'); + kputnum(line); + PGM_FUNC(kputs)(PSTR(": ")); +} + +int PGM_FUNC(__assert)(const char * PGM_ATTR cond, const char * PGM_ATTR file, int line) +{ + klocation(file, line); + PGM_FUNC(kputs)(PSTR("Assertion failed: ")); PGM_FUNC(kputs)(cond); - PGM_FUNC(kputs)(PSTR("\n")); + kputchar('\n'); return 1; } int PGM_FUNC(__invalid_ptr)(void *value, const char * PGM_ATTR name, const char * PGM_ATTR file, int line) { - PGM_FUNC(kputs)(file); - PGM_FUNC(kprintf)(PSTR(":%d: Invalid pointer: "), line); + klocation(file, line); + PGM_FUNC(kputs)(PSTR("Invalid ptr: ")); PGM_FUNC(kputs)(name); - PGM_FUNC(kprintf)(PSTR(" = 0x%x\n"), value); + #if CONFIG_PRINTF + PGM_FUNC(kprintf)(PSTR(" = 0x%x\n"), value); + #else + (void)value; + kputchar('\n'); + #endif return 1; } @@ -329,7 +390,7 @@ void __init_wall(long *wall, int size) } -int __check_wall(long *wall, int size, const char *name, const char *file, int line) +int PGM_FUNC(__check_wall)(long *wall, int size, const char * PGM_ATTR name, const char * PGM_ATTR file, int line) { int i, fail = 0; @@ -337,8 +398,14 @@ int __check_wall(long *wall, int size, const char *name, const char *file, int l { if (wall[i] != WALL_VALUE) { - kprintf("%s:%d: Wall broken: %s[%d] (0x%p) = 0x%lx\n", - file, line, name, i, wall + i, wall[i]); + klocation(file, line); + PGM_FUNC(kputs)(PSTR("Wall broken: ")); + PGM_FUNC(kputs)(name); + #if CONFIG_PRINTF + PGM_FUNC(kprintf)(PSTR("[%d] (0x%p) = 0x%lx\n"), i, wall + i, wall[i]); + #else + kputchar('\n'); + #endif fail = 1; } } @@ -347,6 +414,8 @@ int __check_wall(long *wall, int size, const char *name, const char *file, int l } +#if CONFIG_PRINTF + /*! * Dump binary data in hex */ @@ -359,4 +428,6 @@ void kdump(const void *_buf, size_t len) kputchar('\n'); } +#endif /* CONFIG_PRINTF */ + #endif /* _DEBUG */