Return-path: Received: from oproxy1-pub.bluehost.com ([66.147.249.253]:44224 "HELO oproxy1-pub.bluehost.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1754811Ab0LJVHe (ORCPT ); Fri, 10 Dec 2010 16:07:34 -0500 Date: Fri, 10 Dec 2010 13:07:30 -0800 From: Jesse Barnes To: Jon Mason Cc: linux-pci@vger.kernel.org, Jonathan Corbet , linux-media@vger.kernel.org, Andrew Gallatin , Brice Goglin , netdev@vger.kernel.org, Solarflare linux maintainers , Steve Hodgson , Ben Hutchings , Stephen Hemminger , Ivo van Doorn , Gertjan van Wingerde , linux-wireless@vger.kernel.org, Brian King , Anil Ravindranath , linux-scsi@vger.kernel.org, Jaya Kumar , boyod.yang@siliconmotion.com.cn Subject: Re: PCI: make pci_restore_state return void Message-ID: <20101210130730.29d07c2d@jbarnes-desktop> In-Reply-To: <1291160606-31494-1-git-send-email-jon.mason@exar.com> References: <1291160606-31494-1-git-send-email-jon.mason@exar.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-wireless-owner@vger.kernel.org List-ID: On Tue, 30 Nov 2010 17:43:26 -0600 Jon Mason wrote: > pci_restore_state only ever returns 0, thus there is no benefit in > having it return any value. Also, a large majority of the callers do > not check the return code of pci_restore_state. Make the > pci_restore_state a void return and avoid the overhead. > > Signed-off-by: Jon Mason > --- Applied to linux-next, thanks. -- Jesse Barnes, Intel Open Source Technology Center