{"id":2219701,"url":"http://patchwork.ozlabs.org/api/1.0/patches/2219701/?format=json","project":{"id":67,"url":"http://patchwork.ozlabs.org/api/1.0/projects/67/?format=json","name":"OpenSBI development","link_name":"opensbi","list_id":"opensbi.lists.infradead.org","list_email":"opensbi@lists.infradead.org","web_url":"https://github.com/riscv/opensbi","scm_url":"","webscm_url":""},"msgid":"<20260403202903.3407945-1-david.garcia@aheadcomputing.com>","date":"2026-04-03T20:29:03","name":"lib: utils/regmap: Fix reg_stride calculation in syscon regmap","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"23907a1bbbcd0d238e228f0b0805011f5fa487ca","submitter":{"id":92994,"url":"http://patchwork.ozlabs.org/api/1.0/people/92994/?format=json","name":"David E. Garcia Porras","email":"david.garcia@aheadcomputing.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/opensbi/patch/20260403202903.3407945-1-david.garcia@aheadcomputing.com/mbox/","series":[{"id":498674,"url":"http://patchwork.ozlabs.org/api/1.0/series/498674/?format=json","date":"2026-04-03T20:29:03","name":"lib: utils/regmap: Fix reg_stride calculation in syscon regmap","version":1,"mbox":"http://patchwork.ozlabs.org/series/498674/mbox/"}],"check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/2219701/checks/","tags":{},"headers":{"Return-Path":"\n ","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n secure) header.d=lists.infradead.org header.i=@lists.infradead.org\n header.a=rsa-sha256 header.s=bombadil.20210309 header.b=L6Oh87ov;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n unprotected) header.d=aheadcomputing.com header.i=@aheadcomputing.com\n header.a=rsa-sha256 header.s=selector1 header.b=KpC42Wru;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=none (no SPF record) smtp.mailfrom=lists.infradead.org\n (client-ip=2607:7c80:54:3::133; helo=bombadil.infradead.org;\n envelope-from=opensbi-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org;\n receiver=patchwork.ozlabs.org)"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n [IPv6:2607:7c80:54:3::133])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature ECDSA (secp384r1) server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fnVh15cjVz1xtJ\n\tfor ; Sat, 04 Apr 2026 07:29:32 +1100 (AEDT)","from localhost ([::1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux))\n\tid 1w8l8o-00000002XCf-0X9v;\n\tFri, 03 Apr 2026 20:29:18 +0000","from mail-westusazon11020100.outbound.protection.outlook.com\n ([52.101.85.100] helo=BYAPR05CU005.outbound.protection.outlook.com)\n\tby bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux))\n\tid 1w8l8l-00000002XC4-32YZ\n\tfor opensbi@lists.infradead.org;\n\tFri, 03 Apr 2026 20:29:16 +0000","from DS7P220CA0004.NAMP220.PROD.OUTLOOK.COM (2603:10b6:8:1ca::11) by\n CO6PR22MB2916.namprd22.prod.outlook.com (2603:10b6:303:14f::10) with\n Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9769.17; Fri, 3 Apr\n 2026 20:29:10 +0000","from DM2PEPF00003FC7.namprd04.prod.outlook.com\n (2603:10b6:8:1ca:cafe::f8) by DS7P220CA0004.outlook.office365.com\n (2603:10b6:8:1ca::11) with Microsoft SMTP Server (version=TLS1_3,\n cipher=TLS_AES_256_GCM_SHA384) id 15.20.9769.21 via Frontend Transport; Fri,\n 3 Apr 2026 20:29:10 +0000","from mail.aheadcomputing.com (50.45.236.54) by\n DM2PEPF00003FC7.mail.protection.outlook.com (10.167.23.24) with Microsoft\n SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9769.17\n via Frontend Transport; Fri, 3 Apr 2026 20:29:09 +0000","from davidAhead.lan (unknown [10.8.0.10])\n\tby mail.aheadcomputing.com (Postfix) with ESMTP id 3D39C80FC6;\n\tFri, 3 Apr 2026 13:29:08 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20210309; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:Subject:Cc\n\t:To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From:\n\tResent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:\n\tList-Owner; bh=WmpmWqkfgXxSWfjvIJpLUr3CmZpgPn+e/yQ2DJuFG34=; b=L6Oh87ovkUO2uc\n\tmmM2/syQdUfgc5mLCDeZVxmPwtdfMxDi2aPZU/ykr9/4osyEErS2PBLzbhwHC7TiIoj9cYdhpZyHS\n\tfmJCoStPn/uxVthsLgX4nWOXIYfYKRh9F0akXrdcd6tYukiK1yAgmR1UYTKyAglFhLUx+0mACfoYX\n\tpRJncLoHjFalWfm1t0PL5ippOK84vEuvGA6sQVOyPKj5+hOgPNqafxMPDmBX8hPRC8YmuWigrQMTf\n\tZ8gNM5oc27pfi+fkPgSA7WdZPVRb3m/mDiaimHrHNi8vH2FpNRh7AMocw07F4FpF5+f6ZdM7ot+qY\n\tqFwx0VD8jPwynhB9NiOw==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=aheadcomputing.com;\n s=selector1;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n bh=6X/46RTrRXJTl3cNFZP25l+bb2zaxDJjLb6ApvkFABw=;\n b=KpC42Wruxg948/Zy4iwHSzXdWvDFJigOVGUNud4cqmF5ThETygYhrQP2GR3eIqvlytHfWc0b4RAhDCr5dVVFYCp5g5Euh3GBFb8KgRk+6x1P3A+9Eg2Y6lMEVrpyrOIBVix2s7gtZKZ+fZf/8RNv/Pxz+EcoxMtIoECgqhhBJnjAXza18pfrOwN+mX6qZURmGGzxK2IWGLe2q2Owo86h/yhJNtCJY7ZjCwApHhfH384SNpfCjg+InxO1k+xVlbnJRpR+rUePPDKnVmHqCmldPz2F9FuIo6T8v0p/1APtQtbidBP1VMOpLFvqMd8bHr6+1OfM5lWROrmKya5R9M5wlA=="],"ARC-Seal":"i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none;\n b=OHRrfGropXwGryhrVVzBDJhUOQg502584DlefDrjG2lvZNYHbDwyhq18fQ9TU1pTB7MQKorY2xpG1fZaN6vFWziXTssrYtAZmTCuwr3gGo3Oa9He35y4NFgOO8Mr4scZ7QL6NOf5tpZ1hXZ2exPGxIsuY2EP1uXAxnvE9bTZGikAoaZauDU6TK3aL8cEwt2AWgCr+mI1AquUJMG2NNGU2/k7AgbIinbIJdT+WydW/R4gMngBCN3dA60GeP+XXarEuiL/8RYhmUBbvCM1byg6bH4JVHQJZvzVqsI0jGnPy/sofY5/aFmwDK9a3zro0bOKYvmHf53Aqw5RYm4k983Uqg==","ARC-Message-Signature":"i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com;\n s=arcselector10001;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1;\n bh=6X/46RTrRXJTl3cNFZP25l+bb2zaxDJjLb6ApvkFABw=;\n b=be+CsRrZenky/YiF9Cz876J3j3ptZpOMNXibPjIa+V8S3S86uY8Gcyuu6Y4Zun4VeyzvVeCHflRhJQ/xC1CT5XQ4IN8AOSD9nKh3Dcx4xivxr6zuBBvamdTlCjcNm+KOXYkzP3AbAPyHVG6Y6DoDpwJ23y3ldzJLomuemlsImQLPXkuJ873UCmpjbHy81Ty7owu9tQ5Oe3o5Gyq9Xkb/kGLfkXU+shuukVWL2fFJGPP7wuBh+s06oxXvG7x2XOAk85AG9h3Ztgrx29yWHoOb+b5Oot4rkhcKHhgNxdU72j/1t9obH/mZYBfgtpe6XKZuE8hISn3L6DzzndzHJJOQeg==","ARC-Authentication-Results":"i=1; mx.microsoft.com 1; spf=fail (sender ip is\n 50.45.236.54) smtp.rcpttodomain=aheadcomputing.com\n smtp.mailfrom=aheadcomputing.com; dmarc=fail (p=none sp=none pct=100)\n action=none header.from=aheadcomputing.com; dkim=none (message not signed);\n arc=none (0)","X-MS-Exchange-Authentication-Results":"spf=fail (sender IP is 50.45.236.54)\n smtp.mailfrom=aheadcomputing.com; dkim=none (message not signed)\n header.d=none;dmarc=fail action=none header.from=aheadcomputing.com;","Received-SPF":"Fail (protection.outlook.com: domain of aheadcomputing.com does\n not designate 50.45.236.54 as permitted sender)\n receiver=protection.outlook.com; client-ip=50.45.236.54;\n helo=mail.aheadcomputing.com;","From":"\"David E. Garcia Porras\" ","To":"opensbi@lists.infradead.org","Cc":"\"David E. Garcia Porras\" ","Subject":"[PATCH] lib: utils/regmap: Fix reg_stride calculation in syscon\n regmap","Date":"Fri, 3 Apr 2026 14:29:03 -0600","Message-ID":"<20260403202903.3407945-1-david.garcia@aheadcomputing.com>","X-Mailer":"git-send-email 2.43.0","MIME-Version":"1.0","X-EOPAttributedMessage":"0","X-MS-PublicTrafficType":"Email","X-MS-TrafficTypeDiagnostic":"DM2PEPF00003FC7:EE_|CO6PR22MB2916:EE_","X-MS-Office365-Filtering-Correlation-Id":"71c95d3f-f0b4-4198-bfac-08de91bfa947","X-MS-Exchange-AtpMessageProperties":"SA","X-MS-Exchange-SenderADCheck":"1","X-MS-Exchange-AntiSpam-Relay":"0","X-Microsoft-Antispam":"\n BCL:0;ARA:13230040|82310400026|1800799024|376014|36860700016|56012099003|18002099003|55112099003;","X-Microsoft-Antispam-Message-Info":"\n YEXYg6mqOReHSz1YCOKUuxQJsIps8M9OWjsLFevI0sDowtggGnbXvpRtglWwFM+W6bW+198BEE5tQsGr3f03ZnQ8U40zvTW9pIihvtTsGOPVVj7YI98RT2FF9CxWkGAoqUCGtm6EJhVHcqe5yjjWxLEd7o9EARJ3fjmUuvz2jyN6EbnZZ4sGuglVKnSPNVBdE+68ZKx6sYnc1ZtP5vYPOKmbG5Kn0/OfTOFDNh5fmVLsXdoHczVetczGfZCSJUkxg7ilwXQecUlIn44CmUGMOX0LgHuoffQLngkxjw1MvVNA6wcrs0uk21EYtCqyJnK28iC7nInNcbDJJdAnkPctjfTxNYIov0Hqhv7Bpx4oAbFWWGII4q3zv0jjEU3BHfIHrurZz8HSORLfVkhYsYxwesJ4rVUS1l4Bhh8o8mkv3H91o0YFruKhvRD6DskL0tiKVHZgfqEpRZdfrCcRX1nEhP3RxaFkvM+nbUOFQtvEkHxT1ZEcRUj3bMr57duADPwEuKEM5xtPFNHmFH88we+vCWjr4bE/BIj/TfnFDBCJPNhdVzXuqIhXF+TugDmBt3vevXTxBdSqYxaZROBBDpg17a2L4Hnj1MH+TPqwPnlEjtNsJ74r2YqueI1XBpZJOBInyNaxtsjavs5OwkYLGjlzV1IWAAi1a2JkzqgzyK4ZBAAB/dIHlr5dNrxExlXdVMG3IshxSdxaOPFKUTN+H1kb2MOYupGnqE+isGCr6n9h3VB6vJM95kGSsJuudDJQiHnXIxtmS+MXFobkROuff7CGzw==","X-Forefront-Antispam-Report":"\n CIP:50.45.236.54;CTRY:US;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:mail.aheadcomputing.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(82310400026)(1800799024)(376014)(36860700016)(56012099003)(18002099003)(55112099003);DIR:OUT;SFP:1102;","X-MS-Exchange-AntiSpam-MessageData-ChunkCount":"1","X-MS-Exchange-AntiSpam-MessageData-0":"\n C80wdZO2M+15X+v/6BxgsDZgljrGrNFXiFbW/1SQDTTKuuvW046FAakOnMG2XcFTEd9YSuPZtKwByRdg8rdEJDm4hsZCKIY2UqJS7plOdjeBJkbNS5wm50hN8zBrj/4XFixftuHoN6sRhx04mDrArvlfzeLEXDq17PgUvz2oN7q2ocQXP1XhkRUGJN/EgP8fWukr70PJYWlkpvk+oslDYnO45EjN7fq8WQUJkxBPZreVhc1eSN8RKXIplApubwZU4QKrAgd6ydUsswKUBmDV/pkOCXayy/1cwSW+NcPV/TkIET05h4kxzo9QRZDmgJpOcBEFGnTMO0Gte6cV9RlFgtrzrTFU3dcbBdrwI37J8g5e/iiqyyB49CLvYTGEoJMrhulTBnqnuriaBWMQPfs3NOdCdf/OcEgiQBciwXKYyw7PR6UCxKj35cncfXeJetgl","X-OriginatorOrg":"aheadcomputing.com","X-MS-Exchange-CrossTenant-OriginalArrivalTime":"03 Apr 2026 20:29:09.6733\n (UTC)","X-MS-Exchange-CrossTenant-Network-Message-Id":"\n 71c95d3f-f0b4-4198-bfac-08de91bfa947","X-MS-Exchange-CrossTenant-Id":"53cf747c-7b8f-4d06-8b1e-b22383a3fabf","X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp":"\n TenantId=53cf747c-7b8f-4d06-8b1e-b22383a3fabf;Ip=[50.45.236.54];Helo=[mail.aheadcomputing.com]","X-MS-Exchange-CrossTenant-AuthSource":"\n DM2PEPF00003FC7.namprd04.prod.outlook.com","X-MS-Exchange-CrossTenant-AuthAs":"Anonymous","X-MS-Exchange-CrossTenant-FromEntityHeader":"HybridOnPrem","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"CO6PR22MB2916","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20260403_132915_838221_900A4C49 ","X-CRM114-Status":"UNSURE ( 9.26 )","X-CRM114-Notice":"Please train this message.","X-Spam-Score":"-2.1 (--)","X-Spam-Report":"Spam detection software,\n running on the system \"bombadil.infradead.org\",\n has NOT identified this incoming email as spam. The original\n message has been attached to this so you can view it or label\n similar future email. If you have any questions, see\n the administrator of that system for details.\n Content preview: The reg_stride field represents the address stride in bytes\n between consecutive registers. The Linux kernel regmap framework validates\n register accesses using IS_ALIGNED(reg, map->reg_stride) as an ad [...]\n Content analysis details: (-2.1 points, 5.0 required)\n pts rule name description\n ---- ----------------------\n --------------------------------------------------\n -0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at https://www.dnswl.org/, no\n trust\n [52.101.85.100 listed in list.dnswl.org]\n -0.0 SPF_HELO_PASS SPF: HELO matches SPF record\n -0.0 SPF_PASS SPF: sender matches SPF record\n 0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n not necessarily valid\n -0.1 DKIM_VALID_EF Message has a valid DKIM or DK signature from\n envelope-from domain\n -0.1 DKIM_VALID Message has at least one valid DKIM or DK\n signature\n -0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n author's\n domain\n -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n [score: 0.0000]\n 0.0 RCVD_IN_VALIDITY_SAFE_BLOCKED RBL: ADMINISTRATOR NOTICE: The query to\n Validity was blocked. See\n https://knowledge.validity.com/hc/en-us/articles/20961730681243\n for more information.\n [52.101.85.100 listed in sa-accredit.habeas.com]\n 0.0 RCVD_IN_VALIDITY_CERTIFIED_BLOCKED RBL: ADMINISTRATOR NOTICE: The\n query to Validity was blocked. See\n https://knowledge.validity.com/hc/en-us/articles/20961730681243\n for more information.\n [52.101.85.100 listed in\n sa-trusted.bondedsender.org]\n 0.0 RCVD_IN_MSPIKE_H2 RBL: Average reputation (+2)\n [52.101.85.100 listed in wl.mailspike.net]\n 0.0 RCVD_IN_VALIDITY_RPBL_BLOCKED RBL: ADMINISTRATOR NOTICE: The query to\n Validity was blocked. See\n https://knowledge.validity.com/hc/en-us/articles/20961730681243\n for more information.\n [52.101.85.100 listed in\n bl.score.senderscore.com]","X-BeenThere":"opensbi@lists.infradead.org","X-Mailman-Version":"2.1.34","Precedence":"list","List-Id":"","List-Unsubscribe":",\n ","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n ","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"opensbi\" ","Errors-To":"opensbi-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org"},"content":"The reg_stride field represents the address stride in bytes between\nconsecutive registers. The Linux kernel regmap framework validates\nregister accesses using IS_ALIGNED(reg, map->reg_stride) as an address\nalignment check (drivers/base/regmap/regmap.c). The Linux kernel syscon\ndriver (drivers/mfd/syscon.c) sets reg_stride directly to reg_io_width:\n\n syscon_config.reg_stride = reg_io_width;\n\nThe current OpenSBI code incorrectly multiplies reg_io_width by 8,\nconverting a byte value to bits. Fix this by using reg_io_width directly\nas the stride value, consistent with the Linux kernel.\n\nSigned-off-by: David E. Garcia Porras \n---\n lib/utils/regmap/fdt_regmap_syscon.c | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)","diff":"diff --git a/lib/utils/regmap/fdt_regmap_syscon.c b/lib/utils/regmap/fdt_regmap_syscon.c\nindex 1309a55c..59cd5ee5 100644\n--- a/lib/utils/regmap/fdt_regmap_syscon.c\n+++ b/lib/utils/regmap/fdt_regmap_syscon.c\n@@ -190,7 +190,7 @@ static int regmap_syscon_init(const void *fdt, int nodeoff,\n \n \tsrm->rmap.id = nodeoff;\n \tsrm->rmap.reg_shift = 0;\n-\tsrm->rmap.reg_stride = srm->reg_io_width * 8;\n+\tsrm->rmap.reg_stride = srm->reg_io_width;\n \tsrm->rmap.reg_base = 0;\n \tsrm->rmap.reg_max = size / srm->reg_io_width;\n \tswitch (srm->reg_io_width) {\n","prefixes":[]}