diff --git a/src/include/reef/trace.h b/src/include/reef/trace.h
index cd1178b23..4c3157dc9 100644
--- a/src/include/reef/trace.h
+++ b/src/include/reef/trace.h
@@ -108,7 +108,7 @@ void _trace_event_atomic(uint32_t event);
 void _trace_event_mbox_atomic(uint32_t event);
 void trace_flush(void);
 void trace_off(void);
-void trace_init(struct reef * reef);
+void trace_init(struct reef *reef);
 
 #if TRACE
 
diff --git a/src/ipc/ipc.c b/src/ipc/ipc.c
index 0b52cb207..a6ad21bb7 100644
--- a/src/ipc/ipc.c
+++ b/src/ipc/ipc.c
@@ -374,7 +374,7 @@ int ipc_init(struct reef *reef)
 	/* init ipc data */
 	reef->ipc = rzalloc(RZONE_SYS, SOF_MEM_CAPS_RAM, sizeof(*reef->ipc));
 	reef->ipc->comp_data = rzalloc(RZONE_SYS, SOF_MEM_CAPS_RAM,
-		SOF_IPC_MSG_MAX_SIZE);
+				       SOF_IPC_MSG_MAX_SIZE);
 	reef->ipc->dmat = reef->dmat;
 
 	for (i = 0; i < PLATFORM_MAX_STREAMS; i++)
diff --git a/src/lib/interrupt.c b/src/lib/interrupt.c
index 32b623508..58ab6ae37 100644
--- a/src/lib/interrupt.c
+++ b/src/lib/interrupt.c
@@ -82,7 +82,7 @@ void irq_unregister_child(struct irq_parent *parent, int irq)
 	spin_lock(&parent->lock);
 
 	/* does child already exist ? */
-	if (parent->child[REEF_IRQ_BIT(irq)] == NULL)
+	if (!parent->child[REEF_IRQ_BIT(irq)])
 		goto finish;
 
 	/* free child */
@@ -107,7 +107,7 @@ uint32_t irq_enable_child(struct irq_parent *parent, int irq)
 
 	spin_lock(&parent->lock);
 
-	child =parent->child[REEF_IRQ_BIT(irq)];
+	child = parent->child[REEF_IRQ_BIT(irq)];
 
 	/* already enabled ? */
 	if (child->enabled)
@@ -134,7 +134,7 @@ uint32_t irq_disable_child(struct irq_parent *parent, int irq)
 
 	spin_lock(&parent->lock);
 
-	child =parent->child[REEF_IRQ_BIT(irq)];
+	child = parent->child[REEF_IRQ_BIT(irq)];
 
 	/* already disabled ? */
 	if (!child->enabled)