From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 7AAFCA0588; Thu, 16 Apr 2020 23:20:32 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 5CD381DE43; Thu, 16 Apr 2020 23:20:30 +0200 (CEST) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id E87451DD6D for ; Thu, 16 Apr 2020 23:20:27 +0200 (CEST) IronPort-SDR: ZFHY6gDYp78KRoAMzEu34E+2RjBe8fp9yioozUbr/TqyedI5TQ6Zpfqojy5ms7IyUeCYBDGN7K YDiiJ6WIXlrA== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 16 Apr 2020 14:20:26 -0700 IronPort-SDR: ounZ5MWhFHQa36QiT+5RHYmqUIa/T8Gps8KDgRJxiyxViwKq+XDhPc8NZ6y9oHFdR7/ypZUGZP SmftndgPTx3A== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.72,392,1580803200"; d="scan'208";a="427989745" Received: from orsmsx104.amr.corp.intel.com ([10.22.225.131]) by orsmga005.jf.intel.com with ESMTP; 16 Apr 2020 14:20:26 -0700 Received: from orsmsx113.amr.corp.intel.com (10.22.240.9) by ORSMSX104.amr.corp.intel.com (10.22.225.131) with Microsoft SMTP Server (TLS) id 14.3.439.0; Thu, 16 Apr 2020 14:20:25 -0700 Received: from orsmsx155.amr.corp.intel.com ([169.254.7.107]) by ORSMSX113.amr.corp.intel.com ([169.254.9.188]) with mapi id 14.03.0439.000; Thu, 16 Apr 2020 14:20:25 -0700 From: "Chautru, Nicolas" To: Akhil Goyal , "dev@dpdk.org" CC: "Richardson, Bruce" Thread-Topic: [PATCH v2 05/13] baseband/fpga_5gnr_fec: add device info_get function Thread-Index: AQHWBia13HNPVyTwIUqIoKfwMVogRKh8nzOA//+9n1A= Date: Thu, 16 Apr 2020 21:20:25 +0000 Message-ID: <1183128033837D43A851F70F33ED5C57893CA269@ORSMSX155.amr.corp.intel.com> References: <1585526580-113508-1-git-send-email-nicolas.chautru@intel.com> <1585526580-113508-6-git-send-email-nicolas.chautru@intel.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action x-originating-ip: [10.22.254.140] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v2 05/13] baseband/fpga_5gnr_fec: add device info_get function X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi Akhil,=20 > > Add in the "info_get" function to the driver, to allow us to query the > > device. > > No capability are available yet. > > Linking bbdev-test to support the PMD with null capability. > > > > Signed-off-by: Nicolas Chautru > > --- > > app/test-bbdev/Makefile | 3 ++ > > app/test-bbdev/meson.build | 3 ++ > > drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 62 > > ++++++++++++++++++++++ > > 3 files changed, 68 insertions(+) > > > > diff --git a/app/test-bbdev/Makefile b/app/test-bbdev/Makefile index > > c53982f..e951302 100644 > > --- a/app/test-bbdev/Makefile > > +++ b/app/test-bbdev/Makefile > > @@ -24,5 +24,8 @@ LDLIBS +=3D -lm > > ifeq ($(CONFIG_RTE_LIBRTE_PMD_BBDEV_FPGA_LTE_FEC),y) > > LDLIBS +=3D -lrte_pmd_bbdev_fpga_lte_fec endif > > +ifeq ($(CONFIG_RTE_LIBRTE_PMD_BBDEV_FPGA_5GNR_FEC),y) > > +LDLIBS +=3D -lrte_pmd_bbdev_fpga_5gnr_fec endif > > > > include $(RTE_SDK)/mk/rte.app.mk > > diff --git a/app/test-bbdev/meson.build b/app/test-bbdev/meson.build > > index 4f53a2e..e57e019 100644 > > --- a/app/test-bbdev/meson.build > > +++ b/app/test-bbdev/meson.build > > @@ -10,3 +10,6 @@ deps +=3D ['bbdev', 'bus_vdev'] if > > dpdk_conf.has('RTE_LIBRTE_PMD_BBDEV_FPGA_LTE_FEC') > > deps +=3D ['pmd_bbdev_fpga_lte_fec'] > > endif > > +if dpdk_conf.has('RTE_LIBRTE_PMD_BBDEV_FPGA_5GNR_FEC') > > + deps +=3D ['pmd_bbdev_fpga_5gnr_fec'] > > +endif > > \ No newline at end of file >=20 > Changes in test-bbdev should not be in this patch. This was done so that to be able to run unit-test from that point on since = we can run info_get and pick up supported hw. Similar logic was used with r= aw driver which I used for inspiration (http://patches.dpdk.org/patch/55878= /). I can split into 2 commits still. Thanks