[patch #6768] Addition to util/setbaud.h

[avrs-admin]

Thu 05 Mar 2009 05:21:09 PM CET

I noticed that util/setbaud.h cannot calculate baudrate for sync mode.

I did the following modification to the file. I doubt it will it the next release, but I do beleive it is a good thing to have.

Basic explanation: if BAUD_SYNC is define and equal to 1, a modified macro is use to calculate baud rate. If BAUD_SYNC is define and equal to 0 or not define, macro works as before.

Modification should not break code that use the current version of setbaud.h.

Usage could be: #include <avr/io.h>

#define F_CPU 4000000

static void uart_9600(void) { #define BAUD_SYNC 1 #define BAUD 9600 Device specific declaration #define XCK_DDR DDRB//For ATmega16A #define XCK_DDx DDB0 #include <util/setbaud.h> UBRRH = UBRRH_VALUE; UBRRL = UBRRL_VALUE; UCSRC = _BV(1<<UMSEL); Configure as clock source XCK_DDR |= _BV(1<<XCK_DDx); }

file #17593: setBaud.diff

This issue was migrated from https://savannah.nongnu.org/patch/?6768

[avrs-admin]

Frédéric Nadeau Thu 05 Mar 2009 05:43:04 PM CET

After reread, it should be:

UCSRC |= _BV(1<<UMSEL);

But I guess that most of you already spotted that.

[avrs-admin]

Eric Weddington Fri 13 May 2011 12:28:38 AM CEST

Hi Frédéric,

On line 16 of this patch you are defining BAUD_SYNC to 0 in the header file. Then later on, you check to see if it is not defined, if so then define BAUD_SYNC to 0. Shouldn't that first definition on line 16 be removed?