Вопросы по С/С++ (СИ)

[NStorm]

uldemir, да если еще дальше по ссылке пройти, всё объяснено. Но вы ведь разобрались уже как размер стека увеличить. Раз в вашем камне осталась еще память для кучи, значит и стек можно еще увеличивать.
Вы размер карты на этапе компиляции будете ведь знать - так и задавайте там, чтобы в стеке лежало. Динамическое распределение памяти и malloc() выгодны на ПК, где памяти много и процесс может динамически запрашивать и освобождать память. На МК такую роскошь себе позволить нельзя. Надо понимать, что свопа никто не добавит, когда закончится физическая память. Поэтому буфер должен быть максимально такого размера, сколько может быть данных и хватает памяти. Ротацию данных внутри буфера можно и без динамической памяти устроить.

[uldemir]

Увы и нет. Размер карты выясняется на этапе выполнения (смотреть самый конец сообщения). Но, конечно, я лукавлю. Не нужен мне реально malloc(). Меня интересует метод, как разместить массив в любом месте памяти и с ним комфортно работать. Потому как функция создающая карту использует глобальную переменную для которой статически уже выделена память, затем, эта карта переносится во флеш память для хранения и только потом я из флеш памяти её достаю для отображения. Так что, реально, я могу просто использовать уже выделенный кусок памяти или, хочется научиться использовать данные, которые сохранил во флеш-памяти не вытаскивая оттуда. Ваше предложение создать массив указателей сейчас обдумывается.

Конечно, исходная проблема, как узнать, что размер стека маловат - до сих пор не выяснена. И в той, теме, куда была приведена ссылка, этот вопрос так и не был решен. Была распальцовка, было указание увеличить стек, потом опять распальцовка, а вот как проконтролировать - так и не было сказано. Возможно, это недостаток Кейла, но в нём я так и не нашел репорт, где написано "max stack usage". С другой стороны, если бы Кейл знал это, что ему мешало бы самому выделить под стек необходимую память.

[VladislavS]

uldemir писал(а):

Возможно, это недостаток Кейла, но в нём я так и не нашел репорт, где написано "max stack usage".

--info=stack

[Эйлер Леонард]

Всем добрый вечер. Пытаюсь откомпилировать код, найденный на просторах интернета. Сижу в AtmelStudio. Код для МКонтроллера AVR. МКонтроллер Attiny (малолапчатый)
main.cpp

Спойлер

Код:

#include <avr/io.h>
#include <avr/interrupt.h>
#include "shiftreg.h"
/* macro definition which returns the number of elements in an arraay */
//#define GetArrayElements(x)   sizeof(x)/sizeof(x[0])

/* 7 Segment Codes */
#define ZERO   0x3F
#define ONE   0x06
#define TWO   0x5B
#define THREE   0x4F
#define FOUR   0x66
#define FIVE    0x6D
#define SIX   0x7D
#define SEVEN   0x07
#define EIGHT   0x7F
#define NINE   0x6F

int main(void)
{
   //unsigned short counter = 0;
   //unsigned char temp;
   //unsigned char digits[] = {ZERO,ONE,TWO,THREE,FOUR,FIVE,SIX,SEVEN,EIGHT,NINE};
   //unsigned char display[4];
   SHIFTREG_register reg;
   reg = SHIFTREG_InitRegister(&DDRB, &PORTB, PB0, PB1, PB2);
   sei();
    while (1) {  }
}

shiftreg.h

Спойлер

Код:

/******************************************************************************
* File Name     : shiftreg.h
* Author        : D.Gotic
* Version       : V0.1
* Date          : 03/12/2010
* Description   : This file contains all the functions prototypes
*                 and structure definitions for the 74HC595 shit register driver
******************************************************************************/
/* Define to prevent recursive inclusion ----------*/
#ifndef SHIFTREG_H
#define SHIFTREG_H
/* Includes ----------*/
#include <inttypes.h>
/* a structure which defines the interface  from the uC to the SHIFT REGISTER */
typedef struct _SHIFTREG_register
{
volatile uint8_t *DDR_addr;   /* address of the uC Data Direction Register      */
volatile uint8_t *PORT_addr;   /* address of the uC PORT Register                */
         uint8_t DS_pin;      /* Pin number to which the DS pin is connected      */
         uint8_t ST_CP_pin;      /* Pin number to which the ST_CP pin is connected */
         uint8_t SH_CP_pin;      /* Pin number to which the SH_CP pin is connected */
} SHIFTREG_register;
/*   Function prototypes ----------*/
SHIFTREG_register SHIFTREG_InitRegister(
volatile uint8_t *ddr_addr,
volatile uint8_t *port_addr,
         uint8_t ds_pin,
         uint8_t st_cp_pin,
         uint8_t sh_cp_pin
);

void SHIFTREG_OutputValue(
SHIFTREG_register *reg,
unsigned char *bytes,
unsigned char nbytes
);

#endif   /* SHIFTREG_H */

/************************** END OF FILE **************************/


shiftreg.c

Спойлер

Код:

/****************************************************************************
* File Name    : shiftreg.c
* Author       : D.Gotic
* Version      : V0.1
* Date         : 03/12/2010
* Description  : This file contains all the functions definitions
*                      for the 74HC595 shit register driver
*****************************************************************************/
/* Includes ----------*/
#include "shiftreg.h"
#include "macros.h"

#define BIT_MASK 0x80

/* Private macros-----------------------------------------------------------*/
/* Private functions ----------*/
#define shift_in_pulse(port,reg_pin)     sbi(port,reg_pin); cbi(port,reg_pin)
#define parallel_out_pulse(port,reg_pin) sbi(port,reg_pin); cbi(port,reg_pin)
//
// Function Name : SHIFTREG_InitRegister
// Description   : Initializes the interface from the uC to the shift register
// Input         : DDR adress and PORT adress of the interface and pin numbers
//                 of the uC interface
// Return        : initialized register structure
//
SHIFTREG_register SHIFTREG_InitRegister(
volatile uint8_t *ddr_addr,
volatile uint8_t *port_addr,
         uint8_t ds_pin,
         uint8_t st_cp_pin,
         uint8_t sh_cp_pin)
{
   SHIFTREG_register tmp_reg;
   
   tmp_reg.DDR_addr       = ddr_addr;
   tmp_reg.PORT_addr       = port_addr;
   tmp_reg.DS_pin         = ds_pin;
   tmp_reg.ST_CP_pin      = st_cp_pin;
   tmp_reg.SH_CP_pin      = sh_cp_pin;
   
   sbi(*(tmp_reg.DDR_addr),tmp_reg.DS_pin);            
   sbi(*(tmp_reg.DDR_addr),tmp_reg.ST_CP_pin);      
   sbi(*(tmp_reg.DDR_addr),tmp_reg.SH_CP_pin);      

   return tmp_reg;
}
/****************************************************
* Function Name  : SHIFTREG_OutputValue
* Description    : Transmits nbytes bytes to nbytes shift registers and outputs
*               : the values
* Input          : shift register structure, bytes to transmit, number of bytes
*               : to transmit
* Return         : None
******************************************************/
void SHIFTREG_OutputValue(
SHIFTREG_register *reg,
unsigned char *bytes,
unsigned char nbytes )
{   
      unsigned char mask, i, j;

      for (i=0; i<nbytes; i++)
      {
         mask = BIT_MASK;

         for (j = 0; j< 8; j++)
         {
            /* mask bit
             * if bit 0 output 0
             * else output 1
             * shift bit mask one place right */
            if (bytes[i] & mask)
            {
               sbi(*(reg->PORT_addr),reg->DS_pin);
               shift_in_pulse(*(reg->PORT_addr),reg->SH_CP_pin);
            }
            else
            {
               cbi(*(reg->PORT_addr),reg->DS_pin);
               shift_in_pulse(*(reg->PORT_addr),reg->SH_CP_pin);
            }
            mask = (mask >> 1);
         }
      }
      parallel_out_pulse(*(reg->PORT_addr),reg->ST_CP_pin);
}
/***********************END OF FILE***********************/


macros.h

Спойлер

Код:

/*************************************************************************
* File Name   : shiftreg.h
* Author      : D.Gotic
* Version     : V0.1
* Date        : 03/12/2010
* Description : Helper macros for setting and clearing a bit in a register
**************************************************************************/
#ifndef MACROS_H
#define MACROS_H
#define cbi(addr,bit)   addr &= ~(1 << bit)
#define sbi(addr,bit)   addr |= (1 << bit)
#endif   /* MACROS_H */
/****************************END OF FILE**********************************/


В данном случае хочу разобраться в структурах, типах передаваемых данных, ссылках и указателях в аргументах и параметрах функции.
При компиляции в AtmelStudio ошибка:
Error undefined reference to
"SHIFTREG_InitRegister(unsigned char volatile*, unsigned char volatile*, unsigned char, unsigned char, unsigned char)"
Ошибка неопределенной ссылки на ...
Фрагмент кода с прототипом функции

Код:

/* ---------- Function prototypes ----------*/
SHIFTREG_register SHIFTREG_InitRegister(volatile uint8_t *ddr_addr, volatile uint8_t *port_addr,uint8_t ds_pin, uint8_t st_cp_pin, uint8_t sh_cp_pin);

В функцию необходимо передать в качестве параметров DDRB, PORTB, PB0, PB1, PB3 (порт и пины к которым подключается регистр 74HC595)

ErrorList.png

ErrorList

(17.29 KiB) Скачиваний: 197

Attinu_74HC595.png

(20.23 KiB) Скачиваний: 201

И что за unsigned char - ясно же сказано uint8_t
Может быть преобразование типов в компиляторе происходит не корректно?
Ссылка -> Подобная ошибка, но машинный перевод
P.S. Аксиома "Код, скачаный из интернета, никогда сразу не работает"

Последний раз редактировалось Эйлер Леонард Сб дек 28, 2019 22:45:29, всего редактировалось 2 раз(а).

[WiseLord]

unsigned char и uint8_t - одно и то же.

undefined reference - функция определена в файле shiftreg.c, а раз такая ошибка - то его, похоже, забыли добавить в список для компиляции.

[Эйлер Леонард]

Нет, все файлы подключены.
Файлы проекта ....

2019-12-29_004423.png

(20.08 KiB) Скачиваний: 238

Первоначально файл shiftreg был типом .C , а у меня проект как CPP. Изменил тип файла на CPP и вроде бы как скомпилировалось без ошибок.

[uldemir]

У С и С++ различаются способы подключения хидер-файлов. Я в этом не эксперт, просто сам нарвался, когда хотел править ардуиновский скетч...

Вот пример хидер-файла:

Код:

#ifdef __cplusplus
extern "C" {
#endif

int sumI(int a, int b);
float sumF(float a, float b);

#ifdef __cplusplus
} // end extern "C"
#endif

Эти ifdef/endif разделяют способ, как компиляторы его будут видеть.
компилятор C:

Код:

int sumI(int a, int b);
float sumF(float a, float b);

компилятор C++ :

Код:

extern "C" {

int sumI(int a, int b);
float sumF(float a, float b);

} // end extern "C"

Т.е. main.cpp должен видеть второй вариант.

[Эйлер Леонард]

ОК! Всё фурычит.

Test-30(1)_74HC595.png

Тестирование 74HC595 (счетчик)

(35.8 KiB) Скачиваний: 237

Поправил main.cpp

Спойлер

Код:

#include <avr/io.h>
#include <avr/interrupt.h>
#include "shiftreg.h"
/* macro definition which returns the number of elements in an arraay */
#define GetArrayElements(x)   sizeof(x)/sizeof(x[0])

#define Prescaler_No   (1<<CS00) //No prescaling)
#define Prescaler_8    (1<<CS01)
#define Prescaler_64   (1<<CS01)|(1<<CS00)
#define Prescaler_256  (1<<CS02)
#define Prescaler_1024 (1<<CS00)|(1<<CS02);

/* 7 Segment Codes */
#define ZERO   0x3F
#define ONE      0x06
#define TWO       0x5B
#define THREE   0x4F
#define FOUR   0x66
#define FIVE    0x6D
#define SIX      0x7D
#define SEVEN   0x07
#define EIGHT   0x7F
#define NINE   0x6F

volatile uint8_t increment_counter = 0;

ISR(TIM0_COMPA_vect)
{
   cli();
   increment_counter = 1;
   sei();
}

int main(void)
{   
   unsigned short counter = 0;
   unsigned char temp;
   unsigned char digits[] = {ZERO,ONE,TWO,THREE,FOUR,FIVE,SIX,SEVEN,EIGHT,NINE};
   unsigned char display[4];
   
   SHIFTREG_register reg;
   
   TCNT0 = 0; //Очищаем счетный регистр (на ваякий случай)
   OCR0A = (256-1)-0;//Регистр сравнения, (256-1)-число
   TCCR0A = (1<<WGM01);// Сброс при совпадении.
   TIFR0 = (1<<OCF0A); // Регистр флагов прерываний таймера счетчика
   TIMSK0 = (1<<OCIE0A); //Разрешение прерывания по совпадению таймера/счётчика 0A
   TCCR0B = Prescaler_No;// Делитель частоты
//                                              ds_pin,st_cp_pin,sh_cp_pin
   reg = SHIFTREG_InitRegister(&DDRB, &PORTB, PB0,   PB1,      PB2);
   sei();
    while (1) {
      
if(increment_counter)
{
++counter;
if(counter > 9999){   counter = 0; }
increment_counter = 0;
}
      temp      =  counter / 1000;
      display[0] =  digits[temp];
      temp      =  (counter / 100) % 10;
      display[1] =  digits[temp];
      temp      =  (counter / 10)  % 10;
      display[2] =  digits[temp];
      temp      =  counter % 10;
      display[3] =  digits[temp];   
   
   SHIFTREG_OutputValue(&reg,display,GetArrayElements(display));
   }
return 0;
}

[WiseLord]

Мне думается, что, cli() и sei() внутри ISR - лишнее.

[Эйлер Леонард]

Вот, вот! Существуют разные мнения на сей счет. Остановка прерывания внутри прерывания. Типа того "а вдруг бы чего не вышло не того". Ко мне давно подкрадывался этот вопрос. И, все таки я так же считаю cli() и sei() внутри ISR - лишними. К тому же где то находил убедительные разъяснения по этому поводу.
Я называю это "Копипастинг кусает за пятки".

[ARV]

cli() внутри ISR, вероятнее всего лишнее, а вот sei() может быть оправдано.

[VladislavS]

ARV писал(а):

а вот sei() может быть оправдано.

А можно узнать, как тогда вы попадаете в прерывание, если они у вас были запрещены?

[ARV]

А можно узнать, как тогда вы попадаете в прерывание, если они у вас были запрещены?

любознательность - это очень похвально!
в обработчик прерывания я попадаю уже при запрещенных прерываниях, хотя до момента установки флага запроса прерывания они были разрешены

[Starichok51]

а вот sei() может быть оправдано.

такое может понадобиться, если есть необходимость, чтобы другое прерывание (более важное) могло прервать ход выполняемого прерывания.

[Эйлер Леонард]

Частный случай;

Код:

ISR(INT0_vect)//External Interrupt
{
   ips++;//На порт PB2 подаем импульсы  _П_П_П_П_П_
};
ISR(TIM0_COMPA_vect)//прерывание по совпадению
{   
MAX7219_display_number(ips);//выводим на дисплей цифры 
   ips=0;
};

С внешнего прерывания считаем (инкрементируем) импульсы. Далее, через определенный период тиков, установленных в таймере-компараторе, подаем их на цифровой дисплей, обнуляем инкремент. Сначала я тренировался на чипе MAX7219, но и с 74HC595 так же считаю необходимым подружиться. А в ISR(TIM0_COMPA_vect){ /* необходимо добавить немного if-ов для усреднения значений */}. Ну и какое из прерываний кого хочет перебороть? Да вроде бы всё мирно должно быть.

[VladislavS]

ARV писал(а):

в обработчик прерывания я попадаю уже при запрещенных прерываниях, хотя до момента установки флага запроса прерывания они были разрешены

Уже начал забывать это как страшный сон

[Эйлер Леонард]

-->приоритет прерываний AVR. Пока для меня этот вопрос не столь актуален. Если что, подкуюсь теорией и Гугли.

[WiseLord]

Внутри ISR() прерывания запрещены и так, так что cli() не нужен. Равно как и sei() в конце - после выхода из обработчика прерывания и без этого включатся.

А вот если нужно неблокирующее прерывание (так, чтобы другие были разрешены), есть ISR_NOBLOCK:

Код:

ISR(TIM0_COMPA_vect, ISR_NOBLOCK)

avr-libc обо всём позаботится.

[Эйлер Леонард]

Есть такое дело.

ISR_NOBLOCK.png

ISR_NOBLOCK в interrupt.h

(15.02 KiB) Скачиваний: 204

[ARV]

ISR_NOBLOCK не совсем то же самое, что и sei() внутри обработчика. все дело в моменте разрешения прерываний: макрос делает это сразу после входа, а самодобавленный sei() - когда надо программисту.