Tak więc wziąłem sobie do głowy Wasze rady. Wywaliłem tę lcd_refresh i wróciłem do poprzedniej postaci :
[syntax=c]void lcd_refresh(void)
{
static uint8_t locate_flag = 0; // informuje o konieczności przestawienia kursora lcd
for(uint8_t y=0; y<LCD_ROWS; y++)
{
lcd_locate( 0, y );
for(uint8_t x=0; x<LCD_COLS; x++)
{
if( lcd_buf[y][x] != lcd_buf_old[y][x] )
{
if( !locate_flag )
lcd_locate( x, y );
lcd_char( lcd_buf[y][x] );
lcd_buf_old[y][x] = lcd_buf[y][x];
locate_flag = 1;
}
else
locate_flag = 0;
}
}
}[/syntax]
Tak samo zrobiłem key_init, wyrzuciłem(zakomentowałem) niepotrzebne na razie funkcje, powołałem zmienną volatile uint8_t fRefresh i main.c wygląda następująco :
[syntax=c]#include <avr/io.h>
#include <avr/interrupt.h>
#include <avr/eeprom.h>
#include <avr/pgmspace.h>
#include <stdlib.h>
#include <util/delay.h>
#include <string.h>
#include <stdio.h>
#include "inout.h"
#include "LCD/lcd.h"
#include "LCD/menu.h"
#include "1Wire/onewire.h"
#include "1Wire/ds18x20.h"
#include "1Wire/crc8.h"
#include "IR_DECODE/ir_decode.h"
volatile uint8_t ms10_cnt; // zmienne potrzebne do timerów programowych
volatile uint8_t s1_cnt;
volatile uint8_t fRefresh;
ISR( TIMER0_OVF_vect ) {
if(++ms10_cnt>99) {
ms10_cnt=0;
if(++s1_cnt>59) s1_cnt=0;
}
fRefresh = 1;
}
int main (void) {
DDRD |= (1<<PA7); // ustawienie kierunku linii podświetlenia LCD jako wyjście
PORTD |= (1<<PA7); // stan wysoki LCD - podświetlenie
key_init();
fRefresh=0;
buf_lcd_init(); // inicjalizacja buforowania wyświetlacza LCD
sei();
menu_refresh(); // odświeżanie menu
while(1){
key_down_press();
key_up_press();
key_enter_press();
key_cancel_press();
if (fRefresh == 1){
fRefresh = 0;
lcd_refresh();
}
}
}[/syntax]
w menu.c timer (<- czy jest dobrze opisany?? )
[syntax=c]void buf_lcd_init(void) {
TCNT0 = 0x00; //wartość zliczona przez licznik
TCCR0 |= (1<<CS02)|(1<<CS00);// preskaler = 1024
TIMSK |= (1<<TOIE0); // przerwanie Overflow
lcd_init(); // inicjalizacja LCD
buf_cls();
}[/syntax]
dziobak7 napisał(a):
Inna ważna rzecz. Jeśli masz 3 poziom zagnieżdżenia w funkcji, to jest coś nie tak z Twoim rozumieniem kodu (pominę wyjątki bo to nie jest na tyle skomplikowana aplikacja).
3 poziom zagnieżdżenia funkcji - który fragment masz na myśli?
dziobak7 napisał(a):
Następne... jeśli masz gdzieś zmienną typu flaga, z zasady powinna być ustawiana w 1 miejscu (znanym Ci) i testowana oraz ew. gaszona w 2. Dodatkowo, jej zmiana (czyli zapis), prawie zawsze powinna być zabezpieczona operacją atomową (poczytaj o ATOMIC_BLOCK). A jak jest teraz?
Trochę poczytałem o tym, ale czy to jest tak potrzebne do mojego kodu i nie można pozostać przy timerze? Ewentualnie opierając się na powyższym kodzie z timerem i przerwaniami, jakbyś to zrobił, wykorzystując ATOMIC_BLOCK?
dziobak7 napisał(a):
Dodaj do kompilatora flagi:
-Wall -Wextra -Wshadow -pedantic
Masz to wydzielone w opcjach kompilacji w eclipse. Inne flagi są także bardzo przydatne ale powyższe wystarczą by pokazać Ci kilka(naście) potencjalnych błędów jakie popełniłeś. Przyjmij zasadę (na początek) że kod ma się kompilować bez ostrzeżeń.
Gdzie dokładnie mogę to znaleźć w eclipsie? Co do samej kompilacji kodu, nie wyrzuca mi żadnych błędów i ostrzeżeń.Statystyki: Napisane przez Gammadion — 3 lut 2019, o 13:23
]]>
czyli mechanizm sprawdzania flagi zajętości LCD i przyszyłeś tę wątrobę poza ustrojem, czyli gdzieś w kodzie głównym programu - no masakra 
i ty się dziwisz że coś ci tam nie działa ... a jeszcze tą jakąś funkcją lcd_refresh() chlastasz jak maczetą po całym ciele ... oczywiście ciele programu nawet w przerwaniach ... Panie panie - taki program ma prawo a nawet obowiązek działać dokładnie jak teraz ci działa - czyli nie działa ...
