summaryrefslogtreecommitdiff
path: root/src/soc/intel
diff options
context:
space:
mode:
authorLee Leahy <leroy.p.leahy@intel.com>2017-03-09 10:00:30 -0800
committerMartin Roth <martinroth@google.com>2017-03-13 17:44:24 +0100
commit4430f9f75d81007f8748327526522c8c774d10b3 (patch)
tree348d843dee0c034c55f7ed496bcf0d7cc56ecc1f /src/soc/intel
parent1d20fe77cb854db420d46bb39b01b53e1e02d37c (diff)
downloadcoreboot-4430f9f75d81007f8748327526522c8c774d10b3.tar.xz
soc/intel/apollolake: Remove unnecessary braces
Fix the following warnings detected by checkpatch.pl: WARNING: braces {} are not necessary for any arm of this statement WARNING: braces {} are not necessary for single statement blocks TEST=Build for reef Change-Id: Ifab09c023faa7da215945f1aedd391f4b2a1a04c Signed-off-by: Lee Leahy <leroy.p.leahy@intel.com> Reviewed-on: https://review.coreboot.org/18723 Tested-by: build bot (Jenkins) Reviewed-by: Philippe Mathieu-Daudé <philippe.mathieu.daude@gmail.com> Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/soc/intel')
-rw-r--r--src/soc/intel/apollolake/chip.c5
-rw-r--r--src/soc/intel/apollolake/lpc.c6
-rw-r--r--src/soc/intel/apollolake/northbridge.c3
3 files changed, 5 insertions, 9 deletions
diff --git a/src/soc/intel/apollolake/chip.c b/src/soc/intel/apollolake/chip.c
index 9ac047e142..eacf554105 100644
--- a/src/soc/intel/apollolake/chip.c
+++ b/src/soc/intel/apollolake/chip.c
@@ -154,11 +154,10 @@ static struct device_operations cpu_bus_ops = {
static void enable_dev(device_t dev)
{
/* Set the operations if it is a special bus type */
- if (dev->path.type == DEVICE_PATH_DOMAIN) {
+ if (dev->path.type == DEVICE_PATH_DOMAIN)
dev->ops = &pci_domain_ops;
- } else if (dev->path.type == DEVICE_PATH_CPU_CLUSTER) {
+ else if (dev->path.type == DEVICE_PATH_CPU_CLUSTER)
dev->ops = &cpu_bus_ops;
- }
}
/*
diff --git a/src/soc/intel/apollolake/lpc.c b/src/soc/intel/apollolake/lpc.c
index 5455dedf1d..810c4c9af3 100644
--- a/src/soc/intel/apollolake/lpc.c
+++ b/src/soc/intel/apollolake/lpc.c
@@ -117,9 +117,8 @@ static void loop_resources(struct device *dev)
for (res = dev->resource_list; res; res = res->next) {
- if (res->flags & IORESOURCE_IO) {
+ if (res->flags & IORESOURCE_IO)
lpc_open_pmio_window(res->base, res->size);
- }
if (res->flags & IORESOURCE_MEM) {
/* Check if this is already decoded. */
@@ -143,9 +142,8 @@ static void set_child_resources(struct device *dev)
struct device *child;
for (link = dev->link_list; link; link = link->next) {
- for (child = link->children; child; child = child->sibling) {
+ for (child = link->children; child; child = child->sibling)
loop_resources(child);
- }
}
}
diff --git a/src/soc/intel/apollolake/northbridge.c b/src/soc/intel/apollolake/northbridge.c
index db595f5614..e1b54ec393 100644
--- a/src/soc/intel/apollolake/northbridge.c
+++ b/src/soc/intel/apollolake/northbridge.c
@@ -81,9 +81,8 @@ static int mc_add_imr_resources(device_t dev, int index)
base = read32(mchbar + imr_offset + MCHBAR_IMR0BASE);
mask = read32(mchbar + imr_offset + MCHBAR_IMR0MASK);
- if (is_imr_enabled(base)) {
+ if (is_imr_enabled(base))
imr_resource(dev, index++, base, mask);
- }
}
return index;