Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/131490/?format=api
{ "id": 131490, "url": "https://patches.dpdk.org/api/patches/131490/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/20230915091551.1459606-16-chaoyong.he@corigine.com/", "project": { "id": 1, "url": "https://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<20230915091551.1459606-16-chaoyong.he@corigine.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20230915091551.1459606-16-chaoyong.he@corigine.com", "date": "2023-09-15T09:15:39", "name": "[v3,15/27] net/nfp: refact the rtsym module", "commit_ref": null, "pull_url": null, "state": "changes-requested", "archived": true, "hash": "686e4fdd6be43dc5ee3a33c92783fcf16d0366c4", "submitter": { "id": 2554, "url": "https://patches.dpdk.org/api/people/2554/?format=api", "name": "Chaoyong He", "email": "chaoyong.he@corigine.com" }, "delegate": { "id": 319, "url": "https://patches.dpdk.org/api/users/319/?format=api", "username": "fyigit", "first_name": "Ferruh", "last_name": "Yigit", "email": "ferruh.yigit@amd.com" }, "mbox": "https://patches.dpdk.org/project/dpdk/patch/20230915091551.1459606-16-chaoyong.he@corigine.com/mbox/", "series": [ { "id": 29520, "url": "https://patches.dpdk.org/api/series/29520/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=29520", "date": "2023-09-15T09:15:24", "name": "refact the nfpcore module", "version": 3, "mbox": "https://patches.dpdk.org/series/29520/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/131490/comments/", "check": "success", "checks": "https://patches.dpdk.org/api/patches/131490/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from mails.dpdk.org (mails.dpdk.org [217.70.189.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id 63A06425A3;\n\tFri, 15 Sep 2023 11:18:23 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 6AD6C42C24;\n\tFri, 15 Sep 2023 11:16:48 +0200 (CEST)", "from NAM12-BN8-obe.outbound.protection.outlook.com\n (mail-bn8nam12on2133.outbound.protection.outlook.com [40.107.237.133])\n by mails.dpdk.org (Postfix) with ESMTP id 74E8E4281D\n for <dev@dpdk.org>; Fri, 15 Sep 2023 11:16:46 +0200 (CEST)", "from SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5)\n by CO1PR13MB4773.namprd13.prod.outlook.com (2603:10b6:303:fb::6) with\n Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6792.21; Fri, 15 Sep\n 2023 09:16:44 +0000", "from SJ0PR13MB5545.namprd13.prod.outlook.com\n ([fe80::c05c:6915:1628:70c4]) by SJ0PR13MB5545.namprd13.prod.outlook.com\n ([fe80::c05c:6915:1628:70c4%7]) with mapi id 15.20.6768.029; Fri, 15 Sep 2023\n 09:16:44 +0000" ], "ARC-Seal": "i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;\n b=Z1pA/F+FY1kNYy3U/MN0/kPqIJsTvqn9g6lIcvx0VsQtj1+dh0bvKz9dnJs/SbRPq79Ow33RSvAbMvxxQMkb486sCRD2sHGb57X6acokqiMb0zeT1QbrPKLOEYfa4iZwMc6gtRqt9ouNz2jWioqcAKoxkcQhRyNicF67XJPgkPXncD25LnheFrbLM0zO7LxTtIfk93RiOQH1iYPRtEf4cqlv/OQPh6nb9UscAVtYI4vdsWzgtSSjsowSjDiHP0DKfgyACBx/sc3r4NkFhWGgBBEr3UGlmMzD2/gOElXAxEr2UoW07bpKTal1441WSGX5tY+Rvls/sM892tQFF0yabA==", "ARC-Message-Signature": "i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com;\n s=arcselector9901;\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=OdpEP/UyAGqz4oxHPFVDbYOKsAIafOJKB5CG4krDax0=;\n b=Ht8KnFdIKTQiuEtWLIErmJpXuOtlTnyTnYS4VcwF3yfYzTK3ueO/gU07UF4JrCpuCf28J0pvUUJ8HW7/ksOtOsYGlr93IOR+nTdoJyG1VjiWnqWw5vG8OE7E3A+Z8ohpmsrCTfLS8sEV+gN/CPOEntdPN6S6DaxSJfo4vAZokIKy6z2pbiWea6wHE+i/MGzL9g/sJGrB4EFAQYK6ZYM02PVIytqFil58moEoeHLF3v13j+JUpaNr7V5YEnsh36BccoBKbh4w/w9idTfhB2eGBXW9n/Vkb5HN7U4OED3tdoFgF7cTnhAUwBFRXPNXKXfhBJ/KYFBTMlRuFls8BAXlsw==", "ARC-Authentication-Results": "i=1; mx.microsoft.com 1; spf=pass\n smtp.mailfrom=corigine.com; dmarc=pass action=none header.from=corigine.com;\n dkim=pass header.d=corigine.com; arc=none", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=corigine.onmicrosoft.com; s=selector2-corigine-onmicrosoft-com;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n bh=OdpEP/UyAGqz4oxHPFVDbYOKsAIafOJKB5CG4krDax0=;\n b=W3jcK4Ps7Yq5/1W02pFm2CSjOs6GlkWW7XnAKC26iwl/PK7gWLPI8HSTuEXDQ3EIv5Dj86dQ6Hi6jqHLvTmFcp7PbMeujjYhuccWEFEPwIhjVGDbQK/J00ewE2UhA60QX4ijiG2oGLzMLeTjPmia3kGa9INpBc67HiUrGePc9w4=", "Authentication-Results": "dkim=none (message not signed)\n header.d=none;dmarc=none action=none header.from=corigine.com;", "From": "Chaoyong He <chaoyong.he@corigine.com>", "To": "dev@dpdk.org", "Cc": "oss-drivers@corigine.com, Chaoyong He <chaoyong.he@corigine.com>,\n\t=?utf-8?q?Niklas_S=C3=B6derlund?= <niklas.soderlund@corigine.com>", "Subject": "[PATCH v3 15/27] net/nfp: refact the rtsym module", "Date": "Fri, 15 Sep 2023 17:15:39 +0800", "Message-Id": "<20230915091551.1459606-16-chaoyong.he@corigine.com>", "X-Mailer": "git-send-email 2.39.1", "In-Reply-To": "<20230915091551.1459606-1-chaoyong.he@corigine.com>", "References": "<20230830021457.2064750-1-chaoyong.he@corigine.com>\n <20230915091551.1459606-1-chaoyong.he@corigine.com>", "Content-Type": "text/plain; charset=UTF-8", "Content-Transfer-Encoding": "8bit", "X-ClientProxiedBy": "PH7PR10CA0017.namprd10.prod.outlook.com\n (2603:10b6:510:23d::10) To SJ0PR13MB5545.namprd13.prod.outlook.com\n (2603:10b6:a03:424::5)", "MIME-Version": "1.0", "X-MS-PublicTrafficType": "Email", "X-MS-TrafficTypeDiagnostic": "SJ0PR13MB5545:EE_|CO1PR13MB4773:EE_", "X-MS-Office365-Filtering-Correlation-Id": "5a7e8689-3467-4d80-6973-08dbb5cc7b0b", "X-MS-Exchange-SenderADCheck": "1", "X-MS-Exchange-AntiSpam-Relay": "0", "X-Microsoft-Antispam": "BCL:0;", "X-Microsoft-Antispam-Message-Info": "\n 5mEvGOJNXVcZiMGI/8yavfcYpP0cp2rb1d1E0h1tQeSl9VvQ9HVfemOWA7LMmVVsspzwT4fCjucxHvTW9q8fM0qH4RqZ/Bu+8/7mW3W+1l6Bp8/0ab61fnc/5/XALBtjsPoRFjpQ26Ced+HJiRK4UKcOum0fOlK7WTDAyl7E/H6nOdFP6icQmtIoQz9cmWau7t/IeEtVBfAG1P8Bxc2v9guFfsePmHW0b6N3hQ1s9IlfTLLCw/n3q3qirjod98f36NjBBa41INXzl/NrlW9ZVBJUeJljkcIY/l3qAh+2mMRrhqQGkgsaHBfvFeGerFl8BeRYJdaa2PR5n1YlQNqTc0LAW1NtB+uhnCTLz/DY0j8cngTXU4N+JSYvZGbPa94NR2fjQDfYNYQwi/X63TOqLpZIlVGU/ErBSVTBlCDUQ3RKCPolJcz5PZbJtHdZutIkmLa44YxNE1WAwW3vmlKP43FoPs1ufctSYwvKeihJFEKhwkZTto8Y6F+d1kEUlL2VaBLKShE9xC5FLJwU2Faz2gsb9jgCSR47K4ziVnXOyBbDTE+apmOMjoBxTuCa7jwqSggWKehoFjEuOFC22QSQeYnRSSRbMMdlXIXAzwTeg+34wT36o1aJjZ5/WASVJcyV4sp89+t77DTT4HqprwgAb4osVvVBNQHsNZO1vvGx7uQ=", "X-Forefront-Antispam-Report": "CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:;\n IPV:NLI; SFV:NSPM; H:SJ0PR13MB5545.namprd13.prod.outlook.com; PTR:; CAT:NONE;\n SFS:(13230031)(39840400004)(346002)(136003)(376002)(396003)(366004)(451199024)(186009)(1800799009)(6512007)(107886003)(1076003)(2616005)(6666004)(52116002)(6506007)(6486002)(66946007)(26005)(316002)(41300700001)(66476007)(66556008)(6916009)(54906003)(5660300002)(4326008)(8676002)(38350700002)(8936002)(38100700002)(478600001)(66574015)(83380400001)(36756003)(44832011)(30864003)(86362001)(2906002);\n DIR:OUT; SFP:1102;", "X-MS-Exchange-AntiSpam-MessageData-ChunkCount": "1", "X-MS-Exchange-AntiSpam-MessageData-0": "=?utf-8?q?o1630I8p5MJ/AUZKe1qwBPuRPe6o?=\n\t=?utf-8?q?sT1FV87U6DVCy2sbcN09qz9NKCYLGXBbH6W3fyvSNpGrqMzJWAYsCXTn5w1hw+yAy?=\n\t=?utf-8?q?Gtp1lK8e4BdhZgozqoYj91H1pQ+fVPs99IJBBFLvY31VCZ7TFr2pHS6kYrAEOmWJc?=\n\t=?utf-8?q?uUUAmrjjARW4yvyarE7p0/6pkwF8Xmr1tSqRsEhsUywgJ7Pg/kDbtvbOimuKcqrbR?=\n\t=?utf-8?q?j6sopLSCu+XRpKyJoeGteWJkwOSQpbEDKEhgWa1SEAziNRTh7RIvooM/JaKZi57Y5?=\n\t=?utf-8?q?QHXvu31nA7daHrXRdtkouDl2hEPg8c577P5SdxoDSqRG7FbfGmuZ30A2SueQpX5Kj?=\n\t=?utf-8?q?MeAh+PgLjiWCyyPFM6Gfg4nD/U1KTrIrqQ0Dj3LoUZSvVyolRv107+M4CzkGu08/V?=\n\t=?utf-8?q?VtlGiONn0wjJGyzKav9kUb4S6BITFM1TEkB1o1/KcNHkmbQYowS4qdyaRwegr0Wb8?=\n\t=?utf-8?q?UffjX0OJYEQ7KCj/61LtebLVhQBJK1hPZVBEIsF3dNloDp/YOhdqk1aeYHduAGslr?=\n\t=?utf-8?q?MVYBbzg7VA3nGQTxHCqH/Jvu6zMv4wkROSKPMeS4r0LuK8dQblfgZd9NCbnFxxqrn?=\n\t=?utf-8?q?u0i5NJNVdXyzeoRpolZwYQAECHG80uQ/zVlr5ZCNDBxXqt+PAkQzGBBlZ97nibRGC?=\n\t=?utf-8?q?ylNyiwiDM4xcWSY7ylHYjKBnO5BrJ2kGramv4wbTtj12Szb0Y+6TIY21XW6dJ5Tfm?=\n\t=?utf-8?q?7dtVldDxLKL5BJ2ioh9JNQEIyVTfbf4bVn3Ro4EvuTdIE0FsWLMMj55QmXugSKzoZ?=\n\t=?utf-8?q?6ntSzFpp8/Hxwks1CliRoo+QDk011Tf+WSsl+gxYr/d4ueItHQw6XAEFkYTx1pnLP?=\n\t=?utf-8?q?c/GbJAla87cK4K04K+DgC3+K8HRQZ9Go3t9C+xx0Pdh7N16NS6K23ZEHhJa2d77EQ?=\n\t=?utf-8?q?X6PA5oA9ok/teNg5jpA3VQ5gXyC/BT4UNTs1oVXRE5vyXjlGw3qMEOvFNd1XiLl+0?=\n\t=?utf-8?q?scj1VvBMfRUXSneCvbPbsA98Ob6rDUlu/hSlyJVBYMrx7sC0YYiyG6Poi3TH21Nwj?=\n\t=?utf-8?q?MUn62NmAN980K/UYjDY58F76hTj0HPnzj1DAB4Jx6A6AEx8hTQ3fnc9oqUP70W+Iz?=\n\t=?utf-8?q?zpVBGS10Qa//c+JNthrVstDHYgwVcL/S/0OBrlvyyg1+HSq47as2nIbc9QoREYNFc?=\n\t=?utf-8?q?JZIPoEtbkiUf+qCTiSmwLLx9b2icgvN2vfRn05FaFATGUsNAC7A3zeZld3bQ3YhSq?=\n\t=?utf-8?q?Bp120F6nmdb8WiG7jjHmFMFcmXz+wUIk4xC8hpcXC32/okw6ybRkCQycW3CHkavWI?=\n\t=?utf-8?q?6DC48VPQQ2RXl6knNkaJygn5r5pQBNfIPW9Edd3p7iBtc1jh9aviwgHAzMy3NJRHQ?=\n\t=?utf-8?q?uqVmmbwxlyEOsSjO1eNRQDiN0WfT7/RuDa5ktBMdJU/6uH4x0N4I5KboCmJq24zpm?=\n\t=?utf-8?q?ff79X1Oq9V7CuLqsW8FVkkaKA+i794sDtEi/SvTq1A9zGqpc/w3xWTsa9xaklhPFW?=\n\t=?utf-8?q?9MfwiGY8J0ftKYPusbnGJOtwRQmHUxnp2C36pzgNefGok879q5aJwkXZK6Y2R4Ev6?=\n\t=?utf-8?q?uvBtvGtD1Y4PBU0ixocqrJ05XvSP+oRVDg=3D=3D?=", "X-OriginatorOrg": "corigine.com", "X-MS-Exchange-CrossTenant-Network-Message-Id": "\n 5a7e8689-3467-4d80-6973-08dbb5cc7b0b", "X-MS-Exchange-CrossTenant-AuthSource": "SJ0PR13MB5545.namprd13.prod.outlook.com", "X-MS-Exchange-CrossTenant-AuthAs": "Internal", "X-MS-Exchange-CrossTenant-OriginalArrivalTime": "15 Sep 2023 09:16:44.8291 (UTC)", "X-MS-Exchange-CrossTenant-FromEntityHeader": "Hosted", "X-MS-Exchange-CrossTenant-Id": "fe128f2c-073b-4c20-818e-7246a585940c", "X-MS-Exchange-CrossTenant-MailboxType": "HOSTED", "X-MS-Exchange-CrossTenant-UserPrincipalName": "\n 7QIM5dStcXR++Fyc7PkbIibjDbIPant0N/9CJqOheeOWqQpQ2ptKkaKBJkoSUnXJq1PbXuJzevFCLUl4IDCcqaRuSNUwn44sFCnXc15VnXE=", "X-MS-Exchange-Transport-CrossTenantHeadersStamped": "CO1PR13MB4773", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n <mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dev>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org" }, "content": "Add several read/write APIs and remove the unneeded header file\ninclude statements.\n\nSigned-off-by: Chaoyong He <chaoyong.he@corigine.com>\nReviewed-by: Niklas Söderlund <niklas.soderlund@corigine.com>\n---\n drivers/net/nfp/nfpcore/nfp_rtsym.c | 305 +++++++++++++++++++++++++---\n drivers/net/nfp/nfpcore/nfp_rtsym.h | 53 ++---\n 2 files changed, 299 insertions(+), 59 deletions(-)", "diff": "diff --git a/drivers/net/nfp/nfpcore/nfp_rtsym.c b/drivers/net/nfp/nfpcore/nfp_rtsym.c\nindex 37811ceaeb..2d4100dda2 100644\n--- a/drivers/net/nfp/nfpcore/nfp_rtsym.c\n+++ b/drivers/net/nfp/nfpcore/nfp_rtsym.c\n@@ -8,14 +8,25 @@\n * Interface for accessing run-time symbol table\n */\n \n-#include <stdio.h>\n+#include \"nfp_rtsym.h\"\n+\n #include <rte_byteorder.h>\n-#include \"nfp_cpp.h\"\n+\n #include \"nfp_logs.h\"\n #include \"nfp_mip.h\"\n-#include \"nfp_rtsym.h\"\n #include \"nfp6000/nfp6000.h\"\n \n+enum nfp_rtsym_type {\n+\tNFP_RTSYM_TYPE_NONE,\n+\tNFP_RTSYM_TYPE_OBJECT,\n+\tNFP_RTSYM_TYPE_FUNCTION,\n+\tNFP_RTSYM_TYPE_ABS,\n+};\n+\n+#define NFP_RTSYM_TARGET_NONE 0\n+#define NFP_RTSYM_TARGET_LMEM -1\n+#define NFP_RTSYM_TARGET_EMU_CACHE -7\n+\n /* These need to match the linker */\n #define SYM_TGT_LMEM 0\n #define SYM_TGT_EMU_CACHE 0x17\n@@ -32,6 +43,30 @@ struct nfp_rtsym_entry {\n \tuint32_t size_lo;\n };\n \n+/*\n+ * Structure describing a run-time NFP symbol.\n+ *\n+ * The memory target of the symbol is generally the CPP target number and can be\n+ * used directly by the nfp_cpp API calls. However, in some cases (i.e., for\n+ * local memory or control store) the target is encoded using a negative number.\n+ *\n+ * When the target type can not be used to fully describe the location of a\n+ * symbol the domain field is used to further specify the location (i.e., the\n+ * specific ME or island number).\n+ *\n+ * For ME target resources, 'domain' is an MEID.\n+ * For Island target resources, 'domain' is an island ID, with the one exception\n+ * of \"sram\" symbols for backward compatibility, which are viewed as global.\n+ */\n+struct nfp_rtsym {\n+\tconst char *name; /**< Symbol name */\n+\tuint64_t addr; /**< Address in the domain/target's address space */\n+\tuint64_t size; /**< Size (in bytes) of the symbol */\n+\tenum nfp_rtsym_type type; /**< NFP_RTSYM_TYPE_* of the symbol */\n+\tint target; /**< CPP target identifier, or NFP_RTSYM_TARGET_* */\n+\tint domain; /**< CPP target domain */\n+};\n+\n struct nfp_rtsym_table {\n \tstruct nfp_cpp *cpp;\n \tint num;\n@@ -80,21 +115,8 @@ nfp_rtsym_sw_entry_init(struct nfp_rtsym_table *cache,\n \t\tsw->domain = -1;\n }\n \n-struct nfp_rtsym_table *\n-nfp_rtsym_table_read(struct nfp_cpp *cpp)\n-{\n-\tstruct nfp_mip *mip;\n-\tstruct nfp_rtsym_table *rtbl;\n-\n-\tmip = nfp_mip_open(cpp);\n-\trtbl = __nfp_rtsym_table_read(cpp, mip);\n-\tnfp_mip_close(mip);\n-\n-\treturn rtbl;\n-}\n-\n-struct nfp_rtsym_table *\n-__nfp_rtsym_table_read(struct nfp_cpp *cpp,\n+static struct nfp_rtsym_table *\n+nfp_rtsym_table_read_real(struct nfp_cpp *cpp,\n \t\tconst struct nfp_mip *mip)\n {\n \tint n;\n@@ -162,6 +184,19 @@ __nfp_rtsym_table_read(struct nfp_cpp *cpp,\n \treturn NULL;\n }\n \n+struct nfp_rtsym_table *\n+nfp_rtsym_table_read(struct nfp_cpp *cpp)\n+{\n+\tstruct nfp_mip *mip;\n+\tstruct nfp_rtsym_table *rtbl;\n+\n+\tmip = nfp_mip_open(cpp);\n+\trtbl = nfp_rtsym_table_read_real(cpp, mip);\n+\tnfp_mip_close(mip);\n+\n+\treturn rtbl;\n+}\n+\n /**\n * Get the number of RTSYM descriptors\n *\n@@ -287,7 +322,59 @@ nfp_rtsym_to_dest(struct nfp_cpp *cpp,\n }\n \n static int\n-nfp_rtsym_readl(struct nfp_cpp *cpp,\n+nfp_rtsym_read_real(struct nfp_cpp *cpp,\n+\t\tconst struct nfp_rtsym *sym,\n+\t\tuint8_t action,\n+\t\tuint8_t token,\n+\t\tuint64_t offset,\n+\t\tvoid *buf,\n+\t\tsize_t len)\n+{\n+\tint err;\n+\tuint64_t addr;\n+\tuint32_t cpp_id;\n+\tsize_t length = len;\n+\tuint64_t sym_size = nfp_rtsym_size(sym);\n+\n+\tif (offset > sym_size) {\n+\t\tPMD_DRV_LOG(ERR, \"rtsym '%s' read out of bounds\", sym->name);\n+\t\treturn -ENXIO;\n+\t}\n+\n+\tif (length > sym_size - offset)\n+\t\tlength = sym_size - offset;\n+\n+\tif (sym->type == NFP_RTSYM_TYPE_ABS) {\n+\t\tunion {\n+\t\t\tuint64_t value_64;\n+\t\t\tuint8_t value_8[8];\n+\t\t} tmp;\n+\n+\t\ttmp.value_64 = sym->addr;\n+\t\tmemcpy(buf, &tmp.value_8[offset], length);\n+\n+\t\treturn length;\n+\t}\n+\n+\terr = nfp_rtsym_to_dest(cpp, sym, action, token, offset, &cpp_id, &addr);\n+\tif (err != 0)\n+\t\treturn err;\n+\n+\treturn nfp_cpp_read(cpp, cpp_id, addr, buf, length);\n+}\n+\n+int\n+nfp_rtsym_read(struct nfp_cpp *cpp,\n+\t\tconst struct nfp_rtsym *sym,\n+\t\tuint64_t offset,\n+\t\tvoid *buf,\n+\t\tsize_t len)\n+{\n+\treturn nfp_rtsym_read_real(cpp, sym, NFP_CPP_ACTION_RW, 0, offset, buf, len);\n+}\n+\n+static int\n+nfp_rtsym_readl_real(struct nfp_cpp *cpp,\n \t\tconst struct nfp_rtsym *sym,\n \t\tuint8_t action,\n \t\tuint8_t token,\n@@ -310,8 +397,17 @@ nfp_rtsym_readl(struct nfp_cpp *cpp,\n \treturn nfp_cpp_readl(cpp, cpp_id, addr, value);\n }\n \n+int\n+nfp_rtsym_readl(struct nfp_cpp *cpp,\n+\t\tconst struct nfp_rtsym *sym,\n+\t\tuint64_t offset,\n+\t\tuint32_t *value)\n+{\n+\treturn nfp_rtsym_readl_real(cpp, sym, NFP_CPP_ACTION_RW, 0, offset, value);\n+}\n+\n static int\n-nfp_rtsym_readq(struct nfp_cpp *cpp,\n+nfp_rtsym_readq_real(struct nfp_cpp *cpp,\n \t\tconst struct nfp_rtsym *sym,\n \t\tuint8_t action,\n \t\tuint8_t token,\n@@ -339,6 +435,121 @@ nfp_rtsym_readq(struct nfp_cpp *cpp,\n \treturn nfp_cpp_readq(cpp, cpp_id, addr, value);\n }\n \n+int\n+nfp_rtsym_readq(struct nfp_cpp *cpp,\n+\t\tconst struct nfp_rtsym *sym,\n+\t\tuint64_t offset,\n+\t\tuint64_t *value)\n+{\n+\treturn nfp_rtsym_readq_real(cpp, sym, NFP_CPP_ACTION_RW, 0, offset, value);\n+}\n+\n+static int\n+nfp_rtsym_write_real(struct nfp_cpp *cpp,\n+\t\tconst struct nfp_rtsym *sym,\n+\t\tuint8_t action,\n+\t\tuint8_t token,\n+\t\tuint64_t offset,\n+\t\tvoid *buf,\n+\t\tsize_t len)\n+{\n+\tint err;\n+\tuint64_t addr;\n+\tuint32_t cpp_id;\n+\tsize_t length = len;\n+\tuint64_t sym_size = nfp_rtsym_size(sym);\n+\n+\tif (offset > sym_size) {\n+\t\tPMD_DRV_LOG(ERR, \"rtsym '%s' write out of bounds\", sym->name);\n+\t\treturn -ENXIO;\n+\t}\n+\n+\tif (length > sym_size - offset)\n+\t\tlength = sym_size - offset;\n+\n+\terr = nfp_rtsym_to_dest(cpp, sym, action, token, offset, &cpp_id, &addr);\n+\tif (err != 0)\n+\t\treturn err;\n+\n+\treturn nfp_cpp_write(cpp, cpp_id, addr, buf, length);\n+}\n+\n+int\n+nfp_rtsym_write(struct nfp_cpp *cpp,\n+\t\tconst struct nfp_rtsym *sym,\n+\t\tuint64_t offset,\n+\t\tvoid *buf,\n+\t\tsize_t len)\n+{\n+\treturn nfp_rtsym_write_real(cpp, sym, NFP_CPP_ACTION_RW, 0, offset, buf, len);\n+}\n+\n+static int\n+nfp_rtsym_writel_real(struct nfp_cpp *cpp,\n+\t\tconst struct nfp_rtsym *sym,\n+\t\tuint8_t action,\n+\t\tuint8_t token,\n+\t\tuint64_t offset,\n+\t\tuint32_t value)\n+{\n+\tint err;\n+\tuint64_t addr;\n+\tuint32_t cpp_id;\n+\n+\tif (offset + 4 > nfp_rtsym_size(sym)) {\n+\t\tPMD_DRV_LOG(ERR, \"rtsym '%s' write out of bounds\", sym->name);\n+\t\treturn -ENXIO;\n+\t}\n+\n+\terr = nfp_rtsym_to_dest(cpp, sym, action, token, offset, &cpp_id, &addr);\n+\tif (err != 0)\n+\t\treturn err;\n+\n+\treturn nfp_cpp_writel(cpp, cpp_id, addr, value);\n+}\n+\n+int\n+nfp_rtsym_writel(struct nfp_cpp *cpp,\n+\t\tconst struct nfp_rtsym *sym,\n+\t\tuint64_t offset,\n+\t\tuint32_t value)\n+{\n+\treturn nfp_rtsym_writel_real(cpp, sym, NFP_CPP_ACTION_RW, 0, offset, value);\n+}\n+\n+static int\n+nfp_rtsym_writeq_real(struct nfp_cpp *cpp,\n+\t\tconst struct nfp_rtsym *sym,\n+\t\tuint8_t action,\n+\t\tuint8_t token,\n+\t\tuint64_t offset,\n+\t\tuint64_t value)\n+{\n+\tint err;\n+\tuint64_t addr;\n+\tuint32_t cpp_id;\n+\n+\tif (offset + 8 > nfp_rtsym_size(sym)) {\n+\t\tPMD_DRV_LOG(ERR, \"rtsym '%s' write out of bounds\", sym->name);\n+\t\treturn -ENXIO;\n+\t}\n+\n+\terr = nfp_rtsym_to_dest(cpp, sym, action, token, offset, &cpp_id, &addr);\n+\tif (err != 0)\n+\t\treturn err;\n+\n+\treturn nfp_cpp_writeq(cpp, cpp_id, addr, value);\n+}\n+\n+int\n+nfp_rtsym_writeq(struct nfp_cpp *cpp,\n+\t\tconst struct nfp_rtsym *sym,\n+\t\tuint64_t offset,\n+\t\tuint64_t value)\n+{\n+\treturn nfp_rtsym_writeq_real(cpp, sym, NFP_CPP_ACTION_RW, 0, offset, value);\n+}\n+\n /**\n * Read a simple unsigned scalar value from symbol\n *\n@@ -374,11 +585,11 @@ nfp_rtsym_read_le(struct nfp_rtsym_table *rtbl,\n \n \tswitch (sym->size) {\n \tcase 4:\n-\t\terr = nfp_rtsym_readl(rtbl->cpp, sym, NFP_CPP_ACTION_RW, 0, 0, &val32);\n+\t\terr = nfp_rtsym_readl(rtbl->cpp, sym, 0, &val32);\n \t\tval = val32;\n \t\tbreak;\n \tcase 8:\n-\t\terr = nfp_rtsym_readq(rtbl->cpp, sym, NFP_CPP_ACTION_RW, 0, 0, &val);\n+\t\terr = nfp_rtsym_readq(rtbl->cpp, sym, 0, &val);\n \t\tbreak;\n \tdefault:\n \t\tPMD_DRV_LOG(ERR, \"rtsym '%s' unsupported size: %#lx\",\n@@ -387,8 +598,6 @@ nfp_rtsym_read_le(struct nfp_rtsym_table *rtbl,\n \t\tbreak;\n \t}\n \n-\tif (err)\n-\t\terr = -EIO;\n exit:\n \tif (error != NULL)\n \t\t*error = err;\n@@ -399,6 +608,54 @@ nfp_rtsym_read_le(struct nfp_rtsym_table *rtbl,\n \treturn val;\n }\n \n+/**\n+ * Write an unsigned scalar value to a symbol\n+ *\n+ * Lookup a symbol and write a value to it. Symbol can be 4 or 8 bytes in size.\n+ * If 4 bytes then the lower 32-bits of 'value' are used. Value will be\n+ * written as simple little-endian unsigned value.\n+ *\n+ * @param rtbl\n+ * NFP RTSYM table\n+ * @param name\n+ * Symbol name\n+ * @param value\n+ * Value to write\n+ *\n+ * @return\n+ * 0 on success or error code.\n+ */\n+int\n+nfp_rtsym_write_le(struct nfp_rtsym_table *rtbl,\n+\t\tconst char *name,\n+\t\tuint64_t value)\n+{\n+\tint err;\n+\tuint64_t sym_size;\n+\tconst struct nfp_rtsym *sym;\n+\n+\tsym = nfp_rtsym_lookup(rtbl, name);\n+\tif (sym == NULL)\n+\t\treturn -ENOENT;\n+\n+\tsym_size = nfp_rtsym_size(sym);\n+\tswitch (sym_size) {\n+\tcase 4:\n+\t\terr = nfp_rtsym_writel(rtbl->cpp, sym, 0, value);\n+\t\tbreak;\n+\tcase 8:\n+\t\terr = nfp_rtsym_writeq(rtbl->cpp, sym, 0, value);\n+\t\tbreak;\n+\tdefault:\n+\t\tPMD_DRV_LOG(ERR, \"rtsym '%s' unsupported size: %#lx\",\n+\t\t\t\tname, sym_size);\n+\t\terr = -EINVAL;\n+\t\tbreak;\n+\t}\n+\n+\treturn err;\n+}\n+\n uint8_t *\n nfp_rtsym_map(struct nfp_rtsym_table *rtbl,\n \t\tconst char *name,\ndiff --git a/drivers/net/nfp/nfpcore/nfp_rtsym.h b/drivers/net/nfp/nfpcore/nfp_rtsym.h\nindex fdde1eb75b..f79637ac50 100644\n--- a/drivers/net/nfp/nfpcore/nfp_rtsym.h\n+++ b/drivers/net/nfp/nfpcore/nfp_rtsym.h\n@@ -6,46 +6,13 @@\n #ifndef __NFP_RTSYM_H__\n #define __NFP_RTSYM_H__\n \n-#define NFP_RTSYM_TYPE_NONE 0\n-#define NFP_RTSYM_TYPE_OBJECT 1\n-#define NFP_RTSYM_TYPE_FUNCTION 2\n-#define NFP_RTSYM_TYPE_ABS 3\n-\n-#define NFP_RTSYM_TARGET_NONE 0\n-#define NFP_RTSYM_TARGET_LMEM -1\n-#define NFP_RTSYM_TARGET_EMU_CACHE -7\n-\n-/*\n- * Structure describing a run-time NFP symbol.\n- *\n- * The memory target of the symbol is generally the CPP target number and can be\n- * used directly by the nfp_cpp API calls. However, in some cases (i.e., for\n- * local memory or control store) the target is encoded using a negative number.\n- *\n- * When the target type can not be used to fully describe the location of a\n- * symbol the domain field is used to further specify the location (i.e., the\n- * specific ME or island number).\n- *\n- * For ME target resources, 'domain' is an MEID.\n- * For Island target resources, 'domain' is an island ID, with the one exception\n- * of \"sram\" symbols for backward compatibility, which are viewed as global.\n- */\n-struct nfp_rtsym {\n-\tconst char *name; /**< Symbol name */\n-\tuint64_t addr; /**< Address in the domain/target's address space */\n-\tuint64_t size; /**< Size (in bytes) of the symbol */\n-\tint type; /**< NFP_RTSYM_TYPE_* of the symbol */\n-\tint target; /**< CPP target identifier, or NFP_RTSYM_TARGET_* */\n-\tint domain; /**< CPP target domain */\n-};\n+#include \"nfp_cpp.h\"\n \n+struct nfp_rtsym;\n struct nfp_rtsym_table;\n \n struct nfp_rtsym_table *nfp_rtsym_table_read(struct nfp_cpp *cpp);\n \n-struct nfp_rtsym_table *__nfp_rtsym_table_read(struct nfp_cpp *cpp,\n-\t\tconst struct nfp_mip *mip);\n-\n int nfp_rtsym_count(struct nfp_rtsym_table *rtbl);\n \n const struct nfp_rtsym *nfp_rtsym_get(struct nfp_rtsym_table *rtbl, int idx);\n@@ -53,8 +20,24 @@ const struct nfp_rtsym *nfp_rtsym_get(struct nfp_rtsym_table *rtbl, int idx);\n const struct nfp_rtsym *nfp_rtsym_lookup(struct nfp_rtsym_table *rtbl,\n \t\tconst char *name);\n \n+int nfp_rtsym_read(struct nfp_cpp *cpp, const struct nfp_rtsym *sym,\n+\t\tuint64_t offset, void *buf, size_t len);\n+int nfp_rtsym_readl(struct nfp_cpp *cpp, const struct nfp_rtsym *sym,\n+\t\tuint64_t offset, uint32_t *value);\n+int nfp_rtsym_readq(struct nfp_cpp *cpp, const struct nfp_rtsym *sym,\n+\t\tuint64_t offset, uint64_t *value);\n+\n+int nfp_rtsym_write(struct nfp_cpp *cpp, const struct nfp_rtsym *sym,\n+\t\tuint64_t offset, void *buf, size_t len);\n+int nfp_rtsym_writel(struct nfp_cpp *cpp, const struct nfp_rtsym *sym,\n+\t\tuint64_t offset, uint32_t value);\n+int nfp_rtsym_writeq(struct nfp_cpp *cpp, const struct nfp_rtsym *sym,\n+\t\tuint64_t offset, uint64_t value);\n+\n uint64_t nfp_rtsym_read_le(struct nfp_rtsym_table *rtbl, const char *name,\n \t\tint *error);\n+int nfp_rtsym_write_le(struct nfp_rtsym_table *rtbl, const char *name,\n+\t\tuint64_t value);\n uint8_t *nfp_rtsym_map(struct nfp_rtsym_table *rtbl, const char *name,\n \t\tuint32_t min_size, struct nfp_cpp_area **area);\n \n", "prefixes": [ "v3", "15/27" ] }