staging: wilc1000: clean up sleep wrapper
authorArnd Bergmann <arnd@arndb.de>
Mon, 1 Jun 2015 19:06:39 +0000 (21:06 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 2 Jun 2015 04:45:22 +0000 (13:45 +0900)
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 <arnd@arndb.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wilc1000/wilc_osconfig.h
drivers/staging/wilc1000/wilc_oswrapper.h
drivers/staging/wilc1000/wilc_platform.h
drivers/staging/wilc1000/wilc_sleep.c
drivers/staging/wilc1000/wilc_sleep.h

index d0bf08f89906d79588b7ee6c678d8fe1bb6f1776..6da42c83792856db1362d054b9593d515c0ba471 100644 (file)
@@ -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 */
index 32fe48a6034b0d3d7c1b048c6767861282dc9e8e..2af32fff84aaee324039614482442df81d792a11 100644 (file)
@@ -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
index 36e2e707354beb848afaa669c19df1479b5c3e50..2c66c3f3a2c5789126ca7b1cd0a914ed62dbd8c3 100644 (file)
  * #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 */
 
index b8f45146956b84db605d94f2eded8f916e774eb0..98a079f3d6c97865b838f0a7d94971b01edb8e79 100644 (file)
@@ -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 */
index d640fb553aca56b72c0cdb05628841a3d4c83550..2865c8e44346b5ee4c1f650774b3706befd518b7 100644 (file)
@@ -1,19 +1,6 @@
 #ifndef __WILC_SLEEP_H__
 #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
  *  @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