newaetech/chipwhisperer

Small issue with building CW308_STM32F3 HAL with ARM-GCC 12.2.0

Closed this issue · 2 comments

desertsagebrush commented

Heya,

I was just (re)starting some work involving the chipwhisperer, and I found that I wasn't able to build the simpleserial_base target firmware (develop, commit 276df16), as the build was failing in hal/stm32f3/stm32f3_sysmem.c because the compiler didn't understand caddr_t as a type.

For me, that is defined in sys/types.h, and adding an include for that in that file allows the firmware to successfully build (with a warning about the output ELF file having a load section with rwx permissions). Precise GCC version is

arm-none-eabi-gcc (Arch Repository) 12.2.0

I'm glad to propose a pull request with this change if you think that it would be useful to merge/wouldn't break anything else (and I would also go through the other STM32 HALs to make sure that the issue doesn't show up there as well).

alex-dewar commented

Hi,

Thanks for flagging this. I've replaced caddr_t with char * in 3420466. Let me know if that works for you.

desertsagebrush commented

Looks good to me! Thanks for the quick fix!

(It still has the RWX warning, but that is a different issue and probably can just be ignored)

  • 🎉1