Сложный вопрос AVR на С про volatile to const

alex68md

я попробовал в моем коде заменить все ссылки на указатели и просто скомпилировать без проверки на железе - то да с указателями вроде компелируется. незнаю если работает но компелируется.

ARV, да можно и так правда интервал сброса, и лапку надо еще как аргумент передать туда. так а в чем ошибка в моем коде ? там же элементарно синтаксическая какаято ошибка, мне не хватает знаний это просто увидеть и исправить

Добавлено after 44 minutes 4 seconds:
КРАМ, да можно как ARV проверять перед while что у нас впеременной порта. если ничего то default значение. это один путь. мой путь использовать возможности языка Си а именно, указывать дефолтное значение при объявление аргумента. это другой путь. если физически нельзя передать ссылку на volatile и дефолтное значение тогда буду исправлять логику как ARV или еще както. но я думаю что это возможно просто правильно объявить/передать нужно. но как?!

Реклама

Dimon456

Не понятно для чего передать ссылку на volatile? Она что в коде может поменяться?
И что означает дефолтное значение? Какое у пина порта может быть дефолтное значение?

Добавлено after 28 minutes 24 seconds:
Если я правильно понял, всего-то нужно передать ожидаемый уровень на пине порта

Код: Выделить всё

while((level ^ (PIND&(1 << INT0))));	// где level либо 0 либо 1

Реклама

КРАМ

[uquote="alex68md",url="/forum/viewtopic.php?p=3621134#p3621134"]если ничего то default значение.[/uquote]В порте не может быть "ничего". Там либо ноль, либо единица.
Под пустым значением (NULL) понимают ОПРЕДЕЛЕННЫЙ, специально выделенный код, который не пересекается с иными возможными значениями. В битовых переменных это невозможно принципиально.
В данном случае, физический смысл дефолтного значения порта может быть только один - это ПАССИВНОЕ его состояние. Физически пассивное состояние означает уровень подтяжки пина порта. Его реализуют либо внешним резистором, либо встроенной схемой подтяжки через специальный регистр управления.
Если есть желание создать универсальную функцию на все случаи жизни, можно передать в нее пассивный уровень для данного пина, тогда while будет ожидать ПРОТИВОПОЛОЖНОГО значения.

ARV · Сообщение **ARV** » Пн апр 22, 2019 15:23:03

я так понимаю, что есть желание сделать нечто подобное такому:
procedure proc(param : integer = 1);
к сожалению, я не знаю, как такое оформляется в С++ (потому привел пример из паскаля), но суть такая, что допустимы следующие варианты использования процедуры:
proc(5); // передача параметра 5
proc; // передача параметра 1 (по умолчанию)

то есть если явно параметр функции не указан, используется его "дефолтное" значение. ну а если указан - то фактическое.

зачем это нужно в программе для МК - я не знаю... оно и в "большом" программировании не особо нужно...

Реклама

alex68md

ARV, вы правильно поняли . в с++ так оно и передается. просто как я понял тип переменная(в моём случае 1) , тип конст и тип волатили не дружат с другдругом. т.е. надо менять объявление или одного или другого или всего сразу... в большом очень нужно (как минимум "защита от дурака")

КРАМ, а кто сказал что в порте ? я могу вызвать так soft_reset() без аргумента. и откуда я возьму порт ?

вот оно и нужно дефолтное

друзья а что скажите насчёт такой реализации ? валидно? не валидно - почему? а функция таки да с прицелом на универсальность. не всего и вся но более-менее да

Код: Выделить всё

void soft_reset(const uint8_t wdt_prescale, const volatile uint8_t &port, const uint8_t pin){
	const volatile uint8_t &port_new = (port == NULL) ? 1 :  port;
	const uint8_t pin = (pin == NULL) ? 1 :  pin;
	const uint8_t wdt_prescale = (wdt_prescale == NULL) ? WDTO_15MS :  wdt_prescale;
   wdt_enable(wdt_prescale);
   while(port_new & pin);
   wdt_disable();
}

Добавлено after 7 minutes 42 seconds:
[uquote="Dimon456",url="/forum/viewtopic.php?p=3621182#p3621182"]Если я правильно понял, всего-то нужно передать ожидаемый уровень на пине порта

Код: Выделить всё

while((level ^ (PIND&(1 << INT0))));	// где level либо 0 либо 1

[/uquote]

да оно(состояние порта) в коде может меняться , точнее не в коде а вне кода именно поэтому волтали. да проверка именно такая только через ссылки т.к. порты и пины я подсовываю в функцию разные. у вас же жестко привязано к INT0 и его порту

Реклама

Reflector

[uquote="alex68md",url="/forum/viewtopic.php?p=3621287#p3621287"]друзья а что скажите насчёт такой реализации ? валидно? не валидно - почему?[/uquote]
Подумай над таким вариантом:

Код: Выделить всё

void soft_reset(uint8_t wdt_prescale = WDTO_15MS)
{
   wdt_enable(wdt_prescale);
   while(true) {}
} 

void soft_reset(const volatile uint8_t& port, uint8_t pin, uint8_t wdt_prescale = WDTO_15MS)
{
   wdt_enable(wdt_prescale);
   while(port_new & pin) {}
   wdt_disable();
}

Реклама

Dimon456

alex68md писал(а):да оно(состояние порта) в коде может меняться

Да не вопрос
Создаем две переменных, точнее указатель и переменную

Код: Выделить всё

volatile uint8_t *portd; 
volatile uint8_t int_pin;

Далее, к примеру

Код: Выделить всё

void isPIRAlive(){
   portd = &PIND;
   int_pin = 0;
   isPIRHigh();
   portd = &PINC;
   int_pin = 1;
   isPIRLow();
}

и конечный

Код: Выделить всё

while((level ^ (*portd &(1 << int_pin))));

ARV · Сообщение **ARV** » Пн апр 22, 2019 19:46:01

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

Dimon456

Вот для наглядности

Спойлер

Код: Выделить всё

#include <avr/wdt.h>

#define D_PIN 	PIND
#define D_INT	PIND0
#define C_PIN 	PINC
#define C_INT	PINC1

void soft_reset(const uint8_t wdt_prescale, volatile uint8_t *PORT_set, const uint8_t int_pin, const uint8_t level){ //uint8_t *PIN_set
 
   wdt_enable(wdt_prescale);
   //while((level ^ (*PORT_set &(1 << int_pin))));	// если level=1 то 1 на пине порта ждем
   while(!(level ^ (*PORT_set &(1 << int_pin))));	// если level=1 то 0 на пине порта ждем
   wdt_reset();
   wdt_disable();
}

void isPIRHigh(void){
   soft_reset(WDTO_8S, &D_PIN, D_INT, 1);       //finish when PD0 high 
}

void isPIRLow(void){
   soft_reset(WDTO_8S, &D_PIN, D_INT, 0);               //finish when PD0 low 
}

void isPIRAlive(){
   isPIRHigh();
   isPIRLow();
   
   // для наглядности
   soft_reset(WDTO_8S, &D_PIN, D_INT, 1);
   soft_reset(WDTO_8S, &D_PIN, D_INT, 0);
   soft_reset(WDTO_8S, &C_PIN, C_INT, 1);
   soft_reset(WDTO_8S, &C_PIN, C_INT, 0);
}

void setup(){

   // конфигурация портов на вход на выход
   // назначить вывод порта на ввод
   // включить подтягивающий резистор    
  
isPIRAlive();
}
 
void loop()
 { //
 }

alex68md

ARV , я вам больше скажу. конструкция языка таки не позволяет сделать через одну

точнее как я понял есть 2 пути. один через глобальные переменные как раз Dimon456 в предпоследнем посте упомянул. либо через две наподобе как Reflector упомянул только из первой пустой (без аргументов) вызываем вторую и передаем её дефолтные аргументы - метод перезагрузки функции. на втором методе я пока и остановился но упёрся в строчку

Код: Выделить всё

soft_reset(WDTO_8S, ~port, pin);

~port не хочет передаваться как volatile и пока даже не представляю как решить эту загвоздку ...

ЗЫ: Димон , интересный последний пост , надо обмозговать

Reflector

[uquote="alex68md",url="/forum/viewtopic.php?p=3621886#p3621886"]~port не хочет передаваться как volatile и пока даже не представляю как решить эту загвоздку ...[/uquote]
Тут создается временный объект, причем не константный, потому по обычной ссылке его не передашь, но можно передать по rvalue ссылке:

Код: Выделить всё

void soft_reset(volatile uint8_t&& port);

Единственное, что нужен С++11, не знаю какой там в ардуино...
Другое дело что непонятно зачем в принципе передавать ~port... Думаешь потом будешь читать из порта и там будет инверсное значение или как?

ARV · Сообщение **ARV** » Вт апр 23, 2019 17:28:58

Reflector писал(а):Другое дело что непонятно зачем в принципе

это творить... 2 функции - 1 минута делов и все свободны. нет, надо неделю на форуме людям головы морочить, самому себе голову морочить - а ради чего? какая ценность в этом варианте есть, что стоит этого геморроя?!

Dimon456

Правильно поставленная задача - половина успеха.
Правильно поставленный вопрос - половина ответа.

Нет, опять тресет

alex68md писал(а): на втором методе я пока и остановился но упёрся в строчку
Код:
soft_reset(WDTO_8S, ~port, pin);

~port не хочет передаваться как volatile и пока даже не представляю как решить эту загвоздку ...

Открываем даташит на контроллер, читаем

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

Чего вы пытаетесь ~port туда записать?
Второе: к примеру портД состоит из 8 пинов (8 выводов), надо читать конкретный пин порта, а не полностью порт. Вы можете гарантировать что на состояние остальных пинах (выводах) порта не изменится?
Третье: я перерыл кучу компиляторов, кучу операционных систем, везде константа есть константа, переменная есть переменная.

Константа, переменная — это основополагающие понятия в любом языке программирования.

PS: видать компилятор оказался умнее человека, по этому и не позволяет вам записать в регистр предназначенный для чтения.

ARV · Сообщение **ARV** » Вт апр 23, 2019 19:28:35

Dimon456 писал(а):Открываем даташит на контроллер, читаем

вот именно, читаем (конкретно эта цитата из даташита на atmega328, но такое поведение во всех "новых" МК семейства AVR8):

However, writing '1' to a bit in the PINx Register will result in a toggle in the corresponding bit in the Data Register.

так что запись в PINx очень даже имеет смысл

Dimon456

А, ну да, там же ЛЕОНАРДО....АРДУИНО....
Тогда точнее будет так

Writing a logic one to PINxn toggles the value of PORTxn, independent on the value of DDRxn.
Note that the SBI instruction can be used to toggle one single bit in a port.

То есть другими словами включим подтяжку, выключим подтяжку. Что это изменило?
Нее .. не хороший подход, выводы настроенные на выход.....

ARV · Сообщение **ARV** » Вт апр 23, 2019 20:49:34

я не оцениваю подходы, я указываю на вашу неточность: запись в PINx возможна и имеет определенный смысл. если DDRx=255, то записью в PINx можно получить "частоту" на PORTx, недостижимую для других способов "дрыгонога".

alex68md

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

Reflector, Dimon456, подождите, так что получается эта строчка, это будет запись в порт ? а не просто инверсное чтение ? мне запись категорически нельзя

Код: Выделить всё

 soft_reset(WDTO_8S, ~port, pin);

"Думаешь потом будешь читать из порта и там будет инверсное значение" - да я так и подумал что в WHILE будет сверяться инверсное значение порта через чтение и всё

Оно так и сверяеться когда я на кошках тренировался - просто volatile переменную передавал. а что в случае с регистром так не получиться ?

а как тогда минимумом строк кода передать инверсное realtime чтение порта ?
если я меняю все вызовы на поинтеры ~*port и потом while (*port&pin) то оно в принципе компилируеться но не знаю если будет эфект которого я жду ?

"видать компилятор оказался умнее человека, по этому и не позволяет вам записать в регистр предназначенный для чтения." да причем тут компилятор, именно поэтому const еще используется как защита от записи, поэтому я и пишу const volatile чтоб компилятор считал это как read only переменную. я не знал что ~port пишет чтото в порт. мы же просто читаем инверсию. не понимаю откуда запись

BOB51

Обсуждать программное решение, использующее особенности аппаратной начинки МК,
БЕЗ КОНКРЕТНОЙ СХЕМЫ И ОПИСАНИЯ ХОТЕЛКИ
есть абсолютно бесполезная задача...

Тем более, ежли за основу берем
адуринку и написание программы для оной в рамках ардуиноIDE
а не
"платку с атмегой328Р" в "чистом СИ" хотя-бы в рамках АВР-студии (7й на сегодня вроде).

Reflector

[uquote="alex68md",url="/forum/viewtopic.php?p=3622196#p3622196"]если я меняю все вызовы на поинтеры ~*port и потом while (*port&pin) то оно в принципе компилируеться но не знаю если будет эфект которого я жду ?[/uquote]
У мк есть регистр из которого можно читать инверсное значение порта? Если бы был, то ты бы его вместо PINx и передавал, а сейчас ~*port читает с порта один раз в момент передачи в функцию, потом там в цикле проверяется одно и то же значение. Со ссылками будет то же самое.

подождите, так что получается эта строчка, это будет запись в порт ? а не просто инверсное чтение ? мне запись категорически нельзя
Код: Выделить всё
 soft_reset(WDTO_8S, ~port, pin);

Это просто инверсное чтение, но невозможно получить напрямую ссылку для ~port, так же как нельзя у этой сущности взять адрес, тут будет создан временный объект(в случае передачи по &&) в котором хранится прочитанное значение.

Dimon456

Может все таки так

Спойлер

Код: Выделить всё

#include <avr/wdt.h>

#define D_PIN 	PIND
#define D_INT	0

void soft_reset(const uint8_t wdt_prescale, volatile uint8_t *PORT_set, const uint8_t int_pin, const uint8_t level){ 
 
   wdt_enable(wdt_prescale);
   while((level ^ ((*PORT_set &(1 << int_pin))>>int_pin)));	// 
   wdt_reset();
   wdt_disable();
}

void isPIRHigh(void){
uint8_t level;

level = ((D_PIN&(1<<D_INT))>>D_INT);       	//получение состояния пина порта

if (level==1) {					//если пин порта равен 1 то
   soft_reset(WDTO_8S, &D_PIN, D_INT, 0);	//ждать на пине порта 0
} else {					//иначе если пин порта равен 0 то
   soft_reset(WDTO_8S, &D_PIN, D_INT, 1);	//ждать на пине порта 1
}

}


void isPIRAlive(){
   isPIRHigh();
}

void setup(){

   // назначить вывод порта на ввод
   // включить подтягивающий резистор 
   // конфигурация портов на вход на выход
    
isPIRAlive();
}
 
void loop()
 { 
 }