Правильный способ вызова функции c, которая ожидает const char8 string [] в качестве параметра

Question

Внутри ISR микроконтроллера я пытаюсь вызвать функцию следующим образом:

//adc.c
static volatile char uartBuf[6]={0};

CY_ISR(ISR_ADC)
{

for (uint8_t i=0; i < NS; i++)
  total += adc2_buffer0[i];

uartBuf[0] = total >> 24 & 0xFF;
uartBuf[1] = total >> 16 & 0xFF;
uartBuf[2] = total >> 8 & 0xFF;
uartBuf[3] = total & 0xFF; 
uartBuf[4] = '\n';
UART_1_PutString(uartBuf); //doesn't work
}

//uart.c
void UART_1_PutString(const char8 string[]) 
{
...
}

Однако в функции UART_1_PutString строка всегда указывает на '\ 0', а не на uartBuf? В чем может быть проблема? Теоретически переменная uartBuf не должна оптимизироваться компилятором.

Answer 1

Код кажется правильным

Вы имеете в виду, что string[0] == '\0'?

Может быть (total >> 24 & 0xFF) == 0 всегда (или в большинстве случаев).

РЕДАКТИРОВАТЬ:

Функция должна быть

void UART_1_PutString(const volatile unsigned char buff[]);

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

unsigned потому что "строка" происходит от некоторой битовой обработки без знака после некоторых математических операций, что может привести к недопустимому знаковому значению (вряд ли произойдет сбой, но правило должно быть: char для текста, unsigned char для неизвестных данных (все остальное)).

volatile, потому что если нет, вы отбрасываете спецификатор volatile.Компилятор с достаточно высокими флагами (в GCC: -Wall -Wextra -Werror выделит почти все как ошибки) предупредит вас об этом.Компилятор может подумать, что содержимое функции можно упростить, даже не зная, что вы будете вызывать ее с данными volatile, и, следовательно, оптимизировать то, что не следует оптимизировать.

Вы также можете добавитьrestrict ключевое слово (C11), если никакая другая функция не получит доступ к этому буферу одновременно, так что вы поможете компилятору создать лучший код:

void UART_1_PutString(const volatile unsigned char buff[restrict]);

EDIT2:

Вы должны указать размер буфера, если он будет вызываться только с этим буфером, или также передать размер буфера функции, если нет:

void UART_1_PutString(const volatile unsigned char buff[restrict 6]);

(6, вероятно, должнозаменить на какой-нибудь макрос) или

void UART_1_PutString(size_t size, const volatile unsigned char buff[restrict size]);