From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757336AbbE2U4J (ORCPT ); Fri, 29 May 2015 16:56:09 -0400 Received: from mout.kundenserver.de ([212.227.126.187]:59346 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757047AbbE2Uwz (ORCPT ); Fri, 29 May 2015 16:52:55 -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 07/16] staging: wilc1000: simplify msgqueue code Date: Fri, 29 May 2015 22:52:18 +0200 Message-Id: <1432932747-3739705-8-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:XLV3LFTvhJQJKOaXfsBHwaFlOF8vk6ywulb4R/PFEaveAqRIaEw YJXIvRWZ/TfhVaPujYTe3nBO+WjFSsO0fdj+uSNToNn2gi6dBmCdGXhfzVNEOOvLz/Yk+KJ Ps/Z5pCSkGA66H1bImLehXfo8wSzCJ7BKfvZYuQY+hwftZaY7kHiqTx+gw+7hKJsgtlCfbD TmCb8lMU0Ee0eVZd8R5kA== X-UI-Out-Filterresults: notjunk:1;V01:K0:TOE3i9r/BbE=:/Ge0roSgUcW6BQirLBrrY4 36u1SYDOqpOkSqy2ZRlKvTNI/BlnyJ/Rp4OYv0X6kR1efuB08vK7pOb/ozp+tlu9srj+tYtfw 0pMWtc4waKpm862tsWQmrFfxg6c55d6uzDb78ARxCh+rHMWmdbctxzyr3TXMva8RHNkQ1P5jK 55i3AGGvevecf/7SsxmkxhPDfSk0/hiqXtIHU38jAHvNGtRBonmoLKdxElDHh4EiAIlIhRLHM TB8r8xN8V4i3fI7SHZKqlY/AeieTaOeWimZN8L3VKTUGx+I3VKOLF4erw1hC1BC3mZIc5/ZHY OtEluLBhmzhhKpKyWFEL6dfqdRPYVlNIJ9aDtQSPhXTQtn+jkmmU4xJfW50NZXKUFG0Puz5NZ VqsRKO2QUz8ABwDtBAnzH9xt203z/4e/I5TZeKD77OUfReXHF979XuDs2rgG5WIiSY0N2hNHW +gd2XsCdCJreqwvA2EFpiy6nsuackNuBegwMjdcAv0LkiiuW8u5eAyHcH07Lld6zp7nv88DOc zb5EmtgqFVWZ1zb0f+bqiRRSZlLVwyUvW+QT/BAbCS4oHodk4pm5+in4/by5NCtb4O0+4ULkl 85blXvDVNmBFUOK4PTsfzMjb1Fqy2al8wNztSrM26n54KVIeCYWJ5Ad7bYSz0VnD8RnYqeA/B O5ChhhVc+1ZyS0NEkoxcxNr9F Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The driver contains an abstraction for message queues, with optional unused features, while the driver requires the main feature. This makes the msgqueue code unconditional as it's required but removes the unused parts. A later cleanup should remove the entire msgqueue code and replace it with some normal kernel API. Signed-off-by: Arnd Bergmann --- drivers/staging/wilc1000/wilc_msgqueue.c | 9 --------- drivers/staging/wilc1000/wilc_msgqueue.h | 30 ------------------------------ drivers/staging/wilc1000/wilc_osconfig.h | 3 --- drivers/staging/wilc1000/wilc_oswrapper.h | 2 -- drivers/staging/wilc1000/wilc_platform.h | 12 ------------ 5 files changed, 56 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_msgqueue.c b/drivers/staging/wilc1000/wilc_msgqueue.c index c1d0dabed479..1113092398d1 100644 --- a/drivers/staging/wilc1000/wilc_msgqueue.c +++ b/drivers/staging/wilc1000/wilc_msgqueue.c @@ -1,8 +1,6 @@ #include "wilc_oswrapper.h" #include -#ifdef CONFIG_WILC_MSG_QUEUE_FEATURE - /*! * @author syounan @@ -154,11 +152,6 @@ WILC_ErrNo WILC_MsgQueueRecv(WILC_MsgQueueHandle *pHandle, spin_unlock_irqrestore(&pHandle->strCriticalSection, flags); WILC_SemaphoreFillDefault(&strSemAttrs); - #ifdef CONFIG_WILC_MSG_QUEUE_TIMEOUT - if (pstrAttrs != WILC_NULL) { - strSemAttrs.u32TimeOut = pstrAttrs->u32Timeout; - } - #endif s32RetStatus = WILC_SemaphoreAcquire(&(pHandle->hSem), &strSemAttrs); if (s32RetStatus == WILC_TIMEOUT) { /* timed out, just exit without consumeing the message */ @@ -207,5 +200,3 @@ WILC_ErrNo WILC_MsgQueueRecv(WILC_MsgQueueHandle *pHandle, return s32RetStatus; } - -#endif diff --git a/drivers/staging/wilc1000/wilc_msgqueue.h b/drivers/staging/wilc1000/wilc_msgqueue.h index a48be533aad9..84157368335d 100644 --- a/drivers/staging/wilc1000/wilc_msgqueue.h +++ b/drivers/staging/wilc1000/wilc_msgqueue.h @@ -10,10 +10,6 @@ * @version 1.0 */ -#ifndef CONFIG_WILC_MSG_QUEUE_FEATURE -#error the feature CONFIG_WILC_MSG_QUEUE_FEATURE must be supported to include this file -#endif - /*! * @struct tstrWILC_MsgQueueAttrs * @brief Message Queue API options @@ -22,38 +18,12 @@ * @version 1.0 */ typedef struct { - #ifdef CONFIG_WILC_MSG_QUEUE_IPC_NAME - WILC_Char *pcName; - #endif - - #ifdef CONFIG_WILC_MSG_QUEUE_TIMEOUT - WILC_Uint32 u32Timeout; - #endif - /* a dummy member to avoid compiler errors*/ WILC_Uint8 dummy; } tstrWILC_MsgQueueAttrs; /*! - * @brief Fills the MsgQueueAttrs with default parameters - * @param[out] pstrAttrs structure to be filled - * @sa WILC_TimerAttrs - * @author syounan - * @date 30 Aug 2010 - * @version 1.0 - */ -static void WILC_MsgQueueFillDefault(tstrWILC_MsgQueueAttrs *pstrAttrs) -{ - #ifdef CONFIG_WILC_MSG_QUEUE_IPC_NAME - pstrAttrs->pcName = WILC_NULL; - #endif - - #ifdef CONFIG_WILC_MSG_QUEUE_TIMEOUT - pstrAttrs->u32Timeout = WILC_OS_INFINITY; - #endif -} -/*! * @brief Creates a new Message queue * @details Creates a new Message queue, if the feature * CONFIG_WILC_MSG_QUEUE_IPC_NAME is enabled and pstrAttrs->pcName diff --git a/drivers/staging/wilc1000/wilc_osconfig.h b/drivers/staging/wilc1000/wilc_osconfig.h index f18615e09400..d89864c7697c 100644 --- a/drivers/staging/wilc1000/wilc_osconfig.h +++ b/drivers/staging/wilc1000/wilc_osconfig.h @@ -20,9 +20,6 @@ /* #define CONFIG_WILC_MEMORY_POOLS 1 */ /* #define CONFIG_WILC_MEMORY_DEBUG 1 */ /* #define CONFIG_WILC_ASSERTION_SUPPORT 1 */ -#define CONFIG_WILC_MSG_QUEUE_FEATURE -/* #define CONFIG_WILC_MSG_QUEUE_IPC_NAME */ -/* #define CONFIG_WILC_MSG_QUEUE_TIMEOUT */ /* #define CONFIG_WILC_FILE_OPERATIONS_FEATURE */ /* #define CONFIG_WILC_FILE_OPERATIONS_STRING_API */ /* #define CONFIG_WILC_FILE_OPERATIONS_PATH_API */ diff --git a/drivers/staging/wilc1000/wilc_oswrapper.h b/drivers/staging/wilc1000/wilc_oswrapper.h index c4e97ae03ae0..8b4c3dced981 100644 --- a/drivers/staging/wilc1000/wilc_oswrapper.h +++ b/drivers/staging/wilc1000/wilc_oswrapper.h @@ -78,9 +78,7 @@ typedef WILC_Uint16 WILC_WideChar; #include "wilc_strutils.h" /* Message Queue */ -#ifdef CONFIG_WILC_MSG_QUEUE_FEATURE #include "wilc_msgqueue.h" -#endif /* File operations */ #ifdef CONFIG_WILC_FILE_OPERATIONS_FEATURE diff --git a/drivers/staging/wilc1000/wilc_platform.h b/drivers/staging/wilc1000/wilc_platform.h index b20bbb839e5a..2f6484989565 100644 --- a/drivers/staging/wilc1000/wilc_platform.h +++ b/drivers/staging/wilc1000/wilc_platform.h @@ -50,18 +50,6 @@ #error This feature is not supported by this OS #endif -/* CONFIG_WILC_MSG_QUEUE_FEATURE is implemented */ - -/* remove the following block when implementing its feature */ -#ifdef CONFIG_WILC_MSG_QUEUE_IPC_NAME -#error This feature is not supported by this OS -#endif - -/* remove the following block when implementing its feature */ -/*#ifdef CONFIG_WILC_MSG_QUEUE_TIMEOUT - * #error This feature is not supported by this OS - #endif*/ - /* CONFIG_WILC_FILE_OPERATIONS_FEATURE is implemented */ /* CONFIG_WILC_FILE_OPERATIONS_STRING_API is implemented */ -- 2.1.0.rc2