From a8bab6df818ac0c95ae39a5a68e1e17009609366 Mon Sep 17 00:00:00 2001 From: Pierre-Louis Bossart Date: Wed, 4 Apr 2018 18:46:49 -0500 Subject: [PATCH] clean: fix style warning before big SOF rename Remove checkpatch warnings before the rename to better track real ones Signed-off-by: Pierre-Louis Bossart --- src/include/reef/trace.h | 2 +- src/ipc/ipc.c | 2 +- src/lib/interrupt.c | 6 +++--- 3 files changed, 5 insertions(+), 5 deletions(-) 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)