treewide: apply clang-format

Signed-off-by: Nick Schonning <nschonni@gmail.com>

Author: nschonni

kernel/arch/arm/arch.c

@@ -95,18 +95,18 @@ struct pbl_kernel_debug_layout {
 };
 
 const struct pbl_kernel_debug_layout pbl_kernel_debug_layout __attribute__((used)) = {
-  .version = 1,
-  .thread_sp = offsetof(struct pbl_thread, backend.sp),
-  .thread_all_next = offsetof(struct pbl_thread, backend.all_next),
-  .thread_state = offsetof(struct pbl_thread, backend.state),
-  .thread_id = offsetof(struct pbl_thread, id),
-  .thread_name = offsetof(struct pbl_thread, name),
-  .thread_name_len = PBL_THREAD_NAME_LEN,
-  .ctx_control = offsetof(struct saved_context, control),
-  .ctx_r4 = offsetof(struct saved_context, r4_r11),
-  .ctx_exc_return = offsetof(struct saved_context, exc_return),
-  .ctx_hw = offsetof(struct saved_context, r0),
-  .ctx_fp_extra = NUM_EXTRA_FP_REGS * sizeof(uint32_t),
+    .version = 1,
+    .thread_sp = offsetof(struct pbl_thread, backend.sp),
+    .thread_all_next = offsetof(struct pbl_thread, backend.all_next),
+    .thread_state = offsetof(struct pbl_thread, backend.state),
+    .thread_id = offsetof(struct pbl_thread, id),
+    .thread_name = offsetof(struct pbl_thread, name),
+    .thread_name_len = PBL_THREAD_NAME_LEN,
+    .ctx_control = offsetof(struct saved_context, control),
+    .ctx_r4 = offsetof(struct saved_context, r4_r11),
+    .ctx_exc_return = offsetof(struct saved_context, exc_return),
+    .ctx_hw = offsetof(struct saved_context, r0),
+    .ctx_fp_extra = NUM_EXTRA_FP_REGS * sizeof(uint32_t),
 };
 
 _Static_assert(offsetof(struct pbl_thread, backend.sp) == 0, "saved SP must be first");
@@ -119,7 +119,9 @@ static inline bool prv_fp_active(uint32_t exc_return) {
 
 // ---- threads ----------------------------------------------------------------
 
-static void prv_thread_return(void) { pbl_thread_abort(NULL); }
+static void prv_thread_return(void) {
+  pbl_thread_abort(NULL);
+}
 
 void arch_thread_init(struct pbl_thread *t, void (*entry)(void *), void *arg) {
   uintptr_t top = ((uintptr_t)t->stack + t->stack_size) & ~7u;
@@ -164,7 +166,9 @@ void arch_thread_regions_set(struct pbl_thread *t, const MpuRegion *const *regio
 #endif
 }
 
-void arch_thread_aborted(struct pbl_thread *t) { (void)t; }
+void arch_thread_aborted(struct pbl_thread *t) {
+  (void)t;
+}
 
 void arch_thread_exit(void) {
   // The switch requested by the abort takes over as soon as we get here.
@@ -175,15 +179,19 @@ void arch_thread_exit(void) {
 
 // ---- interrupts -------------------------------------------------------------
 
-bool arch_in_isr(void) { return mcu_state_is_isr(); }
+bool arch_in_isr(void) {
+  return mcu_state_is_isr();
+}
 
 void arch_irq_disable(void) {
   __set_BASEPRI(PBL_IRQ_PRIO_MAX_SYSCALL);
   __DSB();
   __ISB();
 }
 
-void arch_irq_enable(void) { __set_BASEPRI(0); }
+void arch_irq_enable(void) {
+  __set_BASEPRI(0);
+}
 
 void arch_switch_request(void) {
   SCB->ICSR = SCB_ICSR_PENDSVSET_Msk;
@@ -213,7 +221,7 @@ void arch_switch_request(void) {
 #endif
 
 // r1 -> thread's MPU words; clobbers r2, r4-r11.
-#define RESTORE_MPU                                                          \
+#define RESTORE_MPU \
   "  ldr r2, =0xe000ed98 \n"            /* MPU_RNR */                      \
   "  mov r4, #" STR(FIRST_MPU_REGION) " \n"                                  \
   "  str r4, [r2] \n"                                                        \
@@ -226,45 +234,38 @@ __attribute__((naked)) void PendSV_Handler(void) {
       "  mrs r0, psp \n"
       "  isb \n"
       "  ldr r3, =pbl_cur \n"
-      "  ldr r2, [r3] \n"
-      SAVE_FP
-      SAVE_REGS
-      "  str r0, [r2] \n"              /* pbl_cur->backend.sp */
+      "  ldr r2, [r3] \n" SAVE_FP SAVE_REGS
+      "  str r0, [r2] \n" /* pbl_cur->backend.sp */
       "  stmdb sp!, {r3, r14} \n"
       "  mov r0, %0 \n"
       "  msr basepri, r0 \n"
       "  dsb \n"
       "  isb \n"
-      "  bl sched_switch_in \n"        /* returns the next thread */
+      "  bl sched_switch_in \n" /* returns the next thread */
       "  mov r4, r0 \n"
       "  mov r0, #0 \n"
       "  msr basepri, r0 \n"
       "  ldmia sp!, {r3, r14} \n"
-      "  ldr r0, [r4] \n"              /* next->backend.sp */
-      "  add r1, r4, #4 \n"            /* next->backend.arch.mpu */
-      RESTORE_MPU
-      RESTORE_REGS
-      RESTORE_FP
+      "  ldr r0, [r4] \n"   /* next->backend.sp */
+      "  add r1, r4, #4 \n" /* next->backend.arch.mpu */
+      RESTORE_MPU RESTORE_REGS RESTORE_FP
       "  msr psp, r0 \n"
       "  isb \n"
       "  bx r14 \n"
-      "  .ltorg \n"
-      ::"i"(PBL_IRQ_PRIO_MAX_SYSCALL));
+      "  .ltorg \n" ::"i"(PBL_IRQ_PRIO_MAX_SYSCALL));
 }
 
 // Loads the first thread's context. MSP is reset to the top of the ISR stack.
 __attribute__((naked)) static void prv_restore_first_thread(void) {
   __asm volatile(
-      "  ldr r0, =0xE000ED08 \n"       /* VTOR: initial MSP is the first vector */
+      "  ldr r0, =0xE000ED08 \n" /* VTOR: initial MSP is the first vector */
       "  ldr r0, [r0] \n"
       "  ldr r0, [r0] \n"
       "  msr msp, r0 \n"
       "  ldr r3, =pbl_cur \n"
       "  ldr r4, [r3] \n"
       "  ldr r0, [r4] \n"
-      "  add r1, r4, #4 \n"
-      RESTORE_MPU
-      RESTORE_REGS
+      "  add r1, r4, #4 \n" RESTORE_MPU RESTORE_REGS
       "  msr psp, r0 \n"
       "  mov r0, #0 \n"
       "  msr basepri, r0 \n"
@@ -367,7 +368,8 @@ __attribute__((naked)) void SVC_Handler(void) {
 
 // ---- start and tick ---------------------------------------------------------
 
-void arch_init(void) {}
+void arch_init(void) {
+}
 
 void arch_start(void) {
   // PendSV and SysTick at the lowest priority; SVC follows once started.
@@ -392,7 +394,9 @@ void arch_start(void) {
 }
 
 #ifndef CONFIG_SOC_SF32LB52
-void SysTick_Handler(void) { pbl_kernel_tick_isr(); }
+void SysTick_Handler(void) {
+  pbl_kernel_tick_isr();
+}
 #endif
 
 void arch_idle(pbl_tick_t max_ticks) {
@@ -408,16 +412,16 @@ void arch_thread_saved_regs(const struct pbl_thread *t, struct pbl_thread_saved_
   if (t == pbl_cur && arch_in_isr()) {
     // The running thread's registers are live on its stack, not saved.
     const uint32_t *frame = (const uint32_t *)__get_PSP();
-    *regs = (struct pbl_thread_saved_regs){
-      .pc = frame[6], .lr = frame[5], .control = __get_CONTROL() };
+    *regs =
+        (struct pbl_thread_saved_regs){.pc = frame[6], .lr = frame[5], .control = __get_CONTROL()};
     return;
   }
   const struct saved_context *ctx = t->backend.sp;
   const uint32_t *hw = &ctx->r0;
   if (prv_fp_active(ctx->exc_return)) {
     hw += NUM_EXTRA_FP_REGS;
   }
-  *regs = (struct pbl_thread_saved_regs){ .pc = hw[6], .lr = hw[5], .control = ctx->control };
+  *regs = (struct pbl_thread_saved_regs){.pc = hw[6], .lr = hw[5], .control = ctx->control};
 }
 
 void arch_thread_info_regs(const struct pbl_thread *t, uint32_t regs[PBL_THREAD_REG_COUNT]) {

kernel/arch/posix/arch.c

@@ -53,7 +53,8 @@ static void *prv_thread_main(void *arg) {
   return NULL;
 }
 
-void arch_init(void) {}
+void arch_init(void) {
+}
 
 void arch_thread_init(struct pbl_thread *t, void (*entry)(void *), void *arg) {
   pthread_cond_init(&t->backend.arch.wake, NULL);
@@ -97,7 +98,9 @@ static void prv_switch(void) {
   }
 }
 
-void arch_switch_request(void) { s_switch_pending = true; }
+void arch_switch_request(void) {
+  s_switch_pending = true;
+}
 
 void arch_thread_exit(void) {
   // pbl_thread_abort() already switched away and ended this pthread.
@@ -111,9 +114,12 @@ void arch_thread_aborted(struct pbl_thread *t) {
   }
 }
 
-bool arch_in_isr(void) { return s_in_isr; }
+bool arch_in_isr(void) {
+  return s_in_isr;
+}
 
-void arch_irq_disable(void) {}
+void arch_irq_disable(void) {
+}
 
 void arch_irq_enable(void) {
   if (!s_in_isr && s_switch_pending && pbl_kernel_is_started()) {
@@ -123,7 +129,7 @@ void arch_irq_enable(void) {
 
 void arch_thread_saved_regs(const struct pbl_thread *t, struct pbl_thread_saved_regs *regs) {
   (void)t;
-  *regs = (struct pbl_thread_saved_regs){ 0 };
+  *regs = (struct pbl_thread_saved_regs){0};
 }
 
 void arch_thread_info_regs(const struct pbl_thread *t, uint32_t regs[PBL_THREAD_REG_COUNT]) {
@@ -207,7 +213,9 @@ void pbl_test_kernel_stop(void) {
   }
 }
 
-void pbl_test_isr_enter(void) { s_in_isr = true; }
+void pbl_test_isr_enter(void) {
+  s_in_isr = true;
+}
 
 void pbl_test_isr_exit(void) {
   s_in_isr = false;

kernel/debug.c

@@ -10,10 +10,10 @@
 void pbl_thread_foreach(pbl_thread_info_fn fn, void *ctx) {
   for (struct pbl_thread *t = pbl_all_threads; t != NULL; t = t->backend.all_next) {
     struct pbl_thread_info info = {
-      .thread = t,
-      .name = t->name,
-      .id = (uintptr_t)t,
-      .current = (t == pbl_cur),
+        .thread = t,
+        .name = t->name,
+        .id = (uintptr_t)t,
+        .current = (t == pbl_cur),
     };
     if (!info.current) {
       arch_thread_info_regs(t, info.regs);
@@ -33,9 +33,9 @@ void pbl_thread_stack_info(const struct pbl_thread *t, struct pbl_thread_stack_i
     untouched++;
   }
   *info = (struct pbl_thread_stack_info){
-    .start = (uintptr_t)t->stack,
-    .size = t->stack_size,
-    .high_water = untouched * sizeof(uint32_t),
+      .start = (uintptr_t)t->stack,
+      .size = t->stack_size,
+      .high_water = untouched * sizeof(uint32_t),
   };
 }
 
@@ -50,12 +50,12 @@ size_t pbl_thread_stats_snapshot(struct pbl_thread_stats *out, size_t max,
     struct pbl_thread_stack_info stack;
     pbl_thread_stack_info(t, &stack);
     out[n++] = (struct pbl_thread_stats){
-      .thread = t,
-      .name = t->name,
-      .number = t->backend.number,
-      .run_time = t->backend.run_time,
-      .stack_high_water = stack.high_water,
-      .state = (enum pbl_thread_state)t->backend.state,
+        .thread = t,
+        .name = t->name,
+        .number = t->backend.number,
+        .run_time = t->backend.run_time,
+        .stack_high_water = stack.high_water,
+        .state = (enum pbl_thread_state)t->backend.state,
     };
   }
   pbl_irq_unlock();

kernel/irq.c

@@ -17,8 +17,14 @@ void pbl_irq_unlock(void) {
   }
 }
 
-void irq_reset(void) { s_nesting = 0; }
+void irq_reset(void) {
+  s_nesting = 0;
+}
 
-bool pbl_in_isr(void) { return arch_in_isr(); }
+bool pbl_in_isr(void) {
+  return arch_in_isr();
+}
 
-bool pbl_irq_is_locked(void) { return s_nesting > 0; }
+bool pbl_irq_is_locked(void) {
+  return s_nesting > 0;
+}

kernel/msgq.c

@@ -13,7 +13,9 @@ void pbl_msgq_init(struct pbl_msgq *q, void *buf, size_t msg_size, uint32_t max_
   *q = (struct pbl_msgq)PBL_MSGQ_INITIALIZER(buf, msg_size, max_msgs);
 }
 
-void pbl_msgq_deinit(struct pbl_msgq *q) { KERNEL_ASSERT(q->group == NULL); }
+void pbl_msgq_deinit(struct pbl_msgq *q) {
+  KERNEL_ASSERT(q->group == NULL);
+}
 
 static uint8_t *prv_slot(struct pbl_msgq *q, uint32_t index) {
   return (uint8_t *)q->buf + index * q->msg_size;
@@ -114,6 +116,10 @@ void pbl_msgq_purge(struct pbl_msgq *q) {
   pbl_irq_unlock();
 }
 
-uint32_t pbl_msgq_num_used(const struct pbl_msgq *q) { return q->backend.count; }
+uint32_t pbl_msgq_num_used(const struct pbl_msgq *q) {
+  return q->backend.count;
+}
 
-uint32_t pbl_msgq_num_free(const struct pbl_msgq *q) { return q->max_msgs - q->backend.count; }
+uint32_t pbl_msgq_num_free(const struct pbl_msgq *q) {
+  return q->max_msgs - q->backend.count;
+}

kernel/mutex.c

@@ -5,9 +5,13 @@
 
 #include "kernel.h"
 
-void pbl_mutex_init(struct pbl_mutex *m) { *m = (struct pbl_mutex)PBL_MUTEX_INITIALIZER; }
+void pbl_mutex_init(struct pbl_mutex *m) {
+  *m = (struct pbl_mutex)PBL_MUTEX_INITIALIZER;
+}
 
-void pbl_mutex_deinit(struct pbl_mutex *m) { KERNEL_ASSERT(m->count == 0); }
+void pbl_mutex_deinit(struct pbl_mutex *m) {
+  KERNEL_ASSERT(m->count == 0);
+}
 
 int pbl_mutex_lock_lr(struct pbl_mutex *m, pbl_timeout_t timeout, uintptr_t lr) {
   KERNEL_ASSERT(!arch_in_isr());

kernel/poll.c

@@ -27,7 +27,8 @@ void pbl_poll_group_add(struct pbl_poll_group *g, struct pbl_msgq *q) {
 // queue cannot starve the others.
 static struct pbl_msgq *prv_ready_member(struct pbl_poll_group *g) {
   struct pbl_msgq *cursor = g->backend.cursor;
-  for (struct pbl_msgq *m = cursor ? cursor->group_next : g->members; m != NULL; m = m->group_next) {
+  for (struct pbl_msgq *m = cursor ? cursor->group_next : g->members; m != NULL;
+       m = m->group_next) {
     if (m->backend.count > 0) {
       g->backend.cursor = m;
       return m;