From: Ferruh Yigit <ferruh.yigit@intel.com>
To: <jerinj@marvell.com>, <dev@dpdk.org>,
Thomas Monjalon <thomas@monjalon.net>,
Akhil Goyal <gakhil@marvell.com>,
Declan Doherty <declan.doherty@intel.com>,
Ruifeng Wang <ruifeng.wang@arm.com>,
Jan Viktorin <viktorin@rehivetech.com>,
Bruce Richardson <bruce.richardson@intel.com>,
"Ray Kinsella" <mdr@ashroe.eu>,
Radha Mohan Chintakuntla <radhac@marvell.com>,
Veerasenareddy Burru <vburru@marvell.com>,
Nithin Dabilpuram <ndabilpuram@marvell.com>,
Kiran Kumar K <kirankumark@marvell.com>,
"Sunil Kumar Kori" <skori@marvell.com>,
Satha Rao <skoteshwar@marvell.com>,
"Nalla Pradeep" <pnalla@marvell.com>,
Ciara Power <ciara.power@intel.com>,
"Pavan Nikhilesh" <pbhagavatula@marvell.com>,
Shijith Thotton <sthotton@marvell.com>,
Ashwin Sekhar T K <asekhar@marvell.com>,
Anatoly Burakov <anatoly.burakov@intel.com>
Cc: <sburla@marvell.com>, <lironh@marvell.com>
Subject: Re: [dpdk-dev] [PATCH v1] drivers: remove octeontx2 drivers
Date: Mon, 6 Dec 2021 13:35:27 +0000 [thread overview]
Message-ID: <c8bfedb9-5fe2-928d-c9e3-3d0324fa43ce@intel.com> (raw)
In-Reply-To: <20211206083542.3115019-1-jerinj@marvell.com>
On 12/6/2021 8:35 AM, jerinj@marvell.com wrote:
> From: Jerin Jacob<jerinj@marvell.com>
>
> 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<jerinj@marvell.com>
> ---
> 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.
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
Except from above items, agree with change in principal and build test looks good:
Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
next prev parent reply other threads:[~2021-12-06 13:35 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-06 8:35 jerinj
2021-12-06 13:35 ` Ferruh Yigit [this message]
2021-12-07 7:39 ` Jerin Jacob
2021-12-07 11:01 ` Ferruh Yigit
2021-12-07 11:51 ` Kevin Traynor
2022-01-11 16:58 ` Thomas Monjalon
2021-12-07 8:12 ` Akhil Goyal
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=c8bfedb9-5fe2-928d-c9e3-3d0324fa43ce@intel.com \
--to=ferruh.yigit@intel.com \
--cc=anatoly.burakov@intel.com \
--cc=asekhar@marvell.com \
--cc=bruce.richardson@intel.com \
--cc=ciara.power@intel.com \
--cc=declan.doherty@intel.com \
--cc=dev@dpdk.org \
--cc=gakhil@marvell.com \
--cc=jerinj@marvell.com \
--cc=kirankumark@marvell.com \
--cc=lironh@marvell.com \
--cc=mdr@ashroe.eu \
--cc=ndabilpuram@marvell.com \
--cc=pbhagavatula@marvell.com \
--cc=pnalla@marvell.com \
--cc=radhac@marvell.com \
--cc=ruifeng.wang@arm.com \
--cc=sburla@marvell.com \
--cc=skori@marvell.com \
--cc=skoteshwar@marvell.com \
--cc=sthotton@marvell.com \
--cc=thomas@monjalon.net \
--cc=vburru@marvell.com \
--cc=viktorin@rehivetech.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).