From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Mon, 4 Aug 2014 21:01:17 +0200 From: Quentin Lambert To: "Rafael J. Wysocki" , Len Brown , Bjorn Helgaas , Scott Murray , linux-acpi@vger.kernel.org, linux-pci@vger.kernel.org Cc: linux-kernel@vger.kernel.org Subject: [PATCH 2/3] PCI hotplug: fix a simple if coding style issue Message-ID: <20140804190117.GA4243@greed> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Sender: linux-acpi-owner@vger.kernel.org List-ID: Removing curly braces in simple if cases. Signed-off-by: Quentin Lambert --- drivers/pci/hotplug/cpcihp_zt5550.c | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/pci/hotplug/cpcihp_zt5550.c b/drivers/pci/hotplug/cpcihp_zt5550.c index 42a61c2..2b222fc 100644 --- a/drivers/pci/hotplug/cpcihp_zt5550.c +++ b/drivers/pci/hotplug/cpcihp_zt5550.c @@ -182,9 +182,9 @@ static int zt5550_hc_enable_irq(void) { u8 reg; - if (hc_dev == NULL) { + if (hc_dev == NULL) return -ENODEV; - } + reg = readb(csr_int_mask); reg = reg & ~ENUM_INT_MASK; writeb(reg, csr_int_mask); @@ -195,9 +195,8 @@ static int zt5550_hc_disable_irq(void) { u8 reg; - if (hc_dev == NULL) { + if (hc_dev == NULL) return -ENODEV; - } reg = readb(csr_int_mask); reg = reg | ENUM_INT_MASK; @@ -210,9 +209,9 @@ static int zt5550_hc_init_one (struct pci_dev *pdev, const struct pci_device_id int status; status = zt5550_hc_config(pdev); - if (status != 0) { + if (status != 0) return status; - } + dbg("returned from zt5550_hc_config"); memset(&zt5550_hpc, 0, sizeof (struct cpci_hp_controller)); -- 1.9.1