Прерывания в стиле С++

@_SayHello · Регистрация: 30.07.2015

Author24 — интернет-сервис помощи студентам

Решил побаловаться и попробовать написать чего-нибудь на С++ под stm32. Чисто ради любопытства, накорябал класс Led (хотя наверное можно было накидать класс Output, а от него наследоваться =)).
типа того

Кликните здесь для просмотра всего текста

C++

#ifndef LEDS_H_INCLUDED
#define LEDS_H_INCLUDED
 
#include "stm32f30x_gpio.h"
#include "stm32f30x_rcc.h"
 
#define LED_RESET   0
#define LED_SET     1
 
 
class Led
{
private:
    GPIO_TypeDef * LED_PORT;
    uint32_t LED_PIN;
    uint8_t State;
public:
    explicit Led(GPIO_TypeDef * port, uint32_t pin, uint8_t state = LED_RESET);
 
    void Set();
    void Reset();
    void Toogle();
    uint8_t GetState();
};
 
#endif /* LEDS_H_INCLUDED */

Кликните здесь для просмотра всего текста

C++

Led::Led(GPIO_TypeDef * port, uint32_t pin, uint8_t state)
{
    LED_PORT = port;
    LED_PIN = pin;
 
    if(port == GPIOA)
        RCC_AHBPeriphClockCmd(RCC_AHBPeriph_GPIOA, ENABLE);
    else if(LED_PORT == GPIOB)
        RCC_AHBPeriphClockCmd(RCC_AHBPeriph_GPIOB, ENABLE);
    else if(LED_PORT == GPIOC)
        RCC_AHBPeriphClockCmd(RCC_AHBPeriph_GPIOC, ENABLE);
    else if(LED_PORT == GPIOD)
        RCC_AHBPeriphClockCmd(RCC_AHBPeriph_GPIOD, ENABLE);
    else if(LED_PORT == GPIOE)
        RCC_AHBPeriphClockCmd(RCC_AHBPeriph_GPIOE, ENABLE);
    else if(LED_PORT == GPIOF)
        RCC_AHBPeriphClockCmd(RCC_AHBPeriph_GPIOF, ENABLE);
 
    GPIO_InitTypeDef gpio;
 
    gpio.GPIO_Pin = LED_PIN;
    gpio.GPIO_Speed = GPIO_Speed_2MHz;
    gpio.GPIO_Mode = GPIO_Mode_OUT;
    gpio.GPIO_PuPd = GPIO_PuPd_NOPULL;
    GPIO_Init(LED_PORT, &gpio);
 
    if(state == LED_SET)
    {
        GPIO_SetBits(LED_PORT, LED_PIN);
        State = LED_SET;
    }
    else
    {
        GPIO_ResetBits(LED_PORT, LED_PIN);
        State = LED_RESET;
    }
}
 
void Led::Set()
{
    GPIO_SetBits(LED_PORT, LED_PIN);
    State = LED_SET;
}
 
void Led::Reset()
{
    GPIO_ResetBits(LED_PORT, LED_PIN);
    State = LED_RESET;
}
 
void Led::Toogle()
{
    LED_PORT->ODR ^= LED_PIN;
    if(State)
        State = LED_RESET;
    else
        State = LED_SET;
}
 
uint8_t Led::GetState()
{
    return State;
}

Так вот, прикольно, и мне понравилось. Все работает. Но возник вопрос, как например поморгать светодиодом в обработчике прерываний? Если мы объявим объекты в main их будет не видно в обработчиках.
Будучи явно уверенным, что глобально объект объявить нельзя, сделал так:

C++

#include "Leds.h"
 
Led LED1(GPIOE, GPIO_Pin_12);
Led LED2(GPIOE, GPIO_Pin_13);
 
int main(void)
{
    SystemInit();
 
    while(1)
    {
        for(uint32_t i = 0; i < 0xFFFFF; i++){};
        LED1.Toogle();
        LED2.Toogle();
    }
}

И оно заработало. Хотя я почему то был уверен, что функции до main вызывать нельзя и компилятор ругаться будет. Проверил в Qt действительно, и там на вызов перед main не ругается.
Может кто уже с этим разбирался, подскажите как правильно, и чем черевата такая писанина?

@Voland_ · 15.12.2018, 23:01

Сообщение от Void1509

вы утверждали, что операции сдвига - тяжеловесны.

я акцентировал внимание на том, что они тяжеловесны в случае не константного сдвига, а по регистру:

Сообщение от Voland_

структуры динамического сдвига занимают много места и времени в коде

но, в общем, я вижу что в кортексе это не проблема.

Сообщение от Void1509

Сравнивать AVR и ARM - это как сотовый начала 90-х и современный телефон. В смысле и то, и то электроника.

Не по теме:

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

@_SayHello · 15.12.2018, 23:18 **[ТС]**

Void1509, это все конечно приятно и лампово один светодиод один пин один порт. Я тоже так думал когда писал первый проект. Потом светодиодов стало 20 в том же проекте. Потом эти светодиоды перекинули совсем на другие пины. И так далее. Если бы всегда было все статично то вопросов бы не было. А так мне надо одну структуру переопределить и все. А можно и на ходу переделать структуру через интерфейс какой нибудь и сохранить в епроме

@Void1509 · 15.12.2018, 23:38

Боюсь в проблеме которую вы описали - С++ бессилен. Это проблема чисто организационного порядка и никакими средами программирования к сожалению не решается. Заказчик, если его не ставить в рамки договором и техзаданием, имеет неограниченный ассортимент всяческих хотелок и учесть их все просто не возможно.

@_SayHello · 16.12.2018, 00:01 **[ТС]**

Void1509, борьба и идет за упрощение своей работы

@Void1509 · 16.12.2018, 00:10

Как видите, простота есть критерия субъективная. Вам проще через класс описать, мне - дефайном. Но мой способ не генерирует лишний код.

Добавлено через 5 минут
Аж любопытно стало - глянуть на проект на С++. На каком уровне происходит вызов конструкторов в коде ТСа ? На уровне инициализации секции .init ?

@_SayHello · 16.12.2018, 00:10 **[ТС]**

Void1509, но ваш способ не позволяет поменять пин на ходу. не меняя прошивки.

А вообще изначальный вопрос исчерпан. Уже не по теме обсуждение.

@Void1509 · 16.12.2018, 00:14

Сообщение от _SayHello

но ваш способ не позволяет поменять пин на ходу. не меняя прошивки.

Ну, да. К этому и стремился - все что можно - определить на уровне компиляции, а не исполнения.

Добавлено через 1 минуту
Если не секрет, как решили изначальный вопрос ?

@_SayHello · 16.12.2018, 00:17 **[ТС]**

Void1509, в смысле а в чем вопрос? Вопрос решился вызовом конструкторов до main. И обрамлением обработчиков прерываний в extern "C". Это был опыт не более. У меня пока слишком ного библиотек написано а Си, чтобы я резко перешел на С++. Но некие удобства для себя я уже определил

@Void1509 · 16.12.2018, 00:23

Вот ваш код в начале топика

C++

#include "Leds.h"
 
Led LED1(GPIOE, GPIO_Pin_12);
Led LED2(GPIOE, GPIO_Pin_13);
 
int main(void)
{
    SystemInit();
 
    while(1)
    {
        for(uint32_t i = 0; i < 0xFFFFF; i++){};
        LED1.Toogle();
        LED2.Toogle();
    }
}

в данном коде конструкторы объектов класса Led вызываются до SystemInit(); Вопрос в том - как вы разрешили данную коллизию ?

@_SayHello · 16.12.2018, 00:28 **[ТС]**

Void1509, а, если вы про это то вот самый простой вариант

Сообщение от _SayHello

Меня это тоже смутило, на светодиодах это не особо критично, на периферии сложнее уже неприятно. Можно конечно решить, таким образом. Оставить конструктор таким:

C

Led::Led(GPIO_TypeDef * port, uint32_t pin)
{
* * LED_PORT = port;
* * LED_PIN = pin;
}

И вызывать его до main. И добавить метод:

C

void Led::Init(uint8_t State)
{
* * if(port == GPIOA)
* * * * RCC_AHBPeriphClockCmd(RCC_AHBPeriph_GPIOA, ENABLE);
* * else if(LED_PORT == GPIOB)
* * * * RCC_AHBPeriphClockCmd(RCC_AHBPeriph_GPIOB, ENABLE);
* * else if(LED_PORT == GPIOC)
* * * * RCC_AHBPeriphClockCmd(RCC_AHBPeriph_GPIOC, ENABLE);
* * else if(LED_PORT == GPIOD)
* * * * RCC_AHBPeriphClockCmd(RCC_AHBPeriph_GPIOD, ENABLE);
* * else if(LED_PORT == GPIOE)
* * * * RCC_AHBPeriphClockCmd(RCC_AHBPeriph_GPIOE, ENABLE);
* * else if(LED_PORT == GPIOF)
* * * * RCC_AHBPeriphClockCmd(RCC_AHBPeriph_GPIOF, ENABLE);
GPIO_InitTypeDef gpio;
gpio.GPIO_Pin = LED_PIN;
* * gpio.GPIO_Speed = GPIO_Speed_2MHz;
* * gpio.GPIO_Mode = GPIO_Mode_OUT;
* * gpio.GPIO_PuPd = GPIO_PuPd_NOPULL;
* * GPIO_Init(LED_PORT, &gpio);
if(state == LED_SET)
* * {
* * * * GPIO_SetBits(LED_PORT, LED_PIN);
* * * * State = LED_SET;
* * }
* * else
* * {
* * * * GPIO_ResetBits(LED_PORT, LED_PIN);
* * * * State = LED_RESET;
* * } * *
}

И вызывать его в main

@Void1509 · 16.12.2018, 00:32

Благодарю, понял, Вы инициализацию вынесли из конструктора.

@_SayHello · 16.12.2018, 00:33 **[ТС]**

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

@Voland_ 1976 / 1275 / 130 Регистрация: 04.01.2010 Сообщений: 4,607
	15.12.2018, 23:01	21
	Сообщение от Void1509 вы утверждали, что операции сдвига - тяжеловесны. я акцентировал внимание на том, что они тяжеловесны в случае не константного сдвига, а по регистру: Сообщение от Voland_ структуры динамического сдвига занимают много места и времени в коде но, в общем, я вижу что в кортексе это не проблема. Сообщение от Void1509 Сравнивать AVR и ARM - это как сотовый начала 90-х и современный телефон. В смысле и то, и то электроника. Не по теме: :). ну ок, вообще-то это вы слишком далеко их разнесли. И то и то имеет право на жизнь, так что вполне нормально сравнивать то, где существует обсужденная фича. 0

@_SayHello 874 / 535 / 175 Регистрация: 30.07.2015 Сообщений: 1,739
	15.12.2018, 23:18 [ТС]	22
	Void1509, это все конечно приятно и лампово один светодиод один пин один порт. Я тоже так думал когда писал первый проект. Потом светодиодов стало 20 в том же проекте. Потом эти светодиоды перекинули совсем на другие пины. И так далее. Если бы всегда было все статично то вопросов бы не было. А так мне надо одну структуру переопределить и все. А можно и на ходу переделать структуру через интерфейс какой нибудь и сохранить в епроме 0

@Void1509 5 / 5 / 0 Регистрация: 17.09.2018 Сообщений: 43
	16.12.2018, 00:10	25
	Как видите, простота есть критерия субъективная. Вам проще через класс описать, мне - дефайном. Но мой способ не генерирует лишний код. Добавлено через 5 минут Аж любопытно стало - глянуть на проект на С++. На каком уровне происходит вызов конструкторов в коде ТСа ? На уровне инициализации секции .init ? 0

@_SayHello 874 / 535 / 175 Регистрация: 30.07.2015 Сообщений: 1,739
	16.12.2018, 00:10 [ТС]	26
	Void1509, но ваш способ не позволяет поменять пин на ходу. не меняя прошивки. А вообще изначальный вопрос исчерпан. Уже не по теме обсуждение. 0

@Void1509 5 / 5 / 0 Регистрация: 17.09.2018 Сообщений: 43
	16.12.2018, 00:14	27
	Сообщение от _SayHello но ваш способ не позволяет поменять пин на ходу. не меняя прошивки. Ну, да. К этому и стремился - все что можно - определить на уровне компиляции, а не исполнения. Добавлено через 1 минуту Если не секрет, как решили изначальный вопрос ? 0

@Void1509 5 / 5 / 0 Регистрация: 17.09.2018 Сообщений: 43
	15.12.2018, 23:38	23
	Боюсь в проблеме которую вы описали - С++ бессилен. Это проблема чисто организационного порядка и никакими средами программирования к сожалению не решается. Заказчик, если его не ставить в рамки договором и техзаданием, имеет неограниченный ассортимент всяческих хотелок и учесть их все просто не возможно. 0

@_SayHello 874 / 535 / 175 Регистрация: 30.07.2015 Сообщений: 1,739
	16.12.2018, 00:01 [ТС]	24
	Void1509, борьба и идет за упрощение своей работы 0

@_SayHello 874 / 535 / 175 Регистрация: 30.07.2015 Сообщений: 1,739
	16.12.2018, 00:17 [ТС]	28
	Void1509, в смысле а в чем вопрос? Вопрос решился вызовом конструкторов до main. И обрамлением обработчиков прерываний в extern "C". Это был опыт не более. У меня пока слишком ного библиотек написано а Си, чтобы я резко перешел на С++. Но некие удобства для себя я уже определил 0

@Void1509 5 / 5 / 0 Регистрация: 17.09.2018 Сообщений: 43
	16.12.2018, 00:32	31
	Благодарю, понял, Вы инициализацию вынесли из конструктора. 0

@_SayHello 874 / 535 / 175 Регистрация: 30.07.2015 Сообщений: 1,739
	16.12.2018, 00:33 [ТС]	32
	Void1509, для определния пинов и номера интерфейса тактирование особо не важно, и можно их вызвать до настройки системы тактирования, а вот проинициализировать можно и в main. Обычно в работе, точнее при изменениях в проекте перекидываются интерфейсы с одного на другой, дальнейшая их работа остается такой же 0