[ST] Add sample port for H745I-DISCO #49
Conversation
hihigupt
requested review from
tawalke,
wduraes,
jspaith,
danewalton-msft and
ericwolz
| * unexpected behaviour, such as calls to malloc(). */ | ||
| int iMainRand32( void ) | ||
| { | ||
| static UBaseType_t uxlNextRand; /*_RB_ Not seeded. */ |
There was a problem hiding this comment.
There was a problem hiding this comment.
I am not able to follow this link ... Could please provide the link to specific line?
There was a problem hiding this comment.
Sorry - it was the already implemented secure random function instead of implementing a pseudo-random function - likely wrong paste.
| @@ -0,0 +1,71 @@ | |||
| /* USER CODE BEGIN Header */ | |||
There was a problem hiding this comment.
I see inconsistent usage of main.h (e.g. in NXP it is checked in but not used).
Should we get the common APIs required by the sample defined here then include in the sample c file (e.g. the Error_Handler, Random and Time PAL functions).
There was a problem hiding this comment.
ST projects are created using CubeMX and that generates main.h, just to keep it similar to ST experience I kept it same.
Regarding NXP I am not seeing any file name main.h in main branch of this repo (https://github.com/Azure-Samples/iot-middleware-freertos-samples/tree/main/demos/projects/NXP/mimxrt1060), am I missing something here?
main.c is more attached to platform and wanted to make sure it remain same to how that platform provides its sample to boot.
There was a problem hiding this comment.
Ah, OK - a prior version or something happened to my fork - I have main.h in NXP but wasn't referenced.
demos/projects/ST/stm32h745i_discovery/cm7/st_code/lwip/system/arch/sys_arch.h
Show resolved
Hide resolved
hihigupt
force-pushed
the
develop_higupt_premain
branch
from
297e045 to
6ae878b
Compare
hihigupt
force-pushed
the
develop_higupt_premain
branch
from
6ae878b to
f10ad61
Compare
No description provided.