|
Работа с ногами AVR на С++, Проблема переноса с IAR на WinAVR |
|
|
|
Nov 4 2009, 17:42
|

Местный
  
Группа: Свой
Сообщений: 268
Регистрация: 4-11-05
Пользователь №: 10 470

|
Всем доброго времени суток! Понадобилось перенести проект с IAR (EWAVR 530) на WinAVR (20090313) для юридической чистоты... Вылезла такая бяка: Иар успешно работает с такой конструкцией: Код template <unsigned char volatile * port, const char bit> class mypin { public: inline mypin() {*(port-1)|=(1<<bit);} // смещение -1 - регистр DDR - настраиваем на выход. static inline void set() {*port|=(1<<bit);} }; int main() { mypin<&PORTA,4> mypinname; // тут мы обьявляем место подключения и инициализируем одновременно //... mypinname.set(); //... } А WinAVR глючит на конструкции &PORTA в объявлении класса и заявляет что не хватает параметров: Код - wrong number of template arguments (1, should be 2) provided for 'template<volatile uint8_t* port, char bit> class mypin' хотя в другом месте адрес порта успешно изымается: Код volatile uint8_t* portC = &PORTC; *portC &= ~(1<<7); Это глюк компилятора? Как это можно обойти? Подскажите пожалуйста!
|
|
|
|
|
Nov 4 2009, 20:19
|

Местный
  
Группа: Свой
Сообщений: 268
Регистрация: 4-11-05
Пользователь №: 10 470

|
Код template <uint8_t volatile & port, const char bit> class mypin { public: __inline mypin(){*(&port-1)|=(1<<bit);} static __inline void set() {port|=(1<<bit);} };
mypin<PORTA,4> mypinname; mypinname.set(); такой вариант работает в иаре, но ухудшается оптимизация. ВинАВР же опять не доволен строчкой "mypin<PORTA,4> mypinname;": Код - template argument 1 is invalid - `*' cannot appear in a constant-expression - a cast to a type other than an integral or enumeration type cannot appear in a constant-expression звезда там берется из разложения макроса PORTA => (*(volatile uint8_t *)((0x1B) + 0x20))
|
|
|
|
|
Nov 4 2009, 21:20
|

Нечётный пользователь.
     
Группа: Свой
Сообщений: 2 033
Регистрация: 26-05-05
Из: Бровари, Україна
Пользователь №: 5 417

|
Цитата(Genadi Zawidowski @ Nov 4 2009, 23:01)  А не надо делать предположения об адресе регистра управления направлением. Заведите явный параметр для регистра управления направлением и настанет благодать... Шоб это было так просто... Если объявить volatile-переменную и передавать её адрес mypin< &vv >, то и с этим констурктором работает и ожидаемый код генерит, а если даже его убрать а на место &PORTA вручную вписать Код mypin< (volatile uint8_t *)0x1B > mypinname; то всё равно ругается Код p.cpp:13: error: a cast to a type other than an integral or enumeration type cannot appear in a constant-expression p.cpp:13: error: template argument 1 is invalid
--------------------
Ну, я пошёл… Если что – звоните…
|
|
|
|
|
Nov 4 2009, 22:37
|

Нечётный пользователь.
     
Группа: Свой
Сообщений: 2 033
Регистрация: 26-05-05
Из: Бровари, Україна
Пользователь №: 5 417

|
Цитата(Genadi Zawidowski @ Nov 4 2009, 23:26)  Что не так в версии: А я почём знаю, что ему не так. Код ================= E:\>c:\WinAVR/bin/avr-g++ -Wall -Wextra --version 2>&1 avr-g++ (GCC) 4.4.0 20090323 (experimental) Copyright (C) 2009 Free Software Foundation, Inc. This is free software; see the source for copying conditions. There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
E:\>c:\WinAVR/bin/avr-gcc -Wall -Wextra -Os -S -mmcu=atmega64 p.cpp 2>&1 p.cpp: In function 'void foo()': p.cpp:14: error: a cast to a type other than an integral or enumeration type cannot appear in a constant-expression p.cpp:14: error: '*' cannot appear in a constant-expression p.cpp:14: error: a cast to a type other than an integral or enumeration type cannot appear in a constant-expression p.cpp:14: error: '*' cannot appear in a constant-expression p.cpp:14: error: template argument 1 is invalid p.cpp:14: error: template argument 2 is invalid p.cpp:14: error: invalid type in declaration before ';' token p.cpp:15: error: request for member 'set' in 'mypinname', which is of non-class type 'int'
===================
avr-gcc.exe (WinAVR 20090313) 4.3.2 Copyright (C) 2008 Free Software Foundation, Inc. This is free software; see the source for copying conditions. There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
p.cpp: In function 'void foo()': p.cpp:14: error: a cast to a type other than an integral or enumeration type cannot appear in a constant-expression p.cpp:14: error: `*' cannot appear in a constant-expression p.cpp:14: error: a cast to a type other than an integral or enumeration type cannot appear in a constant-expression p.cpp:14: error: `*' cannot appear in a constant-expression p.cpp:14: error: template argument 1 is invalid p.cpp:14: error: template argument 2 is invalid p.cpp:14: error: invalid type in declaration before ';' token p.cpp:15: error: request for member 'set' in 'mypinname', which is of non-class type 'int'
===================
avr-gcc.exe (GCC) 4.2.2 (WinAVR 20071221) Copyright (C) 2007 Free Software Foundation, Inc. This is free software; see the source for copying conditions. There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
p.cpp: In function 'void foo()': p.cpp:14: error: a cast to a type other than an integral or enumeration type cannot appear in a constant-expression p.cpp:14: error: `*' cannot appear in a constant-expression p.cpp:14: error: a cast to a type other than an integral or enumeration type cannot appear in a constant-expression p.cpp:14: error: `*' cannot appear in a constant-expression p.cpp:14: error: template argument 1 is invalid p.cpp:14: error: template argument 2 is invalid p.cpp:14: error: invalid type in declaration before ';' token p.cpp:15: error: request for member 'set' in 'mypinname', which is of non-class type 'int'
--------------------
Ну, я пошёл… Если что – звоните…
|
|
|
|
|
Nov 5 2009, 05:45
|

Местный
  
Группа: Свой
Сообщений: 268
Регистрация: 4-11-05
Пользователь №: 10 470

|
Genadi, даже если уйти от смещений и явно указывать регистры, то проблема остается. Ведь компилятор ругается не на это. Он не хочет принимать указатель как константу. В теме с исходниками нашел такой файлик "avrio.hpp" от Аlex312. Там эта задача реализована через enum-ы: Код enum __bits {__bit_0=0, __bit_1=1, __bit_2=2, __bit_3=3, __bit_4=4, __bit_5=5, __bit_6=6, __bit_7=7}; enum __ports{__port_A,__port_B,__port_C,__port_D,__port_E,__port_F,__port_G}; enum __ports_reg {__reg_PORT, __reg_DDR, __reg_PIN};
// Шаблон функции, определяющей требуемые адреса в пространстве В/В template <__ports port, __ports_reg reg> volatile uint8_t& __ports_ref();
#define AVRLIB_SPECIALIZE_PORT_REF(_port_) \ template <> inline volatile uint8_t& __ports_ref<__port_##_port_, __reg_PORT>() { return PORT##_port_; } \ template <> inline volatile uint8_t& __ports_ref<__port_##_port_, __reg_DDR >() { return DDR##_port_; } \ template <> inline volatile uint8_t& __ports_ref<__port_##_port_, __reg_PIN >() { return PIN##_port_; }
#if defined (PORTA) AVRLIB_SPECIALIZE_PORT_REF(A) #endif #if defined (PORTB) AVRLIB_SPECIALIZE_PORT_REF(B) #endif ..... template <__ports port, __bits bit> class __pin { public: static inline void set(void) { __ports_ref<port, __reg_PORT>()|=bit_mask(bit); } }; __pin<__port_C,__bit_4> out4; out4.set(); Довольно муторно, но работает. Правда на этот раз несварение возникает у Иара. Даже на максимальной оптимизации он упорно отказывается сводить "out4.set();" к ассемблерному "sbi".
|
|
|
|
|
Nov 6 2009, 18:07
|

Местный
  
Группа: Свой
Сообщений: 268
Регистрация: 4-11-05
Пользователь №: 10 470

|
AHTOXA, посмотрел я на ваши труды и попробовал написать подобное для АВР. Сначала было так: Код template<GPIO_mode_t mode, char port, int bit, char activestate> class Pin{ inline volatile uint8_t & PORTx() { #if defined (PORTA) if (port == 'A') return PORTA; #endif #if defined (PORTB) if (port == 'B') return PORTB; #endif //..................такая конструкция для буков С..F и все вместе для DDRх и PINх} inline void On() { if ( mode==BIDIR || mode==OUTPUT ) {activestate == 'L' ? PORTx()&=~mask : PORTx()|=mask;} if ( mode==OUTPUT_OK ) {activestate == 'L' ? DDRx()|=mask : DDRx()&=~mask;} } // и тп } Потом пытался уменьшить количество писанины в части ссылок на регистры: Код template<char port> class Base_Pin;
template<> // такой шаблон для всех буков - можно сделать макрос class Base_Pin<'A'> { public: inline volatile uint8_t & PORTx(){return PORTA;} inline volatile uint8_t & DDRx(){return DDRA;} inline volatile uint8_t & PINx(){return PINA;} };
template<GPIO_mode_t mode, char port, int bit, char activestate> class Pin2 : public Base_Pin<port> { inline void On(){Base_Pin<port>::PORTx()|=(1<<bit);} }; И если дальше "упрощать" создав макрос для объявления базовых классов всех портов, то приходим к необходимости отказаться от символьных констант и воспользоваться enum-ом. В итоге получаем код Alex312. Но я оставил себе первый вариант. Он хоть и громоздкий, но зато легко читаемый\понимаемый. И да кстати. Я так и не понял в чем вкусность переопределения операторов в примере Антона.
|
|
|
|
|
Nov 6 2009, 20:02
|

фанат дивана
     
Группа: Свой
Сообщений: 3 387
Регистрация: 9-08-07
Из: Уфа
Пользователь №: 29 684

|
Цитата(Waso @ Nov 6 2009, 23:07)  Сначала было так: Код template<GPIO_mode_t mode, char port, int bit, char activestate> ... Ну и нормально  Только #if defined (PORTA) - лишнее, ибо не будет вызываться, пока вы не определите Pin<'A'...>. В этом прелесть шаблонов  Цитата Потом пытался уменьшить количество писанины в части ссылок на регистры: Код template<char port> class Base_Pin; ... template<GPIO_mode_t mode, char port, int bit, char activestate> class Pin2 : public Base_Pin<port> ... От наследования тут мало толку, всё равно приходится указывать имя базового класса. Здесь удобнее определить включаемый класс, типа Код // объявление шаблона регистров порта: template<char port>struct port_regs_t; // специализация для каждого порта: template <> struct port_regs_t<'A'> { inline volatile uint8_t & PORTx(){return PORTA;} ... }; ... // и наконец, собственно шаблон ноги: template<GPIO_mode_t mode, char port, int bit, char activestate> class Pin2 { private: typedef port_regs_t<port> regs_t; public: typename regs_t::PORTx PORTx; typename pins_t::DDRx DDRx; typename pins_t::PINx PINx; ... }; В варианте для MSP430 (а именно на него я давал ссылку  ) у меня примерно так и сделано. А вы, похоже, смотрели вариант для stm32? Цитата И да кстати. Я так и не понял в чем вкусность переопределения операторов в примере Антона. Дык, просто что-то не сложилось со ссылками (или просто не догадался). Потому пришлось определять все операции. Если работает со ссылками, то так конечно лучше. Надо будет попробовать... И, кстати, одно замечание по параметрам. Имхо, включать режим работы ножки (GPIO_mode_t) в перечень параметров шаблона - идеологически неверно. Потому что иногда случается, что режим работы ноги надо переключать на ходу.
--------------------
Если бы я знал, что такое электричество...
|
|
|
|
|
Nov 6 2009, 20:43
|

Нечётный пользователь.
     
Группа: Свой
Сообщений: 2 033
Регистрация: 26-05-05
Из: Бровари, Україна
Пользователь №: 5 417

|
Цитата(AHTOXA @ Nov 6 2009, 22:02)  Потому что иногда случается, что режим работы ноги надо переключать на ходу. "я даже больше скажу" - для тех ног, для которых оно в начале программы один раз ставится и всё - для них не жалко и в начале программы несколько строк DDRx = 0xXX написать, а вот если уж начинаешь переключать.... offtopic: Я пока в эти все темы с интересом заглядываю, но использую всё те же "Волковские" макросы, нет времени на эти эксперименты. Зато, похоже, мне нравится placement new
--------------------
Ну, я пошёл… Если что – звоните…
|
|
|
|
1 чел. читают эту тему (гостей: 1, скрытых пользователей: 0)
Пользователей: 0
|
|
|