ATNEL tech-forum 2016-02-15T19:54:12+01:00 https://forum.atnel.pl/feed.php?f=4&t=14376&mode 2016-02-15T19:54:12+01:00 2016-02-15T19:54:12+01:00 https://forum.atnel.pl/viewtopic.php?t=14376&p=153725#p153725 <![CDATA[Re: [Atmega8A]LCD hd44780, RTC DS1307 - zawieszanie się prog]]> Kod programu, który zamieściłeś w Twoim pierszym poście zawiera kilka błędów.

Zobacz:

[syntax=c]#include <avr/io.h>
#include "HD44780.c" [b]//includuje się tylko pliki z rozszerzeniem *.h, a nie z *.c[/b]
#include "DS1307.h"
#include "TWI/i2c_twi.c" [b]//includuje się tylko pliki z rozszerzeniem *.h, a nie z *.c[/b]
#include <string.h>
#include <stdlib.h>
#include <avr/interrupt.h>
#include <util/delay.h>[/syntax]

Powinno to wyglądać raczej tak:
plik main.c:
[syntax=c]#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>

#include <string.h>
#include <stdlib.h>

#include "HD44780.h"
#include "DS1307.h"
#include "TWI/i2c_twi.h"

#define cbi(sfr,bit) (_SFR_BYTE(sfr) &= ~_BV(bit))
#define sbi(sfr,bit) ((_SFR_BYTE(sfr) |= _BV(bit)))

uint8_t _ds1307_bcd_2_dec(uint8_t bcd);

int main(void)
{
//PB1 dioda, PB0 przycisk
//DDRB = 0b11111011; //ustawienie jako wyjsc wszystkiego poza PB2 (on jest wejściem)

char tekst1[]=" : : ";
//char tekst2[]="tescik ";
//int zmienna=0, zmienna2=0;
uint8_t seconds, minutes, hours;

DDRB = 0b11111010;
PORTB = 0b00000101;

i2cSetBitrate( 100 ); //prędkość I2C 100kHz dla DS1307

LCD_Initalize();

DS1307_init();
//SetTime(4,36,00);


sei();

while(1){
_delay_ms(250);
DS1307_GetTime(&hours,&minutes,&seconds);

hours=_ds1307_bcd_2_dec(hours);
minutes=_ds1307_bcd_2_dec(minutes);
seconds=_ds1307_bcd_2_dec(seconds);

if(seconds<10)
{
tekst1[15] = seconds+48;
tekst1[14] = 48;
}

if(seconds >= 10)
{
tekst1[14] = (seconds/10)%10 +48;
tekst1[15] = seconds%10 +48;
}

if(minutes<10)
{
tekst1[12] = minutes+48;
tekst1[11] = 48;
}

if(minutes >= 10)
{
tekst1[11] = (minutes/10)%10 +48;
tekst1[12] = minutes%10 +48;
}
//
if(hours<10)
{
tekst1[9] = hours+48;
tekst1[8] = 48;
}

if(hours >= 10)
{
tekst1[8] = (hours/10)%10 +48;
tekst1[9] = hours%10 +48;
}

//LCD_GoTo(0,0);

LCD_WriteText(tekst1);

}

if(PINB & (0b00000001))
{
cbi(PORTB,7);
cbi(PORTB,1);
}
else
{
sbi(PORTB,7);
sbi(PORTB,1);
}

return 0;
}

uint8_t _ds1307_bcd_2_dec(uint8_t bcd)
{
// uint8_t dec = 10 * (bcd>>4);
// dec += bcd & ~(0b11110000);
// return dec;
return ((((bcd) >> 4) & 0x0F) * 10) + ((bcd) & 0x0F);
}[/syntax]

plik DS1307.h:
[syntax=c]#ifndef DS1307_H_
#define DS1307_H_

#include <avr/io.h>
#include "TWI/i2c_twi.h"
#include <string.h>
#include <stdlib.h>

// ---------------------------------------------------------------------------
// DS1307 RTC ROUTINES

#define DS1307 0xD0 // I2C bus address of DS1307 RTC 0xD0

#define SECONDS_REGISTER 0x00
#define MINUTES_REGISTER 0x01
#define HOURS_REGISTER 0x02
#define DAYOFWK_REGISTER 0x03
#define DAYS_REGISTER 0x04
#define MONTHS_REGISTER 0x05
#define YEARS_REGISTER 0x06
#define CONTROL_REGISTER 0x07
#define RAM_BEGIN 0x08
#define RAM_END 0x3F

#define TW_NACK 0x84 // read data with NACK (last byte)
#define READ 1
#define TW_READY (TWCR & 0x80) // ready when TWINT returns to logic 1.


void DS1307_GetTime(uint8_t *hours, uint8_t *minutes, uint8_t *seconds);
// returns hours, minutes, and seconds in BCD format

void DS1307_GetDate(uint8_t *months, uint8_t *days, uint8_t *years);
// returns months, days, and years in BCD format

void SetTime(uint8_t set_hour, uint8_t set_minute, uint8_t set_second); //time set

void SetDate(uint8_t set_day, uint8_t set_month, uint8_t set_year); //set date

void DS1307_init(void);

#endif /* DS1307_H_ */[/syntax]

Plik DS1307.c:
[syntax=c]#include <avr/io.h>

#include "TWI/i2c_twi.h"
#include "DS1307.h"
//#include <string.h>
//#include <stdlib.h>

uint8_t I2C_ReadNACK(void) // reads a data byte from slave ,dodano (void)
{
TWCR = TW_NACK; // nack = not reading more data
while (!TW_READY); // wait
return TWDR;
}

void I2C_WriteRegister(uint8_t busAddr, uint8_t deviceRegister, uint8_t data) //funkcja zapisu danych po I2C
{
TWI_start(); //start I2C
TWI_write(busAddr); //wyslij adres DS1307
TWI_write(deviceRegister);
TWI_write(data); //wysyłą 3 ardumenty funkcji I2C_WriteRegister
TWI_stop(); //stop I2C
}

uint8_t I2C_ReadRegister(uint8_t busAddr, uint8_t deviceRegister) //funkcja odczytu danych po I2C
{
uint8_t data = 0;

TWI_start(); //start I2C
TWI_write(busAddr); //wyslij adres DS1307
TWI_write(deviceRegister);
TWI_start(); // 2 start I2C ,musi być!!!
TWI_write(DS1307+READ);
data = I2C_ReadNACK(); // read the register data

TWI_stop(); //stop I2C
return data;
}

void DS1307_GetTime(uint8_t *hours, uint8_t *minutes, uint8_t *seconds) // returns hours, minutes, and seconds in BCD format
{
*hours = I2C_ReadRegister(DS1307,HOURS_REGISTER);
*minutes = I2C_ReadRegister(DS1307,MINUTES_REGISTER);
*seconds = I2C_ReadRegister(DS1307,SECONDS_REGISTER);
if (*hours & 0x40) // 12hr mode:
*hours &= 0x1F; // use bottom 5 bits (pm bit = temp & 0x20)
else *hours &= 0x3F; // 24hr mode: use bottom 6 bits
}

void DS1307_GetDate(uint8_t *months, uint8_t *days, uint8_t *years) // returns months, days, and years in BCD format
{
*months = I2C_ReadRegister(DS1307,MONTHS_REGISTER);
*days = I2C_ReadRegister(DS1307,DAYS_REGISTER);
*years = I2C_ReadRegister(DS1307,YEARS_REGISTER);
}

void SetTime(uint8_t set_hour, uint8_t set_minute, uint8_t set_second)
{
I2C_WriteRegister(DS1307,HOURS_REGISTER, set_hour); // add 0x40 for PM ,czyli -> set_hour+0x40 ,dla PM
I2C_WriteRegister(DS1307,MINUTES_REGISTER, set_minute);
I2C_WriteRegister(DS1307,SECONDS_REGISTER, set_second);
}

void SetDate(uint8_t set_day, uint8_t set_month, uint8_t set_year)
{
I2C_WriteRegister(DS1307,MONTHS_REGISTER, set_month);
I2C_WriteRegister(DS1307,DAYS_REGISTER, set_day);
I2C_WriteRegister(DS1307,YEARS_REGISTER, set_year);
}

void DS1307_init(void) // clear the CH bit 0b01111111;
{
TWI_start(); //start I2C
TWI_write(DS1307); //wyslij adres DS1307
TWI_write(SECONDS_REGISTER);
TWI_write(0b01111111); // clear the CH bit ,starts clock
TWI_stop(); //stop I2C
}[/syntax]

Odrazu mówię, że tego nie testowałem. Chcę Ci tylko pokazać, podstawowe błędy jakie robisz.

Poza tym nie ma potrzeby w pętli głównej co 250ms odczytywać czas z DS'a. Przecież na jego nóżce nr7 możesz ustawić cześtotliwość oscylatora na 1Hz i podłączając ją do procka możesz wyzwalać przerwanie od INT0 np zoboczem opadającym.

A jak? Temat pojawiający się pewnie dzięsiątki razy nawet na tym forum.
Choćby tutaj:
http://forum.atnel.pl/post110734.html?hilit=1307#p110734

P.S. ;)
Gorąco polecam Ci też przeczytanie książki p. Mirka:
http://atnel.pl/mikrokontrolery-avr-jezyk-c.html

Statystyki: Napisane przez anshar — 15 lut 2016, o 19:54

]]> 2016-02-15T11:05:10+01:00 2016-02-15T11:05:10+01:00 https://forum.atnel.pl/viewtopic.php?t=14376&p=153694#p153694 <![CDATA[Re: [Atmega8A]LCD hd44780, RTC DS1307 - zawieszanie się prog]]> Statystyki: Napisane przez lukhol — 15 lut 2016, o 11:05

]]> 2016-02-04T11:50:58+01:00 2016-02-04T11:50:58+01:00 https://forum.atnel.pl/viewtopic.php?t=14376&p=152619#p152619 <![CDATA[Re: [Atmega8A]LCD hd44780, RTC DS1307 - zawieszanie się prog]]> Dlatego stawiam, że masz problem z komunikacją z LCD i stajesz na sprawdzaniu flagi BF.
Aby to potwierdzić, to zmień konfigurację swojej biblioteki, by nie był używany pin R/W tylko zwykłe delay-e.

--
Pozdrawiam,
Robert

------------------------ [ Dodano po: 8 minutach ]

Możesz potestować samodzielnie wymuszanie zawieszania poprzez odłączenie linii idącej do sterownika dla LCD.
Powinieneś mieć taki sam efekt.

Czy masz na pewno poprawnie zrobione zasilanie oraz na pewno dobrze stykające kable?

Statystyki: Napisane przez rskup — 4 lut 2016, o 11:50

]]> 2016-02-02T16:37:57+01:00 2016-02-02T16:37:57+01:00 https://forum.atnel.pl/viewtopic.php?t=14376&p=152457#p152457 <![CDATA[Re: [Atmega8A]LCD hd44780, RTC DS1307 - zawieszanie się prog]]>
Aktualnie kod wygląda tak:

[syntax=c]#include <avr/io.h>
#include "HD44780.c"
#include "DS1307.h"
#include "TWI/i2c_twi.c"
#include <string.h>
#include <stdlib.h>
#include <avr/interrupt.h>
#include <util/delay.h>

#define cbi(sfr,bit) (_SFR_BYTE(sfr) &= ~_BV(bit))
#define sbi(sfr,bit) ((_SFR_BYTE(sfr) |= _BV(bit)))

uint8_t _ds1307_bcd_2_dec(uint8_t bcd);

volatile uint8_t cnt=0, flaga = 0;

int main(void){
//PB1 dioda, PB0 przycisk
//DDRB = 0b11111011; //ustawienie jako wyjsc wszystkiego poza PB2 (on jest wejściem)

char tekst1[]=" : : ";
//char tekst2[]="tescik ";
//int zmienna=0, zmienna2=0;
uint8_t seconds, minutes, hours;


DDRB = 0b11111010;
PORTB = 0b00000101;

i2cSetBitrate( 100 ); //prędkość I2C 100kHz dla DS1307

LCD_Initalize();

//DS1307_init();
//SetTime(4,36,00);

TIMSK |= (1<<TOIE0); //Przerwanie overflow (przepełnienie timera)
TCCR0 |= (1<<CS02) | (1<<CS00); // źródłem CLK, preskaler 1024
TCNT0 = 6;

sei();


while(1){
//_delay_ms(250);
DS1307_GetTime(&hours,&minutes,&seconds);

if(flaga)
{
hours=_ds1307_bcd_2_dec(hours);
minutes=_ds1307_bcd_2_dec(minutes);
seconds=_ds1307_bcd_2_dec(seconds);
}

//*******************************************
//sekundy:
if(seconds<10)
{
tekst1[15] = seconds+48;
tekst1[14] = 48;
}

if(seconds >= 10)
{
tekst1[14] = (seconds/10)%10 +48;
tekst1[15] = seconds%10 +48;
}
//*******************************************
//minuty
if(minutes<10)
{
tekst1[12] = minutes+48;
tekst1[11] = 48;
}

if(minutes >= 10)
{
tekst1[11] = (minutes/10)%10 +48;
tekst1[12] = minutes%10 +48;
}
//*******************************************
//godziny
if(hours<10)
{
tekst1[9] = hours+48;
tekst1[8] = 48;
}

if(hours >= 10)
{
tekst1[8] = (hours/10)%10 +48;
tekst1[9] = hours%10 +48;
}


if(flaga)
{
LCD_GoTo(0,0);
LCD_WriteText(tekst1);
}

if(PINB & (0b00000001))
{
cbi(PORTB,1);
}
else
{
sbi(PORTB,1);
}

}



return 0;
}

uint8_t _ds1307_bcd_2_dec(uint8_t bcd)
{
uint8_t dec = 10 * (bcd>>4);
dec += bcd & ~(0b11110000);
return dec;
}

ISR(TIMER0_OVF_vect)
{
TCNT0 = 6; //Początkowa wartość licznika

cnt++; //zwiększa zmienną licznik
if(cnt>1) //jeśli 4 przerwania (czyli ok 1 s)
{
flaga=!flaga;
if(flaga)
sbi(PORTB,1);
else
cbi(PORTB,1);
cnt=0;
}
}[/syntax]

//Edit:
Szybkość migania diody był spowodowany kawałkiem kodu, który obsługiwał zapalenia i gaszenie diody przy aktywnym przycisku. Z tego wniosek program faktycznie wpada gdzieś w pętlę nieskończoną bo przerwania po zatrzymaniu zegara działają prawidłowo.

Statystyki: Napisane przez lukhol — 2 lut 2016, o 16:37

]]> 2016-02-02T10:16:57+01:00 2016-02-02T10:16:57+01:00 https://forum.atnel.pl/viewtopic.php?t=14376&p=152419#p152419 <![CDATA[Re: [Atmega8A]LCD hd44780, RTC DS1307 - zawieszanie się prog]]> rskup napisał(a):

Stawiam, że coś przestaje łączyć na stykówce. A na przykład jak używasz przy LCD pinu R/W, to przy zakłóceniu komunikacji z LCD możesz wejść do nieskończonej pętli.
Wrzuć proste miganie na diodzie LED wykonywane z przerwania timera. Będzie wiadomo czy wiesza się Ci procesor, czy tylko zapętla się gdzieś program w pętli głównej

--
Pozdrawiam,
Robert

Wrzuciłem przerwanie,w którym migam diodą i w przypadku zawieszenia się zegara dioda przestaje migać.

Statystyki: Napisane przez lukhol — 2 lut 2016, o 10:16

]]> 2016-02-01T23:04:26+01:00 2016-02-01T23:04:26+01:00 https://forum.atnel.pl/viewtopic.php?t=14376&p=152377#p152377 <![CDATA[Re: [Atmega8A]LCD hd44780, RTC DS1307 - zawieszanie się prog]]> Wrzuć proste miganie na diodzie LED wykonywane z przerwania timera. Będzie wiadomo czy wiesza się Ci procesor, czy tylko zapętla się gdzieś program w pętli głównej

--
Pozdrawiam,
Robert

Statystyki: Napisane przez rskup — 1 lut 2016, o 23:04

]]> 2016-02-01T21:25:19+01:00 2016-02-01T21:25:19+01:00 https://forum.atnel.pl/viewtopic.php?t=14376&p=152361#p152361 <![CDATA[[Atmega8A]LCD hd44780, RTC DS1307 - zawieszanie się programu]]>


Kod programu:
[syntax=c]#include <avr/io.h>
#include "HD44780.c"
#include "DS1307.h"
#include "TWI/i2c_twi.c"
#include <string.h>
#include <stdlib.h>
#include <avr/interrupt.h>
#include <util/delay.h>

#define cbi(sfr,bit) (_SFR_BYTE(sfr) &= ~_BV(bit))
#define sbi(sfr,bit) ((_SFR_BYTE(sfr) |= _BV(bit)))

uint8_t _ds1307_bcd_2_dec(uint8_t bcd);

int main(void){
//PB1 dioda, PB0 przycisk
//DDRB = 0b11111011; //ustawienie jako wyjsc wszystkiego poza PB2 (on jest wejściem)

char tekst1[]=" : : ";
//char tekst2[]="tescik ";
//int zmienna=0, zmienna2=0;
uint8_t seconds, minutes, hours;


DDRB = 0b11111010;
PORTB = 0b00000101;

i2cSetBitrate( 100 ); //prędkość I2C 100kHz dla DS1307

LCD_Initalize();

//DS1307_init();
//SetTime(4,36,00);

sei();


while(1){
_delay_ms(250);
DS1307_GetTime(&hours,&minutes,&seconds);

hours=_ds1307_bcd_2_dec(hours);
minutes=_ds1307_bcd_2_dec(minutes);
seconds=_ds1307_bcd_2_dec(seconds);


if(seconds<10)
{
tekst1[15] = seconds+48;
tekst1[14] = 48;
}

if(seconds >= 10)
{
tekst1[14] = (seconds/10)%10 +48;
tekst1[15] = seconds%10 +48;
}
//
if(minutes<10)
{
tekst1[12] = minutes+48;
tekst1[11] = 48;
}

if(minutes >= 10)
{
tekst1[11] = (minutes/10)%10 +48;
tekst1[12] = minutes%10 +48;
}
//
if(hours<10)
{
tekst1[9] = hours+48;
tekst1[8] = 48;
}

if(hours >= 10)
{
tekst1[8] = (hours/10)%10 +48;
tekst1[9] = hours%10 +48;
}

//LCD_GoTo(0,0);

LCD_WriteText(tekst1);



}

if(PINB & (0b00000001))
{
cbi(PORTB,7);
cbi(PORTB,1);
}
else
{
sbi(PORTB,7);
sbi(PORTB,1);
}

return 0;
}

uint8_t _ds1307_bcd_2_dec(uint8_t bcd)
{
uint8_t dec = 10 * (bcd>>4);
dec += bcd & ~(0b11110000);
return dec;
}[/syntax]

Kod do biblioteki RTC ds1307

[syntax=c]#ifndef DS1307_H_
#define DS1307_H_


#include <avr/io.h>
#include "TWI/i2c_twi.h"
#include <string.h>
#include <stdlib.h>

// ---------------------------------------------------------------------------
// DS1307 RTC ROUTINES

#define DS1307 0xD0 // I2C bus address of DS1307 RTC 0xD0

#define SECONDS_REGISTER 0x00
#define MINUTES_REGISTER 0x01
#define HOURS_REGISTER 0x02
#define DAYOFWK_REGISTER 0x03
#define DAYS_REGISTER 0x04
#define MONTHS_REGISTER 0x05
#define YEARS_REGISTER 0x06
#define CONTROL_REGISTER 0x07
#define RAM_BEGIN 0x08
#define RAM_END 0x3F

#define TW_NACK 0x84 // read data with NACK (last byte)
#define READ 1
#define TW_READY (TWCR & 0x80) // ready when TWINT returns to logic 1.



void DS1307_GetTime(uint8_t *hours, uint8_t *minutes, uint8_t *seconds);
// returns hours, minutes, and seconds in BCD format

void DS1307_GetDate(uint8_t *months, uint8_t *days, uint8_t *years);
// returns months, days, and years in BCD format

void SetTime(uint8_t set_hour, uint8_t set_minute, uint8_t set_second); //time set

void SetDate(uint8_t set_day, uint8_t set_month, uint8_t set_year); //set date


void DS1307_init(void);

//######################################################################################################################

uint8_t I2C_ReadNACK(void) // reads a data byte from slave ,dodano (void)
{
TWCR = TW_NACK; // nack = not reading more data
while (!TW_READY); // wait
return TWDR;
}





void I2C_WriteRegister(uint8_t busAddr, uint8_t deviceRegister, uint8_t data){ //funkcja zapisu danych po I2C

TWI_start(); //start I2C
TWI_write(busAddr); //wyslij adres DS1307
TWI_write(deviceRegister);
TWI_write(data); //wysyłą 3 ardumenty funkcji I2C_WriteRegister
TWI_stop(); //stop I2C

}




uint8_t I2C_ReadRegister(uint8_t busAddr, uint8_t deviceRegister){ //funkcja odczytu danych po I2C

uint8_t data = 0;

TWI_start(); //start I2C
TWI_write(busAddr); //wyslij adres DS1307
TWI_write(deviceRegister);
TWI_start(); // 2 start I2C ,musi być!!!
TWI_write(DS1307+READ);
data = I2C_ReadNACK(); // read the register data

TWI_stop(); //stop I2C

return data;
}

//#######################################################################################################################



void DS1307_GetTime(uint8_t *hours, uint8_t *minutes, uint8_t *seconds)
// returns hours, minutes, and seconds in BCD format
{
*hours = I2C_ReadRegister(DS1307,HOURS_REGISTER);
*minutes = I2C_ReadRegister(DS1307,MINUTES_REGISTER);
*seconds = I2C_ReadRegister(DS1307,SECONDS_REGISTER);
if (*hours & 0x40) // 12hr mode:
*hours &= 0x1F; // use bottom 5 bits (pm bit = temp & 0x20)
else *hours &= 0x3F; // 24hr mode: use bottom 6 bits
}
void DS1307_GetDate(uint8_t *months, uint8_t *days, uint8_t *years)
// returns months, days, and years in BCD format
{
*months = I2C_ReadRegister(DS1307,MONTHS_REGISTER);
*days = I2C_ReadRegister(DS1307,DAYS_REGISTER);
*years = I2C_ReadRegister(DS1307,YEARS_REGISTER);
}
void SetTime(uint8_t set_hour, uint8_t set_minute, uint8_t set_second)

{

I2C_WriteRegister(DS1307,HOURS_REGISTER, set_hour); // add 0x40 for PM ,czyli -> set_hour+0x40 ,dla PM
I2C_WriteRegister(DS1307,MINUTES_REGISTER, set_minute);
I2C_WriteRegister(DS1307,SECONDS_REGISTER, set_second);
}


void SetDate(uint8_t set_day, uint8_t set_month, uint8_t set_year){

I2C_WriteRegister(DS1307,MONTHS_REGISTER, set_month);
I2C_WriteRegister(DS1307,DAYS_REGISTER, set_day);
I2C_WriteRegister(DS1307,YEARS_REGISTER, set_year);


}



// ---------------------------------------------------------------------------


void DS1307_init(void){
// clear the CH bit 0b01111111;
TWI_start(); //start I2C
TWI_write(DS1307); //wyslij adres DS1307
TWI_write(SECONDS_REGISTER);
TWI_write(0b01111111); // clear the CH bit ,starts clock
TWI_stop(); //stop I2C

}

#endif /* DS1307_H_ */[/syntax]

Biblioteka do wyświetlacza oczywiście edytowane porty pod mój układ pobrałem ze strony radzio.dxp.pl - Sterowanie w trybie 4-bitowym z odczytem flagi zajętości z dowolnym przypisaniem linii sterujących.

Biblioteki do TWI pobrane ze strony p.Mirka w dziale uaktualnienia do DVD.

Obrazek

Statystyki: Napisane przez lukhol — 1 lut 2016, o 21:25

]]>