Return-path: Received: from mail-wm0-f53.google.com ([74.125.82.53]:35238 "EHLO mail-wm0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932864AbcHaLrX (ORCPT ); Wed, 31 Aug 2016 07:47:23 -0400 Received: by mail-wm0-f53.google.com with SMTP id w2so27207634wmd.0 for ; Wed, 31 Aug 2016 04:47:22 -0700 (PDT) From: Jakub Kicinski To: Kalle Valo , Linux Wireless List Cc: Jakub Kicinski Subject: [PATCHv8 0/4] register-field manipulation macros Date: Wed, 31 Aug 2016 12:46:43 +0100 Message-Id: <1472644007-20959-1-git-send-email-jakub.kicinski@netronome.com> (sfid-20160831_134744_123895_16A82E0C) In-Reply-To: <1471625049-12643-1-git-send-email-jakub.kicinski@netronome.com> References: <1471625049-12643-1-git-send-email-jakub.kicinski@netronome.com> Sender: linux-wireless-owner@vger.kernel.org List-ID: Hi! Small improvement suggested by Daniel Borkmann - use two underscore prefix. https://www.mail-archive.com/netdev@vger.kernel.org/msg125423.html -- v6 blurb: This set moves to a global header file macros which I find very useful and worth popularising. The basic problem is that since C bitfields are not very dependable accessing subfields of registers becomes slightly inconvenient. It is nice to have the necessary mask and shift operations wrapped in a macro and have that macro compute shift at compilation time using FFS. My implementation follows what Felix Fietkau has done in mt76. Hannes Frederic Sowa suggested more use of standard Linux/GCC functions. Since the RFC I've also added a compile-time check to validate that the value passed to setters fits in the mask. I attempted the use of static inlines instead of macros but it makes GCC < 6.0 barf at the BUILD_BUG_ON()s. I also noticed that forcing arguments to be u32 for inlines makes the compiler use 32bit arithmetic where it could get away with 64bit before (on 64bit machines, obviously). That's a potential performance concern but probably not a very practical one today. Apart from looking "cleaner" static inlines would have the advantage that we could #undef the auxiliary macros at the end of the header. Build bot caught a build failure with -Os set. AFAICT gcc did not handle temporary variable I put in the macro expression too well. I work around that by defining __BUILD_BUG_ON_NOT_POWER_OF_2 and using it instead of BUILD_BUG_ON(!tmp || is_power_of_2(tmp)). I'm planning to use those macros in another driver soon, Felix may also use them when upstreaming mt76 if he chooses to do so. IMHO if accepted it would be best to push these through Kalle's wireless drivers' tree, since the only existing user is in drivers/net/wireless/. v8: - use two underscores for auxiliary macros (Daniel B). v7: - drop the explicit type marking (u32/u64) - depend on the type of the mask instead; - only allow compilation time constant masks; - barf at "type of register too small to ever match mask" on get; - rename PUT -> PREP. v6: - do a full rename in patch 2; - CC many people. v5: - repost. v4: - add documentation in the header. v3: - don't use variables in statement expressions; - use __BUILD_BUG_ON_NOT_POWER_OF_2. v2: - change Felix's email address. Jakub Kicinski (4): add basic register-field manipulation macros mt7601u: remove redefinition of GENMASK mt7601u: remove unnecessary include mt7601u: use linux/bitfield.h drivers/net/wireless/mediatek/mt7601u/dma.c | 2 +- drivers/net/wireless/mediatek/mt7601u/dma.h | 10 ++- drivers/net/wireless/mediatek/mt7601u/eeprom.c | 12 ++-- drivers/net/wireless/mediatek/mt7601u/init.c | 10 +-- drivers/net/wireless/mediatek/mt7601u/mac.c | 38 +++++----- drivers/net/wireless/mediatek/mt7601u/main.c | 1 - drivers/net/wireless/mediatek/mt7601u/mcu.c | 20 +++--- drivers/net/wireless/mediatek/mt7601u/mt7601u.h | 4 +- drivers/net/wireless/mediatek/mt7601u/phy.c | 44 ++++++------ drivers/net/wireless/mediatek/mt7601u/regs.h | 4 -- drivers/net/wireless/mediatek/mt7601u/tx.c | 19 ++--- drivers/net/wireless/mediatek/mt7601u/util.h | 77 -------------------- include/linux/bitfield.h | 93 +++++++++++++++++++++++++ include/linux/bug.h | 3 + 14 files changed, 177 insertions(+), 160 deletions(-) delete mode 100644 drivers/net/wireless/mediatek/mt7601u/util.h create mode 100644 include/linux/bitfield.h -- 1.9.1