From patchwork Mon Nov 20 14:10:45 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andy Shevchenko X-Patchwork-Id: 167168 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:9910:0:b0:403:3b70:6f57 with SMTP id i16csp2228362vqn; Mon, 20 Nov 2023 06:13:47 -0800 (PST) X-Google-Smtp-Source: AGHT+IER4CSt43gx1x+gW8lXk/WQIwHcVxabwmH7TBKUz3lujbLo4d5lUXk1o2tci8VTzfg1h9WM X-Received: by 2002:a05:6a20:729b:b0:188:75c:e67c with SMTP id o27-20020a056a20729b00b00188075ce67cmr5893906pzk.44.1700489627622; Mon, 20 Nov 2023 06:13:47 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1700489627; cv=none; d=google.com; s=arc-20160816; b=jeNurA3AKiBkzZO5OTJ8/Il2YueRcsPb9bW6n3lHig2fqRe4AzW/hHt2B+n3LF6oGT je11i8bZqBt5OGZjzgBsxvx/uI9r5/CVXwc1k5JRV58fnhvp3NJT/SwOpfeX87/z70MK zl2v3LKhI0aZOLGdU1hKUZ2BpSRenEqVzlo4jVBaWs0/W9HiFYWI+Rley08/5XIxdMc5 EiouGhuR5HJjV4P30C5IqmUTNPtJ44qiZDJXsfLpicmjQItnYg91hP5vpegMpul6ozao /3iWRscLeixWC9A1syAJX/kNGZZFd5spFOMk7mGmNAE+rWSF87airPK+i746oZ5GWOto mLTg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=I2G4bt9m5NQncAk0Bbl8KfnsjVZlVAc18V5Q0gYSy7c=; fh=FKFziR/iyL6nLEsSp1Wcfxi2t3uP2D84uB4BdJKSFcA=; b=nQlq5VDzAIfXCWLsc/6UW5rc7GH0FRMlcjQ9KhVPQo7/PVjig+/v46JI/VOGe6Ik3N v2zsUjDAhjJPVl53tv+2Uaqu6Dz83x9wvAvqLK6N+Z8VyrJmY4Vu+fTAAzZCHqlYSbD7 s8eMrjvue0TAdBIZsZ5c5VPYynn9sXNc0MZ3xoCT9LhRmZbgt2SjfqxmSriZkqwYrD2n 8XIsVjRWL45kWrRXi2y8SmyEfsvO1slZBK3iA8PRs32+aESpfJ8fqwZnes3oCnw/kxz5 OamAkx1a7+9xXgpyYbbYZzZTyTc3bEp0Ey8Zu92iU6riuRb/8pT3zdwh1EmT9d1g8oVE vX7Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=P6c6bdC1; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from agentk.vger.email (agentk.vger.email. [2620:137:e000::3:2]) by mx.google.com with ESMTPS id b21-20020a056a000a9500b006be062ed5c6si8297970pfl.276.2023.11.20.06.13.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 20 Nov 2023 06:13:47 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) client-ip=2620:137:e000::3:2; Authentication-Results: mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=P6c6bdC1; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by agentk.vger.email (Postfix) with ESMTP id 6ADB480407C1; Mon, 20 Nov 2023 06:13:45 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at agentk.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232759AbjKTONT (ORCPT + 27 others); Mon, 20 Nov 2023 09:13:19 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49312 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233099AbjKTONM (ORCPT ); Mon, 20 Nov 2023 09:13:12 -0500 Received: from mgamail.intel.com (mgamail.intel.com [134.134.136.100]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7D6C0CA; Mon, 20 Nov 2023 06:13:08 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1700489588; x=1732025588; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=TuRKMqc5s8ctrxhZQRN1hc+qrqht23N2CXc03YDUn4k=; b=P6c6bdC1IrSiP2WRgGJkAhfhsOD0jsZ2PIvM3mzyR6d8wWZrNYO0caZF 0bYJ+grMhjC6Io09S7Zig2tXYA3mSnbefJPPzd5xbQ4gvNuPr7HP/tqTD Jw3C4GpzwiaZWHgU2hywRaBuz3NEDO4rvpr8fJcMLce1Kx1fwi8A9/haU xeQK55TCfVrHxmpji1g5DZ3/YO2O8bzwxckHtJXkQez9oD6AoTKGrCCFK L5fFEgMJDaLAQ8f5HDZIZBCzOsEUNuaGAajEcNfprSB3cFr/KCGgSyiD8 ypeVCOczodczFcw602lpJVlEQIZ9zoT9YDMFflo26C+fM5xY8eJJExmuG g==; X-IronPort-AV: E=McAfee;i="6600,9927,10900"; a="458121759" X-IronPort-AV: E=Sophos;i="6.04,214,1695711600"; d="scan'208";a="458121759" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Nov 2023 06:12:37 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10900"; a="801184235" X-IronPort-AV: E=Sophos;i="6.04,214,1695711600"; d="scan'208";a="801184235" Received: from black.fi.intel.com ([10.237.72.28]) by orsmga001.jf.intel.com with ESMTP; 20 Nov 2023 06:12:34 -0800 Received: by black.fi.intel.com (Postfix, from userid 1003) id 0C7092CA; Mon, 20 Nov 2023 16:12:32 +0200 (EET) From: Andy Shevchenko To: Andy Shevchenko , linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Tony Luck , Borislav Petkov , James Morse , Mauro Carvalho Chehab , Robert Richter , Qiuxu Zhuo Subject: [PATCH v2 1/4] EDAC, pnd2: Replace custom definition by one from sizes.h Date: Mon, 20 Nov 2023 16:10:45 +0200 Message-ID: <20231120141231.1638240-1-andriy.shevchenko@linux.intel.com> X-Mailer: git-send-email 2.43.0.rc1.1.gbec44491f096 MIME-Version: 1.0 X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (agentk.vger.email [0.0.0.0]); Mon, 20 Nov 2023 06:13:45 -0800 (PST) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1783092611284430000 X-GMAIL-MSGID: 1783092611284430000 The sizes.h provides a set of common size definitions, use it. Reviewed-by: Qiuxu Zhuo Signed-off-by: Andy Shevchenko --- v2: added tag (Qiuxu) drivers/edac/pnd2_edac.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/edac/pnd2_edac.c b/drivers/edac/pnd2_edac.c index 2b306f2cc605..45e3c2913d51 100644 --- a/drivers/edac/pnd2_edac.c +++ b/drivers/edac/pnd2_edac.c @@ -24,6 +24,7 @@ #include #include #include +#include #include #include #include @@ -109,7 +110,6 @@ static struct mem_ctl_info *pnd2_mci; #define MOT_CHAN_INTLV_BIT_1SLC_2CH 12 #define MOT_CHAN_INTLV_BIT_2SLC_2CH 13 #define SELECTOR_DISABLED (-1) -#define _4GB (1ul << 32) #define PMI_ADDRESS_WIDTH 31 #define PND_MAX_PHYS_BIT 39 @@ -587,7 +587,7 @@ static int get_registers(void) /* Get a contiguous memory address (remove the MMIO gap) */ static u64 remove_mmio_gap(u64 sys) { - return (sys < _4GB) ? sys : sys - (_4GB - top_lm); + return (sys < SZ_4G) ? sys : sys - (SZ_4G - top_lm); } /* Squeeze out one address bit, shift upper part down to fill gap */ @@ -643,7 +643,7 @@ static int sys2pmi(const u64 addr, u32 *pmiidx, u64 *pmiaddr, char *msg) /* Give up if address is out of range, or in MMIO gap */ if (addr >= (1ul << PND_MAX_PHYS_BIT) || - (addr >= top_lm && addr < _4GB) || addr >= top_hm) { + (addr >= top_lm && addr < SZ_4G) || addr >= top_hm) { snprintf(msg, PND2_MSG_SIZE, "Error address 0x%llx is not DRAM", addr); return -EINVAL; } From patchwork Mon Nov 20 14:10:46 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andy Shevchenko X-Patchwork-Id: 167166 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:9910:0:b0:403:3b70:6f57 with SMTP id i16csp2228254vqn; Mon, 20 Nov 2023 06:13:38 -0800 (PST) X-Google-Smtp-Source: AGHT+IG450eZnmMyXhfTxpvZqxmQohZh1ipk4/7nXWi2XSS8fm9YLNISVtkNReyRRKRUuPV/6zoO X-Received: by 2002:a05:6808:dc1:b0:3ac:4ee2:337b with SMTP id g1-20020a0568080dc100b003ac4ee2337bmr1921490oic.51.1700489618639; Mon, 20 Nov 2023 06:13:38 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1700489618; cv=none; d=google.com; s=arc-20160816; b=E8VpZcZC838rEX4iUJu4eo0jMtwRNqBSwpvYfiCWG6SlVMWQ8Hns3GMqHsnXMwWD+N 2UWtW1KEx0l05ZuMXV9ybSnL9M3HEjb+ezRffR3ydHPvH8+u01BxFN429OnvpTX7w8aG xK9qm9oXsedsqHPAKond88bgRPGLOxtH43GILCXLQTR4iyHFypxRjW9L55IaloxV4NiL p9ltZvwcMdugTIg8IOP2m7pKqmJndHKLgxZ+ewxW4jYe22uzifPveNLV8xOgIbIPDgSw EmTlpgOx8qk7dJNHZMnEPBl2gRJK33TMZe7KHHY5UQZhKt7wGfnUfAiY+kfwu08PVhJi fwRQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=OL/r1f4aeZXeyiNdFpzr/UP9VKt6MD5yDz/5kQCjndY=; fh=BG3rc2tX6F6U2A1M/aDY9FKtoOLe4vQznm2A17N8JnE=; b=ZEjv9eaMFD6VvLcF62Qi0jblnEE0M4UaVyJUg4/RC6huCVkVaABJUR9b1eEnuYkE7B Znoz75SfS6UfhqtEt+1l80VdbIwOidgHvRe+7BMsySYOO4SbQfVgWH5LdeEfGnuXfOSu /89A8F7etuH0lqL5V1ePIZY5OkAbTXsgPNBx1PGm/QNHif1g2PdBD9hi99VXwFH8jnVE 2KHcO0sQ37pf/Y6wi9P3p+HOm7kRCtAvnv95TebDAli6tNTNaexHNlOcO7AD2yUDstB9 y/Fw8AD/6LqbIwyef0dfU5VcVGWtmFWRtv1HJAWWvKOxLOWlyyNARUNLf8CAM74oOXX1 3gig== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=TxSBSO+9; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from lipwig.vger.email (lipwig.vger.email. [23.128.96.33]) by mx.google.com with ESMTPS id r21-20020a635155000000b0058974d3c296si8432100pgl.815.2023.11.20.06.13.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 20 Nov 2023 06:13:38 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) client-ip=23.128.96.33; Authentication-Results: mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=TxSBSO+9; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by lipwig.vger.email (Postfix) with ESMTP id B961C803890A; Mon, 20 Nov 2023 06:13:35 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at lipwig.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233331AbjKTONQ (ORCPT + 27 others); Mon, 20 Nov 2023 09:13:16 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33838 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232866AbjKTONL (ORCPT ); Mon, 20 Nov 2023 09:13:11 -0500 Received: from mgamail.intel.com (mgamail.intel.com [134.134.136.100]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 21CA3BA; Mon, 20 Nov 2023 06:13:08 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1700489588; x=1732025588; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=dcBH8dkNNXnKF0j8ud1kjx2WkqUBoV7nIP12in5A7ms=; b=TxSBSO+96S4nOTT7Yf2deTYfI8sCLlpIaN6Tlifb2d52DOIOrsHFDZ5o nKk+MscvvOE36MwYo3BbRNbWXsRe0Rld4Yn/nAPiJ8BzjZHHvSI3kHnUB 1kYvK7XlD2ewyuAZD8bU/wx90VuFpCRZFhqK3Emfg3LNEpMNWiCu9wmbG paMzndE/UobhONmQC79En+zvIK4pQZpYKc8/Vs8AZmhWB2JC+VOSVPFf/ mgb02HO81MIblsbjyv5dQiCsuoTGJ6pgzg94soNdjdL1xWZdIYwVV2cF+ +vA9Lo3/bnALAzcHfS41qvoB2zi51jM7b64xW4VtcnLK4PqNAhxL8O97Q Q==; X-IronPort-AV: E=McAfee;i="6600,9927,10900"; a="458121751" X-IronPort-AV: E=Sophos;i="6.04,214,1695711600"; d="scan'208";a="458121751" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Nov 2023 06:12:37 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10900"; a="801184238" X-IronPort-AV: E=Sophos;i="6.04,214,1695711600"; d="scan'208";a="801184238" Received: from black.fi.intel.com ([10.237.72.28]) by orsmga001.jf.intel.com with ESMTP; 20 Nov 2023 06:12:34 -0800 Received: by black.fi.intel.com (Postfix, from userid 1003) id 219BB24F; Mon, 20 Nov 2023 16:12:33 +0200 (EET) From: Andy Shevchenko To: Andy Shevchenko , linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Tony Luck , Borislav Petkov , James Morse , Mauro Carvalho Chehab , Robert Richter Subject: [PATCH v2 2/4] EDAC, pnd2: Apply bit macros and helpers where it makes sense Date: Mon, 20 Nov 2023 16:10:46 +0200 Message-ID: <20231120141231.1638240-2-andriy.shevchenko@linux.intel.com> X-Mailer: git-send-email 2.43.0.rc1.1.gbec44491f096 In-Reply-To: <20231120141231.1638240-1-andriy.shevchenko@linux.intel.com> References: <20231120141231.1638240-1-andriy.shevchenko@linux.intel.com> MIME-Version: 1.0 X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lipwig.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (lipwig.vger.email [0.0.0.0]); Mon, 20 Nov 2023 06:13:35 -0800 (PST) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1783092602423581591 X-GMAIL-MSGID: 1783092602423581591 Apply bit macros (BIT()/BIT_ULL()/GENMASK()/etc) and helpers (for_each_set_bit()/etc) where it makes sense. Signed-off-by: Andy Shevchenko Reviewed-by: Qiuxu Zhuo --- v2: fixed uninitialized variable (lkp, Qiuxu), dropped mask check change (Qiuxu) drivers/edac/pnd2_edac.c | 32 ++++++++++++++------------------ 1 file changed, 14 insertions(+), 18 deletions(-) diff --git a/drivers/edac/pnd2_edac.c b/drivers/edac/pnd2_edac.c index 45e3c2913d51..ee7c9b024354 100644 --- a/drivers/edac/pnd2_edac.c +++ b/drivers/edac/pnd2_edac.c @@ -183,7 +183,7 @@ static int _apl_rd_reg(int port, int off, int op, u32 *data) } P2SB_READ(dword, P2SB_DATA_OFF, data); - ret = (status >> 1) & 0x3; + ret = (status >> 1) & GENMASK(1, 0); out: /* Hide the P2SB device, if it was hidden before */ if (hidden) @@ -307,7 +307,7 @@ static bool two_channels; /* Both PMI channels in one slice enabled */ static u8 sym_chan_mask; static u8 asym_chan_mask; -static u8 chan_mask; +static unsigned long chan_mask; static int slice_selector = -1; static int chan_selector = -1; @@ -598,7 +598,7 @@ static void remove_addr_bit(u64 *addr, int bitidx) if (bitidx == -1) return; - mask = (1ull << bitidx) - 1; + mask = BIT_ULL(bitidx) - 1; *addr = ((*addr >> 1) & ~mask) | (*addr & mask); } @@ -642,7 +642,7 @@ static int sys2pmi(const u64 addr, u32 *pmiidx, u64 *pmiaddr, char *msg) int sym_chan_shift = sym_channels >> 1; /* Give up if address is out of range, or in MMIO gap */ - if (addr >= (1ul << PND_MAX_PHYS_BIT) || + if (addr >= BIT(PND_MAX_PHYS_BIT) || (addr >= top_lm && addr < SZ_4G) || addr >= top_hm) { snprintf(msg, PND2_MSG_SIZE, "Error address 0x%llx is not DRAM", addr); return -EINVAL; @@ -727,10 +727,10 @@ static int sys2pmi(const u64 addr, u32 *pmiidx, u64 *pmiaddr, char *msg) } /* Translate PMI address to memory (rank, row, bank, column) */ -#define C(n) (0x10 | (n)) /* column */ -#define B(n) (0x20 | (n)) /* bank */ -#define R(n) (0x40 | (n)) /* row */ -#define RS (0x80) /* rank */ +#define C(n) (BIT(4) | (n)) /* column */ +#define B(n) (BIT(5) | (n)) /* bank */ +#define R(n) (BIT(6) | (n)) /* row */ +#define RS (BIT(7)) /* rank */ /* addrdec values */ #define AMAP_1KB 0 @@ -1064,9 +1064,9 @@ static int apl_check_ecc_active(void) int i, ret = 0; /* Check dramtype and ECC mode for each present DIMM */ - for (i = 0; i < APL_NUM_CHANNELS; i++) - if (chan_mask & BIT(i)) - ret += check_channel(i); + for_each_set_bit(i, &chan_mask, APL_NUM_CHANNELS) + ret += check_channel(i); + return ret ? -EINVAL : 0; } @@ -1205,10 +1205,7 @@ static void apl_get_dimm_config(struct mem_ctl_info *mci) u64 capacity; int i, g; - for (i = 0; i < APL_NUM_CHANNELS; i++) { - if (!(chan_mask & BIT(i))) - continue; - + for_each_set_bit(i, &chan_mask, APL_NUM_CHANNELS) { dimm = edac_get_dimm(mci, i, 0, 0); if (!dimm) { edac_dbg(0, "No allocated DIMM for channel %d\n", i); @@ -1228,8 +1225,7 @@ static void apl_get_dimm_config(struct mem_ctl_info *mci) } pvt->dimm_geom[i] = g; - capacity = (d->rken0 + d->rken1) * 8 * (1ul << dimms[g].rowbits) * - (1ul << dimms[g].colbits); + capacity = (d->rken0 + d->rken1) * 8 * BIT(dimms[g].rowbits + dimms[g].colbits); edac_dbg(0, "Channel %d: %lld MByte DIMM\n", i, capacity >> (20 - 3)); dimm->nr_pages = MiB_TO_PAGES(capacity >> (20 - 3)); dimm->grain = 32; @@ -1295,7 +1291,7 @@ static void dnv_get_dimm_config(struct mem_ctl_info *mci) continue; } - capacity = ranks_of_dimm[j] * banks * (1ul << rowbits) * (1ul << colbits); + capacity = ranks_of_dimm[j] * banks * BIT(rowbits + colbits); edac_dbg(0, "Channel %d DIMM %d: %lld MByte DIMM\n", i, j, capacity >> (20 - 3)); dimm->nr_pages = MiB_TO_PAGES(capacity >> (20 - 3)); dimm->grain = 32; From patchwork Mon Nov 20 14:10:47 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andy Shevchenko X-Patchwork-Id: 167165 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:9910:0:b0:403:3b70:6f57 with SMTP id i16csp2228087vqn; Mon, 20 Nov 2023 06:13:27 -0800 (PST) X-Google-Smtp-Source: AGHT+IGYnHz8Mt8PF0YffoqlCmHHYwz38PTjHytl+vkh1RtsC/9OcS3A+s9vQ5dUue+dKvNkuvoP X-Received: by 2002:a05:6a20:4305:b0:181:a784:67df with SMTP id h5-20020a056a20430500b00181a78467dfmr19285278pzk.20.1700489606946; Mon, 20 Nov 2023 06:13:26 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1700489606; cv=none; d=google.com; s=arc-20160816; b=WtbeQR947EAq+1vlI140z/wdn9O7mhCZ4NNm6q7AZFTclfpXA7rMDkt9sg1HXGDm40 6iHwR2E+t9FuFaBr1+fo2lzSBjrr3nEHcp7gCMIPBJOl0V8le6eFZETYkuZnTTCgi3rU U4L4pAgacbazhNCo6NPeSD8xyZIiYLe9lRlK9rXXglARG/ApD/xuIeFJPyz5g9dTlO9C U2PkYFbHOaXM8mS9Sx9Ohx409ZRoliBBegqfL7jhJTIEOlazkKn05aw3K8X02LIfQmfx vQSos+yl0foWweOpMCNDglqvuV46TSGhEG1SJGDf9JNrSMzTlT06PWosdGWEaHGys2XP PI9w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=4w7l8+gkL8jy3vFxe/stI/iS+p9FeWAy0tEGqMDjP74=; fh=FKFziR/iyL6nLEsSp1Wcfxi2t3uP2D84uB4BdJKSFcA=; b=Dlsiq/4Jj3gC34n+SJIr2zy/aVIx/5geZPDTb2bEAp+rYdB+Q0EqrnmiuKi5KnzJjK fW3YvGF8cV6N78jv4aZxKbITY6PnPTBlA1RJUARZtCTCW1pbGp9Q4YOxoZsjvUgj2EvW ecCwfEM0Aj3/CRsANoMaNVY5iqmcNfOnH+9m8rat7foawBKeJBbA3hnWaOy7NDpQVje2 DQiZlMOak4e7n+YXEDhqssugqhHUHOL1RAKruehLlA21mhwGKYBbonblicFgqbZ1p5gf /hjlDw4J3MXS6kSqrJTWzAESBxR/UOcacIV12jSGXeG8l1B3h1HHK6pCNbOhU6QHnN3m 4WUQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=js7MhCg2; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from lipwig.vger.email (lipwig.vger.email. [23.128.96.33]) by mx.google.com with ESMTPS id l123-20020a632581000000b00577f7b6e7b4si8330264pgl.624.2023.11.20.06.13.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 20 Nov 2023 06:13:26 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) client-ip=23.128.96.33; Authentication-Results: mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=js7MhCg2; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by lipwig.vger.email (Postfix) with ESMTP id 86D2D8035129; Mon, 20 Nov 2023 06:13:20 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at lipwig.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232947AbjKTONJ (ORCPT + 27 others); Mon, 20 Nov 2023 09:13:09 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33822 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232759AbjKTONI (ORCPT ); Mon, 20 Nov 2023 09:13:08 -0500 Received: from mgamail.intel.com (mgamail.intel.com [134.134.136.100]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5CB21BA; Mon, 20 Nov 2023 06:13:04 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1700489584; x=1732025584; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=1DDuOJefk+3o6ygYL1H6D6JOyUtWSGokeA8lulBT4/o=; b=js7MhCg2iKZhxWkxlLu3Q4w5+PoKPgLwl5qDj1TQxAXUiHX9IF+UG9FP z0dKwd792mJbcjZop+9QAsd6jto6ApdG7S03c0TFPu2+M+em/5c0M5Xng 8Mn57W2wHU4NAPYTuqsjGA69e4gWEJuSJIHu6GQa11sc/lnNWEjSYvFAq 1Gl05nxV+cF9l0kRBrqWWr9wcL6HasOme8GB5dSkHcHWqXFY+jEvfPfnd KTO6n7D1ykrzvszKPoZbus/IcLsLODlNEEDlBqA9VTaElOMlFgcEGf6qD c3SGQexN4V+KxlKxpMmkTWx8dj8F72xb0e8CWNPmge8TMx/ek4P4NU5nw Q==; X-IronPort-AV: E=McAfee;i="6600,9927,10900"; a="458121746" X-IronPort-AV: E=Sophos;i="6.04,214,1695711600"; d="scan'208";a="458121746" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Nov 2023 06:12:37 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10900"; a="801184237" X-IronPort-AV: E=Sophos;i="6.04,214,1695711600"; d="scan'208";a="801184237" Received: from black.fi.intel.com ([10.237.72.28]) by orsmga001.jf.intel.com with ESMTP; 20 Nov 2023 06:12:34 -0800 Received: by black.fi.intel.com (Postfix, from userid 1003) id 2BDE72F0; Mon, 20 Nov 2023 16:12:33 +0200 (EET) From: Andy Shevchenko To: Andy Shevchenko , linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Tony Luck , Borislav Petkov , James Morse , Mauro Carvalho Chehab , Robert Richter , Qiuxu Zhuo Subject: [PATCH v2 3/4] EDAC, pnd2: Correct misleading error message in mk_region_mask() Date: Mon, 20 Nov 2023 16:10:47 +0200 Message-ID: <20231120141231.1638240-3-andriy.shevchenko@linux.intel.com> X-Mailer: git-send-email 2.43.0.rc1.1.gbec44491f096 In-Reply-To: <20231120141231.1638240-1-andriy.shevchenko@linux.intel.com> References: <20231120141231.1638240-1-andriy.shevchenko@linux.intel.com> MIME-Version: 1.0 X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lipwig.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (lipwig.vger.email [0.0.0.0]); Mon, 20 Nov 2023 06:13:20 -0800 (PST) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1783092589993242643 X-GMAIL-MSGID: 1783092589993242643 The mask parameter is expected to be of a sequence of the set bits. It does not mean it must be power of two (only single bit set). Correct misleading error message. Suggested-by: Qiuxu Zhuo Signed-off-by: Andy Shevchenko --- v2: new patch as suggested (Qiuxu) drivers/edac/pnd2_edac.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/edac/pnd2_edac.c b/drivers/edac/pnd2_edac.c index ee7c9b024354..969fb2465edb 100644 --- a/drivers/edac/pnd2_edac.c +++ b/drivers/edac/pnd2_edac.c @@ -329,7 +329,7 @@ static void mk_region_mask(char *name, struct region *rp, u64 base, u64 mask) return; } if (mask != GENMASK_ULL(PND_MAX_PHYS_BIT, __ffs(mask))) { - pr_info(FW_BUG "MOT mask not power of two\n"); + pr_info(FW_BUG "MOT mask is invalid\n"); return; } if (base & ~mask) { From patchwork Mon Nov 20 14:10:48 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andy Shevchenko X-Patchwork-Id: 167167 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:9910:0:b0:403:3b70:6f57 with SMTP id i16csp2228305vqn; Mon, 20 Nov 2023 06:13:43 -0800 (PST) X-Google-Smtp-Source: AGHT+IF7/CkxyZlskJHy56pbcwj+FzetpD2bG680WOimM01dxIQUPh6h9z4eKar7h3OwtXw1Zxp8 X-Received: by 2002:a05:6a20:748d:b0:187:2e65:9cc9 with SMTP id p13-20020a056a20748d00b001872e659cc9mr5557719pzd.56.1700489622822; Mon, 20 Nov 2023 06:13:42 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1700489622; cv=none; d=google.com; s=arc-20160816; b=HelufBUnLUAGc+OBlRsTDbon9202kzZwrimesQBVlyzY+XZ55kNVCD83gutXq3Fqf7 5jkB92oxG3lIZOf0whgrfRo9hsKEmsjEKuP5CHGAEQ3EwDbWIiR2A1xBpNKY/IAQz7S5 IP4IyJBs4S6MeCvtsK4sbyWnsCVoH464cTnsks3H/BuEC0EV3Ds7aHKXsnj4aqWJ9rhd GBVwjb+CUm8PKOdbJWVx6RiGVvXXFGOcgXeqNrNQ7A38jWm7tHrAL2IuQSfmcEdjOSBT ZdBcD/reohgALOEGkLQCndUyDOtaripkowUzNLoJ12idgaPFCo+N+UzV/XOBDfnsSqsG d6Rg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=Pc04N6JfQm72Zg3GmhZEgY4VTHV0ZcNibHrTX+tB2Bk=; fh=FKFziR/iyL6nLEsSp1Wcfxi2t3uP2D84uB4BdJKSFcA=; b=Orx6fVT+aFRFw0qhQy4qQ3tdWQcaOS85fL+Q4jx0GIk2n3q0AU8iyzFtB1lmmowSxK OHDUp6Pf9REsBuvw2HEM4hnnASxAfRoz2TaqIZjWpOpyNDKo1KsQ9x4dKEHvXh1i8u0+ /oWhG4Zxv8gDKRPqu1lIpfayWlGoOLuv/WLnxc56Di/ndHB8yeU/inHbZj183XacWZJu AiA0fqumdppn6Q5ebBnexcdBj039FbTR0W9hUeM4vcvdqt4pxNHKrOF6r2cAMZSaToc9 YZ70+dhTJb+NXWTP25q7TcmKYeQ7R1GDY4nnpevVL2w0a1djm2L7CO9AXmjCzD7c21bS ntgg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=iO19RStE; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.38 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from fry.vger.email (fry.vger.email. [23.128.96.38]) by mx.google.com with ESMTPS id w190-20020a6382c7000000b005a9fde46f98si8023135pgd.52.2023.11.20.06.13.42 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 20 Nov 2023 06:13:42 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.38 as permitted sender) client-ip=23.128.96.38; Authentication-Results: mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=iO19RStE; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.38 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by fry.vger.email (Postfix) with ESMTP id C80798061C16; Mon, 20 Nov 2023 06:13:35 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at fry.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233139AbjKTONN (ORCPT + 27 others); Mon, 20 Nov 2023 09:13:13 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33832 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232759AbjKTONK (ORCPT ); Mon, 20 Nov 2023 09:13:10 -0500 Received: from mgamail.intel.com (mgamail.intel.com [134.134.136.100]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 90F9DA0; Mon, 20 Nov 2023 06:13:07 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1700489587; x=1732025587; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=XKZFQUi+U4ersNqM4YmQM70MnyE4Os/TQpc7j9Ab95o=; b=iO19RStEWjyQLAwIbTLZEi7AOOE4fHs1zJvBjy5sUCbH5AH+5KktXaSm Y7SJzDPSMDuC2qf0xiDtVR/Gq0AVUCv2wjEyN22RRB0nI2Cu5tlr92gBv bVL9HOjngVvrRsPGUwmyfQGFM0dbNsofAuN4rpOYbc+RKuWR0ge7S8UVi At3/LhgF238bvxXwQ1gcSGp4AR/YpNZTA3X8yLLmJ0kCKG0Gg5YYddUXU olmlHWz7ExltKUGiUNQKtxiE/tlrhGRm/p8BqSWyOJbl15YTbRDtjtVar QeQkM+w7sW0DTPPYV83TM+ZgVZrcXtw6OAPGeowQ6S0sATogxUJ32bOIf Q==; X-IronPort-AV: E=McAfee;i="6600,9927,10900"; a="458121755" X-IronPort-AV: E=Sophos;i="6.04,214,1695711600"; d="scan'208";a="458121755" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Nov 2023 06:12:37 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10900"; a="801184239" X-IronPort-AV: E=Sophos;i="6.04,214,1695711600"; d="scan'208";a="801184239" Received: from black.fi.intel.com ([10.237.72.28]) by orsmga001.jf.intel.com with ESMTP; 20 Nov 2023 06:12:34 -0800 Received: by black.fi.intel.com (Postfix, from userid 1003) id 3DA4E4F8; Mon, 20 Nov 2023 16:12:33 +0200 (EET) From: Andy Shevchenko To: Andy Shevchenko , linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Tony Luck , Borislav Petkov , James Morse , Mauro Carvalho Chehab , Robert Richter , Qiuxu Zhuo Subject: [PATCH v2 4/4] EDAC, pnd2: Sort headers alphabetically Date: Mon, 20 Nov 2023 16:10:48 +0200 Message-ID: <20231120141231.1638240-4-andriy.shevchenko@linux.intel.com> X-Mailer: git-send-email 2.43.0.rc1.1.gbec44491f096 In-Reply-To: <20231120141231.1638240-1-andriy.shevchenko@linux.intel.com> References: <20231120141231.1638240-1-andriy.shevchenko@linux.intel.com> MIME-Version: 1.0 X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on fry.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (fry.vger.email [0.0.0.0]); Mon, 20 Nov 2023 06:13:35 -0800 (PST) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1783092606478783013 X-GMAIL-MSGID: 1783092606478783013 Sort the headers in alphabetic order in order to ease the maintenance for this part. Reviewed-by: Qiuxu Zhuo Signed-off-by: Andy Shevchenko --- v2: added tag (Qiuxu) drivers/edac/pnd2_edac.c | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/drivers/edac/pnd2_edac.c b/drivers/edac/pnd2_edac.c index 969fb2465edb..2afcd148fcf8 100644 --- a/drivers/edac/pnd2_edac.c +++ b/drivers/edac/pnd2_edac.c @@ -16,19 +16,20 @@ * rank, bank, row and column using the appropriate "dunit_ops" functions/parameters. */ -#include -#include -#include -#include -#include +#include #include #include -#include -#include -#include -#include +#include #include +#include #include +#include +#include +#include +#include +#include +#include + #include #include