STM32L053 HardFault

[kampri]

В целом или лыжи не едут, или лето настало.
Компилятор KEIL 5.23.
При приеме данных по UART получаю регистры uint16 или uint32 и перед записью в EEPROM вынужден переворачивать их. И тут начинается странное.
void flip_reg(uint16_t* ptr_reg, uint8_t len, uint8_t size)
{uint8_t cnt2;
uint32_t* ptr1, ptr2;
if(size == 2)
{cnt2=len››1;
do
{ptr1 = (uint32_t*)(ptr_reg+(cnt2‹‹1));
*(unsigned short*)ptr1 = swap16(*(unsigned short*)ptr1);
}while(cnt2--);
}
и здесь крашиться в HardFault причем даже в асме подозрений нет.
273: *(unsigned short*)ptr1 = swap16(*(unsigned short*)ptr1);
LDRH r0,[r3,#0x00]
BL.W swap16 (0x08004334)
STRH r0,[r3,#0x00]
274: }while(cnt2--);}
поиск пока ничего не дал. Подскажите куда пинать.
Причем заметил, что ЛЮБОЕ преобразование типа вызывает это прерывание.

[pambaru]

Возможно из-за преобразования uint16_t* в uint32_t*.
Если адрес в ptr_reg не кратен 4.
Попробуйте преобразовать в (__packed uint32_t*).

[pambaru]

Ну, разумеется, при этом ptr1 должна быть объявлена как
__packed uint32_t *ptr1;

Кстати, а ptr2 без знака "*".
Так и надо, или ошибка?

[dgrishin]

Подскажите куда пинать.

ИМХО вы очень сильно злоупотребляете работой с указателями. Самое простое - считать данные по указателю в локальную переменную функции. Поменять байты местами (в локальной переменной) а потом сохранить новое значение по указателю.

[STM32F0]

Компилятор KEIL 5.23.
При приеме данных по UART получаю регистры uint16 или uint32 и перед записью в EEPROM вынужден переворачивать их. И тут начинается странное.

В Ф0 есть причуды при работе с невыравненными данными. В Ф1 может и прокатить, но тоже не всегда. Поэтому меняйте подход в целом.
В кортексах есть команды реверса, но в Ф0 они слегка укошенные. Смотрите в систему команд Ф0 для уточнения.

[j-Roger]

Вот на всякий случай ссылка на STM32F0xxx Cortex-M0 programming manual;

Раздел 3.4.2 LDR and STR, immediate offset
Раздел 3.3.4 Address alignment

[NewWriter]

Экую чухню вы наворотили. Начиная с того, что ptr2 не является указателем, он просто переменная.
Ну и сам обмен байтов в 4-байтной переменной выполяется вот так: (для наглядности пишу пошагово)

  uint32_t abcd = 0x12345678, dcba = 0;
  uint8_t tmp;

  uint8_t *ptr_abcd, *ptr_dcba;

  ptr_abcd = (uint8_t*)&abcd;
  ptr_dcba = (uint8_t*)&dcba;
  ptr_dcba +=3;

  tmp = *(uint8_t*)ptr_abcd++;
  *(uint8_t*)ptr_dcba-- = tmp;

  tmp = *(uint8_t*)ptr_abcd++;
  *(uint8_t*)ptr_dcba-- = tmp;

  tmp = *(uint8_t*)ptr_abcd++;
  *(uint8_t*)ptr_dcba-- = tmp;

  tmp = *(uint8_t*)ptr_abcd;
  *(uint8_t*)ptr_dcba = tmp;

В цикл лучше не заключать - нафик нужна лишняя переменная и работа с ней, если и так всё понятно

Да, если надо определить размерность переменной для "переворота", используем sizeof(), дополнительного параметра размера не надо:

  if (sizeof(abcd) == 4)
  {
	 // ....
  }
  if (sizeof(abcd) == 2)
  {
	 // ....
  }

[H4LF]

kampri, а разве соответствующую инструкцию (rev) нельзя использовать? Или просто нужно чтоб посложнее...?
Раздел 3.5.7 REV, REV16, and REVSH
http://www.st.com/content/ccc/resour...DM00104451.pdf
И есть соответствующие функции в CMSIS:

uint32_t __REV(uint32_t value)
uint32_t __REV16(uint32_t value)
int32_t __REVSH(int32_t value) 

для gcc. И для armcc конечно тоже есть __REV
И вероятность сделать баг на указателях поменьше станет...

[dgrishin]

uint8_t temp;

union swap{
uint32_t temp32;
uint8_t temp8[4];
} var;

var.temp32 = чего_хотим_посвопить;

temp = var.temp8[0];
var.temp8[0] = var.temp8[3];
var.temp8[3] = temp;

temp = var.temp8[1];
var.temp8[1] = var.temp8[2];
var.temp8[2] = temp;

[STM32F0]

Экую чухню вы наворотили.

А сам? "Отпусти меня Микрочип, отпусти!"(с)?

для gcc. И для armcc конечно тоже есть __REV

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

uint8_t temp;

Еще лучше и веселее.