From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f44.google.com (mail-wm1-f44.google.com [209.85.128.44]) (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 86C763F9F2C for ; Tue, 28 Apr 2026 10:43:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.44 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777373027; cv=none; b=sFQh9gavMA5hNrb13/+gMWt3hAhW9XhrzhrF5i7pPdy5HLacqz66AMrGIPGXY3GnT2iwDea4fGdTmHmaJPl9qfYmBnoFrttMtcSCu9Aqceed8Gn+V3+pwXwYESqgsA2vRa3imikHsPWPvM99DIest9NTSGq7tAtrtaTeLDUKpEU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777373027; c=relaxed/simple; bh=Txo9jXdDRugPyX7qmDybQ2dYpzzLy0YS2OMMBLInikE=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=RiKCgGR0u29WYd4hx8Byvkee5VhAk5yjgxAmRFYyiBKQt4bX6MSX2kviXWCMRTB8p/CnNdc7qP2ipaexldAoEYx007PakePnzVaxL9o+4qFEjOF3Iyizp9KYXnq8dWFmEfFisYKbAu+j5c4GFFwTRODVyOnV+LdQUSXzX2X4a2M= 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.44 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-f44.google.com with SMTP id 5b1f17b1804b1-4891f625344so101892685e9.0 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=LbBokfN6w0gqr/cupZengV9OLkS71DYjj69CPEYQo4M+Vo/DTbgNVO1ReZmgjhsFXR RzFnQG4+ghZEaXajtvFgCDgsSBKkRz2YXUzDpLVY+Aeb0B9T9TeupbTYNPl6boWs563p jEruo24moz4B8eqqWbNftQzLDPA+j+SorUaHVuqh81hPqM3Nd/J2cZ9CTZdXFtxwl6uT +kbh4OsQV7xxTFKDcLjMqui0m49M0QwdvwRxwDRa2FLINPiITfA1HHicy/fFLAVdb8Jq ceICEmE0gbDhdo7xOLV3PnQnHJfhhOOmWlM1SbZ3H+yA5Oocxk0vn/ER8BiDFTua7fkj /kGQ== X-Forwarded-Encrypted: i=1; AFNElJ83Sp1LKjk109acbzNS4G8Oay0wK8zMC4/bfuxC/06fGqkzMBqRDMl0rFVZm8utfQ9GhbGua98rtr4gTBg=@vger.kernel.org X-Gm-Message-State: AOJu0Yy5deAeVVG1w4dpuiQCB/IRZlJIYLcytOzGtwDb3sgBMQo7i0YP IDxOXSs9hcEnpYKlR2J4nCSSsNV1QUspUyQBg/6sE3Ca+dVrBo/gHwXC X-Gm-Gg: AeBDieuK/vBT+xkfAIbHXaslWDb29JMLIc0JqJ4fc8Z9/cnvQavco9KlQ/5OJ6V6UjB zdx0bOiOkUhfhyvBYdGlifYLP8msj6TJy9sVlCNVWSH/5ymMk/rfkLBB6j2D+Ulq2Z3MVdDfBoX Na089a/bOE2tVaV0rrtQqwdZlTpG460pdlRKv6TAbUBJnC27XJKCUhZ+dVsob/nCI+/SuAvVBIR +4LJqX5qrB9hMdqpZfoBjubwvPhECAlIUtJnWOhB72oIEaaAJqoNrOaat0AWRBbi4SFh2REgbXn dCgjffu04RP92F9VtAKbS2BMyW80fAAWy0ajF+6t58xRrBNnV03+sl/1dX1mD5v9pGj/jmagkeY Mj+W59D0i4cNdFMz9YtzzkObQ5uTT9DM/Dh7Y2L9YXCod6wFjU1sU0GVafYjoMvDNSobPzOUvMt 3BXzjdM7+6jxM/7WFhuSMNzfwiwABEZJz+eyjwAuvvn8YGQOadwDLpbiDsJ1QcmxUgTNO/tn/bm 7w= 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-kernel@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... >