Re: [uracoli-devel] Missing files?

[Joerg Wunsch]

As address@hidden wrote:

> Sorry, Wuart2 is not ready for release, I removed this target from
> Src/App/SConscript (fixed with revision: 1.11)

OK, updated, wors fine.

I just committed some files to resolve most warnings.  The remaining
warnings are:

Src/App/diagradio.c:150:2: warning: #warning "no radio_config_recall()"
Src/App/wuart.c:139:3: warning: #warning "currently no radio_config_recall()"

(obvious)

Src/App/wgpio.c:181: warning: 'SLEEP_ON_KEY_vect' appears to be a misspelled 
signal handler

This is IMHO a minor mistake in Src/Lib/Inc/boards/board_zgbl.h.  It
has a commented out block

/*
#define SLEEP_ON_KEY_INIT() do{}while(0)
#define SLEEP_ON_KEY() \
        do{\
            EIMSK |= _BV(INT5);\
            set_sleep_mode(SLEEP_MODE_PWR_DOWN);\
            sleep_mode();\
            EIMSK &= ~_BV(INT5);\
        } while(0)

#define SLEEP_ON_KEY_vect INT5_vect
*/

but then still goes on defining:

#define SLEEP_ON_KEY_INIT() do{}while(0)
#define SLEEP_ON_KEY() do{}while(0)

As Src/App/wgpio.c tries to implement the ISR depending on whether
SLEEP_ON_KEY is defined (which /is/ defined), it attempts to declare
an ISR for a non-existent vector.

Src/App/diagradio.c:593: warning: unused variable 'btick'

This is a subtle one.  For some configurations, LED_SET_VALUE()
apparently evaluates to nothing, which would otherwise make use of
this variable.  I don't know of an easy solution for that.

-- 
cheers, J"org               .-.-.   --... ...--   -.. .  DL8DTL

http://www.sax.de/~joerg/                        NIC: JW11-RIPE
Never trust an operating system you don't have sources for. ;-)