Fix warnings with GCC11. cherry-pick:main

Author: tyler-gilbert

include/cortexm/task_types.h

@@ -46,7 +46,7 @@ typedef struct {
   int pid /*! The process ID assigned to the new thread */;
 } task_thread_t;
 
-typedef struct MCU_PACK {
+typedef struct {
   volatile void *sp /*! The task stack pointer */;
   int pid /*! The process id */;
   volatile s8 priority /*! Task priority */;

include/mcu/mmc.h

@@ -20,7 +20,7 @@ typedef struct MCU_PACK {
   u32 value;
 } mmc_event_data_t;
 
-typedef struct MCU_PACK {
+typedef struct {
   mmc_attr_t attr; // default attributes
   u32 port;
 } mmc_config_t;

include/mcu/sdio.h

@@ -20,7 +20,7 @@ typedef struct MCU_PACK {
   u32 value;
 } sdio_event_data_t;
 
-typedef struct MCU_PACK {
+typedef struct {
   sdio_attr_t attr; // default attributes
   u32 port;
 } sdio_config_t;

include/sos/link/types.h

@@ -303,7 +303,7 @@ typedef u32 link_trace_event_id_t;
 typedef u32 link_trace_event_set_t;
 typedef u32 link_mqd_t;
 
-typedef struct MCU_PACK {
+typedef struct {
   u32 status;
   link_trace_event_set_t filter;
   link_trace_attr_t attr;

include/sos/symbols.h

@@ -70,7 +70,7 @@ extern u32 __aeabi_f2uiz();
 extern u32 __aeabi_f2ulz();
 extern u32 __clzsi2();
 extern u32 __clzdi2();
-extern u32 __muldi3();
+extern long long int __muldi3(int,int);
 extern u32 __negdi2();
 extern u32 __cmpdi2();
 extern u32 __ucmpdi2();
@@ -192,9 +192,6 @@ useconds_t _EXFUN(ualarm, (useconds_t __useconds, useconds_t __interval));
 extern void crt_load_data(void * global_reent, int code_size, int data_size);
 extern char ** const crt_import_argv(int argc, char * const argv[]);
 
-extern void sg_api();
-extern void son_api();
-
 #include "sys/socket.h"
 
 #ifndef USE_FLOAT

include/sos/trace.h

@@ -8,14 +8,14 @@ extern "C" {
 #endif
 
 #define SOS_TRACE_MESSAGE(msg)                                                           \
-  sos_trace_event(LINK_POSIX_TRACE_MESSAGE, msg, strnlen(msg, LINK_POSIX_TRACE_DATA_SIZE))
+  sos_trace_event(LINK_POSIX_TRACE_MESSAGE, msg, sizeof(msg))
 #define SOS_TRACE_WARNING(msg)                                                           \
-  sos_trace_event(LINK_POSIX_TRACE_WARNING, msg, strnlen(msg, LINK_POSIX_TRACE_DATA_SIZE))
+  sos_trace_event(LINK_POSIX_TRACE_WARNING, msg, sizeof(msg))
 #define SOS_TRACE_CRITICAL(msg)                                                          \
   sos_trace_event(                                                                       \
-    LINK_POSIX_TRACE_CRITICAL, msg, strnlen(msg, LINK_POSIX_TRACE_DATA_SIZE))
+    LINK_POSIX_TRACE_CRITICAL, msg, sizeof(msg))
 #define SOS_TRACE_FATAL(msg)                                                             \
-  sos_trace_event(LINK_POSIX_TRACE_FATAL, msg, strnlen(msg, LINK_POSIX_TRACE_DATA_SIZE))
+  sos_trace_event(LINK_POSIX_TRACE_FATAL, msg, sizeof(msg))
 
 void sos_trace_event(
   link_trace_event_id_t event_id,

src/sys/sysfs/sysfs.c

@@ -126,7 +126,7 @@ const sysfs_t *sysfs_find(const char *path, bool needs_parent) {
 }
 
 const char *sysfs_stripmountpath(const sysfs_t *fs, const char *path) {
-  path = path + strnlen(fs->mount_path, NAME_MAX);
+  path = path + strnlen(fs->mount_path, SYSFS_MOUNT_PATH_MAX);
   if (path[0] == '/') {
     path++;
   }

src/sys/time/_settimeofday.c

@@ -70,15 +70,15 @@ int settimeofday(const struct timeval *tp, const struct timezone *tzp){
 
 int settimeofday_rtc(const struct timeval *tp) {
   int fd;
-  rtc_time_t cal_time;
 
   fd = open("/dev/rtc", O_RDWR);
   if (fd < 0) {
     return -1;
   }
 
-  gmtime_r((time_t *)&tp, (struct tm *)&cal_time.time);
-  cal_time.useconds = tp->tv_usec;
+  struct link_tm time_struct;
+  gmtime_r((time_t *)&tp, (struct tm *)&time_struct);
+  rtc_time_t cal_time = {.time = time_struct, .useconds = tp->tv_usec};
 
   if (ioctl(fd, I_RTC_SET, &cal_time) < 0) {
     close(fd);

src/sys/time/hibernate.c

@@ -71,9 +71,12 @@ int set_alarm(int seconds) {
 
   fd = open("/dev/rtc", O_RDWR);
   if (fd >= 0) {
-    attr.time.time_t = time(NULL);
+    time_t t = time(NULL);
+    attr.time.time_t = t;
     attr.time.time_t += seconds;
-    gmtime_r((time_t *)&attr.time.time_t, (struct tm *)&attr.time.time);
+    struct link_tm time_struct;
+    gmtime_r(&t, (struct tm *)&time_struct);
+    attr.time.time = time_struct;
     attr.o_flags = RTC_FLAG_IS_ALARM_ONCE | RTC_FLAG_ENABLE_ALARM;
 
     // set the alarm for "seconds" from now

src/sys/trace/posix_trace.c

@@ -68,9 +68,9 @@ static trace_id_handle_t *trace_get_ptr(trace_id_t id) {
   return 0;
 }
 
-static uint32_t calc_checksum(trace_id_t id) {
-  uint32_t check;
-  uint32_t *ptr = (uint32_t *)id;
+static u32 calc_checksum(trace_id_t id) {
+  u32 check;
+  u32 *ptr = (u32 *)id;
   int i;
   check = 0xAA55AA55;
   // go over handle excluding checksum value
@@ -210,8 +210,10 @@ int posix_trace_create(pid_t pid, const trace_attr_t *attr, trace_id_t *id) {
   trace_handle.attr = tmp_attr;
 
   // populate the timestamp
-  clock_gettime(CLOCK_REALTIME, (struct timespec *)&(trace_handle.attr.create_time));
-
+  struct timespec clock_time;
+  clock_gettime(CLOCK_REALTIME, (struct timespec *)&clock_time);
+  trace_handle.attr.create_time.tv_sec = clock_time.tv_sec;
+  trace_handle.attr.create_time.tv_nsec = clock_time.tv_nsec;
   update_checksum(&trace_handle);
 
   *id = trace_find_free();