ATNEL tech-forum

Re: Problem z pamięcią eeprom Atmega88P

2016-06-10T19:51:55+01:00

Ja to widzę tak (i sądzę że w tych linijkach siedzi problem ) nie mam atmegi 8 i nie mam możliwości sprawdzić .
chodzi mi o struktury takie jak

main.c

Code:

 
  // inicjalizacja ustawień fabrycznych w pamięci Flash                                   * ?  a gdzie dane zapisane do struktury? *
 
  const TCFG pgm_cfg __attribute__((__progmem__))={
                Thw,
                Thz,
 
  };                                                                                                                  * tu napisze dalej w opisie *

                    * te 2 linie pominę one powinny być raczej  w camoon.h * 
                                                                                                    
  TCFG eem_cfg EEMEM;                   // dane w pamięci EEPROM
  TCFG ram_cfg;    

common.c

Code:

typedef struct {
        uint8_t thw;
        uint8_t thz;
 
} TCFG;
 
// ustawienia domylne (fabryczne)
#define Thw 6               //                               *  coś tu chyba nie tak jest? *
#define Thz 12                  //                            *  coś tu chyba nie tak jest? *

a czy deklaracja z main.c nie powinna być w common.h ?

a w miejsce linijek które mogą ci mieszać i to bardzo w kodzie
nie lepiej w strukturze umieszczonej w PROGMEM zapisać na stałe te wartości zamiast

Code:

#define Thw 6     
#define Thz 12 

przecież i tak nie masz możliwości ich zmiany ? a po

Code:

void check_and_load_defaults( void ) {
        uint8_t i, len = sizeof( ram_cfg );
        uint8_t * ram_wsk = (uint8_t*)&ram_cfg;
 
        copy_eem_to_ram();
        for(i=0; i                if( 0xff == *ram_wsk++ ) continue;
                break;
        }
 
        if( i == len ) {
                load_defaults();                                           *   chodzi mi o to co tu się wyrabia  :)  *
        }
 
}

a dokładniej TU

Code:

void load_defaults( void ) {
        copy_pgm_to_ram();                   * OO tu  tu *
        copy_ram_to_eem();
}

czym zapełniasz dane z pgm ? przecież w odwołaniach do struktur pre-procesor już ci po podmieniał dane po # definach

// *** Pomieszał a ja jeszcze bardziej ***// dopiero zauważyłem pomylone opisy plików common.c <-> common.h

nie będę zmieniał tylko 1 linijkę wrzucę

Code:

extern const TCFG pgm_cfg __attribute__((__progmem__));         // dane w pamięci FLASH

a skąd ci się to wzięło ?
nie # include main.c skąd ma wiedzieć kompilator co to jest ?
rozumiał bym jak by struktura była w cammon.c zamiast w main.c

Według mnie tu masz takie zagwozdki które mogą powodować małe / lub / spore kłopoty sprawdź je reszta niby ok
nie mam procka ani bibliotek ds18b20 do których się odwołujesz

Statystyki: Napisane przez SIND — 10 cze 2016, o 19:51

Re: Problem z pamięcią eeprom Atmega88P

2016-06-10T09:56:37+01:00

Wrócę z pracy to sprawdzę to w taki sposób, najdziwniejsze jest to że kilka razy też dobrze wczyta a z którymś razem thz 255.

Statystyki: Napisane przez Kulfi27 — 10 cze 2016, o 09:56

Re: Problem z pamięcią eeprom Atmega88P

2016-06-10T09:42:58+01:00

Sprawdzałem ten twój kod, ale bez obsługi czujników temperatury. No niestety nie posiadam biblioteki: DS18B20/ds18b20_v1.h
Oczywiście po za komentowaniu kilku linijek związanych z tą biblioteką i wgraniu do procka, wszystko jest OK. To znaczy obydwie zmienne thw i thz wyświetlane są na wyświetlaczu LCD prawidłowo. Po zmianie ich wartości przyciskami g1-g3 i resecie procka, zmienne te odczytywane są prawidłowo.
Wniosek nasuwa mi się taki, że coś musisz mieć nie tak z tą biblioteką lub jej obsługą.
Sprawdź sam, za komentuj tą bibliotekę, wyskoczy masa błędów, za komentuj więc te linijki kodu, gdzie te błędy występują, skompiluj i wgraj do procka. Zobaczysz, że odczyt i zapis do EEPROM'a działa w porządku.
Pozdrawiam.

Statystyki: Napisane przez anshar — 10 cze 2016, o 09:42

Re: Problem z pamięcią eeprom Atmega88P

2016-06-10T08:02:21+01:00

Ten fragment ma za zadanie sprawdzić czy zawartość pamięci EE to same 255 jeśli tak to to wczytaj wartości zapisane w pamięci
flash. jeśli chodzi o weryfikacje samej pamięci EE to nie robiłem tego wrócę z pracy to sprawdzę co tam się zapisuje, dobry pomysł

Statystyki: Napisane przez Kulfi27 — 10 cze 2016, o 08:02

Re: Problem z pamięcią eeprom Atmega88P

2016-06-10T07:27:40+01:00

Co robi ten fragment w funkcji void check_and_load_defaults( void ) ?
[syntax=c]for(i=0; i if( 0xff == *ram_wsk++ ) continue;
break;
}[/syntax]
W jaki sposób weryfikujesz zawartość pamięci EEPROM? Odczytujesz ją za pomocą programatora?

Statystyki: Napisane przez Sparrow-hawk — 10 cze 2016, o 07:27

Problem z pamięcią eeprom Atmega88P

2016-06-09T22:34:07+01:00

Witam kolegów, zdaje sobie sprawy że temat już był wałowany kilkakrotnie ale mimo to nie mogłem znaleźć rozwiązania,
mam problem związany z eepromem napisałem sobie program, sterownik solara. Zapisuje sobie dwa parametry temperatura histerezy załączania i wyłączania, korzystam ze typu strukturalnego tak jak to pokazał Mirek w swoim poradniku no i problem mam taki iż po odłączeniu zasilania pierwszy parametr thw jest odczytany z EE prawidłowo ale drugi thz to już losowo wczytuje wartość 255 czasmi kilka razy wczyta dobrze. Jeszcze nie ogarniam tego wszystkiego za dobrze więc proszę o podpowiedz co jest nie tak. Poniżej przedstawiam kod [syntax=c]#include
#include
#include
#include
#include
#include "LCD/lcd44780.h"
#include "DS18B20/ds18b20_v1.h"
#include "common.h"
#include
#include

#define g1 (1<#define g2 (1<#define g3 (1<#define d1 (1<#define d2 (1<
#define relly_on PORTD |= d1
#define relly_off PORTD &=~d1
#define led_on PORTC |= d2
#define led_off PORTC &=~d2
#define menu !(PINC & g1)
#define minus !(PINC & g2)
#define pplus !(PINC & g3)

uint16_t Tb, Ts ;
uint8_t key_loc, menu_1,flaga_1,flaga_2 ;
void wys_tem(void);
void kontrola(void);
void buttun(void);
void poziom (uint8_t mm);
void Pomiar_DS18B20( char numer_ukladu);
uint16_t Odczyt_DS18B20( char numer_ukladu);
char str[20];

// inicjalizacja ustawień fabrycznych w pamięci Flash

const TCFG pgm_cfg __attribute__((__progmem__))={
Thw,
Thz,

};

TCFG eem_cfg EEMEM;// dane w pamięci EEPROM
TCFG ram_cfg; // dane w pamięci RAM

int main(void)
{
DDRC &= ~(g1 | g2 | g3 );
PORTC |= g1 | g2 | g3 ;
DDRC |= d2;
PORTC &= ~d1;

DDRD |= d1;
PORTD &= ~d1;

///////////////////////////////////////////////
//wyłączenie watchdoga po resecie
MCUSR = 0;
WDTCSR = (1< WDTCSR = 0;

///////////////////////////////////////////////

sei();
lcd_init();
led_on;
lcd_locate(0,4);
lcd_str("SOLAR" );
lcd_locate(1,4);
lcd_str("KULFI");
_delay_ms(4000);
led_off;
lcd_cls();
soft_timer_init();
copy_eem_to_ram(); //kopiowanie eepromu do ramu
check_and_load_defaults(); //sprawdzenie czy w pamieci eeprom jest cos zapisane
// wdt_enable( WDTO_120MS);

while(1)
{
// wdt_reset();
buttun();
wys_tem();
kontrola();

if(ram_cfg.thz <= ram_cfg.thw)++ram_cfg.thz;
if(ram_cfg.thw<1)ram_cfg.thw=1;

}
}

////////////////////////////////////////////////////////////

void kontrola(void)
{ //sprawdzanie czy temp. solara + tem. histerezy jest wieksza od tem. bojlera
if (Ts>=Tb+ram_cfg.thz)
{
lcd_locate(1,6);
lcd_str(">> on ");
relly_on;
}
if(Ts<=Tb+ram_cfg.thw)
{
lcd_locate(1,6);
lcd_str("<< off ");
relly_off;
}
lcd_locate(1,1);
lcd_str("Pom.");

}

/////////////////////////////////////////////////////////////

void buttun(void){
// obsługa guzika menu_1
register uint8_t key_press = !key_loc && menu; // jesli wcisnięty włącz poświetlenie
if(key_press){
led_on;
Timer3 = 120;
}
else
if(!Timer3)led_off; // odlicz czas i wyłącz podświetlenie

if(!Timer1){
if(key_press ){ // jeśli guzik menu_1 wcisniety zwiekszaj flage
++flaga_1;

if(flaga_1==2){
key_loc=1;

if(key_loc==1){
++menu_1;
lcd_cls();
}
flaga_1=0;
}
}
key_loc=0;

if(menu_1){
Timer1=1;
poziom(menu_1); //wywołanie funkcji poziom i przekazanie do niej jako parametr menu_1
}
else
Timer1=2;
}
if(!key_press) flaga_1=0;

}

void poziom(uint8_t mm){

// w zależności od wartości menu_1 ustawiaj tem. histerezy załączania lub tem. histerezy wyłaczania
if(mm==1){
if(minus)-- ram_cfg.thw;
}
if(mm==1){
if(pplus)++ ram_cfg.thw;
}

if(mm==2){
if(minus)-- ram_cfg.thz;
}
if(mm==2){
if(pplus)++ ram_cfg.thz;
}

switch (mm)
{
case 1:

lcd_locate(0,0);
lcd_str("Thw: ");
lcd_int(ram_cfg.thw);

break;
case 2:

lcd_locate(0,0);
lcd_str("Thz: ");
lcd_int(ram_cfg.thz);

break;
case 3:
copy_ram_to_eem(); // kopiuj wartoś ram do eeprom i wyjdz z menu
menu_1=0;
lcd_locate(0,0);
lcd_str(" ");
break;

default:
break;
}
}

/////////////////////////////////////////////////////////////

void wys_tem(void)
{ //wyswietlanie oraz pomiar tem. z DS
if(!Timer2)flaga_2=1;
if (flaga_2)
{
Pomiar_DS18B20(6);
Pomiar_DS18B20(7);
Timer2=4;
flaga_2=0;
}
else if(Timer2==2)Tb=Odczyt_DS18B20(6);

else if(Timer2==1)Ts=Odczyt_DS18B20(7);

if(!menu_1){
lcd_locate(0,0);
sprintf(str,"Tb:%2d\xdf""C",Tb);
lcd_str(str);
lcd_locate(0,9);
sprintf(str,"Ts:%2d\xdf""C",Ts);
lcd_str(str);
}
}[/syntax]

tutaj wrzucam pliki wydzielone common.h
[syntax=c]/*
* common.c
#include
#include
#include
#include
#include "DS18B20/ds18b20_v1.h"

#include "common.h"

#include "LCD/lcd44780.h"

volatile uint8_t Timer1, Timer2, Timer3; // Timer4;/* timery programowe 100Hz */

void soft_timer_init(void) {
TCCR1B |= (1<TCCR1B |= (1<OCR1A = 15624; // 65535-34285 = 31250*0,000032 = 1s
TIMSK1 |= (1<}

void copy_eem_to_ram( void ) {
eeprom_read_block( &ram_cfg, &eem_cfg, sizeof(ram_cfg) );
}

void copy_ram_to_eem( void ) {
eeprom_write_block( &ram_cfg, &eem_cfg, sizeof(ram_cfg) );
}

void copy_pgm_to_ram( void ) {
memcpy_P( &ram_cfg, &pgm_cfg, sizeof(ram_cfg) );
}

void load_defaults( void ) {
copy_pgm_to_ram();
copy_ram_to_eem();
}

void check_and_load_defaults( void ) {
uint8_t i, len = sizeof( ram_cfg );
uint8_t * ram_wsk = (uint8_t*)&ram_cfg;

copy_eem_to_ram();
for(i=0; iif( 0xff == *ram_wsk++ ) continue;
break;
}

if( i == len ) {
load_defaults();
}

}

void Pomiar_DS18B20(char numer_ukladu)
{
char sprawdz;
sprawdz=RESET_PULSE(numer_ukladu);
if(sprawdz==1)
{
send_byte(DS18B20_ROM_SKIP, numer_ukladu);
send_byte(DS18B20_CMD_CONVERT, numer_ukladu);
}
}

uint16_t Odczyt_DS18B20(char numer_ukladu)
{
char sprawdz;
uint8_t lsb=0,msb=0;
uint16_t temp=0;
sprawdz=RESET_PULSE(numer_ukladu);
if(sprawdz==1)
{
send_byte(DS18B20_ROM_SKIP, numer_ukladu);
send_byte(DS18B20_CMD_READ_SCRATCHPAD, numer_ukladu);
lsb=read_byte(numer_ukladu);
msb=read_byte(numer_ukladu);
sprawdz=RESET_PULSE(numer_ukladu);// koniec transmisji
temp =(lsb+(msb << 8))/16;
}
return temp;
}

ISR( TIMER1_COMPA_vect) {
uint8_t n;

n = Timer1;/* 100Hz Timer1 */
if (n) Timer1 = --n;
n = Timer2;/* 100Hz Timer2 */
if (n) Timer2 = --n;
n = Timer3;/* 100Hz Timer3 */
if (n) Timer3 = --n;
//n = Timer4;/* 100Hz Timer4 */
//if (n) Timer4 = --n;

}[/syntax]
oraz common.c
[syntax=c]* common.h
*
#include
#include
#include "DS18B20/ds18b20_v1.h"

#ifndef COMMON_H_
#define COMMON_H_

typedef struct {
uint8_t thw;
uint8_t thz;

} TCFG;

// ustawienia domylne (fabryczne)
#define Thw 6 //
#define Thz 12//

extern const TCFG pgm_cfg __attribute__((__progmem__));// dane w pamięci FLASH
extern TCFG eem_cfg EEMEM;// dane w pamięci EEPROM
extern TCFG ram_cfg;// dane w pamięci RAM

extern volatile uint8_t Timer1, Timer2, Timer3;// Timer4;/* timery programowe 100Hz */

extern char bufor[100];

void check_and_load_defaults( void );
void copy_eem_to_ram( void );
void copy_ram_to_eem( void );
void copy_pgm_to_ram( void );
void load_defaults( void );

void soft_timer_init( void );
void pokaz_alarmy( void );

void Pomiar_DS18B20(char numer_ukladu);
uint16_t Odczyt_DS18B20(char numer_ukladu);

#endif /* COMMON_H_ */[/syntax]

Statystyki: Napisane przez Kulfi27 — 9 cze 2016, o 22:34