From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759750Ab3BMDBO (ORCPT ); Tue, 12 Feb 2013 22:01:14 -0500 Received: from mout.gmx.net ([212.227.17.21]:55437 "EHLO mout.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759597Ab3BMDBN (ORCPT ); Tue, 12 Feb 2013 22:01:13 -0500 X-Authenticated: #12255092 X-Provags-ID: V01U2FsdGVkX18osj1ygu8oSpWMpWsACF54k5Yd0CJlIwkWpaLDfW JpzxD9swnvq5k4 From: Peter Huewe To: Greg Kroah-Hartman Cc: Peter Huewe , Andrew Miller , Maxim Mikityanskiy , YAMANE Toshiaki , devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: [PATCH 6/9] staging/rtl8187se: Remove temporary variable for return value Date: Wed, 13 Feb 2013 04:05:23 +0100 Message-Id: <1360724726-2997-6-git-send-email-peterhuewe@gmx.de> X-Mailer: git-send-email 1.7.8.6 In-Reply-To: <1360724302-2794-4-git-send-email-peterhuewe@gmx.de> References: <1360724302-2794-4-git-send-email-peterhuewe@gmx.de> X-Y-GMX-Trusted: 0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The simple PlatformIORead1Byte function doesn't need to store its return value in a temporary variable; rather simply return the value directly. By moving the function to the top we can also get rid of the forward declaration. The wrapper functions could be easily replaced by direct calls to read_nic_byte but is kept for readability. Since this functions is local only we can mark it as static. Signed-off-by: Peter Huewe --- drivers/staging/rtl8187se/r8185b_init.c | 17 +++++------------ 1 files changed, 5 insertions(+), 12 deletions(-) diff --git a/drivers/staging/rtl8187se/r8185b_init.c b/drivers/staging/rtl8187se/r8185b_init.c index a3b836c..1dfa78e 100644 --- a/drivers/staging/rtl8187se/r8185b_init.c +++ b/drivers/staging/rtl8187se/r8185b_init.c @@ -115,6 +115,11 @@ static u8 OFDM_CONFIG[] = { *--------------------------------------------------------------- */ +static u8 PlatformIORead1Byte(struct net_device *dev, u32 offset) +{ + return read_nic_byte(dev, offset); +} + void PlatformIOWrite1Byte(struct net_device *dev, u32 offset, u8 data) { write_nic_byte(dev, offset, data); @@ -127,8 +132,6 @@ void PlatformIOWrite2Byte(struct net_device *dev, u32 offset, u16 data) read_nic_word(dev, offset); /* To make sure write operation is completed, 2005.11.09, by rcnjko. */ } -u8 PlatformIORead1Byte(struct net_device *dev, u32 offset); - void PlatformIOWrite4Byte(struct net_device *dev, u32 offset, u32 data) { if (offset == PhyAddr) { @@ -172,16 +175,6 @@ void PlatformIOWrite4Byte(struct net_device *dev, u32 offset, u32 data) } } -u8 PlatformIORead1Byte(struct net_device *dev, u32 offset) -{ - u8 data = 0; - - data = read_nic_byte(dev, offset); - - - return data; -} - u16 PlatformIORead2Byte(struct net_device *dev, u32 offset) { u16 data = 0; -- 1.7.8.6