From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <dev-bounces@dpdk.org>
Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124])
	by inbox.dpdk.org (Postfix) with ESMTP id 1C8E4A0546;
	Fri, 30 Apr 2021 07:11:24 +0200 (CEST)
Received: from [217.70.189.124] (localhost [127.0.0.1])
	by mails.dpdk.org (Postfix) with ESMTP id 8BCF440693;
	Fri, 30 Apr 2021 07:11:23 +0200 (CEST)
Received: from mail-pg1-f174.google.com (mail-pg1-f174.google.com
 [209.85.215.174])
 by mails.dpdk.org (Postfix) with ESMTP id C71E140395
 for <dev@dpdk.org>; Fri, 30 Apr 2021 07:11:21 +0200 (CEST)
Received: by mail-pg1-f174.google.com with SMTP id s22so27564231pgk.6
 for <dev@dpdk.org>; Thu, 29 Apr 2021 22:11:21 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;
 h=mime-version:references:in-reply-to:from:date:message-id:subject:to
 :cc; bh=cZTWux/HVY9kVRiaSSupTSMeyN6AFCPXI+855wQ82/A=;
 b=kW2ozMMe73J5O/2m+bBAhfB+X9F6JK4JPmeKB0ZzTEJZxqyb1shJTgUiTsvqdywUio
 fnz8Hct3R3XShSVl60ve+f62X9rsQh2IXJ7rjsaIqGgKEnjCG5B6P7dDHSv+LdTP7cei
 3M7MuhcxJVf6c6o0Ce/i4/VqkL3qf78GhnaphMKX0pd1IGUi6U6tOWqpyIdMjyDbbGqE
 q1wgQ3A/VJ46burZf+c7eSM7oIbLqOYC3K+enbs2zy5MCsaTCx6O8532jXWP+voR3606
 ryoZqt2BgcoWPdbe7jYcd/izLowl+uYIpXYVip5W7lWxAcbXdXcNsf5ywMbhl6HjzKgN
 hI7Q==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=1e100.net; s=20161025;
 h=x-gm-message-state:mime-version:references:in-reply-to:from:date
 :message-id:subject:to:cc;
 bh=cZTWux/HVY9kVRiaSSupTSMeyN6AFCPXI+855wQ82/A=;
 b=pJg4ZleQopNF/FpXVEmY/l8YDj7BcYXXy6+Uv6HPHwwWV3kjVmyEu3Ymsjr9gXwlzS
 h7kdT0/uAf7yQgFacE+KHW/Z3ENjkTTtKtXyqqcxTv2fw03CuB+nSJh8LkUoZWOQlhcn
 DbMQuXfkPxqz8juUr2sNllWDg/zhQXFHU67+nVzruGhXFfRq8N6tCalW111Istf59TA6
 dIXzAuB0mQjsHSZTlsBVLTEDsxLgKn2D5EhpEQPMNChPAsZ63eTpSTDiqdcmfWCcIFus
 3tg3AiEBx/hqAztGBO8yz28nTmre1cOQN/SK5FBrENG7r+w63hfswcYjydgNf6KXHpId
 8LSQ==
X-Gm-Message-State: AOAM530j7iSkt/sC7uKK4xuuQwMvu/UYJdmaoaDJb4CXRUHN0z0FTBis
 265WOhi4WwAS2X4YHpo8Kqh7bEozbpdrgxuXueY=
X-Google-Smtp-Source: ABdhPJye85dzBZrzoIZURnST1BWSR/TCR2GsGksl8GfxQYGmqefT+yEv8uYKxvsjJPAzm5SKoM7oPnICcHZcdBJdqHA=
X-Received: by 2002:a63:6545:: with SMTP id z66mr3160136pgb.292.1619759480690; 
 Thu, 29 Apr 2021 22:11:20 -0700 (PDT)
MIME-Version: 1.0
References: <20210306162942.6845-1-pbhagavatula@marvell.com>
 <20210426174441.2302-1-pbhagavatula@marvell.com>
In-Reply-To: <20210426174441.2302-1-pbhagavatula@marvell.com>
From: Jerin Jacob <jerinjacobk@gmail.com>
Date: Fri, 30 Apr 2021 10:41:04 +0530
Message-ID: <CALBAE1OB4zD2poyOZ24qGbiw+vZycEN42KcCMLxdfgAEEzOBRQ@mail.gmail.com>
To: Pavan Nikhilesh <pbhagavatula@marvell.com>
Cc: Jerin Jacob <jerinj@marvell.com>, dpdk-dev <dev@dpdk.org>
Content-Type: text/plain; charset="UTF-8"
Subject: Re: [dpdk-dev] [PATCH v2 00/33] Marvell CNXK Event device Driver
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>,
 <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>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

On Mon, Apr 26, 2021 at 11:14 PM <pbhagavatula@marvell.com> wrote:
>
> From: Pavan Nikhilesh <pbhagavatula@marvell.com>
>
> This patchset adds support for Marvell CN106XX SoC based on 'common/cnxk'
> driver. In future, CN9K a.k.a octeontx2 will also be supported by same
> driver when code is ready and 'event/octeontx2' will be deprecated.
>
> v2 Changes:
> - Split Rx/Tx adapter into seperate patch set to remove dependency on net/cnxk
> - Add missing xStats patch.
> - Fix incorrect head wait operation.


# Please check the following checkpatch issues[1]
# Please fix the copyright to Copyright(c) 2021 Marvell[2]
# Please fix documentation to have synergy with rest of doc[2]



[1]
[for-main][dpdk-next-eventdev] $ ./devtools/checkpatches.sh -n 33

### event/cnxk: add devargs to control SSO HWGRP QoS

WARNING:TYPO_SPELLING: 'Aka' may be misspelled - perhaps 'A.k.a.'?
#17:
Qx  -> Event queue Aka SSO GGRP.
                   ^^^

total: 0 errors, 1 warnings, 154 lines checked

### event/cnxk: add port config functions

WARNING:TYPO_SPELLING: 'aka' may be misspelled - perhaps 'a.k.a.'?
#6:
Add SSO HWS aka event port setup and release functions.
            ^^^

WARNING:TYPO_SPELLING: 'aka' may be misspelled - perhaps 'a.k.a.'?
#500: FILE: drivers/event/cnxk/cnxk_eventdev.h:103:
+/* Event port aka GWS */
               ^^^

total: 0 errors, 2 warnings, 526 lines checked

### event/cnxk: support event timer

WARNING:TYPO_SPELLING: 'aka' may be misspelled - perhaps 'a.k.a.'?
#6:
Add event timer adapter aka TIM initialization on SSO probe.
                        ^^^

WARNING:TYPO_SPELLING: 'aka' may be misspelled - perhaps 'a.k.a.'?
#21: FILE: doc/guides/eventdevs/cnxk.rst:40:
+- Up to 256 TIM rings aka event timer adapters.
                       ^^^

total: 0 errors, 2 warnings, 136 lines checked



[2]

[for-main]dell[dpdk-next-eventdev] $ git diff
diff --git a/MAINTAINERS b/MAINTAINERS
index d1ae33d48f..5a2297e999 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1224,7 +1224,7 @@ M: Jerin Jacob <jerinj@marvell.com>
 F: drivers/event/octeontx2/
 F: doc/guides/eventdevs/octeontx2.rst

-Marvell OCTEON CNXK
+Marvell cnxk
 M: Pavan Nikhilesh <pbhagavatula@marvell.com>
 M: Shijith Thotton <sthotton@marvell.com>
 F: drivers/event/cnxk/
diff --git a/doc/guides/eventdevs/cnxk.rst b/doc/guides/eventdevs/cnxk.rst
index c42784a3be..27cc32e26e 100644
--- a/doc/guides/eventdevs/cnxk.rst
+++ b/doc/guides/eventdevs/cnxk.rst
@@ -1,8 +1,8 @@
 ..  SPDX-License-Identifier: BSD-3-Clause
-    Copyright(c) 2021 Marvell International Ltd.
+    Copyright(c) 2021 Marvell.

-OCTEON CNXK SSO Eventdev Driver
-==========================
+Marvell cnxk SSO Eventdev Driver
+================================

 The SSO PMD (**librte_event_cnxk**) and provides poll mode
 eventdev driver support for the inbuilt event device found in the
@@ -11,7 +11,7 @@ eventdev driver support for the inbuilt event device
found in the
 More information about OCTEON CNXK SoC can be found at `Marvell
Official Website
 <https://www.marvell.com/embedded-processors/infrastructure-processors/>`_.

-Supported OCTEON CNXK SoCs
+Supported OCTEON cnxk SoCs
 --------------------------

 - CN9XX
>
> Pavan Nikhilesh (16):
>   event/cnxk: add build infra and device setup
>   event/cnxk: add platform specific device probe
>   event/cnxk: add common configuration validation
>   event/cnxk: allocate event inflight buffers
>   event/cnxk: add devargs to configure getwork mode
>   event/cnxk: add SSO HW device operations
>   event/cnxk: add SSO GWS fastpath enqueue functions
>   event/cnxk: add SSO GWS dequeue fastpath functions
>   event/cnxk: add SSO selftest and dump
>   event/cnxk: add event port and queue xstats
>   event/cnxk: add devargs to disable NPA
>   event/cnxk: allow adapters to resize inflights
>   event/cnxk: add TIM bucket operations
>   event/cnxk: add timer arm routine
>   event/cnxk: add timer arm timeout burst
>   event/cnxk: add timer cancel function
>
> Shijith Thotton (17):
>   event/cnxk: add device capabilities function
>   event/cnxk: add platform specific device config
>   event/cnxk: add event queue config functions
>   event/cnxk: add devargs for inflight buffer count
>   event/cnxk: add devargs to control SSO HWGRP QoS
>   event/cnxk: add port config functions
>   event/cnxk: add event port link and unlink
>   event/cnxk: add device start function
>   event/cnxk: add device stop and close functions
>   event/cnxk: support event timer
>   event/cnxk: add timer adapter capabilities
>   event/cnxk: create and free timer adapter
>   event/cnxk: add timer adapter info function
>   event/cnxk: add devargs for chunk size and rings
>   event/cnxk: add timer stats get and reset
>   event/cnxk: add timer adapter start and stop
>   event/cnxk: add devargs to control timer adapters
>
>  MAINTAINERS                                 |    6 +
>  app/test/test_eventdev.c                    |   14 +
>  doc/guides/eventdevs/cnxk.rst               |  162 ++
>  doc/guides/eventdevs/index.rst              |    1 +
>  drivers/common/cnxk/roc_sso.c               |   63 +
>  drivers/common/cnxk/roc_sso.h               |   19 +
>  drivers/event/cnxk/cn10k_eventdev.c         |  509 ++++++
>  drivers/event/cnxk/cn10k_worker.c           |  115 ++
>  drivers/event/cnxk/cn10k_worker.h           |  175 +++
>  drivers/event/cnxk/cn9k_eventdev.c          |  578 +++++++
>  drivers/event/cnxk/cn9k_worker.c            |  236 +++
>  drivers/event/cnxk/cn9k_worker.h            |  297 ++++
>  drivers/event/cnxk/cnxk_eventdev.c          |  647 ++++++++
>  drivers/event/cnxk/cnxk_eventdev.h          |  253 +++
>  drivers/event/cnxk/cnxk_eventdev_adptr.c    |   67 +
>  drivers/event/cnxk/cnxk_eventdev_selftest.c | 1570 +++++++++++++++++++
>  drivers/event/cnxk/cnxk_eventdev_stats.c    |  289 ++++
>  drivers/event/cnxk/cnxk_tim_evdev.c         |  538 +++++++
>  drivers/event/cnxk/cnxk_tim_evdev.h         |  275 ++++
>  drivers/event/cnxk/cnxk_tim_worker.c        |  191 +++
>  drivers/event/cnxk/cnxk_tim_worker.h        |  601 +++++++
>  drivers/event/cnxk/cnxk_worker.h            |  101 ++
>  drivers/event/cnxk/meson.build              |   23 +
>  drivers/event/cnxk/version.map              |    3 +
>  drivers/event/meson.build                   |    1 +
>  25 files changed, 6734 insertions(+)
>  create mode 100644 doc/guides/eventdevs/cnxk.rst
>  create mode 100644 drivers/event/cnxk/cn10k_eventdev.c
>  create mode 100644 drivers/event/cnxk/cn10k_worker.c
>  create mode 100644 drivers/event/cnxk/cn10k_worker.h
>  create mode 100644 drivers/event/cnxk/cn9k_eventdev.c
>  create mode 100644 drivers/event/cnxk/cn9k_worker.c
>  create mode 100644 drivers/event/cnxk/cn9k_worker.h
>  create mode 100644 drivers/event/cnxk/cnxk_eventdev.c
>  create mode 100644 drivers/event/cnxk/cnxk_eventdev.h
>  create mode 100644 drivers/event/cnxk/cnxk_eventdev_adptr.c
>  create mode 100644 drivers/event/cnxk/cnxk_eventdev_selftest.c
>  create mode 100644 drivers/event/cnxk/cnxk_eventdev_stats.c
>  create mode 100644 drivers/event/cnxk/cnxk_tim_evdev.c
>  create mode 100644 drivers/event/cnxk/cnxk_tim_evdev.h
>  create mode 100644 drivers/event/cnxk/cnxk_tim_worker.c
>  create mode 100644 drivers/event/cnxk/cnxk_tim_worker.h
>  create mode 100644 drivers/event/cnxk/cnxk_worker.h
>  create mode 100644 drivers/event/cnxk/meson.build
>  create mode 100644 drivers/event/cnxk/version.map
>
> --
> 2.17.1
>