PCI: exynos: Remove unnecessary return statement
This patch fixes the following checkpatch warning: WARNING: void function return statements are not generally useful Signed-off-by: Jingoo Han <jg1.han@samsung.com> Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
This commit is contained in:
parent
70b3e89aaf
commit
bc512c5a8c
|
@ -312,7 +312,6 @@ static void exynos_pcie_assert_reset(struct pcie_port *pp)
|
||||||
if (exynos_pcie->reset_gpio >= 0)
|
if (exynos_pcie->reset_gpio >= 0)
|
||||||
devm_gpio_request_one(pp->dev, exynos_pcie->reset_gpio,
|
devm_gpio_request_one(pp->dev, exynos_pcie->reset_gpio,
|
||||||
GPIOF_OUT_INIT_HIGH, "RESET");
|
GPIOF_OUT_INIT_HIGH, "RESET");
|
||||||
return;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static int exynos_pcie_establish_link(struct pcie_port *pp)
|
static int exynos_pcie_establish_link(struct pcie_port *pp)
|
||||||
|
@ -388,7 +387,6 @@ static void exynos_pcie_clear_irq_pulse(struct pcie_port *pp)
|
||||||
|
|
||||||
val = exynos_elb_readl(exynos_pcie, PCIE_IRQ_PULSE);
|
val = exynos_elb_readl(exynos_pcie, PCIE_IRQ_PULSE);
|
||||||
exynos_elb_writel(exynos_pcie, val, PCIE_IRQ_PULSE);
|
exynos_elb_writel(exynos_pcie, val, PCIE_IRQ_PULSE);
|
||||||
return;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static void exynos_pcie_enable_irq_pulse(struct pcie_port *pp)
|
static void exynos_pcie_enable_irq_pulse(struct pcie_port *pp)
|
||||||
|
@ -400,7 +398,6 @@ static void exynos_pcie_enable_irq_pulse(struct pcie_port *pp)
|
||||||
val = IRQ_INTA_ASSERT | IRQ_INTB_ASSERT |
|
val = IRQ_INTA_ASSERT | IRQ_INTB_ASSERT |
|
||||||
IRQ_INTC_ASSERT | IRQ_INTD_ASSERT,
|
IRQ_INTC_ASSERT | IRQ_INTD_ASSERT,
|
||||||
exynos_elb_writel(exynos_pcie, val, PCIE_IRQ_EN_PULSE);
|
exynos_elb_writel(exynos_pcie, val, PCIE_IRQ_EN_PULSE);
|
||||||
return;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static irqreturn_t exynos_pcie_irq_handler(int irq, void *arg)
|
static irqreturn_t exynos_pcie_irq_handler(int irq, void *arg)
|
||||||
|
@ -429,7 +426,6 @@ static void exynos_pcie_msi_init(struct pcie_port *pp)
|
||||||
val = exynos_elb_readl(exynos_pcie, PCIE_IRQ_EN_LEVEL);
|
val = exynos_elb_readl(exynos_pcie, PCIE_IRQ_EN_LEVEL);
|
||||||
val |= IRQ_MSI_ENABLE;
|
val |= IRQ_MSI_ENABLE;
|
||||||
exynos_elb_writel(exynos_pcie, val, PCIE_IRQ_EN_LEVEL);
|
exynos_elb_writel(exynos_pcie, val, PCIE_IRQ_EN_LEVEL);
|
||||||
return;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static void exynos_pcie_enable_interrupts(struct pcie_port *pp)
|
static void exynos_pcie_enable_interrupts(struct pcie_port *pp)
|
||||||
|
@ -438,8 +434,6 @@ static void exynos_pcie_enable_interrupts(struct pcie_port *pp)
|
||||||
|
|
||||||
if (IS_ENABLED(CONFIG_PCI_MSI))
|
if (IS_ENABLED(CONFIG_PCI_MSI))
|
||||||
exynos_pcie_msi_init(pp);
|
exynos_pcie_msi_init(pp);
|
||||||
|
|
||||||
return;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void exynos_pcie_readl_rc(struct pcie_port *pp,
|
static inline void exynos_pcie_readl_rc(struct pcie_port *pp,
|
||||||
|
@ -448,7 +442,6 @@ static inline void exynos_pcie_readl_rc(struct pcie_port *pp,
|
||||||
exynos_pcie_sideband_dbi_r_mode(pp, true);
|
exynos_pcie_sideband_dbi_r_mode(pp, true);
|
||||||
*val = readl(dbi_base);
|
*val = readl(dbi_base);
|
||||||
exynos_pcie_sideband_dbi_r_mode(pp, false);
|
exynos_pcie_sideband_dbi_r_mode(pp, false);
|
||||||
return;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void exynos_pcie_writel_rc(struct pcie_port *pp,
|
static inline void exynos_pcie_writel_rc(struct pcie_port *pp,
|
||||||
|
@ -457,7 +450,6 @@ static inline void exynos_pcie_writel_rc(struct pcie_port *pp,
|
||||||
exynos_pcie_sideband_dbi_w_mode(pp, true);
|
exynos_pcie_sideband_dbi_w_mode(pp, true);
|
||||||
writel(val, dbi_base);
|
writel(val, dbi_base);
|
||||||
exynos_pcie_sideband_dbi_w_mode(pp, false);
|
exynos_pcie_sideband_dbi_w_mode(pp, false);
|
||||||
return;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static int exynos_pcie_rd_own_conf(struct pcie_port *pp, int where, int size,
|
static int exynos_pcie_rd_own_conf(struct pcie_port *pp, int where, int size,
|
||||||
|
|
Loading…
Reference in a new issue