Return-path: Received: from smtp-out4.electric.net ([192.162.216.195]:54602 "EHLO smtp-out4.electric.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751427AbcGELHf convert rfc822-to-8bit (ORCPT ); Tue, 5 Jul 2016 07:07:35 -0400 From: David Laight To: 'Jakub Kicinski' , "kvalo@codeaurora.org" , "linux-wireless@vger.kernel.org" CC: "netdev@vger.kernel.org" , "hannes@stressinduktion.org" , "nbd@nbd.name" , "linux-kernel@vger.kernel.org" Subject: RE: [PATCHv3 wl-drv-next 1/2] add basic register-field manipulation macros Date: Tue, 5 Jul 2016 10:56:13 +0000 Message-ID: <063D6719AE5E284EB5DD2968C1650D6D5F4F06BF@AcuExch.aculab.com> (sfid-20160705_130740_414201_AAC23B01) References: <1467408409-24224-1-git-send-email-jakub.kicinski@netronome.com> <1467408409-24224-2-git-send-email-jakub.kicinski@netronome.com> In-Reply-To: <1467408409-24224-2-git-send-email-jakub.kicinski@netronome.com> Content-Type: text/plain; charset="Windows-1252" MIME-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: From: Jakub Kicinski > Sent: 01 July 2016 22:27 > > C bitfields are problematic and best avoided. Developers > interacting with hardware registers find themselves searching > for easy-to-use alternatives. Common approach is to define > structures or sets of macros containing mask and shift pair. > Operations on the register are then performed as follows: > > field = (reg >> shift) & mask; > > reg &= ~(mask << shift); > reg |= (field & mask) << shift; > > Defining shift and mask separately is tedious. Ivo van Doorn > came up with an idea of computing them at compilation time > based on a single shifted mask (later refined by Felix) which > can be used like this: > > #define REG_FIELD 0x000ff000 > > field = FIELD_GET(REG_FIELD, reg); > > reg &= ~REG_FIELD; > reg |= FIELD_PUT(REG_FIELD, field); My problem with these sort of 'helpers' is that they make it much harder to read code unless you happen to know exactly what the helpers do. Unexpected issues (like values being sign extended) can be hard to spot. A lot of the time you can make things simpler by not doing the shifts (ie define shifted constants). David