Помощь - Поиск - Пользователи - Календарь
Полная версия этой страницы: деление на ноль
Форум разработчиков электроники ELECTRONIX.ru > Микроконтроллеры (MCs) > ARM
Jenya7
надоело все время высчитывать делитель для таймера - решил автоматизировать процесс:
CODE
#define F_CPU 72000000UL
#define F_CPU_MHZ 72
#define SYSTEM_TICK_NS ((1 /F_CPU_MHZ) * 1000UL)

uint32_t GetPrescaleForDelay(uint32_t delay, char * units)
{
uint32_t prescale;
uint32_t multiplyer;
uint32_t ticks;

if(strcmp(units,"ns")==0)
{ multiplyer = 1; }
else if(strcmp(units,"us")==0)
{ multiplyer = 1000; }
else if(strcmp(units,"ms")==0)
{ multiplyer = 1000000; }
//else
//{ return 0; }

//how many ticks needed for delay
// if(SYSTEM_TICK_NS > 0) //не помогает
ticks = (delay * multiplyer) / SYSTEM_TICK_NS;

prescale = F_CPU / ticks;
return prescale;
}


компайлер честно предупреждает - Warning[Pe039]: division by zero на строчке ticks = (delay * multiplyer) / SYSTEM_TICK_NS;

но это как то действует на нервы. как бороться с делением на ноль?
_Артём_
Цитата(Jenya7 @ Feb 19 2014, 14:20) *
компайлер честно предупреждает - Warning[Pe039]: division by zero на строчке ticks = (delay * multiplyer) / SYSTEM_TICK_NS;

Попробуйте подавить выдачу предупреждения - компиляторы обычно дают такую возможность.
Сергей Борщ
У вас SYSTEM_TICK_NS будет равен нулю для любого F_CPU_MHZ больше 1.
Исправьте SYSTEM_TICK_NS на

Код
#define SYSTEM_TICK_NS (1000UL /F_CPU_MHZ)


Цитата(_Артём_ @ Feb 19 2014, 12:35) *
Попробуйте подавить выдачу предупреждения
Ой.
Jenya7
Цитата(Сергей Борщ @ Feb 19 2014, 15:43) *
У вас SYSTEM_TICK_NS будет равен нулю для любого F_CPU_MHZ больше 1.
Исправьте SYSTEM_TICK_NS на

Код
#define SYSTEM_TICK_NS (1000UL /F_CPU_MHZ)


Ой.


ой спасибо - так не ругается.
scifi
Цитата(Jenya7 @ Feb 19 2014, 15:23) *
ой спасибо - так не ругается.

Вы бы лучше разобрались, как работает целочисленная арифметика в языке Си. Там таких граблей - море.
Jenya7
Цитата(scifi @ Feb 19 2014, 17:47) *
Вы бы лучше разобрались, как работает целочисленная арифметика в языке Си. Там таких граблей - море.

понял - без явного приведения типов компайлер округлит это выражение #define SYSTEM_TICK_NS ((1 /F_CPU_MHZ) * 1000UL) до нуля.
интересно а так можно?
#define SYSTEM_TICK_NS ((float) (1000UL /F_CPU_MHZ)
Andreas1
Цитата(Jenya7 @ Feb 19 2014, 17:06) *
#define SYSTEM_TICK_NS ((float) (1000UL /F_CPU_MHZ)

Сначала округлит, затем во флоат переведет.
Надо писать не 1000UL, а 1000.0 тогда все вычисления будут во флоате и только где уже нужен результат пишете (u32)....
Jenya7
или так #define SYSTEM_TICK_NS ((1.0 / F_CPU_MHZ) * 1000.0)
Сергей Борщ
Я бы делал
Код
#define TIMER_TICKS(time_ms)   (((time_ms) * F_CPU + 500)/ 1000)
....
ticks = TIMER_TICKS(delay * multiplyer);
Jenya7
Цитата(Сергей Борщ @ Feb 19 2014, 18:43) *
Я бы делал
Код
#define TIMER_TICKS(time_ms)   (((time_ms) * F_CPU + 500)/ 1000)
....
ticks = TIMER_TICKS(delay * multiplyer);


Сергей как всегда крут. a14.gif
Andreas1
Вроде по умолчанию S16 числа? Переполнение не выходит?
Jenya7
Цитата(Andreas1 @ Feb 19 2014, 19:10) *
Вроде по умолчанию S16 числа? Переполнение не выходит?

это интересное замечание, надо проверить.
Для просмотра полной версии этой страницы, пожалуйста, пройдите по ссылке.
Invision Power Board © 2001-2025 Invision Power Services, Inc.