From patchwork Fri Jul 1 21:26:49 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jakub Kicinski X-Patchwork-Id: 9210507 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 3483A607D8 for ; Fri, 1 Jul 2016 21:27:49 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 282B1286B3 for ; Fri, 1 Jul 2016 21:27:49 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 1CB5E286CA; Fri, 1 Jul 2016 21:27:49 +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 E251F286B3 for ; Fri, 1 Jul 2016 21:27:46 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932127AbcGAV10 (ORCPT ); Fri, 1 Jul 2016 17:27:26 -0400 Received: from mail-wm0-f48.google.com ([74.125.82.48]:37827 "EHLO mail-wm0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752351AbcGAV1Y (ORCPT ); Fri, 1 Jul 2016 17:27:24 -0400 Received: by mail-wm0-f48.google.com with SMTP id a66so44453647wme.0 for ; Fri, 01 Jul 2016 14:27:23 -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=clF1om1KbCvYJkz+ZH4MH7ZIVwgtc6nlpVrlL64ybO7KMR5wblD+9jkDi+DzInhlIw HQ5FSrMR6CYPvnIa240cmD6ZfoZT2n03p2Tvg+hHUdFWL0EM2RpgCi6wMCtdwHhP0F71 txGuCvsoC/1uwnEcjIDuZcokspD3iPwwPv3D+AxmJ6fd4S1s0dhGH+pICYzedhSB6yUj OkTKpzfv+IUKSn/PO/RJxsr5ofVzr9B6lnx3CSpz5PJWpq0/A3UaZJC6QiSzFNUbiM+O 6xfGXZTmVmV62AK9E23lFPYk3e50YGNF6blEYdnFrxP/fNg25QuUI2WXs+1a1bYSJ0Oa E8WQ== 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=ZlphmV51GXtVrZK6VO3yRqn0f0NZkEdn1tPIOfy6xiovRmzDKF0qk+AB3VQpX1jVJv bM5//+reAWImdY1CQeUDdurhFyTyxp6scOZf2iWeVG1Betgb0RlAJAESD89P6q05oA/5 d7Gfa4hCOX6kJxPpIVYfB4/Ljh0MVIzsoeXO5EN1K/oVkmnQHf4s+1gTLBKDSWhW5Rfz oMbztwL6v+V9E+wiLOzDQ4wn9gbdVDTEpzGnfaK1HPL7DDWS+Y/kLuE0KU8ZPecQDxQY UlYE26XbCXCP2hGdAGATLxoOwHWZ8S4o37WYi+N4yWRq2TNxBnWwH/m4we/8t70CqsP3 mAUA== X-Gm-Message-State: ALyK8tINVR/CiDzBt6vPj6Yb92POGqqzlNet2gpUPgL2L5iCHi+RdLMyXRj/p3+ZSGQ9UQPB X-Received: by 10.28.136.21 with SMTP id k21mr269659wmd.24.1467408442138; Fri, 01 Jul 2016 14:27:22 -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 a84sm1590018wma.0.2016.07.01.14.27.21 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Fri, 01 Jul 2016 14:27:21 -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: [PATCHv3 wl-drv-next 2/2] mt7601u: use linux/bitfield.h Date: Fri, 1 Jul 2016 22:26:49 +0100 Message-Id: <1467408409-24224-3-git-send-email-jakub.kicinski@netronome.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1467408409-24224-1-git-send-email-jakub.kicinski@netronome.com> References: <1467408409-24224-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