From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rene Herman Subject: Re: [patch 09/15] PNP: make resource assignment functions return 0 (success) or -EBUSY (failure) Date: Sun, 01 Jun 2008 23:59:06 +0200 Message-ID: <48431BAA.4040902@keyaccess.nl> References: <20080530224853.976744229@ldl.fc.hp.com> <20080530224933.163750253@ldl.fc.hp.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from smtpq1.tilbu1.nb.home.nl ([213.51.146.200]:60676 "EHLO smtpq1.tilbu1.nb.home.nl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751726AbYFAVzt (ORCPT ); Sun, 1 Jun 2008 17:55:49 -0400 In-Reply-To: <20080530224933.163750253@ldl.fc.hp.com> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Bjorn Helgaas Cc: Len Brown , linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, Adam Belay , Adam M Belay , Li Shaohua , Matthieu Castet , Thomas Renninger , Jaroslav Kysela , Andrew Morton , Takashi Iwai On 31-05-08 00:49, Bjorn Helgaas wrote: > This patch doesn't change any behavior; it just makes the return > values more conventional. > > This changes pnp_assign_dma() from a void function to one that > returns an int, just like the other assignment functions. For > now, at least, pnp_assign_dma() always returns 0 (success), so > it appears to never fail, just like before. > > Signed-off-by: Bjorn Helgaas Acked-by: Rene Herman I believe it would a bit nicer to (as long as we're pretending it matters in the first place) pass the return value back up from pnp_check_foo() instead of returning -EBUSY always but Rene.