ATNEL tech-forum

Re: PCF8583 zegar stoi w miescu

2014-09-02T22:21:54+01:00

Witam,

Sorry, faktycznie miało być 0x00 - późno już było... . Masz podpięte przyciski do PD6 i PD7 ? W takim razie:

[syntax=c]DDRD = DDRD | (1<
po co ustawiasz te linie jako wyjścia ??? Klawiatura Ci pewnie przez to działa bez sensu - nie wiadomo, co czytasz. przestaw DDR tak, aby PD6 i PD7 były wejściami.

Pozdrawiam, QuadMan.

Statystyki: Napisane przez QuadMan — 2 wrz 2014, o 22:21

Re: PCF8583 zegar stoi w miescu

2014-09-02T20:37:09+01:00

To ja też bym spróbował je wyświetlić i zobaczyć jak się RTC zachowuje

Ale zwykle, jeśli masz problem, staraj się rozbić go na mniejsze podproblemy, potem złączysz je w całosć i rozwiażesz problem.
Chodzi mi o to, byś zakomentował funkcję ustawiania czasu(pomińmy póki co jaka godzina jest ustawiona - na pewno jakaś jest) i obsługi przycisków. DOdatkowo wyłączył bym póki co przerwanie i w while'u z delayem odczytywałbym czas i wrzucał go na LCD. Wtedy jeśli to ruszy, dopiero dodawaj przyciski( po jednym), potem przerwanie. Po drodze na pewno problem się sam odnajdzie.

Wstaw proszę w nowym poście kod po mojej radzie (jeśli uważasz ją za słuszną oczywiście )

Statystyki: Napisane przez Sheep — 2 wrz 2014, o 20:37

Re: PCF8583 zegar stoi w miescu

2014-09-02T20:31:41+01:00

Sheep napisał(a):

A czy tutaj, nie powinieneś podać kolejnych numerów do odpowiedniego wskazania indeksu tablicy?
[syntax=c]// wskazujący nam później na odpowiednie indeksy w tablicy (buforze)

enum {ss=1, mm=2, hh=3};[/syntax]
No i dlaczego nie numerujesz od zera?

Zmieniłem na
[syntax=c]enum {ss=1, mm=2, hh=3};[/syntax]
A później dla pewności na
[syntax=c]sekundy = bcd2dec( bufor[1] );
minuty = bcd2dec( bufor[2] );
godziny = bcd2dec( bufor[3] );[/syntax]
Ale jest bez zmian.

Nie numeruje od zera bo pod indeksem zero są zwracane setne sekundy, których nie wyświetlam na LCD.

Statystyki: Napisane przez look-ashek — 2 wrz 2014, o 20:31

Re: PCF8583 zegar stoi w miescu

2014-09-02T20:18:01+01:00

Statystyki: Napisane przez Sheep — 2 wrz 2014, o 20:18

Re: PCF8583 zegar stoi w miescu

2014-09-02T20:04:12+01:00

Niestety nie pomogło.

1) Skopiowałem kod z Erraty
2) Odpiąłem EEPRON, przez wyciągniecie zworki J13
3) Zmieniłem funkcje konwertujące (za radą sheep)
4) Zresetowałem rejestr konfiguracyjny (za radą QuadMan). Z tym, ze wywołałem funkcję TWI_write_buf( PCF8583_ADDR, 0x00, 1, bufor ), czyli od adresu 0x00 a nie tak jak napisał QuadMan
5) Dla pewności do ustawiania czasu zrezygnowałem z funkcji konwertujących i wpisuje bezpośrednio wartości binarne

Reset i ustawienie czasu przeniosłem do funkcji wywoływanych po wciśnięciu przycisków (PD6 i PD7)

Teraz sytuacja wygląda tak.
W zależnosći od tego jaki czas uwawie to zegar tyka lub nie. Ale jak tyka to wartosci są losowe. Np.
12:12:12
12:12:10
12:12:00
12:12:10

I uwaga ciekawostka.
Jak wcisnę przycisk który wywołuje funkcję reset_zegara() i trzymam go wcisnięty to pojawia się "prawidłowa" godzina i zegar tyka poprawnie.

Ktoś ma pomysł co się dzieje.

[syntax=c]/*
* main.c
*
* Created on: 2010-09-07
* Autor: Mirosław Kardaś
*
* Program korzysta ze sprzętowej obsługi I2C/TWI
* Umożliwia zapis danych do RTC oraz EEPROM a także
* oczyt z tych urządzeń
*
*/
#include
#include

#include

#include "LCD/lcd44780.h"
#include "I2C_TWI/i2c_twi.h"

#define PCF8583_ADDR 0xA2

#define WCISNIETY_PRZYCISK1_1 (1<#define WCISNIETY_PRZYCISK1_2 (1<#define LED_PIN (1<#define LED_TOG PORTC ^= LED_PIN// makrodefinicja – zmiana stanu diody

// utworzenie typu u08, odpowiednika uint8_t aby kod był bardziej czytelny
typedef unsigned char u08;
typedef unsigned short u16;

volatile uint8_t int0_flag=1;// flaga zmieniana w przerwaniu i sprawdzana w pętli głównej

// konwersja liczby dziesiętnej na BCD
uint8_t dec2bcd(uint8_t dec);
// konwersja liczby BCD na dziesiętną
uint8_t bcd2dec(uint8_t bcd);

uint8_t przycisk_wcisniety(uint8_t klawisz);
void reset_zegara(void);
void ustaw_czas(void);

int main(void) {

DDRC |= LED_PIN;// kierunek pinu PC7 – wyjściowy
PORTC |= LED_PIN;// wyłączenie diody LED

DDRD = DDRD | (1<PORTD = PORTD | (1<

// Przerwanie INT0
MCUCR |= (1<GICR |= (1<PORTD |= (1<

// definiujemy sobie dla polepszenia czytelności programu typ wyliczeniowy
// wskazujący nam później na odpowiednie indeksy w tablicy (buforze)
enum {ss=1, mm, hh};
uint8_t bufor[4];// rezerwacja bufora 4 bajty

uint8_t war_resetu[1];// rezerwacja bufora 4 bajty

uint8_t sekundy, minuty, godziny;

lcd_init();

i2cSetBitrate(100);

sei();

while(1) {

if ( int0_flag ) {

//odczyt 4 bajtów do bufora od adresu 0x01 z pamięci RAM naszego RTC
TWI_read_buf( PCF8583_ADDR, 0x01, 4, bufor );

sekundy = bcd2dec( bufor[ss] );
minuty = bcd2dec( bufor[mm] );
godziny = bcd2dec( bufor[hh] );

// wyświetlenie czasu na LCD
lcd_locate(1,0);
if( godziny < 10 ) lcd_str("0");
lcd_int(godziny);
lcd_str(":");
if( minuty < 10 ) lcd_str("0");
lcd_int(minuty);
lcd_str(":");
if( sekundy < 10 ) lcd_str("0");
lcd_int(sekundy);

TWI_read_buf( PCF8583_ADDR, 0x00, 1, war_resetu );
lcd_locate(0,0);
lcd_int(war_resetu[0]);

LED_TOG;// diida do debugowania wskazuje czy jest przerwania
int0_flag=0;
}

if ( przycisk_wcisniety(WCISNIETY_PRZYCISK1_1) ) reset_zegara();
if ( przycisk_wcisniety(WCISNIETY_PRZYCISK1_2) ) ustaw_czas();

}
}

// procedura obsługi przerwania INT 0
ISR( INT0_vect ) {
int0_flag = 1;
}

// konwersja liczby dziesiętnej na BCD
uint8_t bcd2dec(uint8_t c)
{
return ((c/16)*10)+(c%16);
}
uint8_t dec2bcd(uint8_t num)
{
return ((num/10 * 16) + (num % 10));
}

uint8_t przycisk_wcisniety(uint8_t klawisz)
{
if (!(PIND & klawisz))
{
_delay_ms(80);
if (!(PIND & klawisz)) return 1;
}

return 0;
}

void reset_zegara(void)
{
uint8_t bufor[4];
bufor[0] = 0;
TWI_write_buf( PCF8583_ADDR, 0x00, 1, bufor );

}
void ustaw_czas(void)
{
uint8_t bufor[5];
bufor[0] = 0; // reset zegara
bufor[1] = 0;// setne części sekundy
bufor[2] = 0b0010010;// sekundy liczba 12 w formacie BCD
bufor[3] = 0b0010010;// minutyliczba 12 w formacie BCD
bufor[4] = 0b0010010;// godzinyliczba 12 w formacie BCD
// zapis 5 bajtów z bufora pod adres 0x01 w pamięci RAM naszego RTC
TWI_write_buf( PCF8583_ADDR, 0x00, 5, bufor );

}[/syntax]

Statystyki: Napisane przez look-ashek — 2 wrz 2014, o 20:04

Re: PCF8583 zegar stoi w miescu

2014-09-02T06:10:43+01:00

Widzę że definiujesz też eeprom. Rozumię że działasz na ATB. Kiedyś miałem podobny przypadek. Zanim coś pozmieniasz to zobacz tak czy ci zadziała RTC.

Wyłącz zasilanie.
Wypnij eeprom.
Włącz zasilanie.

Jak RTC rudzy to może zworka przy eeprom źle ustawiona lub eeprom nie ten albo padnięty.

Namazane ze smarta.

Statystyki: Napisane przez majster — 2 wrz 2014, o 06:10

Re: PCF8583 zegar stoi w miescu

2014-09-02T00:53:25+01:00

Witam,

Najprawdopodobniej coś przy ustawianiu czasu poszło nie tak i niechcący coś wpisałeś do rejestru konfiguracyjnego PCF8583, wystarczy jesli wpiszesz 0x00 i zegar ruszy . Poniżej masz kod:
[syntax=c]bufor[0] = 0;// dlaczego zero ?? - odsyłam do PDF-a
// zapis 1 bajtu z bufora pod adres 0x00 - to jest właśnie ten rejestr w pamięci RAM naszego RTC
TWI_write_buf( PCF8583_ADDR, 0x01, 1, bufor );[/syntax]

Umieść to przed wejściem do pętli głównej, zaraz po sei();. W zasadzie możesz to zostawić na stałe - jeśli coś znów pójdzie nie tak, to zegar się odblokuje .

Pozdrawiam, QuadMan.

Statystyki: Napisane przez QuadMan — 2 wrz 2014, o 00:53

Re: PCF8583 zegar stoi w miescu

2014-09-02T00:18:07+01:00

Hej,

Zobacz proszę, czy działa Tobie Twój kod, ale z tymi funkcjami konwertującymi:
[syntax=c]uint8_t bcd2dec(uint8_t c)
{
return ((c/16)*10)+(c%16);
}
uint8_t dec2bcd(uint8_t num)
{
return ((num/10 * 16) + (num % 10));
}[/syntax]

@@EDIT

look-ashek napisał(a):

ale zegar stoi w miejscu

Nie ma czegoś takiego, jak zegar, który się zatrzymał. Nie wiem jak jest w przypadku PCF'a, ale z tego co na szybko wyczytałem też jest bateria zewnętrzna jako podtrzymanie zastosowana, mi się wydaję, że zegar cały czas tyka (możesz to sprawdzić?), tylko kwestia z odbiorem tych danych po I2C.

Jesteś w stanie wrócić do kodu, w którym jest sam odbiór i wyświetlenie tego na LCD?

Statystyki: Napisane przez Sheep — 2 wrz 2014, o 00:18

Re: PCF8583 zegar stoi w miescu

2014-09-01T22:18:51+01:00

podałeś kod gdzie nie zmieniasz już ustawiania godziny. Na tym kodzie rtc też nie działa ? skopiuj dokładnie kod z erraty do BB i spróbuj odpalić nic nie grzebiąc w kodzie, czasami można coś przeoczyć albo niechcący usunąć. Pozdrawiam

Statystyki: Napisane przez Nondzu — 1 wrz 2014, o 22:18

PCF8583 zegar stoi w miescu

2014-09-01T20:21:32+01:00

Witam serdecznie.

Mam problem z prawidłowym działaniem układu RTC. (Kodz niebieskiej książki dla rozdziału4.10.1 RTC - sprzętowa obsługa I2C)
Używam zestawu ATB 1.04

Problem polega na tym że cały czas na LCD wyświetla się ta sama godzina.

A tera jak do tego doszło.
1)Najpierw lekko zmodyfikowałem i wgrałem kod z książki(z DVD)
Program zadział i na LCD wyświetlił się godzina zmieniając się co sekundę.

2)Ale godzina był nieprawidłowa, więc postanowiłem ją zmienić. W kodzie był za komentowany fragment kodu do ustawiania czasu.
Odkomentowałem go, wpisałem nowe wartości i wysłalem do AVR.
Następnie od razu zakomentowalem ten fragment i wgrałem poprzedni kod (ten który tylko odczytuje czas)
Czas się zmienił na prawidłowy, ale zaczął się dziwnie zmieniać. Sekundy zmieniały się od 0 do 7 i z powrotem 0 .. Minuty w ogóle się nie zmieniały.

3) Dzisiaj postanowiłem zmienić tak kod, żeby po naciśnięciu przycisku godzina zmieniła się na 20:00:00.
kod wgrałem godzina się zmieniła, ale zegar stoi w miejscu

Sprawdziłem, że przerywania się wykonują.

Załączam kod, który jest aktualnie wgany
[syntax=c]/*
* main.c
*
* Created on: 2010-09-07
* Autor: Mirosław Kardaś
*
* Program korzysta ze sprzętowej obsługi I2C/TWI
* Umożliwia zapis danych do RTC oraz EEPROM a także
* oczyt z tych urządzeń
*
* UWAGA!!! programowa wersja dla F_CPU = 8 MHz
* jeli zmienione zostanie taktowanie na np: 16 MHz
* wtedy trzeba sobie przerobić opóźnienia w bibliotece
*
*/
#include
#include
#include
#include

#include "LCD/lcd44780.h"
#include "I2C_TWI/i2c_twi.h"

// **** UWAGA - wybierz właciwy adres jak niżej **************
//#define PCF8583_ADDR 0xA0// gdy A1 --> GND
#define PCF8583_ADDR 0xA2// gdy A1 --> VCC

#define _24C04_ADDR 0xA8

// dioda LED
#define LED_PIN (1<#define LED_TOG PORTC ^= LED_PIN// makrodefinicja – zmiana stanu diody

// utworzenie typu u08, odpowiednika uint8_t aby kod był bardziej czytelny
typedef unsigned char u08;
typedef unsigned short u16;

volatile uint8_t int0_flag=1;// flaga zmieniana w przerwaniu i sprawdzana w pętli głównej

// konwersja liczby dziesiętnej na BCD
uint8_t dec2bcd(uint8_t dec);
// konwersja liczby BCD na dziesiętną
uint8_t bcd2dec(uint8_t bcd);

int main(void) {

DDRC |= LED_PIN;// kierunek pinu PC7 – wyjściowy
PORTC |= LED_PIN;// wyłączenie diody LED

// Przerwanie INT0
MCUCR |= (1<GICR |= (1<PORTD |= (1<

// definiujemy sobie dla polepszenia czytelności programu typ wyliczeniowy
// wskazujący nam później na odpowiednie indeksy w tablicy (buforze)
enum {ss=1, mm, hh};
uint8_t bufor[4];// rezerwacja bufora 4 bajty
uint8_t sekundy, minuty, godziny;

lcd_init();

i2cSetBitrate( 100 ); // USTAWIAMY prędkość 100 kHz na magistrali I2C

sei();

while(1) {

if ( int0_flag ) {
//odczyt 4 bajtów do bufora od adresu 0x01 z pamięci RAM naszego RTC
TWI_read_buf( PCF8583_ADDR, 0x01, 4, bufor );

sekundy = bcd2dec( bufor[ss] );
minuty = bcd2dec( bufor[mm] );
godziny = bcd2dec( bufor[hh] );

// wyświetlenie czasu na LCD
lcd_locate(1,0);
if( godziny < 10 ) lcd_str("0");
lcd_int(godziny);
lcd_str(":");
if( minuty < 10 ) lcd_str("0");
lcd_int(minuty);
lcd_str(":");
if( sekundy < 10 ) lcd_str("0");
lcd_int(sekundy);

LED_TOG;// zmień stan diody na przeciwny

int0_flag=0;
}

}
}

// procedura obsługi przerwania INT 0
ISR( INT0_vect ) {
int0_flag = 1;
}

// konwersja liczby dziesiętnej na BCD
uint8_t dec2bcd(uint8_t dec) {
return ((dec / 10)<<4) | (dec % 10);
}

// konwersja liczby BCD na dziesiętną
uint8_t bcd2dec(uint8_t bcd) {
return ((((bcd) >> 4) & 0x0F) * 10) + ((bcd) & 0x0F);
}[/syntax]

Czy macie pomysł co moglem zrobić źle?

Statystyki: Napisane przez look-ashek — 1 wrz 2014, o 20:21