From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f43.google.com (mail-wm1-f43.google.com [209.85.128.43]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8CA3D3F9F36 for ; Tue, 28 Apr 2026 10:43:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.43 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777373028; cv=none; b=Z14BFoAuye6OPuuf1+IYk4OBoL2zrLn43HyyargpQD7vYcvInQ3kjuWla2SGBj6jiMZCAOp0R0dQYNjkI+3mv4UZQCrzjM46Z7+6iNzCuBvj4PlT9KjX3ydck0rSSyM+ntIby2B/AZVy3TphbkV9rNgGOjP+tV2YOFmXj0NINek= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777373028; c=relaxed/simple; bh=Txo9jXdDRugPyX7qmDybQ2dYpzzLy0YS2OMMBLInikE=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=SnHlNfVc10way5r92TlpbwLIsVdVrSF4rtrPwdmzFTy8a32/kL6NsRw85idOPfrCQp2nrIsi84H3eT6IujgFvBkdnRRTdOItApXAaQcwU33m+5rzB7BBetuaLoux1685EIygrFLaJdWU5G01P9l71JCM9QAWyTIShpIJPFfWK6w= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=OnXstZmN; arc=none smtp.client-ip=209.85.128.43 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="OnXstZmN" Received: by mail-wm1-f43.google.com with SMTP id 5b1f17b1804b1-48a3e9862f0so60193075e9.1 for ; Tue, 28 Apr 2026 03:43:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1777373025; x=1777977825; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=hKtRqSF01B3I4gaZJfxuG2QF2TOpMdK++wbIpuZOUFw=; b=OnXstZmN7ww1Q4bhga8AwQWndl6j+ndOBx2NRQaG/ke7OEbOxNum8W/rv6iVgWWpsP q3CqBLQEPXY6iz2D7jDqte1G8jU5tZRYGnO5sYLmLOYq0HhrVh8lUNG2Pa+PTDRUbBj8 yXvz1CKuvbjoMO/5hlzvifIjmBs1huNuA2I2nkMOI0UGUJStbkpc26sEi5r1jR65caUI rKz1BFO0+/M/uhxlM+PjQdZhsnYgbUuAwIfTbMeQZ7QefPDTYw6nsVY264kH03ze1LtD lTRdJn7H0otmTRynmseltG+hfSWaQ6x3nMXcRRDxxFzlXhbdE1cKgLGYcmyphB1lBzLY ZXsg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1777373025; x=1777977825; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=hKtRqSF01B3I4gaZJfxuG2QF2TOpMdK++wbIpuZOUFw=; b=VOIsyM95jHgUAyYd4T7SN2rywuhBpceBYpV8kh5r/BccCFa4HSeBRo0w8oiSWh352r 6qtm7hjr6jliJBmUwTbrJqk42q3Qb77JQqNBCojTIWgEUFif2x6HN7yDkYIIh2BGrTZb zPfAXp3k6mQ5O4cFL8g+Ony2Wt1BDPCKypM6qxicbMHGtk9FuirxupqoIce2o/EhxGev unwV8VjAPQhY8IHZ1XIU39MVwf6C224v6MFW9CN8YmIAmd+q4CApFRopur1d0LJOuq/8 ifg7Ka0ykSFPb4MSG7i4uneNbrTMqHtcAUEN5BCpEU/+eywE24/l3SOQNu7jgTB49OPp XQbw== X-Forwarded-Encrypted: i=1; AFNElJ/YRJP9i5hCSsjbqwFlR5Glpg5blJ/KEbovQeFupu/hOu4DQbAbP+GkfWpFudTTGKYYHLtSLr4gXvc=@vger.kernel.org X-Gm-Message-State: AOJu0Yz6yD/VZ7HxL2dhwBMr3xRga2vzJdambd0CQoAS69L/UH0A/Cc5 jJrsu0QeHk8Vy2J/ZEKh89xHIa3t76JjFTyB6m7eWuS1CM8HUKaointM X-Gm-Gg: AeBDiesuPEGBWFXIepGJaP3pL2iroJuG4tS8+2xIop+XB08+9A443K6V6+4FWu9Vv/X iGqAW0CP1Ea8eTSdLX2lAgwhdM7qRZNx8nyrOJYwKlgxeSO3VbkTyCHxw+qgy01kYhbfZN4hkzh G9jcD7lMYRxlC+/eBD50cWmwdWQhp91L8Iw/hmHG9PXUba+QHuttbfkva9R2NNujMpbpG6N2kvc YEN/d6wUcHarQQ2imDOPJUyDyZmmybo6IhqSdmcuH108Qshyn/7wcAgjE0bT6u+ED/qCovFepXd rOwYYjGWucVUmnmLxE+GMt0gKUyaciDPn473WyeSCivMbkVZwB5BqJ3Eyv9aUQRjq/4SCVBkQd9 LXyg30/HIcjtunznb/n9r1NLyoa71eJZ2FUi5CjL5QXTIPnk4TCn3ssH+NHR8Ayl167NqAmYky8 aXlsn2EyKfrjwa6v3NyyOWDSyfW6A2VnqG88hL7R69fDV16qGpf0B5OVo9wPrFDkeubIKyzJxlB m8= X-Received: by 2002:a05:600c:a08b:b0:48a:53cb:8604 with SMTP id 5b1f17b1804b1-48a78a536acmr28625155e9.14.1777373024680; Tue, 28 Apr 2026 03:43:44 -0700 (PDT) Received: from pumpkin (82-69-66-36.dsl.in-addr.zen.co.uk. [82.69.66.36]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-48a77af28e6sm41696855e9.6.2026.04.28.03.43.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 28 Apr 2026 03:43:44 -0700 (PDT) Date: Tue, 28 Apr 2026 11:43:42 +0100 From: David Laight To: Andy Shevchenko Cc: Yury Norov , Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , "H. Peter Anvin" , Andy Lutomirski , Peter Zijlstra , Jonathan Cameron , David Lechner , Johannes Berg , Nuno =?UTF-8?B?U8Oh?= , Andy Shevchenko , Ping-Ke Shih , Richard Cochran , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Alexandre Belloni , Yury Norov , Rasmus Villemoes , Hans de Goede , Linus Walleij , Sakari Ailus , Salah Triki , Achim Gratz , Ben Collins , x86@kernel.org, linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org, linux-wireless@vger.kernel.org, netdev@vger.kernel.org, linux-rtc@vger.kernel.org Subject: Re: [PATCH v2 7/9] wifi: rtw89: switch to using FIELD_GET_SIGNED() Message-ID: <20260428114342.0d674fd8@pumpkin> In-Reply-To: References: <20260427214127.406067-1-ynorov@nvidia.com> <20260427214127.406067-8-ynorov@nvidia.com> X-Mailer: Claws Mail 4.1.1 (GTK 3.24.38; arm-unknown-linux-gnueabihf) Precedence: bulk X-Mailing-List: linux-rtc@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Tue, 28 Apr 2026 10:10:22 +0300 Andy Shevchenko wrote: > On Mon, Apr 27, 2026 at 05:41:24PM -0400, Yury Norov wrote: > > Switch from sign_extend32(FIELD_GET()) to the dedicated > > FIELD_GET_SIGNED() and don't calculate the fields length explicitly. > > ... > > > for (i = 0; i < ADDC_T_AVG; i++) { > > tmp = rtw89_phy_read32_mask(rtwdev, R_DBG32_D, MASKDWORD); > > - dc_re += sign_extend32(FIELD_GET(0xfff000, tmp), 11); > > - dc_im += sign_extend32(FIELD_GET(0xfff, tmp), 11); > > + dc_re += FIELD_GET_SIGNED(0xfff000, tmp); > > + dc_im += FIELD_GET_SIGNED(0xfff, tmp); > > In the same driver the GENMASK() is being used, why not doing it here while at it? To me those bit masks look more readable than the GENMASK() calls would be. David > > > } > > ... > > > for (i = 0; i < ADDC_T_AVG; i++) { > > tmp = rtw89_phy_read32_mask(rtwdev, R_DBG32_D, MASKDWORD); > > - dc_re += sign_extend32(FIELD_GET(0xfff000, tmp), 11); > > - dc_im += sign_extend32(FIELD_GET(0xfff, tmp), 11); > > + dc_re += FIELD_GET_SIGNED(0xfff000, tmp); > > + dc_im += FIELD_GET_SIGNED(0xfff, tmp); > > } > > Ditto, and it even looks like the same piece repeating twice in different > compilation units of the same driver... >