From: travisutk Date: Sun, 17 Apr 2011 22:23:46 +0000 (+0000) Subject: AVR baud rates are good, but the values are hackish. X-Git-Url: http://git.rot13.org/?p=goodfet;a=commitdiff_plain;h=7254ebd4731d5c3ce94bbe4e16a5928a5089e6c4;hp=5ac4c1769e9976b39e919bb5f5ac9e97f62e7d0f AVR baud rates are good, but the values are hackish. git-svn-id: https://svn.code.sf.net/p/goodfet/code/trunk@998 12e2690d-a6be-4b82-a7b7-67c4a43b65c8 --- diff --git a/firmware/apps/monitor/monitor.c b/firmware/apps/monitor/monitor.c index 73aa61c..701de9e 100644 --- a/firmware/apps/monitor/monitor.c +++ b/firmware/apps/monitor/monitor.c @@ -146,9 +146,11 @@ void monitor_handle_fn(uint8_t const app, break; case MONITOR_CONNECTED: - msp430_init_dco_done(); - txdata(app,verb,0); - break; + #ifdef MSP430 + msp430_init_dco_done(); + #endif + txdata(app,verb,0); + break; } } diff --git a/firmware/lib/arduino.c b/firmware/lib/arduino.c index 503492c..698aa9e 100644 --- a/firmware/lib/arduino.c +++ b/firmware/lib/arduino.c @@ -13,20 +13,7 @@ void arduino_init(){ /* set PORTB for output*/ DDRB = 0xFF; - /* - while (1) - { - //LED on - PORTB = 0x20; - - _delay_ms(1000); - - //LED off - PORTB = 0x00; - - _delay_ms(1000); - } -*/ + avr_init_uart0(); } #endif diff --git a/firmware/lib/atmega168.c b/firmware/lib/atmega168.c index 1ed8840..c6516ea 100644 --- a/firmware/lib/atmega168.c +++ b/firmware/lib/atmega168.c @@ -51,15 +51,14 @@ void setbaud0(unsigned char rate){ break; } - uint16_t bittimer=( F_CPU / SPEED / 16 ) - 1; - UBRR0H = (unsigned char) (bittimer >> 8); - UBRR0L = (unsigned char) bittimer; +#define F_CPU 16000000L +#define BAUD 230400L +#include + UBRR0H = UBRRH_VALUE; + UBRR0L = UBRRL_VALUE; - - /* set the framing to 8N1 */ - UCSR0C = (3 << UCSZ00); - /* Engage! */ - UCSR0B = (1 << RXEN0) | (1 << TXEN0); + UCSR0C = _BV(UCSZ01) | _BV(UCSZ00); + UCSR0B = _BV(RXEN0) | _BV(TXEN0); return; } @@ -88,22 +87,13 @@ void setbaud1(unsigned char rate){ } -void msp430_init_uart0(){ -} - - -void msp430_init_uart1(){ +void avr_init_uart0(){ + setbaud0(0); } - -//! Initialization is correct. -void msp430_init_dco_done(){ - //Nothing to do for the AVR w/ xtal. +void avr_init_uart1(){ } -void msp430_init_dco() { - // -}