Форум разработчиков электроники ELECTRONIX.ru > проект в иар студио разбиваю на файлы дает ошибку

Помощь - Поиск - Пользователи - Календарь

Полная версия этой страницы: проект в иар студио разбиваю на файлы дает ошибку

Форум разработчиков электроники ELECTRONIX.ru > Сайт и форум > В помощь начинающему > Программирование

loggonlink

Jul 30 2010, 07:15

что то я туплю не могу вспомнить как правильно разбивать на файлы в проектах
обрисую ситуацию
есть программа робочая хочу разбить маин .с на несколько файлов
создал _LCD.H и LCD.C
файлы в проект добавлены
где ошибаюсь ))
просьба не судить строго давно делал такие вещи )

main.c

"

Код

#include <msp430f5515.h>
#include "_LCD.H"
void main(void)
{
  WDTCTL = WDTPW+WDTHOLD;                   // Stop watchdog timer
  LCD_main();
__bis_SR_register(LPM0_bits + GIE);       // CPU off, enable interrupts
}

#pragma vector=USCI_B1_VECTOR
__interrupt void USCI_B1_ISR(void)//LCD_Interupt
{
LCD_interupt();
}

"

_LCD.H
"

Код

#ifndef _LCD_H
#define _LCD_H
unsigned int k=0,q=0,s=0,a=0;
unsigned int LCD_Data = 0;
void datalcd(int i);
void test(int i);
void head(void);
void spi_init(void);
void LCD_port_config(void);
void LCD_main(void);
void LCD_interupt(void);

#endif

"

начало _LCD.C
"

Код

#include "_LCD.H"
#include <msp430f5515.h>

......
......
"
пишет ошибку при компиляции
Error[e27]: Entry "LCD_Data" in module _LCD ( C:\Documents and Settings\Admin\Рабочий стол\весы\Debug\Obj\_LCD.r43 ) redefined in module main

если переношу из _LCD.H
обьявление переменных в _LCD.C
то пишет
Error[e16]: Segment INTVEC (size: 0x5c align: 0x1) is too long for segment definition. At least 0x3c more bytes needed. The problem occurred while processing the segment placement command "-Z(CODE)INTVEC=FFE0-FFFF", where at the moment of placement the available memory ranges were "CODE:ffe0-ffff"
Reserved ranges relevant to this placement: ffe0-ffff INTVEC

MrYuran

Jul 30 2010, 07:27

Цитата(loggonlink @ Jul 30 2010, 11:15)

если переношу из _LCD.H
обьявление переменных в _LCD.C
то пишет
Error[e16]: Segment INTVEC (size: 0x5c align: 0x1) is too long for segment definition. At least 0x3c more bytes needed. The problem occurred while processing the segment placement command "-Z(CODE)INTVEC=FFE0-FFFF", where at the moment of placement the available memory ranges were "CODE:ffe0-ffff"
Reserved ranges relevant to this placement: ffe0-ffff INTVEC

Объявление в *.c правильное, а ругается линкер.
Что-то не то с секциями.
Проверьте, правильный ли выбран кристалл в опциях проекта.
По умолчанию может какой-нибудь Generic стоять.

PS: в следующий раз (да и в этот не поздно) оформляйте исходники в тегах

Код

...

, иначе читать тяжко.
Это кнопочка такая есть на панели при редактировании поста.

loggonlink

Jul 30 2010, 07:41

Цитата(MrYuran @ Jul 30 2010, 11:27)

извиняюсь но не понял ничего...
но не думаю что дело в опциях проекта, работал проект до разбивки.
или я совсем не так понял вас.

Цитата(MrYuran @ Jul 30 2010, 11:27)

PS: в следующий раз (да и в этот не поздно) оформляйте исходники в тегах

Код

...

, иначе читать тяжко.
Это кнопочка такая есть на панели при редактировании поста.

в след раз так и буду спс за конструктивную критику

проблему решил
прописал
через статик

Код

#ifndef _LCD_H
#define _LCD_H
static unsigned int k=0,q=0,s=0,a=0;
static unsigned int LCD_Data = 0;
void datalcd(int i);
void test(int i);
void head(void);
void spi_init(void);
void LCD_port_config(void);
void LCD_main(void);
void LCD_interupt(void);
#endif

MrYuran

Jul 30 2010, 07:48

Цитата(loggonlink @ Jul 30 2010, 11:41)

извиняюсь но не понял ничего...

Объявление переменных правильно делать в файлах .c, а в .h - определять внешние связи (то есть предварительные объявления функций и extern-объявления переменных, видимых снаружи программного модуля.)

Компилятор съел ваши исходники нормально, а ругается линкер, который не может скомпилированные куски программы разместить в памяти.
В частности, что-то случилось с секцией векторов прерываний.

Цитата(loggonlink @ Jul 30 2010, 11:41)

проблему решил
прописал
через статик

Это нужно делать для всех переменных и функций, которые используются только внутри модуля и не выглядывают наружу.
Это способствует лучшей оптимизации, да и нагляднее.
Кстати, с местом у вас там как, не жмёт?

dimka76

Jul 30 2010, 07:57

Нельзя в заголовочных файлах объявлять переменные. Иначе они будут плодиться во всех файлах реализации ( .с) причем с одинаковыми именами. И, как следствие, расходовать ресурсы ОЗУ.

loggonlink

Jul 30 2010, 08:01

Цитата(dimka76 @ Jul 30 2010, 11:57)

дык у меня стоит защита

Код

#ifndef _LCD_H
#define _LCD_H

или я не понимаю точно что оно делает ?

dimka76

Jul 30 2010, 08:05

Цитата(loggonlink @ Jul 30 2010, 12:01)

дык у меня стоит защита

Код

#ifndef _LCD_H
#define _LCD_H

или я не понимаю точно что оно делает ?

Защищает от повторного включения в один и тот же файл. А вы в разные включаете файлы.

SSerge

Jul 30 2010, 08:10

Как тут уже верно заметили проблема в объявлении переменных в .h-файле. Поскольку каждый .с-файл компилируется отдельно, то в каждом из них отводится место для этих переменных, а потом линкер обнаруживает что в разных модулях объявлены переменные с одинаковыми именами и ругается.
Объявление переменных как static приводит к тому, что теперь эти переменные будут видны только внутри .с-файла, линкер про них даже не узнает, и в каждом файле это будут свои, отдельные переменные. Однако, это явно не то чего Вы хотели добиться.

Вообще, в .h-файл стоит помещать определение констант (#define и enum{} ), прототипы функций, определение типов, и реализацию inline-функций.
Чтобы получить доступ к глобальной переменной, объявленной в другом файле, нужно описать её у себя как extern.

loggonlink

Jul 30 2010, 09:10

принял к сведению спс поправил.

вопрос по поводу дисплея тут могу выложить или надо отдельный топик создавать?

Для просмотра полной версии этой страницы, пожалуйста, пройдите по ссылке.