From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 48E5EA034F; Tue, 7 Dec 2021 08:40:25 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D2E0041144; Tue, 7 Dec 2021 08:40:24 +0100 (CET) Received: from mail-io1-f51.google.com (mail-io1-f51.google.com [209.85.166.51]) by mails.dpdk.org (Postfix) with ESMTP id 13A7F4114F for ; Tue, 7 Dec 2021 08:40:23 +0100 (CET) Received: by mail-io1-f51.google.com with SMTP id z26so15933644iod.10 for ; Mon, 06 Dec 2021 23:40:23 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=snrW3iajYQr5g+hP63WL5LiAT/MwpdR/pPHNredSOWA=; b=j/lWWV2L/1qcNL7QjH3uoN2Tft3QHzWopEu/fK+GCc6M5WzUnQUVchqPyiEE/y/Qv4 Hrts5h+pBvQ5/zlqUp6yvW/JEwXUWAFslLmrvb5k8sQBix6cTP0ugB0HjCB6CLb5e3iE AkH1MNxV/b//HHfsq8F4tTM/GRJwuVUFz6EJ3yV/yYgbOdH80pG/qrL68rj6z8T2QAVx 1/2NtN33UcTS6o+cmas93LIhVpC9O/fHerGhsmlqAa3EHs0khxq5+fPY3knP1r6juy5a EFt3vYnOxLUlwX0G8rHUrD9BTYneGLZSKIgeTDVTk2OBb3o3FJwWivD2lesMIosgYNt6 i4FA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=snrW3iajYQr5g+hP63WL5LiAT/MwpdR/pPHNredSOWA=; b=A4fVJbBuKPu/A0rYuuLysqrmQJg4AYC+Z14pdYQ889526YDO9P1LmqCLF8hRcnE/d+ SAtX6HiLZuKTz79RRT30M6kaHXCHBi+2wOBM5MvYcv6ejqoXAnLjr2z2sedGvEJqJ2Cg mKhFIr5cxTbx7VKJmphdq72E0KP6wHCcJ//VNwqwaMX+Umg9yK9dQ1k/o443jNwZJXnq bmRllTEsUKw3JsaWaq2fCG9RVQoHCspCxTiQm8DmQYIG/ENm97xVhQMgI8id2Bb5wU3X ucRjMBnqeom5oraxiWNH4y53jonC2hAnWqnvOwgNmy6Xwp+7kEsB4lgVxCa1fZdDEtkB U7Eg== X-Gm-Message-State: AOAM532FxScNPLzIfzhlJdZCXYpnn1kGpSVgRHMKOqKYKQX0Qtm9oKmG SpjySkXVbRhOGX3RqM26AcVXCcopKe1fdRBQInU= X-Google-Smtp-Source: ABdhPJyoKjcExrTZ+2eZE51mVX25zYGV5P3noQtBdNVnmDx24eD1oRp9y8cBOhA1E8FQa9jt3W2C6MWelMdiAM/NhRM= X-Received: by 2002:a05:6638:2395:: with SMTP id q21mr46866091jat.48.1638862822377; Mon, 06 Dec 2021 23:40:22 -0800 (PST) MIME-Version: 1.0 References: <20211206083542.3115019-1-jerinj@marvell.com> In-Reply-To: From: Jerin Jacob Date: Tue, 7 Dec 2021 13:09:56 +0530 Message-ID: Subject: Re: [dpdk-dev] [PATCH v1] drivers: remove octeontx2 drivers To: Ferruh Yigit Cc: Jerin Jacob , dpdk-dev , Thomas Monjalon , Akhil Goyal , Declan Doherty , Ruifeng Wang , Jan Viktorin , Bruce Richardson , Ray Kinsella , Radha Mohan Chintakuntla , Veerasenareddy Burru , Nithin Dabilpuram , Kiran Kumar K , Sunil Kumar Kori , Satha Rao , Nalla Pradeep , Ciara Power , Pavan Nikhilesh , Shijith Thotton , Ashwin Sekhar T K , Anatoly Burakov , Satananda Burla , Liron Himi Content-Type: text/plain; charset="UTF-8" X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org On Mon, Dec 6, 2021 at 7:05 PM Ferruh Yigit wrote: > > On 12/6/2021 8:35 AM, jerinj@marvell.com wrote: > > From: Jerin Jacob > > > > As per the deprecation notice, In the view of enabling unified driver > > for octeontx2(cn9k)/octeontx3(cn10k), removing drivers/octeontx2 > > drivers and replace with drivers/cnxk/ which > > supports both octeontx2(cn9k) and octeontx3(cn10k) SoCs. > > > > This patch does the following > > > > - Replace drivers/common/octeontx2/ with drivers/common/cnxk/ > > - Replace drivers/mempool/octeontx2/ with drivers/mempool/cnxk/ > > - Replace drivers/net/octeontx2/ with drivers/net/cnxk/ > > - Replace drivers/event/octeontx2/ with drivers/event/cnxk/ > > - Replace drivers/crypto/octeontx2/ with drivers/crypto/cnxk/ > > - Rename config/arm/arm64_octeontx2_linux_gcc as > > config/arm/arm64_cn9k_linux_gcc > > - Update the documentation and MAINTAINERS to reflect the same. > > - Change the reference to OCTEONTX2 as OCTEON 9. The kernel related > > documentation is not accounted for this change as kernel documentation > > still uses OCTEONTX2. > > > > Depends-on: series-20804 ("common/cnxk: add REE HW definitions") > > Signed-off-by: Jerin Jacob > > --- > > MAINTAINERS | 37 - > > app/test/meson.build | 1 - > > app/test/test_cryptodev.c | 7 - > > app/test/test_cryptodev.h | 1 - > > app/test/test_cryptodev_asym.c | 17 - > > app/test/test_eventdev.c | 8 - > > config/arm/arm64_cn10k_linux_gcc | 1 - > > ...teontx2_linux_gcc => arm64_cn9k_linux_gcc} | 3 +- > > config/arm/meson.build | 10 +- > > devtools/check-abi.sh | 4 + > > doc/guides/cryptodevs/features/octeontx2.ini | 87 - > > doc/guides/cryptodevs/index.rst | 1 - > > doc/guides/cryptodevs/octeontx2.rst | 188 - > > doc/guides/dmadevs/cnxk.rst | 2 +- > > doc/guides/eventdevs/features/octeontx2.ini | 30 - > > doc/guides/eventdevs/index.rst | 1 - > > doc/guides/eventdevs/octeontx2.rst | 178 - > > doc/guides/mempool/index.rst | 1 - > > doc/guides/mempool/octeontx2.rst | 92 - > > doc/guides/nics/cnxk.rst | 4 +- > > doc/guides/nics/features/octeontx2.ini | 97 - > > doc/guides/nics/features/octeontx2_vec.ini | 48 - > > doc/guides/nics/features/octeontx2_vf.ini | 45 - > > doc/guides/nics/index.rst | 1 - > > doc/guides/nics/octeontx2.rst | 465 --- > > doc/guides/nics/octeontx_ep.rst | 4 +- > > doc/guides/platform/cnxk.rst | 12 + > > .../octeontx2_packet_flow_hw_accelerators.svg | 2804 -------------- > > .../img/octeontx2_resource_virtualization.svg | 2418 ------------ > > doc/guides/platform/index.rst | 1 - > > doc/guides/platform/octeontx2.rst | 520 --- > > doc/guides/rel_notes/deprecation.rst | 17 - > > doc/guides/rel_notes/release_19_08.rst | 12 +- > > doc/guides/rel_notes/release_19_11.rst | 6 +- > > doc/guides/rel_notes/release_20_02.rst | 8 +- > > doc/guides/rel_notes/release_20_05.rst | 4 +- > > doc/guides/rel_notes/release_20_08.rst | 6 +- > > doc/guides/rel_notes/release_20_11.rst | 8 +- > > doc/guides/rel_notes/release_21_02.rst | 10 +- > > doc/guides/rel_notes/release_21_05.rst | 6 +- > > doc/guides/rel_notes/release_21_11.rst | 2 +- > > Not sure about updating old release notes files, using 'octeontx2' still can make > sense for the context of those releases. OK. I will send v2 with keeping octeontx2 in OLD release notes. > > Also search still gives some instances of 'octeontx2', like 'devtools/check-abi.sh' > one, can you please confirm if OK to have them: > $git grep -i octeontx2 This change to skip octeontx2 driver for ABI check as it is removed. This change is needed. if grep -qE "\ > Except from above items, agree with change in principal and build test looks good: > Acked-by: Ferruh Yigit