From patchwork Tue Jul 5 11:42:59 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jakub Kicinski X-Patchwork-Id: 9214135 X-Patchwork-Delegate: kvalo@adurom.com Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 844C46088F for ; Tue, 5 Jul 2016 11:44:03 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 755E428A1C for ; Tue, 5 Jul 2016 11:44:03 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 69D0628A1E; Tue, 5 Jul 2016 11:44:03 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=2.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_HI,T_DKIM_INVALID autolearn=unavailable version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 3DF2828A1C for ; Tue, 5 Jul 2016 11:44:02 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932982AbcGELnd (ORCPT ); Tue, 5 Jul 2016 07:43:33 -0400 Received: from mail-wm0-f42.google.com ([74.125.82.42]:38077 "EHLO mail-wm0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932965AbcGELn3 (ORCPT ); Tue, 5 Jul 2016 07:43:29 -0400 Received: by mail-wm0-f42.google.com with SMTP id r201so148808424wme.1 for ; Tue, 05 Jul 2016 04:43:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=netronome-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=uCYM2LfkNJqWipCkRicrAMiTAPfBqHAw3a4AiQMt760=; b=nROpgcFkqOyELkXczQLL9CEfyB78DzPotM6H7VBkvwyQAuYrtZdKKBHt9FC85Yw1Zj o6/QfF6wuQaE2pql/0Z8ecdSvupNpGqKsCbrdPdei3ECOnx4GZlS2DB7Zq7kYy+EAeNi MTvZIshuAMgolir91JvF0bZHPva2Uro9+Vy8ElT3VfzNdw06nob8V1QtHsIUYbVzI6Qh KcVr9haPuHLrCVfG3QUVR5hHW1/k1qC3wRO7+NWB8/vZ2uLvItfg3HJd7Hd3qdcBowVi bp0juKEBx0vLy8frrqccSfEafyQARDNcEYV+jHaB8MHfhSjS0qHViKj1XJxWyRMCv2wF jrGA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=uCYM2LfkNJqWipCkRicrAMiTAPfBqHAw3a4AiQMt760=; b=lVsz7o9MRJLm1xeeNqX+e2EtcjJoBFChrtHv+jWCT4gP0pLzMT9/OXtki/WydjCaNQ ZjEAnG6TOZ0L2YU6x4nhrMpMYdf024OhrZ5R6CmDB0Az1hOocdzRTvKuFdnEVEsyABFp IpIl+NKmoLgxDsLpExbwq7X3qnoW43u01GxVQ0d20I7/CET7CNXrREKDLwX44LfC6d5z QlX+M2tgnyqyFUAIR74El88YoAGNBDFDIS0DPwKupaKqAo2tE/xpXitwxcjLLhZF3zXK 8+UgSDFE6arX5BumATimmB58ToWQIHjCHjoKH8pC2U7JgwYW3IdYVhvzWljWAlxCqzSH fbeA== X-Gm-Message-State: ALyK8tK7vUR4FivnOMiZYVdmNxF0N5KNaBu/FI9/MaDya/NKtMZQ8LWiWHVzWFWNyh7HGQb1 X-Received: by 10.28.45.15 with SMTP id t15mr10870807wmt.37.1467719007624; Tue, 05 Jul 2016 04:43:27 -0700 (PDT) Received: from jkicinski-Precision-T1700.netronome.com (host-79-78-33-110.static.as9105.net. [79.78.33.110]) by smtp.gmail.com with ESMTPSA id e69sm354446wma.2.2016.07.05.04.43.24 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Tue, 05 Jul 2016 04:43:26 -0700 (PDT) From: Jakub Kicinski To: kvalo@codeaurora.org, linux-wireless@vger.kernel.org Cc: netdev@vger.kernel.org, hannes@stressinduktion.org, nbd@nbd.name, linux-kernel@vger.kernel.org, Jakub Kicinski Subject: [PATCHv4 wl-drv-next 2/2] mt7601u: use linux/bitfield.h Date: Tue, 5 Jul 2016 12:42:59 +0100 Message-Id: <1467718979-20029-3-git-send-email-jakub.kicinski@netronome.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1467718979-20029-1-git-send-email-jakub.kicinski@netronome.com> References: <1467718979-20029-1-git-send-email-jakub.kicinski@netronome.com> Sender: linux-wireless-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Use the newly added linux/bitfield.h. Signed-off-by: Jakub Kicinski --- drivers/net/wireless/mediatek/mt7601u/dma.h | 2 - drivers/net/wireless/mediatek/mt7601u/mt7601u.h | 5 +- drivers/net/wireless/mediatek/mt7601u/util.h | 77 ------------------------- 3 files changed, 4 insertions(+), 80 deletions(-) delete mode 100644 drivers/net/wireless/mediatek/mt7601u/util.h diff --git a/drivers/net/wireless/mediatek/mt7601u/dma.h b/drivers/net/wireless/mediatek/mt7601u/dma.h index 978e8a90b87f..166ac71905d2 100644 --- a/drivers/net/wireless/mediatek/mt7601u/dma.h +++ b/drivers/net/wireless/mediatek/mt7601u/dma.h @@ -18,8 +18,6 @@ #include #include -#include "util.h" - #define MT_DMA_HDR_LEN 4 #define MT_RX_INFO_LEN 4 #define MT_FCE_INFO_LEN 4 diff --git a/drivers/net/wireless/mediatek/mt7601u/mt7601u.h b/drivers/net/wireless/mediatek/mt7601u/mt7601u.h index 428bd2f10b7b..5ef62e02ce66 100644 --- a/drivers/net/wireless/mediatek/mt7601u/mt7601u.h +++ b/drivers/net/wireless/mediatek/mt7601u/mt7601u.h @@ -15,6 +15,7 @@ #ifndef MT7601U_H #define MT7601U_H +#include #include #include #include @@ -24,7 +25,6 @@ #include #include "regs.h" -#include "util.h" #define MT_CALIBRATE_INTERVAL (4 * HZ) @@ -282,6 +282,9 @@ struct mt7601u_rxwi; extern const struct ieee80211_ops mt7601u_ops; +#define MT76_SET FIELD_PUT +#define MT76_GET FIELD_GET + void mt7601u_init_debugfs(struct mt7601u_dev *dev); u32 mt7601u_rr(struct mt7601u_dev *dev, u32 offset); diff --git a/drivers/net/wireless/mediatek/mt7601u/util.h b/drivers/net/wireless/mediatek/mt7601u/util.h deleted file mode 100644 index b89140bf1210..000000000000 --- a/drivers/net/wireless/mediatek/mt7601u/util.h +++ /dev/null @@ -1,77 +0,0 @@ -/* - * Copyright (C) 2014 Felix Fietkau - * Copyright (C) 2004 - 2009 Ivo van Doorn - * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License version 2 - * as published by the Free Software Foundation - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - */ - -#ifndef __MT76_UTIL_H -#define __MT76_UTIL_H - -/* - * Power of two check, this will check - * if the mask that has been given contains and contiguous set of bits. - * Note that we cannot use the is_power_of_2() function since this - * check must be done at compile-time. - */ -#define is_power_of_two(x) ( !((x) & ((x)-1)) ) -#define low_bit_mask(x) ( ((x)-1) & ~(x) ) -#define is_valid_mask(x) is_power_of_two(1LU + (x) + low_bit_mask(x)) - -/* - * Macros to find first set bit in a variable. - * These macros behave the same as the __ffs() functions but - * the most important difference that this is done during - * compile-time rather then run-time. - */ -#define compile_ffs2(__x) \ - __builtin_choose_expr(((__x) & 0x1), 0, 1) - -#define compile_ffs4(__x) \ - __builtin_choose_expr(((__x) & 0x3), \ - (compile_ffs2((__x))), \ - (compile_ffs2((__x) >> 2) + 2)) - -#define compile_ffs8(__x) \ - __builtin_choose_expr(((__x) & 0xf), \ - (compile_ffs4((__x))), \ - (compile_ffs4((__x) >> 4) + 4)) - -#define compile_ffs16(__x) \ - __builtin_choose_expr(((__x) & 0xff), \ - (compile_ffs8((__x))), \ - (compile_ffs8((__x) >> 8) + 8)) - -#define compile_ffs32(__x) \ - __builtin_choose_expr(((__x) & 0xffff), \ - (compile_ffs16((__x))), \ - (compile_ffs16((__x) >> 16) + 16)) - -/* - * This macro will check the requirements for the FIELD{8,16,32} macros - * The mask should be a constant non-zero contiguous set of bits which - * does not exceed the given typelimit. - */ -#define FIELD_CHECK(__mask) \ - BUILD_BUG_ON(!(__mask) || !is_valid_mask(__mask)) - -#define MT76_SET(_mask, _val) \ - ({ \ - FIELD_CHECK(_mask); \ - (((u32) (_val)) << compile_ffs32(_mask)) & _mask; \ - }) - -#define MT76_GET(_mask, _val) \ - ({ \ - FIELD_CHECK(_mask); \ - (u32) (((_val) & _mask) >> compile_ffs32(_mask)); \ - }) - -#endif