ATNEL tech-forum 2021-07-03T13:13:32+01:00 https://forum.atnel.pl/feed.php?f=4&t=23835&mode 2021-07-03T13:13:32+01:00 2021-07-03T13:13:32+01:00 https://forum.atnel.pl/viewtopic.php?t=23835&p=232925#p232925 <![CDATA[Re: Zapis PWM w EEPROM]]>

Andrews
Wszystko co piszesz ma logiczny sens. Niestety ja nie jestem programistą z wykształcenia, bazuję głównie na książkach i poradnikach Mirka, i jeszcze dużo muszę się nauczyć.
W międzyczasie przeniosłem projekt na ATmege32, poniżej zamieszczam kod, który działa właśnie na ATmedze32, jest krótki i pozwala zrozumieć cały ten mechanizm. Myślę, że wielu osobom pomoże. Jeszcze raz dziękuję i pozdrawiam.
[syntax=c]#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
#include <avr/pgmspace.h>
#include <avr/eeprom.h>

#define LED1(1<<PC7)// dioda podłączona do pinu PC7

#define LED1_ON PORTC |= LED1;
#define LED1_OFF PORTC &= ~LED1;
#define LED1_TOG PORTC ^= LED1;

#define K_PIN PINB
#define KEY1 (1<<PB0) //definicja klawisza 1
#define KEY2 (1<<PB1)//definicja klawisza 2
#define KEY3 (1<<PB2)//definicja klawisza 3

uint16_t pwm1;
uint16_t pwm2;

typedef struct {
uint16_t pwm1;
uint16_t pwm2;
} TCFG;

TCFG ram_cfg;// dane w pamieci ram
TCFG eem_cfg EEMEM;// dane w pamieci eeprom
const TCFG pgm_cfg PROGMEM = {// dane w pamieci flash
256, 512
};

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

uint8_t przycisk(uint8_t klawisz);


int main(void) {

_delay_ms(20);

// ustawienie klawiszy
DDRB &= ~(KEY1 | KEY2 | KEY3); //ustawia wejscia
PORTB |=(KEY1 | KEY2 | KEY3); //podciaganie do VCC

DDRC |=LED1;// PC7 - jako wyjście
LED1_OFF;// wylaczmy diode

DDRD |= (1 << PD5);// PWM output on PA6 (OC1A)
DDRD |= (1 << PD4);// PWM output on PA5 (OC1B)

// Timer Counter1 10-bit fast PWM mode
TCCR1A =(1<<COM1A1) | (1<<COM1B1) | (1<<WGM11) | (1<<WGM10);// fast PWM enable
TCCR1B =(1<<WGM12);// Waveform Generation Mode
TCCR1B =(1<<CS10);// prescaler clk / 1

copy_eem_to_ram();// kopiowanie eeprom do ram
pwm1 = ram_cfg.pwm1;// przypisujemy do zmiennej pwm1 odczytana wartosc z EEPROM-a

while(1) {


OCR1A = pwm1;

if (!(K_PIN & KEY1) && pwm1<1023) { //pwm up
pwm1++;

_delay_ms(20);
}
if(!(K_PIN & KEY2) && pwm1>0) { //pwm down
pwm1--;

_delay_ms(20);
}

if (!(K_PIN & KEY3)) {
ram_cfg.pwm1 = pwm1;// przypisujemy wartosc zmiennej pwm1 do struktury w RAM
copy_ram_to_eem();// kopiowanie ram do eeprom
LED1_ON;
_delay_ms(250);
LED1_OFF;
}

}
}
uint8_t przycisk(uint8_t klawisz) {
if (!(K_PIN & klawisz)) {
_delay_ms(25);
if (!(K_PIN & klawisz)) return 1;
}
return 0;
}

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

void copy_eem_to_ram (void) {
eeprom_read_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();
}[/syntax]

Statystyki: Napisane przez marian253 — 3 lip 2021, o 13:13

]]> 2021-07-03T11:16:49+01:00 2021-07-03T11:16:49+01:00 https://forum.atnel.pl/viewtopic.php?t=23835&p=232924#p232924 <![CDATA[Re: Zapis PWM w EEPROM]]> marian253 napisał(a):

Może jeszcze brakuje czegoś przed pętlą while(1) żeby dane wczytały się ramu.

Znasz przyczynę, tyko nie potrafisz znaleźć rozwiązania?

Do rejestru OCR1A wpisujesz wartość zmiennej pwm1 (linijka 66).

Zmienna pwm1 jest zadeklarowana globalnie i nie jest zainicjowana żadną wartością, więc podczas startu programu zostanie wyzerowana. Jeśli więc wpiszesz jej wartość do OCR1A (linijka 66), to PWM wystartuje od zera.

Zapisaną w EEPROM wartość zmiennej eem_cfg.pwm1 wczytujesz przed pętlą while do zmiennej w RAM ram_cfg.pwm1 (linijka 60), a do rejestru OCR1A wpisujesz (wyzerowaną) zmienną pwm1. Nie masz za to nigdzie przypisania do zmiennej pwm1 odczytanej z EEPROM wartości ram_cfg.pwm1 i dlatego Ci to nie działa. Powinieneś po linijce 60 (copy_eem_to_ram();) dodać linijkę
pwm1 = ram_cfg.pwm1;

marian253 napisał(a):

Zrobiłem tak jak Mirek sugerował, czyli pod klawiszem dorzuciłem: ram_cfg.pwm1 = pwm1;

Skoro zrobiłeś tak, jak Mirek zalecił to jeszcze usuń tę linijkę 64 (eem_cfg.pwm1 = pwm1;)

Statystyki: Napisane przez andrews — 3 lip 2021, o 11:16

]]> 2021-07-02T22:31:49+01:00 2021-07-02T22:31:49+01:00 https://forum.atnel.pl/viewtopic.php?t=23835&p=232923#p232923 <![CDATA[Re: Zapis PWM w EEPROM]]>
andrews napisał(a):

Gdybyś więc w linijce 64 zamiast eem_cfg.pwm1 = pwm1; napisał ram_cfg.pwm1 = pwm1;, to zapis do EEPROM miałby szansę zadziałać prawidłowo, czyli zgodnie z Twoimi oczekiwaniami

Zmieniłem zapis w linii 64 ale nic to nie dało.

Zrobiłem tak jak Mirek sugerował, czyli pod klawiszem dorzuciłem: ram_cfg.pwm1 = pwm1; ale dalej nie działa, pwm cały czas startuje od zera.
[syntax=c]if (!(K_PIN & KEY3)) {
ram_cfg.pwm1 = pwm1;
copy_ram_to_eem();// kopiowanie ram do eeprom
LED1_ON;
_delay_ms(250);
LED1_OFF;
}[/syntax]
Może jeszcze brakuje czegoś przed pętlą while(1) żeby dane wczytały się ramu.

Statystyki: Napisane przez marian253 — 2 lip 2021, o 22:31

]]> 2021-07-02T14:32:48+01:00 2021-07-02T14:32:48+01:00 https://forum.atnel.pl/viewtopic.php?t=23835&p=232922#p232922 <![CDATA[Re: Zapis PWM w EEPROM]]> marian253 napisał(a):

    while(1) {
 
        eem_cfg.pwm1 = pwm1;


Dobrze, że nie udał ci się ten zapis - bo próbowałeś go robić (zastanowiłeś się ile razy? .... no cały pierdyliard razy na sekundę - więc szybko zajechałbyś na śmierć biednego eeproma ! :(

Panie pisałeś że pod klawiszem niby zapisujesz ? a gdzie to widać w kodzie ? ;)

No właśnie tu musisz :
Quote:

if (!(K_PIN & KEY3)) {
ram_cfg.pwm1 = pwm1; - toż zapis parametru do struktury w RAM .. a później zapis tej struktury do EEPROM ;)
copy_ram_to_eem(); // kopiowanie ram do eeprom

Statystyki: Napisane przez mirekk36 — 2 lip 2021, o 14:32

]]> 2021-07-02T12:51:52+01:00 2021-07-02T12:51:52+01:00 https://forum.atnel.pl/viewtopic.php?t=23835&p=232921#p232921 <![CDATA[Re: Zapis PWM w EEPROM]]> marian253 napisał(a):

[syntax=c]eem_cfg.pwm1 = pwm1;[/syntax]

W linijce 64 napisałeś jak powyżej.
Po pierwsze nie możesz w ten sposób przypisać wartości do zmiennej w EEPROM, powinieneś użyć do tego funkcji (np. eeprom_write_word ()).
Po drugie, podejrzewam, że chciałeś w tym miejscu skopiować wartość zmiennej pwm1 do zmiennej ram_cfg.pwm1, bo tak faktycznie nigdzie indziej tego nie robisz, więc jak zapisujesz ram_cfg do eem_cfg, to tam nie ma aktualnej wartości. Gdybyś więc w linijce 64 zamiast eem_cfg.pwm1 = pwm1; napisał ram_cfg.pwm1 = pwm1;, to zapis do EEPROM miałby szansę zadziałać prawidłowo, czyli zgodnie z Twoimi oczekiwaniami.
Nie sprawdzałem dokładnie, czy nie masz innych błędów, na razie zacznij od tego, co napisałem.

Statystyki: Napisane przez andrews — 2 lip 2021, o 12:51

]]> 2021-07-02T11:39:28+01:00 2021-07-02T11:39:28+01:00 https://forum.atnel.pl/viewtopic.php?t=23835&p=232920#p232920 <![CDATA[Zapis PWM w EEPROM]]> [syntax=c]#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
#include <avr/pgmspace.h>
#include <avr/eeprom.h>

#define LED1(1<<PA0)

#define LED1_ON PORTA |= LED1;
#define LED1_OFF PORTA &= ~LED1;
#define LED1_TOG PORTA ^= LED1;

#define K_PIN PINB
#define KEY1 (1<<PB0) //definicja klawisza 1
#define KEY2 (1<<PB1)//definicja klawisza 2
#define KEY3 (1<<PB2)//definicja klawisza 3

uint16_t pwm1;
uint16_t pwm2;

typedef struct {
uint16_t pwm1;
uint16_t pwm2;
} TCFG;

TCFG ram_cfg;// dane w pamieci ram
TCFG eem_cfg EEMEM;// dane w pamieci eeprom
const TCFG pgm_cfg PROGMEM = {// dane w pamieci flash
256, 512
};

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

uint8_t przycisk(uint8_t klawisz);


int main(void) {

_delay_ms(20);

// ustawienie klawiszy
DDRB &= ~(KEY1 | KEY2 | KEY3); //ustawia wejscia
PORTB |=(KEY1 | KEY2 | KEY3); //podciaganie do VCC

DDRA |=LED1;// PB1 - jako wyjście
LED1_OFF;// PB1 OFF

DDRA |= (1 << PA6);// PWM output on PA6 (OC1A)
DDRA |= (1 << PA5);// PWM output on PA5 (OC1B)

// Timer Counter1 10-bit fast PWM mode
TCCR1A =(1<<COM1A1) | (1<<COM1B1) | (1<<WGM11) | (1<<WGM10);// fast PWM enable
TCCR1B =(1<<WGM12);// Waveform Generation Mode
TCCR1B =(1<<CS10);// prescaler clk / 1

copy_eem_to_ram();// kopiowanie eeprom do ram

while(1) {

eem_cfg.pwm1 = pwm1;

OCR1A = pwm1;

if (!(K_PIN & KEY1) && pwm1<1023) { //pwm up
pwm1++;

_delay_ms(50);
}
if(!(K_PIN & KEY2) && pwm1>0) { //pwm down
pwm1--;

_delay_ms(50);
}

if (!(K_PIN & KEY3)) {
copy_ram_to_eem();// kopiowanie ram do eeprom
LED1_ON;
_delay_ms(250);
LED1_OFF;
}

}
}
uint8_t przycisk(uint8_t klawisz) {
if (!(K_PIN & klawisz)) {
_delay_ms(25);
if (!(K_PIN & klawisz)) return 1;
}
return 0;
}

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

void copy_eem_to_ram (void) {
eeprom_read_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();
}[/syntax]

Statystyki: Napisane przez marian253 — 2 lip 2021, o 11:39

]]>