Почему не работает конструкция на С

[amkuznecov]

Задержка получается неверная

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

int set_portA(char delay){
		         port_A=port_A|1<<pinX;
		         _delay_us(delay);		
		         return 0;
		         }

[Мастер Ломастер]

миллион раз было сказано, в документации написано: никогда не вызывайте _delay_us или _delay_ms с параметром-переменной!!! ТОЛЬКО КОНСТАНТА В КАЧЕСТВЕ ПАРАМЕТРА ОБЕСПЕЧИВАЕТ ЗАЯВЛЕННУЮ ТОЧНОСТЬ ЗАДЕРЖКИ и компактный код!

[Аlex]

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

#define set_portA(pinX, delay)  do{  \
 port_A |=1<<(pinX);                  \
 _delay_us(delay);                    \
 }while(0)

[Мастер Ломастер]

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

#define set_portA(pinX, delay)  do{  \
 port_A |=1<<(pinX);                  \
 _delay_us(delay);                    \
 }while(0)

это совершенно ничего не меняет, если будет делаться так, как раньше:

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

#define set_portA(pinX, delay)  do{  \
 port_A |=1<<(pinX);                  \
 _delay_us(delay);                    \
 }while(0)

int var = 100;
set_portA(5, var); // все равно в _delay_us попадает ПЕРЕМЕННАЯ

[Аlex]

Ну естественно, если использовать переменную, то не меняет.
Возможно что ТС передаёт в свою ф-ию константу.

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

[amkuznecov]

Ну естественно, если использовать переменную, то не меняет.
Возможно что ТС передаёт в свою ф-ию константу.

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

Я передаю именно переменную которая вычисляется, ну не писать же 50 одинаковых функций, _delay_us(10); _delay_us(20);.........
Либо писать тогда свой delay

[Мастер Ломастер]

Я передаю именно переменную которая вычисляется, ну не писать же 50 одинаковых функций, _delay_us(10); _delay_us(20);.........

если вы делаете то, что нельзя делать (об этом недвусмысленно сказано в документации) - кто ж вам виноват?

[amkuznecov]

если вы делаете то, что нельзя делать (об этом недвусмысленно сказано в документации) - кто ж вам виноват?

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

[Мастер Ломастер]

если вы делаете то, что нельзя делать (об этом недвусмысленно сказано в документации) - кто ж вам виноват?

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

ну, если супер-точность не требуется, то вот вариант:

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

static void delay_MS(uint16_t ms){
   for(; ms; ms--) _delay_ms(1);
}

[pyzhman]

Единичную задержку просчитать ручками на бумажке карандашиком и - юзать.

[ChipKiller]

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

... зависит от диапазона задержки, о чем нет ни слова...

[DX168B]

Тоже заметил, что _delay_us(); и _delay_ms(); имеют ограничения на максимальное значение.
Только вот не помню на сколько. И передача туда переменной, траблы вызывала. Я вообще МК на АСМе программирую. В С\С++ у меня под ПК лучше получается.

Может попробуйте так:

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

void SetPort_A(unsigned long int delay)
{
    ////////////
    _delay_us(delay);
    ////////////
}

[Мастер Ломастер]

Может попробуйте так:

может, не будете советовать то, что априори неверно?

хотя бы прочли доку к функции _delay_ms или _delay_us: там сказано, что во-первых, аргумент этой функции имеет тип float, а во-вторых, должен быть константой, чтобы все вычисления были сделаны на этапе компиляции.

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

[DX168B]

Я уже проверил. Нифига оно не работает.
Да уж, придётся свою функцию писать, или библиотеки искать.

[ibiza11]

вот вам библиотека на асм, поддерживающая переменные со значениями от 0 до 65535

этот файл назвать "mydelay.s" и подключить его к проекту:

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

NAME mydelay_lib
  PUBLIC mydelay
  RSEG CODE
mydelay:
  subi r16,1
  sbci r17,0
  push r16
  or r16,r17
  breq exit_delay
  pop r16
  rjmp mydelay
exit_delay:
  pop r16
  ret  
  end

этот файл назвать "mydelay.h" и #include его в основную программу :

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

#ifndef mydelay_lib
#define mydelay_lib
void mydelay(unsigned int);
#endif

вызывать так:

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

#include "mydelay.h"
/***********/
mydelay(ticks);

[Goldsmith]

Если используете GCC и хотите избежать дорогостоящей возни с числами с плавающей точкой, воспользуйтесь готовыми библиотечными функциями _delay_loop_1 и _delay_loop_2. Правда, аргументы для них задаются не в микро(милли)секундах, а в тройках тактов CPU, но при известной тактовой частоте пересчитать не проблема.

Их определения находятся в <util/delay_basic.h>.