ATNEL tech-forum

Re: UNIA jako wygodny dostep do struktury i takie tam inne .

2015-08-03T22:44:28+01:00

Problem polegał na tym że inicjalizowałem tzn. ustawiałem źródło i podział dla TIMER0 w programie głównym, i od tego momentu rozpoczynało się zliczanie przepełniając pewnie wiele razy TIMER0 co powodowało ustawienie flagi TOV0 w rejestrze TIFR. W obsłudze przerwania odbioru znaku tylko ustawiałem licznik na wartość początkową i włączałem przerwanie od przepełnienia TIMER0 i w tym momencie następował obsługa przerwania (bo wcześniej była ustawiona flaga TOV0). Dalsze znaki odbierane były jakby w "drugiej paczce" bo od tego momentu TIMER0 już był opanowany i liczył tak jak zakładałem. Wyjście to wyzerowanie flagi TOV0 w TIFR oczywiście TIFR = (1<Inicjalizacja parametrów TIMERA w programie głównym w wielu innych przypadkach może nie wpływać na dziwne zachowanie się programu (jedna więcej pętla obsługi zdarzeń itp.) tu jednak był dość duży reżim jeśli chodzi o poprawną jego pracę.
pozdr Jurek
temat można zamknąć

Statystyki: Napisane przez roll99 — 3 sie 2015, o 22:44

Re: UNIA jako wygodny dostep do struktury i takie tam inne .

2015-08-03T15:29:03+01:00

OK masz rację ...
pozdr Jurek

Statystyki: Napisane przez roll99 — 3 sie 2015, o 15:29

Re: UNIA jako wygodny dostep do struktury i takie tam inne .

2015-08-03T15:15:18+01:00

roll99 napisał(a):

chyba że źle czytam.

Źle czytasz i kompletnie nie rozumiem po co się upierasz .... Tym bardziej, że sam producent procków tobie to pisze...

Robiąc operację OR możesz niechcący kasować inne flagi przerwań w tym samym rejestrze

a ten cytat z "so there is no risk" odnosi się właśnie do = a nie do |=

Statystyki: Napisane przez mirekk36 — 3 sie 2015, o 15:15

Re: UNIA jako wygodny dostep do struktury i takie tam inne .

2015-08-03T14:40:40+01:00

mirekk36 napisał(a):

Widzę, że niedokładnie przeczytałeś link który ci podałem bo tu w ogóle nie chodzi o wydajność (oszczędność instrukcji)

No tak ale poza tym OR (plus odczyt stanu poprzedniego) różnicy w efekcie i ewentualnych działaniach ubocznych nie ma - chyba że źle czytam.

Quote:

... so there is no risk of any race condition that might accidentally clear another interrupt request bit. So instead of writing TIFR |= _BV(TOV0); /* wrong! */ simply use TIFR = _BV(TOV0);

pozr Jurek

Statystyki: Napisane przez roll99 — 3 sie 2015, o 14:40

Re: UNIA jako wygodny dostep do struktury i takie tam inne .

2015-08-03T14:23:15+01:00

mirekk36 napisał(a):

No tak jak pisałem, obie metody są OK, Twoja o jedną instrukcję wydajniejsza,

Widzę, że niedokładnie przeczytałeś link który ci podałem bo tu w ogóle nie chodzi o wydajność (oszczędność instrukcji)

Statystyki: Napisane przez mirekk36 — 3 sie 2015, o 14:23

Re: UNIA jako wygodny dostep do struktury i takie tam inne .

2015-08-03T14:07:44+01:00

No tak jak pisałem, obie metody są OK, Twoja o jedną instrukcję wydajniejsza, bardziej interesuje mnie dlaczego jak nie wyzeruję tej flagi to po odebraniu pierwszego znaku odpala mi przerwanie od przepełnienia TIMER0 , długo nad tym siedziałem - efekt był taki ze slave odpowiadał dwiema ramkami raz po pierwszym znaku a drugi raz po reszcie odebranej ramki (ramki odbiorczej nie analizowałem tylko sprawdzałem czy cos zostało odebrane czyli czy przerwanie od TIMER0 zostało odpalone a flaga stanu ustawiona na MB_DONE_RX a ustawiała się ona tylko w procedurze obsługi TIMER0 po przepełnieniu czyli po upływie 3-4 ms od odbioru ostatniego znaku.

pozdrawiam Jurek

Statystyki: Napisane przez roll99 — 3 sie 2015, o 14:07

Re: UNIA jako wygodny dostep do struktury i takie tam inne .

2015-08-03T13:37:44+01:00

Sam do niedawna często zapominałem o tym i pisałem bodajże nawet w książce kasowanie flag przerwań z tym OR ... ale to akurat w tym przypadku jest niestety błędne

tu: http://www.atmel.com/webdoc/AVRLibcRefe ... tbits.html masz dokładniejsze wyjaśnienie

------------------------ [ Dodano po: 26 minutach ]

roll99 napisał(a):

(przynajmniej w ATMEGA 8)

w KAŻDYM AVR przy kasowaniu flag przerwań

Statystyki: Napisane przez mirekk36 — 3 sie 2015, o 13:37

Re: UNIA jako wygodny dostep do struktury i takie tam inne .

2015-08-03T13:31:23+01:00

Dziękuje za pochwałę Mistrza (nie ironicznie )dotyczącą unii - prace trwają więc sprawa rozwojowa .

a co jest złe w zapisie ?
[syntax=c]TIFR |= (1<
czy nie ustawi to flagi TOV0 na jeden, wiem że nie można zapisać do rejestru TIFR zera (przynajmniej w ATMEGA 8) i pewnie zapis TIFR = (1<
pozdrawiam Jurek

Statystyki: Napisane przez roll99 — 3 sie 2015, o 13:31

Re: UNIA jako wygodny dostep do struktury i takie tam inne .

2015-08-03T13:21:52+01:00

roll99 napisał(a):

Czy taka konstrukcja jakoś bardzo pogarsza wydajność programu?

Wręcz przeciwnie - to jest WZORCOWE PODEJŚCIE aż się miło patrzy. Tzn mówię o takim zastosowaniu unii i struktur - nie wnikam w stronę merytoryczną.

roll99 napisał(a):

TIFR |= (1<

UWAGA! tak się nie kasuje flag przerwań - KONIECZNIE napisz to tak:

[syntax=c]TIFR = (1<Statystyki: Napisane przez mirekk36 — 3 sie 2015, o 13:21

UNIA jako wygodny dostep do struktury i takie tam inne ...

2015-08-03T11:28:53+01:00

Witam na forum

Piszę w ramach wprawki prostą bibliotekę modbus (ATMEGA 8), i mam dwa pytanka :
- w celu ułatwienia dostępu do struktury w której trzymam ramkę odebraną/do nadania używam wielopoziomowej unii. Czy taka konstrukcja jakoś bardzo pogarsza wydajność programu?

[syntax=c]typedef struct {
uint16_t startAdrr;
uint16_t numberRegs;
} TMBRxFn03;

typedef struct {
uint16_t startAdrr;
uint16_t numberCoils;
} TMBRxFn01;

typedef struct {
uint8_t numberBytes;
uint8_t data[ MB_DATA_SIZE_8_BIT -1 ];
} TMBTxFn01;

typedef struct {
uint8_t exceptionCode;
} TMBTxEception;

// struktura ramki MB
typedef union {
uint8_t mbFrame[ MB_MAX_FRAME_SIZE ];
struct {
uint8_tslaveAddr;
uint8_tcodeFunction;
union {
uint8_tdata[ MB_DATA_SIZE_8_BIT ];
uint16_treg16[ MB_DATA_SIZE_16_BIT ];
TMBRxFn03RxFn03;
TMBRxFn01RxFn01;
TMBTxFn01TxFn01;
TMBTxEception exception;
};
uint16_tcrc;
};
} TMBFrame ;[/syntax]

- w przerwaniu generowanym po odbiorze danej przez UART, przy pierwszej danej odebranej ustawiam parametry dla TIMER0 tak aby generował przerwanie po około 3ms, jeśli nie wykonam kasowania (wpisanie 1) rejestru TIFR |= (1<[syntax=c]void initUART( void ) {

UBRRH = ( uint8_t ) ( UART_UBRR >> 8 );
UBRRL = ( uint8_t ) ( UART_UBRR );
UCSRC = (1<UART_TX_ENABLE;
UART_RX_ENABLE;

}

ISR(SIG_UART_RECV){

if( MB_PROTOCOL_STATE.state == MB_IS_IDDLE) {
MB_PROTOCOL_STATE.state = MB_IS_RX;
MB_RX_BUFFER_INDEX = 0;
T35_TOV0_RESET;
T35_TIMER_RESET;
T35_TIMER_ENABLE;

}

if( MB_PROTOCOL_STATE.state == MB_IS_RX ) {
if( MB_RX_BUFFER_INDEX < MB_MAX_FRAME_SIZE ) {
MB_FRAME_BUFFER.mbFrame[ MB_RX_BUFFER_INDEX++ ] = UDR;
T35_TIMER_RESET;

}
else {
MB_PROTOCOL_STATE.state = MB_ERR_RX;
MB_PROTOCOL_STATE.codeError = MB_ERR_BUFFER_OVERLOAD;
}

}

}[/syntax]

[syntax=c]void initTimer( void ) {
TCCR0 |= (1<T35_TIMER_RESET;
}[/syntax]

i makra

[syntax=c]#define T35_TIMER 220 //12MHz 36*85us
#define T35_TIMER_RESET TCNT0 = T35_TIMER
#define T35_TOV0_RESETTIFR |= (1<#define T35_TIMER_ENABLETIMSK |= (1<< TOIE0); //Włączenie przerwanie overflow (przepełnienie timera)
#define T35_TIMER_DISABLETIMSK &= ~(1<< TOIE0); //Wyłączenie przerwanie overflow (przepełnienie timera)[/syntax]

pozdrawiam Jurek

Statystyki: Napisane przez roll99 — 3 sie 2015, o 11:28