From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg Dietsche Subject: Re: [PATCH] ata: remove unnecessary code Date: Tue, 19 Jul 2011 21:04:57 -0500 Message-ID: <4E2637C9.2080205@cuw.edu> References: <1308242361-28859-1-git-send-email-Gregory.Dietsche@cuw.edu> <4E263769.9040500@cuw.edu> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from que21.charter.net ([209.225.8.22]:57515 "EHLO que21.charter.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752770Ab1GTCN2 (ORCPT ); Tue, 19 Jul 2011 22:13:28 -0400 In-Reply-To: <4E263769.9040500@cuw.edu> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Greg Dietsche Cc: jgarzik@pobox.com, linux-ide@vger.kernel.org, kernel-janitors@vger.kernel.org, tj@kernel.org, trivial@kernel.org On 07/19/2011 09:03 PM, Greg Dietsche wrote: > > > On 06/16/2011 11:39 AM, Greg Dietsche wrote: >> Compile tested. >> remove unnecessary code that matches this coccinelle pattern >> if (...) >> return ret; >> return ret; >> >> Signed-off-by: Greg Dietsche > > Acked-by: Tejun Heo > > >> --- >> drivers/ata/pata_acpi.c | 8 +------- >> 1 files changed, 1 insertions(+), 7 deletions(-) >> >> diff --git a/drivers/ata/pata_acpi.c b/drivers/ata/pata_acpi.c >> index 91949d9..54145ed 100644 >> --- a/drivers/ata/pata_acpi.c >> +++ b/drivers/ata/pata_acpi.c >> @@ -195,8 +195,6 @@ static int pacpi_port_start(struct ata_port *ap) >> struct pci_dev *pdev = to_pci_dev(ap->host->dev); >> struct pata_acpi *acpi; >> >> - int ret; >> - >> if (ap->acpi_handle == NULL) >> return -ENODEV; >> >> @@ -205,11 +203,7 @@ static int pacpi_port_start(struct ata_port *ap) >> return -ENOMEM; >> acpi->mask[0] = pacpi_discover_modes(ap,&ap->link.device[0]); >> acpi->mask[1] = pacpi_discover_modes(ap,&ap->link.device[1]); >> - ret = ata_bmdma_port_start(ap); >> - if (ret< 0) >> - return ret; >> - >> - return ret; >> + return ata_bmdma_port_start(ap); >> } >> >> static struct scsi_host_template pacpi_sht = { > This was acked by TJ, but seems to be lost in the shuffle thus far. > I'm adding trivial@kernel.org to the cc list since this is trivial and > no one has picked up this patch to date. > > Greg > > Sorry for the noise, but i forgot to add trivial to the cc :/ This mail fixes that omission. Greg