мне нужно умножать два 16bit числа, потом результат поделить на 4095
галочка в ИАРе в опциях стоит - hardware multiplyer
но приведенный ниже код не работает!
может лучше на ассемблере сделать эту фунцкию?


unsigned int get_code ( unsigned int num_value, unsigned int k )
{
tmp = num_value * k ;
asm ("nop") ;
asm ("nop") ;
asm ("nop") ;
tmp = tmp >> 12 ;
return tmp ;
}

В том коде, который приведён, ничего работать не будет. tmp не объявлена.
Кстати, сдвиг на 12 эквивалентен делению на 4096, а не на 4095...
попробуйте так

А зачем для столь простых операций целочисленного умножения и деления отдельную функцию городить? Это же Си, а не ассемблер. Если в свойствах проекта указано о том, что нужно использовать аппаратный умножитель, то компилятор сам подставит вызов именно той функции, которая его использует.

я сделал так:

unsigned int dac_code = 0 ;
unsigned int HV_value = 0 ;
........

unsigned int get_code ( unsigned int num_value, unsigned int k )
{
unsigned long t = num_value * k ;
asm ( " nop " ) ;
asm ( " nop " ) ;
asm ( " nop " ) ;
asm ( " nop " ) ;
return ( t >> 12 ) ;
}

.........

HV_value = 500 ;

..........

// DAC8512 code = HV_value * 4000/5000 = HV_value * 3276/4095
dac_code = get_code ( HV_value, 3276 ) ;

.........

// отсылаю значение dac_code в ПК через RS-232
query_buf [ 6 ] = dac_code ;
query_buf [ 7 ] = dac_code >> 8 ;

В итоге на компе вижу цифру 15
Откуда!?
Должно быть 400.....

так железно должно работать:

unsigned int t = 0 ;

unsigned int get_code ( unsigned int num_value, unsigned int k )
{
stack1 [ 0 ] = num_value ;
stack1 [ 1 ] = num_value >> 8 ;
stack2 [ 0 ] = k ;
stack2 [ 1 ] = k >> 8 ;

result [ 3 ] = stack1 [ 1 ] * stack2 [ 1 ] ;
result [ 2 ] = stack1 [ 1 ] * stack2 [ 0 ] ;
result [ 1 ] = stack1 [ 0 ] * stack2 [ 1 ] ;
result [ 0 ] = stack1 [ 0 ] * stack2 [ 0 ] ;

t = ( result [ 3 ] << 16 ) + result [ 0 ] + ( ( result [ 2 ] + result [ 1 ] ) << 8 ) ;
return ( t >> 12 ) ;
}

но компилятор ругаеца - Warning[Pe063]: shift count is too large
что собственно естественно - у MSP430 размер регистров 16 бит

Вы опять наступаете на те же грабли. Без явного объявления (по-умолчанию) автоматическая переменная имеет тип int, для MSP430 int = 16 бит. Либо объявите переменные как unsigned long, либо делайте в каждом выражении явное приведение типа. Например,

Ну как, нравится? Думаю, что нет. Так что определяйте типы переменных заранее и не вые надейтесь на "авось". Законы Мерфи-то работают!

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

А надо так:


Тогда будет всё в порядке. Кстати, без "nop"-ов тоже будет работать :-)

Так вроде то же самое написано...

Попробую догадаться, почему так происходит:
В первом случае сначала происходит перемножение num_value и k и происходит переполнение, поэтому когда результат присваивается t, он ошибочный.
Во втором случае переменной t присваивается значение num_value и потом только происходит умножение t на k и при этом переполнения не возникает.
Так?

Да. В первом случае сначала перемножаются два int-а, результат тоже int.
Во втором случае - сначала присвоение long-у, потом умножение long-а на int, результат - long.

Можно также написать:

Так работает !!!!!!!!!!!! Ура !!!!!!!!!

P.S. Опять наступил на грабли приведения типов в С...