From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1422831AbbE2UzZ (ORCPT ); Fri, 29 May 2015 16:55:25 -0400 Received: from mout.kundenserver.de ([212.227.126.131]:56006 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757333AbbE2Uw6 (ORCPT ); Fri, 29 May 2015 16:52:58 -0400 From: Arnd Bergmann To: Greg KH Cc: Rachel Kim , Dean Lee , Chris Park , devel@driverdev.osuosl.org, nicolas.ferre@atmel.com, Johnny Kim , linux-kernel@vger.kernel.org, Arnd Bergmann Subject: [PATCH 10/16] staging: wilc1000: clean up sleep wrapper Date: Fri, 29 May 2015 22:52:21 +0200 Message-Id: <1432932747-3739705-11-git-send-email-arnd@arndb.de> X-Mailer: git-send-email 2.1.0.rc2 In-Reply-To: <1432932747-3739705-1-git-send-email-arnd@arndb.de> References: <1432932747-3739705-1-git-send-email-arnd@arndb.de> X-Provags-ID: V03:K0:hqMpYVNdNZQ5yLWWLSRtYLWbUARr9HZNgN0Wa83JR1MWA7Gv4WK oataqGk/66GYBTbM61ESOruzrUxEq2J0Utb7bC6kuH83Jlsq4lqJnLGrWtDB1Jb8vo7YxwZ M0ifO41LA67OXgX9wRYXD0OCHB1z8w/V37xq4kn9kKOxa4JjAFwyY7+qAQc6OqLhWz1fe9b /kqP0rH1UzbHknJWnNLzQ== X-UI-Out-Filterresults: notjunk:1;V01:K0:cuYgqusIXHs=:zcvHMOWGmzYZNAJT1VDtm2 6bynG8IQD9lPyF+tjdmQv2LkYaJ7vBlIpq1ja+Niijsu8rCTTjJQvqlPdnmy+bYAckWKpca17 qA2ZHScxDW+atMlk39hAPD0q/sesy5amf/bOvYTsOgFAV+pVlpPrQYDR0tIug4cWFdSvvR+xc mnJ4TQNlSnODFZeGFer+BsMCqAd866yXkjjOS5zG+X250n0gYrJNIycT774gq5J77DnJApjWM A2Kx2yFHZZQTjb0v7cQndR3ad/hVkWqwOQ0myl2y0wG9EH5QXmaUgYq/Pes28/vTvUlAc3w8m o3zkhEyaEPi9frvJQz5J32JXyfbkGK6HjsbR39QlRXBtKkvrx/AHIZJ+FHjXTy4J5snwGKpRN dgOhtKLfFbrDPb0166Ff5jHLuELpVuMioQLciuQX8hGagTnIj4FYNHnciEjoEAgOnvwcyifPi DoWsxqdN07QI8ObBZMPtMDmQUz+Eb/AEkZetKZFeZIoIn4Ej/FbbaZL4jBMHJWQ1DVcPPz32b sWMmLRbULWT7thCbnPK6r6zDmCwugRsO52Bmm8YLsgzSaVh+QWN5ieMd7FUmUNnfx5Z69ut4a 0F9tgvYu0dx16uWmAvzaX/rwflzpv//SojN5vwUO0imzLqX+YxO61AhHxWGheYVNqzt5Y/S4w Mf6cNMYSNjFvR3aPc0G+C52C2 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The driver has a simple wrapper around msleep, as well as a more advanced sleep function that is unused. This removes the unused code and the options to turn the feature on or off. A follow-up should rework the code to use msleep directly. Signed-off-by: Arnd Bergmann --- drivers/staging/wilc1000/wilc_osconfig.h | 2 -- drivers/staging/wilc1000/wilc_oswrapper.h | 2 -- drivers/staging/wilc1000/wilc_platform.h | 6 ------ drivers/staging/wilc1000/wilc_sleep.c | 10 ---------- drivers/staging/wilc1000/wilc_sleep.h | 30 +----------------------------- 5 files changed, 1 insertion(+), 49 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_osconfig.h b/drivers/staging/wilc1000/wilc_osconfig.h index d0bf08f89906..6da42c837928 100644 --- a/drivers/staging/wilc1000/wilc_osconfig.h +++ b/drivers/staging/wilc1000/wilc_osconfig.h @@ -11,8 +11,6 @@ /* OS features supported */ /* #define CONFIG_WILC_SEMAPHORE_TIMEOUT 1 */ -#define CONFIG_WILC_SLEEP_FEATURE 1 -#define CONFIG_WILC_SLEEP_HI_RES 1 #define CONFIG_WILC_TIMER_FEATURE 1 /* #define CONFIG_WILC_TIMER_PERIODIC 1 */ /* #define CONFIG_WILC_ASSERTION_SUPPORT 1 */ diff --git a/drivers/staging/wilc1000/wilc_oswrapper.h b/drivers/staging/wilc1000/wilc_oswrapper.h index 32fe48a6034b..2af32fff84aa 100644 --- a/drivers/staging/wilc1000/wilc_oswrapper.h +++ b/drivers/staging/wilc1000/wilc_oswrapper.h @@ -58,9 +58,7 @@ typedef WILC_Uint16 WILC_WideChar; #include "wilc_semaphore.h" /* Sleep support */ -#ifdef CONFIG_WILC_SLEEP_FEATURE #include "wilc_sleep.h" -#endif /* Timer support */ #ifdef CONFIG_WILC_TIMER_FEATURE diff --git a/drivers/staging/wilc1000/wilc_platform.h b/drivers/staging/wilc1000/wilc_platform.h index 36e2e707354b..2c66c3f3a2c5 100644 --- a/drivers/staging/wilc1000/wilc_platform.h +++ b/drivers/staging/wilc1000/wilc_platform.h @@ -22,12 +22,6 @@ * #error This feature is not supported by this OS #endif*/ -/* CONFIG_WILC_SLEEP_FEATURE is implemented */ - -/* remove the following block when implementing its feature */ -/* #ifdef CONFIG_WILC_SLEEP_HI_RES */ -/* #error This feature is not supported by this OS */ -/* #endif */ /* CONFIG_WILC_TIMER_FEATURE is implemented */ diff --git a/drivers/staging/wilc1000/wilc_sleep.c b/drivers/staging/wilc1000/wilc_sleep.c index b8f45146956b..98a079f3d6c9 100644 --- a/drivers/staging/wilc1000/wilc_sleep.c +++ b/drivers/staging/wilc1000/wilc_sleep.c @@ -1,8 +1,6 @@ #include "wilc_oswrapper.h" -#ifdef CONFIG_WILC_SLEEP_FEATURE - /* * @author mdaftedar * @date 10 Aug 2010 @@ -18,11 +16,3 @@ void WILC_Sleep(WILC_Uint32 u32TimeMilliSec) } } -#endif - -/* #ifdef CONFIG_WILC_SLEEP_HI_RES */ -void WILC_SleepMicrosec(WILC_Uint32 u32TimeMicoSec) -{ - usleep_range(u32TimeMicoSec, u32TimeMicoSec); -} -/* #endif */ diff --git a/drivers/staging/wilc1000/wilc_sleep.h b/drivers/staging/wilc1000/wilc_sleep.h index d640fb553aca..2865c8e44346 100644 --- a/drivers/staging/wilc1000/wilc_sleep.h +++ b/drivers/staging/wilc1000/wilc_sleep.h @@ -2,19 +2,6 @@ #define __WILC_SLEEP_H__ /*! - * @file wilc_sleep.h - * @brief Sleep OS Wrapper functionality - * @author syounan - * @sa wilc_oswrapper.h top level OS wrapper file - * @date 10 Aug 2010 - * @version 1.0 - */ - -#ifndef CONFIG_WILC_SLEEP_FEATURE -#error the feature WILC_OS_FEATURE_SLEEP must be supported to include this file -#endif - -/*! * @brief forces the current thread to sleep until the given time has elapsed * @param[in] u32TimeMilliSec Time to sleep in Milli seconds * @sa WILC_SleepMicrosec @@ -24,22 +11,7 @@ * @note This function offers a relatively innacurate and low resolution * sleep, for accurate high resolution sleep use u32TimeMicoSec */ +/* TODO: remove and open-code in callers */ void WILC_Sleep(WILC_Uint32 u32TimeMilliSec); -#ifdef CONFIG_WILC_SLEEP_HI_RES -/*! - * @brief forces the current thread to sleep until the given time has elapsed - * @param[in] u32TimeMicoSec Time to sleep in Micro seconds - * @sa WILC_Sleep - * @author syounan - * @date 10 Aug 2010 - * @version 1.0 - * @note This function offers an acurare high resolution sleep, depends on - * the feature WILC_OS_FEATURE_SLEEP_HI_RES and may not be supported - * on all Operating Systems - */ -void WILC_SleepMicrosec(WILC_Uint32 u32TimeMicoSec); -#endif - - #endif -- 2.1.0.rc2