ATNEL tech-forum

Re: ATtiny13 problem z zapisem EEPROM

2013-03-09T15:07:20+01:00

barney napisał(a):

Hmm... no faktycznie, macie rację
Nie wiem tylko czy będę w stanie dokładnie wytłumaczyć jak ten kod działa (zdolności dydaktycznych nie mam wcale, a konstrukcje są trochę zakręcone).

Postaraj się wytłumaczyć w ogóle co to za urządzenie, jak działa - jego zalety, przewaga nad podobnymi rozwiązaniami - o takie rzeczy szczególnie - żeby mogło być to też inspiracją dla innych

Statystyki: Napisane przez mirekk36 — 9 mar 2013, o 15:07

Re: ATtiny13 problem z zapisem EEPROM

2013-03-09T14:50:51+01:00

Hmm... no faktycznie, macie rację
Nie wiem tylko czy będę w stanie dokładnie wytłumaczyć jak ten kod działa (zdolności dydaktycznych nie mam wcale, a konstrukcje są trochę zakręcone).

Statystyki: Napisane przez barney — 9 mar 2013, o 14:50

Re: ATtiny13 problem z zapisem EEPROM

2013-03-09T14:43:27+01:00

Jaglarz napisał(a):

No to skoro uzyskałeś TUTAJ pomoc to może wypadałoby opublikować to TUTAJ w dziale DIY?

dokładnie - sam bym z chęcią sobie o tym u nas poczytał ....

Statystyki: Napisane przez mirekk36 — 9 mar 2013, o 14:43

Re: ATtiny13 problem z zapisem EEPROM

2013-03-09T14:42:11+01:00

No to skoro uzyskałeś TUTAJ pomoc to może wypadałoby opublikować to TUTAJ w dziale DIY?

Statystyki: Napisane przez Jaglarz — 9 mar 2013, o 14:42

Re: ATtiny13 problem z zapisem EEPROM

2013-03-09T14:33:54+01:00

Jeżeli ktoś jest ciekaw, co się narodziło z projektu to zapraszam do wątku:
http://www.swiatelka.pl/viewtopic.php?t=10224

Statystyki: Napisane przez barney — 9 mar 2013, o 14:33

Re: ATtiny13 problem z zapisem EEPROM

2013-02-22T15:17:48+01:00

Zgadza się. Przy operacjach bitowych następuje promocja do typu int o długości 16-bit. Zastosowałeś rzutowanie, ale ono ma wyższy priorytet niż przesunięcie i w efekcie rzutowałeś uint8_t na uint8_t, a nie wynik przesunięcia. Tak powinno być w tym przykładzie:
[syntax=c]if ((u08) (cfg_ram.ktory_tryb<<7) )[/syntax]
osobiście sądzę że lepszy jest taki zapis:
[syntax=c]if (cfg_ram.ktory_tryb & 0x01 )[/syntax]

Statystyki: Napisane przez Krauser — 22 lut 2013, o 15:17

Re: ATtiny13 problem z zapisem EEPROM

2013-02-22T14:53:34+01:00

Dlaczego taki zapis działa jak powinien:
[syntax=c]licznik=cfg_ram.ktory_tryb<<7;
if (licznik) ... [/syntax]
natomiast zapisany już w ten sposób:
[syntax=c]if (cfg_ram.ktory_tryb<<7) ...[/syntax]
nie działa prawidłowo? Obie zmienne są oczywiście u08. Konstrukcja ta ma wykonywać 2 różne operacje w zależności od stanu najmłodszego bitu (a pozostałe pomijać). Przy drugim zapisie jeżeli cfg_ram.ktory_tryb jest większy od 0b00000001 zawsze wychodzi z warunku 1. Dokonuje się niejawne rzutowanie? Dodam, że taki zapis:
[syntax=c]if ((u08) cfg_ram.ktory_tryb<<7) ...[/syntax]
nie rozwiązuje problemu

Statystyki: Napisane przez barney — 22 lut 2013, o 14:53

Re: ATtiny13 problem z zapisem EEPROM

2013-02-19T22:29:11+01:00

Wielkie dzięki

W obydwu kodach ten sam błąd
Pierwszy zadziałał od kopa, drugi dopiero po przypisaniu wartości początkowych do EEPROMu, ale ważne, że już działa

Statystyki: Napisane przez barney — 19 lut 2013, o 22:29

Re: ATtiny13 problem z zapisem EEPROM

2013-02-19T20:57:41+01:00

Tam ma być != (wykrzyknik i równa się, czyli operator porównania różne) a nie |= (pionowa kreska i równa się)

Po wyjściu z main jest cli(), a potem pętla nieskończona. Sprawdź sobie w pliku wynikowym. Dlatego to sei() na nic się zda. Wstaw jednak to while(1); to jeden rozkaz.

Statystyki: Napisane przez Krauser — 19 lut 2013, o 20:57

Re: ATtiny13 problem z zapisem EEPROM

2013-02-19T20:04:19+01:00

No faktycznie ta linia

Code:

if (cfg_ram.eprOK|=15) // Sprawdzenie poprawnosci EEPROM

jest dosyć radosną konstrukcją, ale powinna działać.
O ile dobrze pamiętam z http://mirekk36.blogspot.com/2012/11/avr-eeprom-struktury-poradnik-do-ksiazek.html to wyzerowany EEPROM przyjmuje wartość 0xFF, więc ten zapis miał w sposób "garażowy" sprawdzać poprawność EEPROMu.
Jeżeli starczy mi miejsca to pewnie wrzucę sprawdzanie tego typu:
[syntax=c]// Autor: barney
#include
#include
#include
#include
#include
#include "setup.h"
#define AMC (1<#define PWM OCR0B
typedef uint8_t u08;
typedef uint16_t u16;

typedef struct
{
u08 tryb[9];
u16 strb_czasy[2];
u08 rower;
u08 strb;
u08 ktory_tryb;
u08 ignoruj_klik;
u16 crc;
} TCFG;

const TCFG cfg_pgm PROGMEM =
{
{LVL1, LVL2, LVL3, LVL4, LVL5, LVL6, LVL7, LVL8, LVL9},
{STRB_OFF, STRB_ON},
0,
0,
5,
1,
0
};
TCFG cfg_ram;
TCFG cfg_epr EEMEM;

void epr_to_ram (void);
void ram_to_epr (void);

int main (void)
{
epr_to_ram();// Odczytanie zawartosci EEPROM
/*if (cfg_ram.ignoruj_klik) cfg_ram.ignoruj_klik=0; else //Obsluga klikniec
{
cfg_ram.ktory_tryb++;
if (cfg_ram.ktory_tryb==TRYBY) cfg_ram.ktory_tryb=0;
}*/
cfg_ram.ktory_tryb++;
if (cfg_ram.ktory_tryb==TRYBY) cfg_ram.ktory_tryb=0;
ram_to_epr();
DDRB |= AMC; // kierunek pinu PB1 wyjsciowy
TCCR0A |= (1<TCCR0A |= (1<TCCR0B |= (1<PWM=cfg_ram.tryb[cfg_ram.ktory_tryb];// Ustawienie trybu
/*_delay_ms (TTS);// Czekanie do zapamietania trybu
cfg_ram.ignoruj_klik=1;
ram_to_epr();*/
return 0;
}

void epr_to_ram (void)
{
u08 temp; u16 crc_check;

eeprom_read_block(&cfg_ram, &cfg_epr, sizeof(cfg_ram));

crc_check=cfg_ram.rower+cfg_ram.strb+cfg_ram.ktory_tryb+cfg_ram.ignoruj_klik;
for (temp=0; temp<=1; temp++) crc_check+=cfg_ram.strb_czasy[temp];
for (temp=0; temp<=8; temp++) crc_check+=cfg_ram.tryb[temp];
if (crc_check |= cfg_ram.crc) memcpy_P(&cfg_ram, &cfg_pgm, sizeof(cfg_ram));
}
void ram_to_epr (void)
{
u08 temp;

cfg_ram.crc=cfg_ram.rower+cfg_ram.strb+cfg_ram.ktory_tryb+cfg_ram.ignoruj_klik;
for (temp=0; temp<=1; temp++) cfg_ram.crc+=cfg_ram.strb_czasy[temp];
for (temp=0; temp<=8; temp++) cfg_ram.crc+=cfg_ram.tryb[temp];

eeprom_write_block(&cfg_ram, &cfg_epr, sizeof(cfg_ram));
}
[/syntax]
ale to zajmuje ~100B FLASHa, a jest go tutaj cały 1KB więc nie mam za bardzo czym szastać.
Nie wiem czemu, ale przy takiej konstrukcji układ działa trochę inaczej.
Przy starej od razu po zaprogramowaniu dioda zaczynała świecić zaprogramowana jasnością, a teraz dopiero po jednym restarcie (po zaprogramowaniu zachowuje się jakby nie mógł w ogóle nic sczytać) ...
Przy obydwu oczywiście nie można tej jasności normalnie zmienić...

A sei(); będzie potrzebny w przyszłości (ochrona ogniwa, itp.), ale faktycznie obecnie jest niepotrzebny.

Statystyki: Napisane przez barney — 19 lut 2013, o 20:04

Re: ATtiny13 problem z zapisem EEPROM

2013-02-19T18:41:39+01:00

Ta linia jest podejrzana:
[syntax=c]if (cfg_ram.eprOK|=15) // Sprawdzenie poprawnosci EEPROM[/syntax]
PS.
niepotrzebe jest sei();

Statystyki: Napisane przez Krauser — 19 lut 2013, o 18:41

Re: ATtiny13 problem z zapisem EEPROM

2013-02-19T18:32:16+01:00

ok popatrzę wieczorkiem

Statystyki: Napisane przez mirekk36 — 19 lut 2013, o 18:32

Re: ATtiny13 problem z zapisem EEPROM

2013-02-19T17:45:29+01:00

Chyba się nie zrozumieliśmy...
PCB wygląda mniej-więcej tak: http://www.swiatelka.pl/upload_img/obra ... 5d6012.jpg
Włącznik podłącza i odłącza baterie od całego układu (razem z uC).

Cały więc myk polega na tym, aby program ten analizując odpowiednie sekwencje kliknięć (przy czym kliknięciem jest tu odcięcie zasilania od uC) wrzucił odpowiednie wypełnienie na ster AMC7135.

Chyba że ja czegoś nie zrozumiałem i to:

int main (void)
{
epr_to_ram();
(...)
return 0;
}
Nie jest głównym programem...
Pętli w stylu while (1)... Nie ma bo program ma (bardzo) szybko wykonać jakieś kombinacje po tej strukturze, zapisać ją, ustawić odpowiednie wypełnienie i zacząć idlować.

Statystyki: Napisane przez barney — 19 lut 2013, o 17:45

Re: ATtiny13 problem z zapisem EEPROM

2013-02-19T17:25:07+01:00

Panie, Panie .... haaaalo stop bo wybiegłeś na pole ale jak to się mówi pługa zapomniałeś .... a gdzie u ciebie pętla główna (czyli ten pług) ???

w tył zwrot i idź go zabrać i używać

bo teraz to w tym programie jakimś klawiszem to może MATRIX by zdążył

to nie jest w ogóle program

Statystyki: Napisane przez mirekk36 — 19 lut 2013, o 17:25

ATtiny13 problem z zapisem EEPROM

2013-02-19T16:36:33+01:00

Witam wszystkich

Mam problem z (najprawdopodobniej) zapisem EEPROMu.
Kod ten ma trafić do procka w latarce LED (włącznik jedynie załącza i odcina baterie).

main.c
[syntax=c]// Autor: barney
#include
#include
#include
#include
#include
#include "setup.h"
#define AMC (1<#define PWM OCR0B
typedef uint8_t u08;
typedef uint16_t u16;

typedef struct
{
u08 eprOK;
u08 tryb[9];
u16 strb_czasy[2];
u08 rower_clicks;
u08 rower;
u08 strb_clicks;
u08 strb;
u08 ktory_tryb;
u08 ignoruj_klik;
} TCFG;

const TCFG cfg_pgm PROGMEM =
{
15,
{LVL1, LVL2, LVL3, LVL4, LVL5, LVL6, LVL7, LVL8, LVL9},
{STRB_OFF, STRB_ON},
ROWER,
0,
STRB,
0,
4,
1
};
TCFG cfg_ram;
TCFG cfg_epr EEMEM;

void epr_to_ram (void);
void ram_to_epr (void);
void pgm_to_ram (void);

int main (void)
{
epr_to_ram();// Odczytanie zawartosci EEPROM
if (cfg_ram.eprOK|=15) // Sprawdzenie poprawnosci EEPROM
{
pgm_to_ram();
ram_to_epr();
}
/*if (cfg_ram.ignoruj_klik) cfg_ram.ignoruj_klik=0; else //Obsluga klikniec
{
cfg_ram.ktory_tryb++;
if (cfg_ram.ktory_tryb==TRYBY) cfg_ram.ktory_tryb=0;
}*/
cfg_ram.ktory_tryb++;
if (cfg_ram.ktory_tryb==TRYBY) cfg_ram.ktory_tryb=0;
ram_to_epr();
sei();// Odblokowanie przerwan
DDRB |= AMC; // kierunek pinu PB1 wyjsciowy
TCCR0A |= (1<TCCR0A |= (1<TCCR0B |= (1<//PWM=64;
PWM=cfg_ram.tryb[cfg_ram.ktory_tryb];// Ustawienie trybu
/*_delay_ms (TTS);// Czekanie do zapamietania trybu
cfg_ram.ignoruj_klik=1;
ram_to_epr();*/
return 0;
}

void epr_to_ram (void)
{
eeprom_read_block(&cfg_ram, &cfg_epr, sizeof(cfg_ram));
}
void ram_to_epr (void)
{
eeprom_write_block(&cfg_ram, &cfg_epr, sizeof(cfg_ram));
}
void pgm_to_ram (void)
{
memcpy_P(&cfg_ram, &cfg_pgm, sizeof(cfg_ram));
}
[/syntax]

setup.h
[syntax=c]#define TRYBY 7 // Ile trybow ma miec driver (max 9)

#define LVL1 255// Jasnosci trybow od tego do ktorego bedzie opadac
#define LVL2 8// max 255
#define LVL3 16
#define LVL4 32
#define LVL5 64
#define LVL6 128
#define LVL7 255
#define LVL8 0
#define LVL9 0

#define STRB_OFF 100// Czasy stroboskopu w ms (max 65535)
#define STRB_ON 100

#define ROWER 15// Ile klikow do trybu rowerowego (255 wylacza)
#define STRB 10// Ile klikow do trybu stroboskopowego (255 wylacza)

#define TTS500// Czas w ms do zapamietania trybu
[/syntax]

Klikanie włącznikiem (tzn. odcinanie zasilania) nie powoduje zmiany jasności...
Obecnie jasność zmienia się tylko jeżeli w definicji stałej struktury ładowanej do PROGMEM zmienię wartość u08 ktory_tryb.
Kompilacja w Eclipse przechodzi bez żadnych ostrzeżeń.
Aby mieć dostęp do EEPROMu trzeba ustawić jakieś FUSY?

Statystyki: Napisane przez barney — 19 lut 2013, o 16:36