ATNEL tech-forum

Re: podział projektu na moduły

2016-03-10T22:45:59+01:00

gaza napisał(a):

nie musiałem robić extern
Wystarczyło ,ze w common.h

Czyli zainspirowało ciebie to co pisałem (podałeś cytat) a zrobiłeś CAŁKOWICIE ODWROTNIE czyli tragicznie bo zrobiłeś definicje zmiennych w pliku *.h

uint16_t tempPeriodTime; //liczba sekund pomiedzy odczytami temperatury
uint16_t totalConsumption;

i w cytacie jak widzisz piszę że DEFINICJE mają być w *.c

Proszę poczytaj czym różni się DEFINICJA i DEKLARACJA bo brniesz w ślepy zaułek a to że teraz ci się dobrze skompilowało wcale nie oznacza że jest dobrze zrobione i że czkawką ci nie zacznie się niedługo odbijać

czyli to czego się NIGDY nie robi w C i od czego będziesz miał w dalszych swoich losach same problemy z działaniem różnych swoich projektów i językiem C jako takim ponieważ ..... ponieważ nadal nie rozumiesz (z tego właśnie widać) czym różni się DEFINICJA od DEKLARACJI.

Statystyki: Napisane przez mirekk36 — 10 mar 2016, o 22:45

Re: podział projektu na moduły

2016-03-10T22:06:11+01:00

jak mawiał mój dobry nauczyciel od matematyki ze szkoły podstawoej " umiesz liczyć .... "

A tak poważnie , to zainspirował mnie ten post http://forum.atnel.pl/topic14683.html
i wypowiedz Mirka

Quote:

Bo to są DEKLARACJE zmiennych

zaś w pliku d_led.c masz bez extern i to są DEFINICJE tych zmiennych

nie zwróciłeś uwagi w Bluebooku na to jak naciskam mocno na to aby czytelnik zrozumiał czym różni się DEKLARACJA od DEFINICJI ? .... brak zrozumienia tego zagadnienia będzie powodował, że wciąż będzie się miało problemy ze zrozumieniem różnych części kodu w C

nie musiałem robić extern
Wystarczyło ,ze w common.h [syntax=c]enum heatState {PERMANENT, VARY, OFF};
enum heatState currentHeatState;
enum heatState previousHeatState;

uint16_t tempPeriodTime; //liczba sekund pomiedzy odczytami temperatury
uint16_t totalConsumption;[/syntax]

a w main.c
je zainicłowałem
[syntax=c]totalConsumption = 0;
tempPeriodTime = 60;
currentHeatState = VARY;
previousHeatState = OFF;[/syntax]

Kompilacja przeszła bez błędów
Jutro wgram hex-a i zobaczę jak działa

Dzięki za poświęcony czas i pomoc

Pozdr.

Statystyki: Napisane przez gaza — 10 mar 2016, o 22:06

Re: podział projektu na moduły

2016-03-10T21:37:53+01:00

Jak masz w błędach napisane 'undeclared' to znaczy, że program ich nie widzi, albo podałeś złą ścieżkę dostępu do pliku .h,
albo dopisz extern przed zmienną. Przed zmiennymi typu "enum" też.

Statystyki: Napisane przez Rafał555 — 10 mar 2016, o 21:37

Re: podział projektu na moduły

2016-03-10T20:39:12+01:00

plagiatu się nie obawiam, a jedynie może trochę wstyd to pokazywać

[syntax=c]/*
* main.c
*
* Created on: 2016-03-09
* Author: GL
*/
#include
#include
#include
#include
#include
#include
#include "MKUART/mkuart.h"
#include "1Wire/ds18x20.h"
#include "common.h"
#include "ATCOMMANDS/atcommands.h"

//deklaracje zmiennych
volatile uint16_t Timer1;
volatile uint8_t secFlag;
volatile uint8_t seconds;
volatile uint8_t int1_flag= 0;
volatile uint8_t heatFlag=0; //definicja flagi zdalnego włączenia grzania (RS)
volatile uint8_t int1_start=0; // zmienna przechowująca sekundę wystapienia int1
uint8_t cntSensor = 0; // liczba termometrów
uint8_t z; //zmienna petli
uint8_t thermostatKey = 0; //stan reguloatora pokojowego
uint8_t key1_lock, key2_lock; //zmienne pomocnicze obsługi drgań styków
uint8_t subzero, cel, cel_fract_bits;
char bufor[100];// bufor na potrzeby odebranych danych z UART

typedef struct {
char devPath[128];
char devID[16];
char tempData[6];
} TDS18B20;

#define RELAY_PIN (1<#define RELAY_TOG PORTD ^= RELAY_PIN // makrodefinicja zmiana stanu diody
#define RELAY_ON PORTD |= (1<#define RELAY_OFF PORTD &= ~(1<
#define LED_HALL_PIN (1<#define LED_HALL_OFF PORTD |= (1<#define LED_HALL_ON PORTD &= ~(1<
#define LED_MANUAL_PIN (1<#define LED_MANUAL_OFF PORTD |= (1<#define LED_MANUAL_ON PORTD &= ~(1<
#define LED_HEAT_PIN (1<#define LED_HEAT_OFF PORTD |= (1<#define LED_HEAT_ON PORTD &= ~(1<
//przycisk zrzucenia totalConsumption
#define KEY_FLUSH_PIN (1<#define KEY_FLUSH_DOWN !(PINC & KEY_FLUSH_PIN)
//sterownik pokojowy
#define KEY_HEAT_CONTROLLER_PIN (1<#define KEY_HEAT_CONTROLLER_DOWN !(PINC & KEY_HEAT_CONTROLLER_PIN)

//#define DS2408 "2912345678ABC" //8 switch
//#defineDS2423 "1D12345678ABC" //counter

void sendConsumption(void){
uart_puts("C|");// wyślij tekst
uart_puts(DS2423);
uart_puts("|");
uart_putint(1, 10);
uart_putc('\n');// wyślij znak LF (nowa linia)
}

void sendTemp(uint8_t x){
uint8_t status;
uint8_t i;
status = DS18X20_read_meas(gSensorIDs[x], &subzero, &cel, &cel_fract_bits) ;
if( DS18X20_OK == status){
uart_puts("T|");// wyślij tekst
for (i = 0; i < 8; i++)
uart_putint(gSensorIDs[x][i],16);
uart_puts("#");
uart_putint(cel,10);
uart_puts(".");
uart_putint(cel_fract_bits, 10);
uart_puts("C");
uart_putc('\n');// nowa linia
}else {
uart_puts("T|");
for (i = 0; i < 8; i++)
uart_putint(gSensorIDs[x][i],16);
uart_puts("#error:");// wyślij tekst
uart_putint(status, 10);
uart_putc('\n');// nowa linia
}

}

void setup(void)
{
//konfiguracja pinów
DDRD |= RELAY_PIN | LED_HALL_PIN | LED_MANUAL_PIN | LED_HEAT_PIN;
PORTD |= LED_HALL_PIN | LED_MANUAL_PIN | LED_HEAT_PIN;

//przycisk
DDRC &= ~KEY_FLUSH_PIN | ~KEY_HEAT_CONTROLLER_PIN;
PORTC |= KEY_FLUSH_PIN | KEY_HEAT_CONTROLLER_PIN;
// Przerwanie INT1
DDRD &= ~(1<PORTD |= (1<EICRA |= (1<EIMSK |= (1<

//konfiguracja TIMER2 dla F_CPU 1159200
TCCR2A |= (1<TCCR2B |= (1<OCR2A = 78;
TIMSK2 = (1<cntSensor = search_sensors();
}

//************ główna funkcja main() programu *****************************
int main(void) {

setup();
// inicjalizacja UART
USART_Init(__UBRR);
register_uart_str_rx_event_callback( parse_uart_data );

sei();// globalne odblokowanie przerwać

// przedstawienie się układu przez UART po starcie
uart_puts("I|CoManager ON\r\n");
// przedstawienie liczby znalezionych sensorów [ S ]
uart_puts("S|");
uart_putint(cntSensor, 10);
uart_putc('\n');// nowa linia*/

LED_MANUAL_OFF;

// petla główna programu
_delay_ms(1000);

while(1) {
UART_RX_STR_EVENT(bufor);
// ******* KLAWISZ FLUSH
if( !key1_lock && !KEY_FLUSH_DOWN ) {
key1_lock=1;
sendTotalConsumption(totalConsumption);
totalConsumption = 0;
} else if( key1_lock && KEY_FLUSH_DOWN ) key1_lock++;

// ******* Regulator pokojowy
if( !key2_lock && !KEY_HEAT_CONTROLLER_DOWN ) {
// regulator pokojowy "wcisniety"
key2_lock=1;
uart_puts("|HC|");//29
uart_puts(DS2408);
uart_puts("|OFF");
uart_putc('\n');
thermostatKey = 0;
LED_MANUAL_OFF; //zielona
}
else if( key2_lock && KEY_HEAT_CONTROLLER_DOWN )
{
if( !++key2_lock ) {
// reakcja na PUSH_UP (zwolnienie przycisku)
uart_puts("|HC|");
uart_puts(DS2408);
uart_puts("|ON");
uart_putc('\n');
thermostatKey = 1;
LED_MANUAL_ON; //zielona
}
}

switch (currentHeatState) {
case PERMANENT :
if(currentHeatState != previousHeatState) {
RELAY_ON;
uart_puts("|CHS|"); //Central Heating Stove - piec centralnego ogrzewania
uart_puts(DS2408);
uart_puts("|ON|MANUAL");
uart_putc('\n');
previousHeatState = currentHeatState;
}
break;
case VARY :
if (thermostatKey) {
if(currentHeatState != previousHeatState) {
RELAY_ON;
uart_puts("|CHS|"); //Central Heating Stove - piec centralnego ogrzewania
uart_puts(DS2408);
uart_puts("|ON|AUTO");
uart_putc('\n');
previousHeatState = currentHeatState;
}
}else{
currentHeatState = OFF;
}
break;
case OFF :
if (!thermostatKey) {
if(currentHeatState != previousHeatState) {
RELAY_OFF;
uart_puts("|CHS|"); //Central Heating Stove - piec centralnego ogrzewania
uart_puts(DS2408);
uart_puts("|CHS|OFF");
uart_putc('\n');
previousHeatState = currentHeatState;
}
}else{
currentHeatState = VARY;
}
break;

}

//co minutę odczytujemy termometry i wysyłamy ich wynik
if (secFlag ){
if (seconds % tempPeriodTime == 0)
DS18X20_start_meas(DS18X20_POWER_EXTERN, NULL);
if (seconds % tempPeriodTime == 1)
for (z = 0; z < cntSensor; z++){ sendTemp(z); };
secFlag = 0;

/*
* obsluga dekrementacji zmiennej z liczba sekund po ktorej mozna
* powtornie obslugiwac przerwanie.
* Gdy licznik sekund zero - wylaczamy diode ( pomaranczowa )
*/
if ( int1_start > 0 )
int1_start--;
else if (int1_start == 0)
LED_HALL_OFF;

}

/*
* Gdy jest flag przerwania i dodatkowo licznik sekund jest wyzerowany
* to wysyłamy info o zuzyciu jednostkowy, zwiekszamy licznik calkowitego
* zuzycia i zerujemy flage przerwania.
* Na koniec ustawiamy licznik sekund na 10 co spowoduje mozliwosc
* obslugi przerwania dopiero po 10 sekundach
*/
if (int1_flag && int1_start == 0 ) {
sendConsumption();
int1_flag = 0;
totalConsumption++;
int1_start=10;
}
}

}
//************************* koniec main() **********************

ISR(TIMER2_COMPA_vect)
{
static uint8_t cnt=0; /* statyczna zmienna cnt do odliczania setnych ms */
uint16_t x;
if(++cnt>99) { /* gdy licznik ms > 99 (mineła 1 sekunda) */
cnt=0; /* wyzeru licznik setnych ms */
secFlag=1; /* ustaw flagę tyknięcia sekundy */
seconds++; /* zwiększ licznik sekund */
if(seconds>59)
seconds=0; /* jeżeli ilość sekund > 59 - wyzeruj */

}

x =Timer1;
if (x) Timer1 = --x;

}

// procedura obsługi przerwania INT 0
ISR( INT1_vect ) {
int1_flag = 1; // flaga obslugi przerwania
LED_HALL_ON; // zapalenie diody sygnalizacyjnej - pomaranczowa
//_delay_ms(40); //mało optymalne, ale bezpieczne
}[/syntax]

do tego common.h
[syntax=c]/*
* common.h
*
* Created on: 9 mar 2016
* Author: GL
*/

#ifndef COMMON_H_
#define COMMON_H_

#define DS2408 "2912345678ABC" //8 switch
#defineDS2423 "1D12345678ABC" //counter

enum heatState {PERMANENT, VARY, OFF};
enum heatState currentHeatState = VARY;
enum heatState previousHeatState = OFF;

volatile uint16_t tempPeriodTime = 60; //liczba sekund pomiedzy odczytami temperatury
volatile uint16_t totalConsumption = 0;

void sendTotalConsumption(uint16_t consumption);

#endif /* COMMON_H_ */[/syntax]

i common.c
[syntax=c]/*
* common.c
*
* Created on: 9 mar 2016
* Author: GL
*/

#include
#include
#include
#include "common.h"
#include "MKUART/mkuart.h"

void sendTotalConsumption(uint16_t consumption){
uart_puts("TC|");// wyślij tekst
uart_puts(DS2423);
uart_puts("|");
uart_putint(consumption, 10);
uart_putc('\n');// wyślij znak LF (nowa linia)
}[/syntax]

no i na koniec dwie funkcje z atcommands
[syntax=c]/*
* komendy_at.c
*
* Created on: 13-03-2012
* Author: Mirosław Kardaś
*/

#include
#include
#include
#include
#include
#include
#include
#include "../MKUART/mkuart.h"
#include "atcommands.h"
#include "../common.h"

#define AT_CNT 6// iloć poleceń AT

//----------- tablica z poleceniami AT i wskaźnikami funkcji do ich obsługi --------------------
const TATCMD polecenia_at[AT_CNT] PROGMEM = {
// { at_cmd } , { wskaźnik do funkcji obsługi at },
{"AT", at_service},
{"ATI", ati_service},
{"AT+RST", at_rst_service},
{"AT+LED", at_led_service},
{"AT+TC", at_tc_service},
{"AT+STATE",at_state_service},
};

int8_t at_tc_service(uint8_t inout, char * params) {
if( !inout ){
//sendTotalConsumption(13);
;
}
return 0;
}

int8_t at_state_service(uint8_t inout, char * params) {
if( 1 == inout) {
//ustawienie statusu
if( params[0] < '0' || params[0] > '1' ) return -1;
if( '1' == params[0] )
currentHeatState = PERMANENT;
else
currentHeatState = VARY;;

//currentHeatState = 'PERMANENT';
} else if( !inout ){
//odczyt statusu
uart_puts("+STATE: ");
switch (currentHeatState) {
case PERMANENT : uart_puts("PERMANENT\r\n"); break;
case VARY : uart_puts("VARY\r\n"); break;
case OFF : uart_puts("OFF\r\n"); break;
}
} else if( 2 == inout ){
uart_puts("AT+STATE = (0-1)\r\n");
}

return 0;
}[/syntax]

Nie załączam całego kodu atcommands.c bo to nie open-source

Statystyki: Napisane przez gaza — 10 mar 2016, o 20:39

Re: podział projektu na moduły

2016-03-10T18:39:32+01:00

Witam
Jak się napisze do ogółu to nikt nie przeczyta, jak do konkretnej osoby ( może inni zobaczą ) to troszkę niesmacznie brzmi, ale " my tu staramy się pomóc w oparciu o jakieś podstawy programowe a nie raporty co jest źle - nie da się wróżyć"

gaza napisał(a):

czyli common musi byc załączony

Nie wiadomo co tam masz, ale może do tych zmiennych które mają też być w main.c dopisz extern.
Jeśli nie obawiasz się plagiatu ani tego że ktoś Ci to podprowadzi to zamieść cały kod; może ktoś z nudów doprowadzi to do "kompilowalności"

Pozdr.

Statystyki: Napisane przez kicajek — 10 mar 2016, o 18:39

Re: podział projektu na moduły

2016-03-10T18:02:21+01:00

rzeczywiście przeoczenie, bo ten kod był dzisiaj maglowany kilka razy.
w main były jeszcze raz te definicje, zakomentowałem i teraz mam inne błędy

Quote:

17:53:54 **** Incremental Build of configuration Release for project usart ****
make all
Building file: ../main.c
Invoking: AVR Compiler
avr-gcc -Wall -Os -fpack-struct -fshort-enums -ffunction-sections -fdata-sections -std=gnu99 -funsigned-char -funsigned-bitfields -mmcu=atmega88 -DF_CPU=11059200UL -MMD -MP -MF"main.d" -MT"main.d" -c -o "main.o" "../main.c"
Finished building: ../main.c

Building target: z_uart.elf
Invoking: AVR C Linker
avr-gcc -Wl,-Map,z_uart.map -mmcu=atmega88 -o "z_uart.elf" ./MKUART/mkuart.o ./ATCOMMANDS/atcommands.o ./1Wire/crc8.o ./1Wire/ds18x20.o ./1Wire/onewire.o ./common.o ./main.o
./main.o:(.bss.totalConsumption+0x0): multiple definition of `totalConsumption'
makefile:53: polecenia dla obiektu 'z_uart.elf' nie powiodły się
./ATCOMMANDS/atcommands.o:(.bss.totalConsumption+0x0): first defined here
./main.o:(.data.currentHeatState+0x0): multiple definition of `currentHeatState'
./ATCOMMANDS/atcommands.o:(.data.currentHeatState+0x0): first defined here
./main.o:(.data.previousHeatState+0x0): multiple definition of `previousHeatState'
./ATCOMMANDS/atcommands.o:(.data.previousHeatState+0x0): first defined here
./main.o:(.data.tempPeriodTime+0x0): multiple definition of `tempPeriodTime'
./ATCOMMANDS/atcommands.o:(.data.tempPeriodTime+0x0): first defined here
collect2: error: ld returned 1 exit status
make: *** [z_uart.elf] Błąd 1

17:53:55 Build Finished (took 508ms)

Jak usunę include z common.h z pliku main.c to mam tak

Quote:

17:57:13 **** Incremental Build of configuration Release for project usart ****
make all
Building file: ../main.c
Invoking: AVR Compiler
avr-gcc -Wall -Os -fpack-struct -fshort-enums -ffunction-sections -fdata-sections -std=gnu99 -funsigned-char -funsigned-bitfields -mmcu=atmega88 -DF_CPU=11059200UL -MMD -MP -MF"main.d" -MT"main.d" -c -o "main.o" "../main.c"
../main.c: In function ‘main’:
../main.c:166:25: error: ‘totalConsumption’ undeclared (first use in this function)
sendTotalConsumption(totalConsumption);
^
../main.c:166:25: note: each undeclared identifier is reported only once for each function it appears in
subdir.mk:21: polecenia dla obiektu 'main.o' nie powiodły się
../main.c:195:11: error: ‘currentHeatState’ undeclared (first use in this function)
switch (currentHeatState) {
^
../main.c:196:8: error: ‘PERMANENT’ undeclared (first use in this function)
case PERMANENT :
^
../main.c:197:27: error: ‘previousHeatState’ undeclared (first use in this function)
if(currentHeatState != previousHeatState) {
^
../main.c:206:8: error: ‘VARY’ undeclared (first use in this function)
case VARY :
^
../main.c:217:24: error: ‘OFF’ undeclared (first use in this function)
currentHeatState = OFF;
^
../main.c:239:18: error: ‘tempPeriodTime’ undeclared (first use in this function)
if (seconds % tempPeriodTime == 0)
^
make: *** [main.o] Błąd 1

17:57:13 Build Finished (took 205ms)

czyli common musi byc załączony w main.c ,ale wtedy mam info, że multiple definition

Statystyki: Napisane przez gaza — 10 mar 2016, o 18:02

Re: podział projektu na moduły

2016-03-10T17:48:02+01:00

gaza napisał(a):

bo przy kompilacji mam taką listę błędów

A czytasz ty chociaż po kolei te błędy co ci się pokazują ? Bo coś mi się wydaje, że nie za bardzo - bo nawet gdybyś chciał powiedzieć, że nie znasz angielskiego to wystarczyłoby użyć google translate .... a błędy masz podane jak na talerzu, proszę pierwszy:

Quote:

../main.c:18:19: error: redefinition of ‘tempPeriodTime’

co to znaczy ? .... no że w pliku main.c masz powtórnie zdefiniowaną zmienną o podanej tu nazwie. Żeby tego było mało w kolejnych linijkach kompilator cię informuje DOKŁADNIE,

Quote:

In file included from ../main.c:15:0:
../common.h:15:19: note: previous definition of ‘tempPeriodTime’ was here
volatile uint16_t tempPeriodTime = 60;

gdzie ją masz wcześniej zdefiniowaną czyli w pliku common.h ... no to powiedz sam - czego więcej oczekiwać ? Skoro widać, że zdublowałeś zmienne ?

Statystyki: Napisane przez mirekk36 — 10 mar 2016, o 17:48

Re: podział projektu na moduły

2016-03-10T17:37:17+01:00

poprawione

Statystyki: Napisane przez gaza — 10 mar 2016, o 17:37

Re: podział projektu na moduły

2016-03-10T17:22:43+01:00

gaza napisał(a):

PS. dlaczego po zmianie znaczników z code na syntax zginęły wartości includów ?

Bo źle wstawiłeś kod

Statystyki: Napisane przez Jaglarz — 10 mar 2016, o 17:22

Re: podział projektu na moduły

2016-03-10T17:00:51+01:00

gaza napisał(a):

jak zorganizować projekt aby było przejrzyście i zgodnie ze sztuką ?

Zacznij może od poprawnego wstawienia kodu na forum aby było przejrzyście i zgodnie ze sztuką

Statystyki: Napisane przez BIGjack — 10 mar 2016, o 17:00

podział projektu na moduły

2016-03-10T17:36:53+01:00

witam.
Zrobiłem sobie urządzenie do podglądu działań pieca CO. Piec retortowy opalany miałem ,ale to mniej istotne.
Urządzenie ma mieć możliwość współpracy z regulatorem pokojowym, liczyć ilość ruchów retory (szuflady ), mierzyć i wysyłać temperatury w różnych punktach instalacji.
To tytułem wstępu.

Program korzysta z bibliotek [syntax=c]#include
#include
#include
#include
#include
#include
#include "MKUART/mkuart.h"
#include "1Wire/ds18x20.h"
#include "common.h"
#include "ATCOMMANDS/atcommands.h"[/syntax]

i tu zaczyna się problem, bo
w main korzystam między innymi z kilku zmiennych i funkcji
[syntax=c]enum heatState {PERMANENT, VARY, OFF};
enum heatState currentHeatState = VARY;
enum heatState previousHeatState = OFF;

volatile uint16_t tempPeriodTime = 60; //liczba sekund pomiedzy odczytami temperatury
volatile uint16_t totalConsumption = 0;

void sendTotalConsumption(uint16_t consumption);[/syntax]

Program tych samych zmiennych musi korzystać w bibliotece ATCOMMANDS.
Zmienne na początku były zdefiniowane w main ( jak jeszcze nie korzystałem z ATCOMMANDS), ale potem okazało się że muszą być zdefiniowane gdzieś globalnie aby i w ATCOMMANDS były dostępne. Utowrzyłem zatem plik common.h i tam je wrzuciłem, po czym zainkludowałem je do main.c i atcommands.c [syntax=c]#include
#include
#include
#include
#include
#include
#include
#include "../MKUART/mkuart.h"
#include "atcommands.h"
#include "../common.h"[/syntax]

i tu pojawia się problem.
bo przy kompilacji mam taką listę błędów

Quote:

make all
Building file: ../ATCOMMANDS/atcommands.c
Invoking: AVR Compiler
avr-gcc -Wall -Os -fpack-struct -fshort-enums -ffunction-sections -fdata-sections -std=gnu99 -funsigned-char -funsigned-bitfields -mmcu=atmega88 -DF_CPU=11059200UL -MMD -MP -MF"ATCOMMANDS/atcommands.d" -MT"ATCOMMANDS/atcommands.d" -c -o "ATCOMMANDS/atcommands.o" "../ATCOMMANDS/atcommands.c"
Finished building: ../ATCOMMANDS/atcommands.c

Building file: ../1Wire/crc8.c
Invoking: AVR Compiler
avr-gcc -Wall -Os -fpack-struct -fshort-enums -ffunction-sections -fdata-sections -std=gnu99 -funsigned-char -funsigned-bitfields -mmcu=atmega88 -DF_CPU=11059200UL -MMD -MP -MF"1Wire/crc8.d" -MT"1Wire/crc8.d" -c -o "1Wire/crc8.o" "../1Wire/crc8.c"
Finished building: ../1Wire/crc8.c

Building file: ../1Wire/ds18x20.c
Invoking: AVR Compiler
avr-gcc -Wall -Os -fpack-struct -fshort-enums -ffunction-sections -fdata-sections -std=gnu99 -funsigned-char -funsigned-bitfields -mmcu=atmega88 -DF_CPU=11059200UL -MMD -MP -MF"1Wire/ds18x20.d" -MT"1Wire/ds18x20.d" -c -o "1Wire/ds18x20.o" "../1Wire/ds18x20.c"
Finished building: ../1Wire/ds18x20.c

Building file: ../1Wire/onewire.c
Invoking: AVR Compiler
avr-gcc -Wall -Os -fpack-struct -fshort-enums -ffunction-sections -fdata-sections -std=gnu99 -funsigned-char -funsigned-bitfields -mmcu=atmega88 -DF_CPU=11059200UL -MMD -MP -MF"1Wire/onewire.d" -MT"1Wire/onewire.d" -c -o "1Wire/onewire.o" "../1Wire/onewire.c"
Finished building: ../1Wire/onewire.c

Building file: ../common.c
Invoking: AVR Compiler
avr-gcc -Wall -Os -fpack-struct -fshort-enums -ffunction-sections -fdata-sections -std=gnu99 -funsigned-char -funsigned-bitfields -mmcu=atmega88 -DF_CPU=11059200UL -MMD -MP -MF"common.d" -MT"common.d" -c -o "common.o" "../common.c"
Finished building: ../common.c

../main.c:18:19: error: redefinition of ‘tempPeriodTime’
volatile uint16_t tempPeriodTime = 60; //liczba sekund pomiedzy odczytami temperatury
^
In file included from ../main.c:15:0:
../common.h:15:19: note: previous definition of ‘tempPeriodTime’ was here
volatile uint16_t tempPeriodTime = 60; //liczba sekund pomiedzy odczytami temperatury
Building file: ../main.c
Invoking: AVR Compiler
avr-gcc -Wall -Os -fpack-struct -fshort-enums -ffunction-sections -fdata-sections -std=gnu99 -funsigned-char -funsigned-bitfields -mmcu=atmega88 -DF_CPU=11059200UL -MMD -MP -MF"main.d" -MT"main.d" -c -o "main.o" "../main.c"
^
../main.c:19:19: error: redefinition of ‘totalConsumption’
volatile uint16_t totalConsumption = 0;
^
In file included from ../main.c:15:0:
../common.h:16:19: note: previous definition of ‘totalConsumption’ was here
subdir.mk:21: polecenia dla obiektu 'main.o' nie powiodły się
volatile uint16_t totalConsumption = 0;
^
make: *** [main.o] Błąd 1

jak zorganizować projekt aby było przejrzyście i zgodnie ze sztuką ?

Pozdr

Statystyki: Napisane przez gaza — 10 mar 2016, o 16:56