Skip to content

Commit e66f9ee

Browse files
bulislaw0xc0170
authored andcommitted
Fix IAR coding style and thread/mutex namingin mbed_boot.c
1 parent b01f13d commit e66f9ee

File tree

1 file changed

+52
-48
lines changed

1 file changed

+52
-48
lines changed

rtos/mbed_boot.c

Lines changed: 52 additions & 48 deletions
Original file line numberDiff line numberDiff line change
@@ -342,15 +342,15 @@ void _main_init (void) __attribute__((section(".ARM.Collect$$$$000000FF")));
342342
void $Super$$__cpp_initialize__aeabi_(void);
343343

344344
void _main_init (void) {
345-
mbed_set_stack_heap();
346-
/* Copy the vector table to RAM only if uVisor is not in use. */
345+
mbed_set_stack_heap();
346+
/* Copy the vector table to RAM only if uVisor is not in use. */
347347
#if !(defined(FEATURE_UVISOR) && defined(TARGET_UVISOR_SUPPORTED))
348-
mbed_cpy_nvic();
348+
mbed_cpy_nvic();
349349
#endif
350-
mbed_sdk_init();
351-
osKernelInitialize();
352-
mbed_start_main();
353-
for (;;);
350+
mbed_sdk_init();
351+
osKernelInitialize();
352+
mbed_start_main();
353+
for (;;);
354354
}
355355

356356
void $Sub$$__cpp_initialize__aeabi_(void)
@@ -362,13 +362,14 @@ void $Sub$$__cpp_initialize__aeabi_(void)
362362

363363
void pre_main()
364364
{
365-
singleton_mutex_attr.attr_bits = osMutexRecursive;
366-
singleton_mutex_attr.cb_size = sizeof(singleton_mutex_obj);
367-
singleton_mutex_attr.cb_mem = &singleton_mutex_obj;
368-
singleton_mutex_id = osMutexNew(&singleton_mutex_attr);
365+
singleton_mutex_attr.name = "singleton_mutex";
366+
singleton_mutex_attr.attr_bits = osMutexRecursive;
367+
singleton_mutex_attr.cb_size = sizeof(singleton_mutex_obj);
368+
singleton_mutex_attr.cb_mem = &singleton_mutex_obj;
369+
singleton_mutex_id = osMutexNew(&singleton_mutex_attr);
369370

370-
$Super$$__cpp_initialize__aeabi_();
371-
main();
371+
$Super$$__cpp_initialize__aeabi_();
372+
main();
372373
}
373374

374375
#else /******************** ARMC ********************/
@@ -540,14 +541,15 @@ void pre_main(void)
540541
#pragma required=__vector_table
541542
void __iar_program_start( void )
542543
{
543-
__iar_init_core();
544-
__iar_init_vfp();
544+
__iar_init_core();
545+
__iar_init_vfp();
545546

546-
uint8_t low_level_init_needed_local;
547+
uint8_t low_level_init_needed_local;
548+
549+
low_level_init_needed_local = __low_level_init();
550+
if (low_level_init_needed_local) {
551+
__iar_data_init3();
547552

548-
low_level_init_needed_local = __low_level_init();
549-
if (low_level_init_needed_local) {
550-
__iar_data_init3();
551553
/* Copy the vector table to RAM only if uVisor is not in use. */
552554
#if !(defined(FEATURE_UVISOR) && defined(TARGET_UVISOR_SUPPORTED))
553555
mbed_cpy_nvic();
@@ -574,71 +576,73 @@ static mbed_rtos_storage_mutex_t std_mutex_file[_FOPEN_MAX] = {0};
574576

575577
void __iar_system_Mtxinit(__iar_Rmtx *mutex) /* Initialize a system lock */
576578
{
577-
osMutexAttr_t attr;
578-
uint32_t index;
579-
for (index = 0; index < _MAX_LOCK; index++) {
580-
if (0 == std_mutex_id_sys[index]) {
581-
attr.cb_mem = &std_mutex_sys[index];
582-
attr.cb_size = sizeof(std_mutex_sys[index]);
583-
attr.attr_bits = osMutexRecursive;
584-
std_mutex_id_sys[index] = osMutexNew(&attr);
585-
*mutex = (__iar_Rmtx*)&std_mutex_id_sys[index];
586-
return;
579+
osMutexAttr_t attr;
580+
uint32_t index;
581+
for (index = 0; index < _MAX_LOCK; index++) {
582+
if (0 == std_mutex_id_sys[index]) {
583+
attr.name = "system_mutex";
584+
attr.cb_mem = &std_mutex_sys[index];
585+
attr.cb_size = sizeof(std_mutex_sys[index]);
586+
attr.attr_bits = osMutexRecursive;
587+
std_mutex_id_sys[index] = osMutexNew(&attr);
588+
*mutex = (__iar_Rmtx*)&std_mutex_id_sys[index];
589+
return;
590+
}
587591
}
588-
}
589592

590-
/* This should never happen */
591-
error("Not enough mutexes\n");
593+
/* This should never happen */
594+
error("Not enough mutexes\n");
592595
}
593596

594597
void __iar_system_Mtxdst(__iar_Rmtx *mutex) /* Destroy a system lock */
595598
{
596-
osMutexDelete(*(osMutexId_t*)*mutex);
597-
*mutex = 0;
599+
osMutexDelete(*(osMutexId_t*)*mutex);
600+
*mutex = 0;
598601
}
599602

600603
void __iar_system_Mtxlock(__iar_Rmtx *mutex) /* Lock a system lock */
601604
{
602-
osMutexAcquire(*(osMutexId_t*)*mutex, osWaitForever);
605+
osMutexAcquire(*(osMutexId_t*)*mutex, osWaitForever);
603606
}
604607

605608
void __iar_system_Mtxunlock(__iar_Rmtx *mutex) /* Unlock a system lock */
606609
{
607-
osMutexRelease(*(osMutexId_t*)*mutex);
610+
osMutexRelease(*(osMutexId_t*)*mutex);
608611
}
609612

610613
void __iar_file_Mtxinit(__iar_Rmtx *mutex) /* Initialize a file lock */
611614
{
612615
osMutexAttr_t attr;
613616
uint32_t index;
614617
for (index = 0; index < _FOPEN_MAX; index++) {
615-
if (0 == std_mutex_id_file[index]) {
616-
attr.cb_mem = &std_mutex_file[index];
617-
attr.cb_size = sizeof(std_mutex_file[index]);
618-
attr.attr_bits = osMutexRecursive;
619-
std_mutex_id_file[index] = osMutexNew(&attr);
620-
*mutex = (__iar_Rmtx*)&std_mutex_id_file[index];
621-
return;
622-
}
618+
if (0 == std_mutex_id_file[index]) {
619+
attr.name = "file_mutex";
620+
attr.cb_mem = &std_mutex_file[index];
621+
attr.cb_size = sizeof(std_mutex_file[index]);
622+
attr.attr_bits = osMutexRecursive;
623+
std_mutex_id_file[index] = osMutexNew(&attr);
624+
*mutex = (__iar_Rmtx*)&std_mutex_id_file[index];
625+
return;
626+
}
623627
}
624628
/* The variable _FOPEN_MAX needs to be increased */
625629
error("Not enough mutexes\n");
626630
}
627631

628632
void __iar_file_Mtxdst(__iar_Rmtx *mutex) /* Destroy a file lock */
629633
{
630-
osMutexDelete(*(osMutexId_t*)*mutex);
631-
*mutex = 0;
634+
osMutexDelete(*(osMutexId_t*)*mutex);
635+
*mutex = 0;
632636
}
633637

634638
void __iar_file_Mtxlock(__iar_Rmtx *mutex) /* Lock a file lock */
635639
{
636-
osMutexAcquire(*(osMutexId_t*)*mutex, osWaitForever);
640+
osMutexAcquire(*(osMutexId_t*)*mutex, osWaitForever);
637641
}
638642

639643
void __iar_file_Mtxunlock(__iar_Rmtx *mutex) /* Unlock a file lock */
640644
{
641-
osMutexRelease(*(osMutexId_t*)*mutex);
645+
osMutexRelease(*(osMutexId_t*)*mutex);
642646
}
643647

644648
#endif

0 commit comments

Comments
 (0)