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 C906AA0C46;
	Fri, 17 Sep 2021 10:33:18 +0200 (CEST)
Received: from [217.70.189.124] (localhost [127.0.0.1])
	by mails.dpdk.org (Postfix) with ESMTP id 50380407FF;
	Fri, 17 Sep 2021 10:33:18 +0200 (CEST)
Received: from EUR03-DB5-obe.outbound.protection.outlook.com
 (mail-eopbgr40054.outbound.protection.outlook.com [40.107.4.54])
 by mails.dpdk.org (Postfix) with ESMTP id 84BDD406B4
 for <dev@dpdk.org>; Fri, 17 Sep 2021 10:33:17 +0200 (CEST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;
 b=aVoTaqUcrIvYamASHf6MTLIR06lrrJlBRxw/Ghwj/rttUNzdPQ5m0rjYSz3tFjWI2FnM3A4RcLIsToYu+b0tAU6A03YEbVtPh+E+xlLv1ikIqvOO49Mhogd2gQKKnPmtCGD4A+PtjdDhHhPqMHMCqTY96DdLoOZKBpp9jn+f63A8V4CLRR5JtpNcGHlGmTmq8l6COyD6OD4GMO9bJBFD61Q55JbV9kG7CVF20fcRLlgGlk+dYd3P9lzO14AwGfMQ/Ok+6z5gURFm3ANl1uoxdbX1MCCVDvbKNAIX5AvDk25ZHFkdWQP5w0BlfVm7pjP/WvoNS99f+Q0ZPP7nRKqPqg==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; 
 s=arcselector9901;
 h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; 
 bh=dZlj7f5yQF4Asmm92xdOMH6XecXfAreVNygzNbY/u0k=;
 b=YYw+95HoS79UdmQF/si0bZzS63wRwV0r3zFAJGUtCO6i+2pUkmPbS3p4okEbHayudklPv6bxQGUzjsFe5FWZQUE54ZREVfCA1kgjd9xIOv7FV2cDSDPuJufgerdph+7dlkHxd9y80ZE8ZsAUWreitHsuXWnMa3/aVJdJcCuOjG8N6OD43MabobDDfSxYZgCIuZqYsYWDIgxFfUGtTcPuM//5y0Na9wwur7Y4Wg+dkufGbGqQvviTpZ6vNtSmDOZkostsvVR/dkfhUd1rS3dVqluUSRLMgvVMr6tKgSKLP3zHenckcAG1UOaQhLho0uFj++y4RL55wPd4SZT9IR/1Tw==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass
 smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass
 header.d=nxp.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; 
 h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;
 bh=dZlj7f5yQF4Asmm92xdOMH6XecXfAreVNygzNbY/u0k=;
 b=HnSsykyVw65jggqytB2le9/HYo9y2xyTYjKGU9GMF72obOEQoPhLCFggcaDV5mEwRZDmmzxX4X7I8wGQWp5HAYSbwV30rMXEl9jO9TWZAq5D8/hJ/qzZx+jksCnTpO8TJe86JnRDtnk1J+MOuYske0ko5smQzcFTEfG+fKwDymw=
Received: from DB9PR04MB8429.eurprd04.prod.outlook.com (2603:10a6:10:24e::23)
 by DB3PR0402MB3707.eurprd04.prod.outlook.com (2603:10a6:8:2::21) with
 Microsoft SMTP Server (version=TLS1_2,
 cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4523.17; Fri, 17 Sep
 2021 08:33:15 +0000
Received: from DB9PR04MB8429.eurprd04.prod.outlook.com
 ([fe80::dcda:93c2:47e7:19b4]) by DB9PR04MB8429.eurprd04.prod.outlook.com
 ([fe80::dcda:93c2:47e7:19b4%9]) with mapi id 15.20.4523.017; Fri, 17 Sep 2021
 08:33:15 +0000
From: Nipun Gupta <nipun.gupta@nxp.com>
To: "Chautru, Nicolas" <nicolas.chautru@intel.com>, "dev@dpdk.org"
 <dev@dpdk.org>, "gakhil@marvell.com" <gakhil@marvell.com>
CC: "david.marchand@redhat.com" <david.marchand@redhat.com>, Hemant Agrawal
 <hemant.agrawal@nxp.com>
Thread-Topic: [PATCH v5 5/9] baseband/la12xx: add queue and modem config
 support
Thread-Index: AQHXp8/qjTHmq/lmCkGZTGJ+YWngg6uiUnsAgAWapSA=
Date: Fri, 17 Sep 2021 08:33:15 +0000
Message-ID: <DB9PR04MB8429C2F8B1DB14BB3C5AA8E7E6DD9@DB9PR04MB8429.eurprd04.prod.outlook.com>
References: <20210318063421.14895-1-hemant.agrawal@nxp.com>
 <20210912121510.22699-1-nipun.gupta@nxp.com>
 <20210912121510.22699-6-nipun.gupta@nxp.com>
 <BY5PR11MB4451B4DA3E29BC9608FB0813F8D99@BY5PR11MB4451.namprd11.prod.outlook.com>
In-Reply-To: <BY5PR11MB4451B4DA3E29BC9608FB0813F8D99@BY5PR11MB4451.namprd11.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
authentication-results: intel.com; dkim=none (message not signed)
 header.d=none;intel.com; dmarc=none action=none header.from=nxp.com;
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: d5c6d37e-60ef-4acf-6327-08d979b5cb35
x-ms-traffictypediagnostic: DB3PR0402MB3707:
x-ms-exchange-transport-forked: True
x-microsoft-antispam-prvs: <DB3PR0402MB3707EC750FEF14F748C537C6E6DD9@DB3PR0402MB3707.eurprd04.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:390;
x-ms-exchange-senderadcheck: 1
x-ms-exchange-antispam-relay: 0
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: vsaCFpFNnCP8Xj6fHJhSUi9cOrFPYb+I3nZDwaBbG341rBahxXf7Od/KYwUJnJkG9h7pnlGBmTspYCdkLBsPDha3QHXWCe+AFkwn/4NWIBixt5IsKpWXCCrcJP2OTax+w4bKXNQwDU+JVsma6L+r+pHPgi74Y0vL75VB8G4+w0VvX0xvuyj7+awl6prgGmzQBCfLNs6seS8NLkUPJPYmt+ErPjyhIQeH1v0/tONFq6HEjsMQeOy+pORVCxDnGUI4bvjbrALs1HoPTc4ikXUsTwJBUUDIb81TLGLG50Uci1206xJbo62t4EhJj/uoT7hFW6EMb9T0dAhC88SLBFspZ4NGulY7v0mKz6L0LXzUPWeegujqxaoesxYjb4aFd/rTaXLWa3XZfBOXWXXDcb6aHUKGGrpI5/rUI2VPcPUMpvxIl2EBp9Cgj4A3m7yOesX8rPHkTSCU/oIj9L95wePt1/RDJyva2mjIGfsaMuIuy5LHhM6OgXpicfuC7f+gjHLTb5iJM9aOP23VGCrXzR8YC7aHIR+Bh9sMX6Fs8rt73XPUoTDSKr3qUoDv0FFJsmHdC7o+puFK/b0NfDk9yUFwADy0uBhL3j4IgeH9aMDk1h4dzzhiiXHakyU3Bg3rEAdozHaolihQUTcGvPxD1BIofidCa+hlvGVy8DQNNiSEP+stci9Pa5OMgZo6DdSxn6xotcEmHnl9OPKLEnX/ehbAPSVXjpj0H+AE7hR5IYskaFBrYvBNsdgHUQjO+1BGxwRbfqpmSKg9VFfB9XwEV23Wg5/h+nhPpXouHGGnNva27JE=
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:;
 IPV:NLI; SFV:NSPM; H:DB9PR04MB8429.eurprd04.prod.outlook.com; PTR:; CAT:NONE;
 SFS:(4636009)(136003)(346002)(376002)(366004)(39860400002)(396003)(86362001)(71200400001)(2906002)(66476007)(8936002)(53546011)(76116006)(64756008)(66446008)(6506007)(26005)(83380400001)(30864003)(55236004)(8676002)(66946007)(7696005)(186003)(66556008)(38070700005)(52536014)(38100700002)(5660300002)(122000001)(44832011)(478600001)(55016002)(4326008)(9686003)(33656002)(316002)(110136005)(54906003)(559001)(579004);
 DIR:OUT; SFP:1101; 
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?gtecdBMiYlBLNWBY7E3xhlxUXfkeGgQS4V0YuU5JGsKYQSUWqTR0Px2RU4/T?=
 =?us-ascii?Q?sPMtUQzsol27GNMz4ym/bmjHViNsjMkiUdGiCri2Nnqxzu6/tu4FWNvzKU2C?=
 =?us-ascii?Q?mnzlJZb2EUDFohEb/hEHK0sDvoV3yXw3xHAXIyJTcD48ZlsNFufrPLHIZEor?=
 =?us-ascii?Q?1TTzTchmZrNaSG39pH1UHfSewLUdseY+BRzEpJdfA3WN+OGQJwdwfahrnYpV?=
 =?us-ascii?Q?TK3O8mP53Iq43SOLjc9pwTt7Ut2e4IjFr2XfWkoYGFYamWys8UbOmM5IxA3p?=
 =?us-ascii?Q?ozGzflPSiy+3GDjEnrpGTUoeK+zLUNAS4tNPzekzo05fJVAm728zuEXrVjJi?=
 =?us-ascii?Q?+wq5xypaJqo8CyP3BZz1f/uDSyTGbxJQQArfXD05jTL1cYY/vZ6kvtGUQ6Im?=
 =?us-ascii?Q?AIxSK4PnNIBtytqgEWyCuVBNYsPBU5s5tkP/SPR6jTFffEcgZN4Ae57IYRmM?=
 =?us-ascii?Q?3Fswc8ivkQnNgevXLywQLa0R8x/tUTf77w2bPss72fpW1pAYM6rmz1LAoBJC?=
 =?us-ascii?Q?2plLNyqEx8fl+hZb87gMMvrnyfhsviXqk3iCHpK1qcC6ZqaHJyB31SY7+kzh?=
 =?us-ascii?Q?WuA1NyeLa788W9q08Hw20cZ4fDAM4QSw9RLv4VObMvJl/1oo10+tsU+znu4z?=
 =?us-ascii?Q?O9vMmWFKkD6BWMaGFxHlMjT6VHs4dGPsLQbm/q0mgqZ3pNAfaM2Q5bLOOEvJ?=
 =?us-ascii?Q?2Mv85s9XAUk7tsrNy39bt6kjBYVaW8AnKxcJ3xgDRPd50zukaRoOGCyi7KyG?=
 =?us-ascii?Q?Sx0JAx7uMeAyMn5rNs6wS1psBTyJwQ4sTSV2QiQhITnPfLBzCHpBLHciRhwl?=
 =?us-ascii?Q?izHQxIiFa7xYob9UO0Ly3We/ZVOMv/xBihYz6sRgkrFwskDgd1KcPeaMbGEY?=
 =?us-ascii?Q?5NOpPvbcGmOmRzcvQDq+zLhsNVoeW6rDzNTU2s5vmyFk84yhDb3XS6Ha4u4S?=
 =?us-ascii?Q?d1syUAVESGZGK9RLQgnOW+/5e21rfULD4KNC6oRNQw2bo8VExugFRnG7ZMsQ?=
 =?us-ascii?Q?YG1yv4gijRTsr+nt1PLwUmr409/3lqRKFSEczcHxgYEuPAd4hETsLBjGh2w7?=
 =?us-ascii?Q?WGj993rpKkwl12IvxuXAZU1CDuOO+qHDoBKaL2SYX3518zhl+R7CHN1IQvxT?=
 =?us-ascii?Q?E1nn05tTbknGxElFdsvb6VgSLL9iHxeuQ13x5QweLOX2PedSiW6HnfMkg74s?=
 =?us-ascii?Q?tH1299gVEl2oqAE4TZe8cRzgIg0Eesxr9m5VfmtUvvRKDEVqX40e33bV7O4r?=
 =?us-ascii?Q?DFMwJEQvz1LVLheuxID/xpxL52ie006h4/+ZeVIf3+oRhkbdEFkv8m7Rgy9g?=
 =?us-ascii?Q?eUF8BqjF3Y048ekeBqDK+FJ2?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DB9PR04MB8429.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: d5c6d37e-60ef-4acf-6327-08d979b5cb35
X-MS-Exchange-CrossTenant-originalarrivaltime: 17 Sep 2021 08:33:15.3497 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 9nHnbIgLQz+EV+xHEgeg5XPVWcUQFuvR/oeG+TgpuKltqQcEwLh1EMpzl6h5Bl1psMFxue5nM+j6syUF3WNofw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB3PR0402MB3707
Subject: Re: [dpdk-dev] [PATCH v5 5/9] baseband/la12xx: add queue and modem
 config support
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>



> -----Original Message-----
> From: Chautru, Nicolas <nicolas.chautru@intel.com>
> Sent: Tuesday, September 14, 2021 12:26 AM
> To: Nipun Gupta <nipun.gupta@nxp.com>; dev@dpdk.org; gakhil@marvell.com
> Cc: david.marchand@redhat.com; Hemant Agrawal <hemant.agrawal@nxp.com>
> Subject: RE: [PATCH v5 5/9] baseband/la12xx: add queue and modem config
> support
>=20
>=20
>=20
> > -----Original Message-----
> > From: Nipun Gupta <nipun.gupta@nxp.com>
> > Sent: Sunday, September 12, 2021 5:15 AM
> > To: dev@dpdk.org; gakhil@marvell.com; Chautru, Nicolas
> > <nicolas.chautru@intel.com>
> > Cc: david.marchand@redhat.com; hemant.agrawal@nxp.com; Nipun Gupta
> > <nipun.gupta@nxp.com>
> > Subject: [PATCH v5 5/9] baseband/la12xx: add queue and modem config
> > support
> >
> > From: Hemant Agrawal <hemant.agrawal@nxp.com>
> >
> > This patch add support for connecting with modem and creating the ipc
> > channel as queues with modem for the exchange of data.
> >
> > Signed-off-by: Nipun Gupta <nipun.gupta@nxp.com>
> > Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>
> > ---
> >  MAINTAINERS                                |   1 +
> >  doc/guides/bbdevs/index.rst                |   1 +
> >  doc/guides/bbdevs/la12xx.rst               |  81 +++
> >  doc/guides/rel_notes/release_21_11.rst     |   5 +
> >  drivers/baseband/la12xx/bbdev_la12xx.c     | 553 ++++++++++++++++++++-
> >  drivers/baseband/la12xx/bbdev_la12xx.h     |  11 +-
> >  drivers/baseband/la12xx/bbdev_la12xx_ipc.h | 189 ++++++-
> >  7 files changed, 831 insertions(+), 10 deletions(-)  create mode 10064=
4
> > doc/guides/bbdevs/la12xx.rst
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index a63e672c9e..2c243c10fe 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -1295,6 +1295,7 @@ NXP LA12xx driver
> >  M: Hemant Agrawal <hemant.agrawal@nxp.com>
> >  M: Nipun Gupta <nipun.gupta@nxp.com>
> >  F: drivers/baseband/la12xx/
> > +F: doc/guides/bbdevs/la12xx.rst
> >
> >
> >  Rawdev Drivers
> > diff --git a/doc/guides/bbdevs/index.rst b/doc/guides/bbdevs/index.rst
> > index 4445cbd1b0..cedd706fa6 100644
> > --- a/doc/guides/bbdevs/index.rst
> > +++ b/doc/guides/bbdevs/index.rst
> > @@ -14,3 +14,4 @@ Baseband Device Drivers
> >      fpga_lte_fec
> >      fpga_5gnr_fec
> >      acc100
> > +    la12xx
> > diff --git a/doc/guides/bbdevs/la12xx.rst b/doc/guides/bbdevs/la12xx.rs=
t
> > new file mode 100644 index 0000000000..3c9ac5c047
> > --- /dev/null
> > +++ b/doc/guides/bbdevs/la12xx.rst
> > @@ -0,0 +1,81 @@
> > +..  SPDX-License-Identifier: BSD-3-Clause
> > +    Copyright 2021 NXP
> > +
> > +NXP LA12xx Poll Mode Driver
> > +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
> > +
> > +The BBDEV LA12xx poll mode driver (PMD) supports an implementation for
> > +offloading High Phy processing functions like LDPC Encode / Decode 5GN=
R
> > +wireless acceleration function, using PCI based LA12xx Software define=
d
> > radio.
> > +
> > +More information can be found at `NXP Official Website
> >
> +<https://eur01.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Fwww=
.
> nxp.com%2Fproducts%2Fprocessors-and-microcontrollers%2Farm-
> &amp;data=3D04%7C01%7Cnipun.gupta%40nxp.com%7C92cc367c64324a156f730
> 8d976e8373d%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C6376715
> 62002990761%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjo
> iV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=3D4HYKiRKBu
> Y4VkVvN73UKM8ZP13NcBEx81ZbME9LiWhI%3D&amp;reserved=3D0
> > processors/layerscape-processors/layerscape-access-la1200-programmable-
> > baseband-processor:LA1200>`_.
> > +
> > +Features
> > +--------
> > +
> > +LA12xx PMD supports the following features:
> > +
> > +- Maximum of 8 UL queues
> > +- Maximum of 8 DL queues
> > +- PCIe Gen-3 x8 Interface
> > +- MSI-X
> > +
> > +Installation
> > +------------
> > +
> > +Section 3 of the DPDK manual provides instructions on installing and
> > compiling DPDK.
> > +
> > +DPDK requires hugepages to be configured as detailed in section 2 of t=
he
> > DPDK manual.
> > +
> > +Initialization
> > +--------------
> > +
> > +The device can be listed on the host console with:
> > +
> > +
> > +Use the following lspci command to get the multiple LA12xx processor
> > +ids. The device ID of the LA12xx baseband processor is "1c30".
> > +
> > +.. code-block:: console
> > +
> > +  sudo lspci -nn
> > +
> > +...
> > +0001:01:00.0 Power PC [0b20]: Freescale Semiconductor Inc Device
> > +[1957:1c30] ( rev 10) ...
> > +0002:01:00.0 Power PC [0b20]: Freescale Semiconductor Inc Device
> > +[1957:1c30] ( rev 10)
> > +
> > +
> > +Prerequisites
> > +-------------
> > +
> > +Currently supported by DPDK:
> > +
> > +- NXP LA1224 BSP **1.0+**.
> > +- NXP LA1224 PCIe Modem card connected to ARM host.
> > +
> > +- Follow the DPDK :ref:`Getting Started Guide for Linux <linux_gsg>` t=
o
> > setup the basic DPDK environment.
> > +
> > +* Use dev arg option ``modem=3D0`` to identify the modem instance for =
a
> > +given
> > +  device. This is required only if more than 1 modem cards are attache=
d to
> > host.
> > +  this is optional and the default value is 0.
> > +  e.g. ``--vdev=3Dbaseband_la12xx,modem=3D0``
> > +
> > +* Use dev arg option ``max_nb_queues=3Dx`` to specify the maximum
> > number
> > +of queues
> > +  to be used for communication with offload device i.e. modem. default=
 is
> > 16.
> > +  e.g. ``--vdev=3Dbaseband_la12xx,max_nb_queues=3D4``
> > +
> > +Enabling logs
> > +-------------
> > +
> > +For enabling logs, use the following EAL parameter:
> > +
> > +.. code-block:: console
> > +
> > +   ./your_bbdev_application <EAL args> --log-level=3Dla12xx:<level>
> > +
> > +Using ``bb.la12xx`` as log matching criteria, all Baseband PMD logs ca=
n
> > +be enabled which are lower than logging ``level``.
> > diff --git a/doc/guides/rel_notes/release_21_11.rst
> > b/doc/guides/rel_notes/release_21_11.rst
> > index 675b573834..a0e0ebbeb8 100644
> > --- a/doc/guides/rel_notes/release_21_11.rst
> > +++ b/doc/guides/rel_notes/release_21_11.rst
> > @@ -62,6 +62,11 @@ New Features
> >    * Added bus-level parsing of the devargs syntax.
> >    * Kept compatibility with the legacy syntax as parsing fallback.
> >
> > +* **Added NXP LA12xx baseband PMD.**
> > +
> > +  * Added a new baseband PMD driver for NXP LA12xx Software defined
> > radio.
> > +  * See the :doc:`../bbdevs/la12xx` for more details.
> > +
> >
> >  Removed Items
> >  -------------
> > diff --git a/drivers/baseband/la12xx/bbdev_la12xx.c
> > b/drivers/baseband/la12xx/bbdev_la12xx.c
> > index f26f3f2a08..57e957a93a 100644
> > --- a/drivers/baseband/la12xx/bbdev_la12xx.c
> > +++ b/drivers/baseband/la12xx/bbdev_la12xx.c
> > @@ -3,6 +3,11 @@
> >   */
> >
> >  #include <string.h>
> > +#include <unistd.h>
> > +#include <fcntl.h>
> > +#include <sys/ioctl.h>
> > +#include <sys/mman.h>
> > +#include <dirent.h>
> >
> >  #include <rte_common.h>
> >  #include <rte_bus_vdev.h>
> > @@ -31,11 +36,550 @@ struct bbdev_la12xx_params {
> >  #define LA12XX_VDEV_MODEM_ID_ARG	"modem"
> >  #define LA12XX_MAX_MODEM 4
> >
> > +#define LA12XX_MAX_CORES	4
> > +#define LA12XX_LDPC_ENC_CORE	0
> > +#define LA12XX_LDPC_DEC_CORE	1
> > +
> > +#define LA12XX_MAX_LDPC_ENC_QUEUES	4
> > +#define LA12XX_MAX_LDPC_DEC_QUEUES	4
> > +
> >  static const char * const bbdev_la12xx_valid_params[] =3D {
> >  	LA12XX_MAX_NB_QUEUES_ARG,
> >  	LA12XX_VDEV_MODEM_ID_ARG,
> >  };
> >
> > +static const struct rte_bbdev_op_cap bbdev_capabilities[] =3D {
> > +	{
> > +		.type   =3D RTE_BBDEV_OP_LDPC_ENC,
> > +		.cap.ldpc_enc =3D {
> > +			.capability_flags =3D
> > +					RTE_BBDEV_LDPC_CRC_24A_ATTACH
>=20
> Are you sure you genuinely don't support any Ratematching (not a typo)? T=
his is
> LDPC Encoder only? How do you redefine TB mode without deRMing supported,
> ie. what concatenation assumption?

Rate Matcher and de-matcher is supported. Will add it in the next spin.

>=20
> > |
> > +
> > 	RTE_BBDEV_LDPC_CRC_24B_ATTACH,
> > +			.num_buffers_src =3D
> > +
> > 	RTE_BBDEV_LDPC_MAX_CODE_BLOCKS,
> > +			.num_buffers_dst =3D
> > +
> > 	RTE_BBDEV_LDPC_MAX_CODE_BLOCKS,
> > +		}
> > +	},
> > +	{
> > +		.type   =3D RTE_BBDEV_OP_LDPC_DEC,
> > +		.cap.ldpc_dec =3D {
> > +			.capability_flags =3D
> > +				RTE_BBDEV_LDPC_CRC_TYPE_24A_CHECK |
> > +
> > 	RTE_BBDEV_LDPC_CRC_TYPE_24B_CHECK |
>=20
> Same comment as above, this is purely decoder?
> From documentation, there was support for MSI-X, still you don't expose t=
hat
> capability here?

Ill remove MSI-x capability. It is supported, but not yet implemented.

>=20
>=20
> > +
> > 	RTE_BBDEV_LDPC_CRC_TYPE_24B_DROP,
> > +			.num_buffers_src =3D
> > +
> > 	RTE_BBDEV_LDPC_MAX_CODE_BLOCKS,
> > +			.num_buffers_hard_out =3D
> > +
> > 	RTE_BBDEV_LDPC_MAX_CODE_BLOCKS,
> > +			.llr_size =3D 8,
> > +			.llr_decimals =3D 1,
> > +		}
> > +	},
> > +	RTE_BBDEV_END_OF_CAPABILITIES_LIST()
> > +};
> > +
> > +static struct rte_bbdev_queue_conf default_queue_conf =3D {
> > +	.queue_size =3D MAX_CHANNEL_DEPTH,
> > +};
> > +
> > +/* Get device info */
> > +static void
> > +la12xx_info_get(struct rte_bbdev *dev __rte_unused,
> > +		struct rte_bbdev_driver_info *dev_info) {
> > +	PMD_INIT_FUNC_TRACE();
> > +
> > +	dev_info->driver_name =3D RTE_STR(DRIVER_NAME);
> > +	dev_info->max_num_queues =3D LA12XX_MAX_QUEUES;
> > +	dev_info->queue_size_lim =3D MAX_CHANNEL_DEPTH;
> > +	dev_info->hardware_accelerated =3D true;
> > +	dev_info->max_dl_queue_priority =3D 0;
> > +	dev_info->max_ul_queue_priority =3D 0;
> > +	dev_info->default_queue_conf =3D default_queue_conf;
> > +	dev_info->capabilities =3D bbdev_capabilities;
> > +	dev_info->cpu_flag_reqs =3D NULL;
> > +	dev_info->min_alignment =3D 64;
> > +
> > +	rte_bbdev_log_debug("got device info from %u", dev->data-
> > >dev_id); }
> > +
> > +/* Release queue */
> > +static int
> > +la12xx_queue_release(struct rte_bbdev *dev, uint16_t q_id) {
> > +	RTE_SET_USED(dev);
> > +	RTE_SET_USED(q_id);
> > +
> > +	PMD_INIT_FUNC_TRACE();
> > +
> > +	/* TODO: Implement */
>=20
> Why is there a TODO here still? The implementation is not complete in tha=
t
> version of the code?

Ill remove this TODO. No queue release is supported.

>=20
>=20
> > +
> > +	return 0;
> > +}
> > +
> > +#define HUGEPG_OFFSET(A) \
> > +		((uint64_t) ((unsigned long) (A) \
> > +		- ((uint64_t)ipc_priv->hugepg_start.host_vaddr)))
> > +
> > +static int ipc_queue_configure(uint32_t channel_id,
> > +		ipc_t instance, struct bbdev_la12xx_q_priv *q_priv) {
> > +	ipc_userspace_t *ipc_priv =3D (ipc_userspace_t *)instance;
> > +	ipc_instance_t *ipc_instance =3D ipc_priv->instance;
> > +	ipc_ch_t *ch;
> > +	void *vaddr;
> > +	uint32_t i =3D 0;
> > +	uint32_t msg_size =3D sizeof(struct bbdev_ipc_enqueue_op);
> > +
> > +	PMD_INIT_FUNC_TRACE();
> > +
> > +	rte_bbdev_log_debug("%x %p", ipc_instance->initialized,
> > +		ipc_priv->instance);
> > +	ch =3D &(ipc_instance->ch_list[channel_id]);
> > +
> > +	rte_bbdev_log_debug("channel: %u, depth: %u, msg size: %u",
> > +		channel_id, q_priv->queue_size, msg_size);
> > +
> > +	/* Start init of channel */
> > +	ch->md.ring_size =3D rte_cpu_to_be_32(q_priv->queue_size);
> > +	ch->md.pi =3D 0;
> > +	ch->md.ci =3D 0;
> > +	ch->md.msg_size =3D msg_size;
> > +	for (i =3D 0; i < q_priv->queue_size; i++) {
> > +		vaddr =3D rte_malloc(NULL, msg_size, RTE_CACHE_LINE_SIZE);
> > +		if (!vaddr)
> > +			return IPC_HOST_BUF_ALLOC_FAIL;
> > +		/* Only offset now */
> > +		ch->bd_h[i].modem_ptr =3D
> > +			rte_cpu_to_be_32(HUGEPG_OFFSET(vaddr));
> > +		ch->bd_h[i].host_virt_l =3D lower_32_bits(vaddr);
> > +		ch->bd_h[i].host_virt_h =3D upper_32_bits(vaddr);
> > +		q_priv->msg_ch_vaddr[i] =3D vaddr;
> > +		/* Not sure use of this len may be for CRC*/
> > +		ch->bd_h[i].len =3D 0;
> > +	}
> > +	ch->host_ipc_params =3D
> > +		rte_cpu_to_be_32(HUGEPG_OFFSET(q_priv-
> > >host_params));
> > +
> > +	rte_bbdev_log_debug("Channel configured");
> > +	return IPC_SUCCESS;
> > +}
> > +
> > +static int
> > +la12xx_e200_queue_setup(struct rte_bbdev *dev,
> > +		struct bbdev_la12xx_q_priv *q_priv)
> > +{
> > +	struct bbdev_la12xx_private *priv =3D dev->data->dev_private;
> > +	ipc_userspace_t *ipc_priv =3D priv->ipc_priv;
> > +	struct gul_hif *mhif;
> > +	ipc_metadata_t *ipc_md;
> > +	ipc_ch_t *ch;
> > +	int instance_id =3D 0, i;
> > +	int ret;
> > +
> > +	PMD_INIT_FUNC_TRACE();
> > +
> > +	switch (q_priv->op_type) {
> > +	case RTE_BBDEV_OP_LDPC_ENC:
> > +		q_priv->la12xx_core_id =3D LA12XX_LDPC_ENC_CORE;
> > +		break;
> > +	case RTE_BBDEV_OP_LDPC_DEC:
> > +		q_priv->la12xx_core_id =3D LA12XX_LDPC_DEC_CORE;
> > +		break;
> > +	default:
> > +		rte_bbdev_log(ERR, "Unsupported op type\n");
> > +		return -1;
> > +	}
> > +
> > +	mhif =3D (struct gul_hif *)ipc_priv->mhif_start.host_vaddr;
> > +	/* offset is from start of PEB */
> > +	ipc_md =3D (ipc_metadata_t *)((uint64_t)ipc_priv-
> > >peb_start.host_vaddr +
> > +		mhif->ipc_regs.ipc_mdata_offset);
> > +	ch =3D &ipc_md->instance_list[instance_id].ch_list[q_priv->q_id];
> > +
> > +	if (q_priv->q_id < priv->num_valid_queues) {
> > +		ipc_br_md_t *md =3D &(ch->md);
> > +
> > +		q_priv->feca_blk_id =3D rte_cpu_to_be_32(ch->feca_blk_id);
> > +		q_priv->feca_blk_id_be32 =3D ch->feca_blk_id;
> > +		q_priv->host_pi =3D rte_be_to_cpu_32(md->pi);
> > +		q_priv->host_ci =3D rte_be_to_cpu_32(md->ci);
> > +		q_priv->host_params =3D (host_ipc_params_t *)
> > +			(rte_be_to_cpu_32(ch->host_ipc_params) +
> > +			((uint64_t)ipc_priv->hugepg_start.host_vaddr));
> > +
> > +		for (i =3D 0; i < q_priv->queue_size; i++) {
> > +			uint32_t h, l;
> > +
> > +			h =3D ch->bd_h[i].host_virt_h;
> > +			l =3D ch->bd_h[i].host_virt_l;
> > +			q_priv->msg_ch_vaddr[i] =3D (void *)join_32_bits(h, l);
> > +		}
> > +
> > +		rte_bbdev_log(WARNING,
> > +			"Queue [%d] already configured, not configuring
> > again",
> > +			q_priv->q_id);
> > +		return 0;
> > +	}
> > +
> > +	rte_bbdev_log_debug("setting up queue %d", q_priv->q_id);
> > +
> > +	/* Call ipc_configure_channel */
> > +	ret =3D ipc_queue_configure(q_priv->q_id, ipc_priv, q_priv);
> > +	if (ret) {
> > +		rte_bbdev_log(ERR, "Unable to setup queue (%d) (err=3D%d)",
> > +		       q_priv->q_id, ret);
> > +		return ret;
> > +	}
> > +
> > +	/* Set queue properties for LA12xx device */
> > +	switch (q_priv->op_type) {
> > +	case RTE_BBDEV_OP_LDPC_ENC:
> > +		if (priv->num_ldpc_enc_queues >=3D
> > LA12XX_MAX_LDPC_ENC_QUEUES) {
> > +			rte_bbdev_log(ERR,
> > +				"num_ldpc_enc_queues reached max
> > value");
> > +			return -1;
> > +		}
> > +		ch->la12xx_core_id =3D
> > +			rte_cpu_to_be_32(LA12XX_LDPC_ENC_CORE);
> > +		ch->feca_blk_id =3D rte_cpu_to_be_32(priv-
> > >num_ldpc_enc_queues++);
> > +		break;
> > +	case RTE_BBDEV_OP_LDPC_DEC:
> > +		if (priv->num_ldpc_dec_queues >=3D
> > LA12XX_MAX_LDPC_DEC_QUEUES) {
> > +			rte_bbdev_log(ERR,
> > +				"num_ldpc_dec_queues reached max
> > value");
> > +			return -1;
> > +		}
> > +		ch->la12xx_core_id =3D
> > +			rte_cpu_to_be_32(LA12XX_LDPC_DEC_CORE);
> > +		ch->feca_blk_id =3D rte_cpu_to_be_32(priv-
> > >num_ldpc_dec_queues++);
> > +		break;
> > +	default:
> > +		rte_bbdev_log(ERR, "Not supported op type\n");
> > +		return -1;
> > +	}
> > +	ch->op_type =3D rte_cpu_to_be_32(q_priv->op_type);
> > +	ch->depth =3D rte_cpu_to_be_32(q_priv->queue_size);
> > +
> > +	/* Store queue config here */
> > +	q_priv->feca_blk_id =3D rte_cpu_to_be_32(ch->feca_blk_id);
> > +	q_priv->feca_blk_id_be32 =3D ch->feca_blk_id;
> > +
> > +	return 0;
> > +}
> > +
> > +/* Setup a queue */
> > +static int
> > +la12xx_queue_setup(struct rte_bbdev *dev, uint16_t q_id,
> > +		const struct rte_bbdev_queue_conf *queue_conf) {
> > +	struct bbdev_la12xx_private *priv =3D dev->data->dev_private;
> > +	struct rte_bbdev_queue_data *q_data;
> > +	struct bbdev_la12xx_q_priv *q_priv;
> > +	int ret;
> > +
> > +	PMD_INIT_FUNC_TRACE();
> > +
> > +	/* Move to setup_queues callback */
> > +	q_data =3D &dev->data->queues[q_id];
> > +	q_data->queue_private =3D rte_zmalloc(NULL,
> > +		sizeof(struct bbdev_la12xx_q_priv), 0);
> > +	if (!q_data->queue_private) {
> > +		rte_bbdev_log(ERR, "Memory allocation failed for qpriv");
> > +		return -ENOMEM;
> > +	}
> > +	q_priv =3D q_data->queue_private;
> > +	q_priv->q_id =3D q_id;
> > +	q_priv->bbdev_priv =3D dev->data->dev_private;
> > +	q_priv->queue_size =3D queue_conf->queue_size;
> > +	q_priv->op_type =3D queue_conf->op_type;
> > +
> > +	ret =3D la12xx_e200_queue_setup(dev, q_priv);
> > +	if (ret) {
> > +		rte_bbdev_log(ERR, "e200_queue_setup failed for qid: %d",
> > +				     q_id);
> > +		return ret;
> > +	}
> > +
> > +	/* Store queue config here */
> > +	priv->num_valid_queues++;
> > +
> > +	return 0;
> > +}
> > +
> > +static int
> > +la12xx_start(struct rte_bbdev *dev)
> > +{
> > +	struct bbdev_la12xx_private *priv =3D dev->data->dev_private;
> > +	ipc_userspace_t *ipc_priv =3D priv->ipc_priv;
> > +	int ready =3D 0;
> > +	struct gul_hif *hif_start;
> > +
> > +	PMD_INIT_FUNC_TRACE();
> > +
> > +	hif_start =3D (struct gul_hif *)ipc_priv->mhif_start.host_vaddr;
> > +
> > +	/* Set Host Read bit */
> > +	SET_HIF_HOST_RDY(hif_start, HIF_HOST_READY_IPC_APP);
> > +
> > +	/* Now wait for modem ready bit */
> > +	while (!ready)
> > +		ready =3D CHK_HIF_MOD_RDY(hif_start,
> > HIF_MOD_READY_IPC_APP);
> > +
> > +	return 0;
> > +}
> > +
> > +static const struct rte_bbdev_ops pmd_ops =3D {
> > +	.info_get =3D la12xx_info_get,
> > +	.queue_setup =3D la12xx_queue_setup,
> > +	.queue_release =3D la12xx_queue_release,
> > +	.start =3D la12xx_start
> > +};
> > +static struct hugepage_info *
> > +get_hugepage_info(void)
> > +{
> > +	struct hugepage_info *hp_info;
> > +	struct rte_memseg *mseg;
> > +
> > +	PMD_INIT_FUNC_TRACE();
> > +
> > +	/* TODO - find a better way */
> > +	hp_info =3D rte_malloc(NULL, sizeof(struct hugepage_info), 0);
> > +	if (!hp_info) {
> > +		rte_bbdev_log(ERR, "Unable to allocate on local heap");
> > +		return NULL;
> > +	}
> > +
> > +	mseg =3D rte_mem_virt2memseg(hp_info, NULL);
> > +	hp_info->vaddr =3D mseg->addr;
> > +	hp_info->paddr =3D rte_mem_virt2phy(mseg->addr);
> > +	hp_info->len =3D mseg->len;
> > +
> > +	return hp_info;
> > +}
> > +
> > +static int open_ipc_dev(int modem_id)
> > +{
> > +	char dev_initials[16], dev_path[PATH_MAX];
> > +	struct dirent *entry;
> > +	int dev_ipc =3D 0;
> > +	DIR *dir;
> > +
> > +	dir =3D opendir("/dev/");
> > +	if (!dir) {
> > +		rte_bbdev_log(ERR, "Unable to open /dev/");
> > +		return -1;
> > +	}
> > +
> > +	sprintf(dev_initials, "gulipcgul%d", modem_id);
> > +
> > +	while ((entry =3D readdir(dir)) !=3D NULL) {
> > +		if (!strncmp(dev_initials, entry->d_name,
> > +		    sizeof(dev_initials) - 1))
> > +			break;
> > +	}
> > +
> > +	if (!entry) {
> > +		rte_bbdev_log(ERR, "Error: No gulipcgul%d device",
> > modem_id);
> > +		return -1;
> > +	}
> > +
> > +	sprintf(dev_path, "/dev/%s", entry->d_name);
> > +	dev_ipc =3D open(dev_path, O_RDWR);
> > +	if (dev_ipc  < 0) {
> > +		rte_bbdev_log(ERR, "Error: Cannot open %s", dev_path);
> > +		return -errno;
> > +	}
> > +
> > +	return dev_ipc;
> > +}
> > +
> > +static int
> > +setup_la12xx_dev(struct rte_bbdev *dev) {
> > +	struct bbdev_la12xx_private *priv =3D dev->data->dev_private;
> > +	ipc_userspace_t *ipc_priv =3D priv->ipc_priv;
> > +	struct hugepage_info *hp =3D NULL;
> > +	ipc_channel_us_t *ipc_priv_ch =3D NULL;
> > +	int dev_ipc =3D 0, dev_mem =3D 0, i;
> > +	ipc_metadata_t *ipc_md;
> > +	struct gul_hif *mhif;
> > +	uint32_t phy_align =3D 0;
> > +	int ret;
> > +
> > +	PMD_INIT_FUNC_TRACE();
> > +
> > +	if (!ipc_priv) {
> > +		/* TODO - get a better way */
> > +		/* Get the hugepage info against it */
> > +		hp =3D get_hugepage_info();
> > +		if (!hp) {
> > +			rte_bbdev_log(ERR, "Unable to get hugepage info");
> > +			ret =3D -ENOMEM;
> > +			goto err;
> > +		}
> > +
> > +		rte_bbdev_log_debug("%lx %p %lx",
> > +				hp->paddr, hp->vaddr, hp->len);
> > +
> > +		ipc_priv =3D rte_zmalloc(0, sizeof(ipc_userspace_t), 0);
> > +		if (ipc_priv =3D=3D NULL) {
> > +			rte_bbdev_log(ERR,
> > +				"Unable to allocate memory for ipc priv");
> > +			ret =3D -ENOMEM;
> > +			goto err;
> > +		}
> > +
> > +		for (i =3D 0; i < IPC_MAX_CHANNEL_COUNT; i++) {
> > +			ipc_priv_ch =3D rte_zmalloc(0,
> > +				sizeof(ipc_channel_us_t), 0);
> > +			if (ipc_priv_ch =3D=3D NULL) {
> > +				rte_bbdev_log(ERR,
> > +					"Unable to allocate memory for
> > channels");
> > +				ret =3D -ENOMEM;
> > +			}
> > +			ipc_priv->channels[i] =3D ipc_priv_ch;
> > +		}
> > +
> > +		dev_mem =3D open("/dev/mem", O_RDWR);
> > +		if (dev_mem < 0) {
> > +			rte_bbdev_log(ERR, "Error: Cannot open
> > /dev/mem");
> > +			ret =3D -errno;
> > +			goto err;
> > +		}
> > +
> > +		ipc_priv->instance_id =3D 0;
> > +		ipc_priv->dev_mem =3D dev_mem;
> > +
> > +		rte_bbdev_log_debug("hugepg input %lx %p %lx",
> > +			hp->paddr, hp->vaddr, hp->len);
> > +
> > +		ipc_priv->sys_map.hugepg_start.host_phys =3D hp->paddr;
> > +		ipc_priv->sys_map.hugepg_start.size =3D hp->len;
> > +
> > +		ipc_priv->hugepg_start.host_phys =3D hp->paddr;
> > +		ipc_priv->hugepg_start.host_vaddr =3D hp->vaddr;
> > +		ipc_priv->hugepg_start.size =3D hp->len;
> > +
> > +		rte_free(hp);
> > +	}
> > +
> > +	dev_ipc =3D open_ipc_dev(priv->modem_id);
> > +	if (dev_ipc < 0) {
> > +		rte_bbdev_log(ERR, "Error: open_ipc_dev failed");
> > +		goto err;
> > +	}
> > +	ipc_priv->dev_ipc =3D dev_ipc;
> > +
> > +	ret =3D ioctl(ipc_priv->dev_ipc, IOCTL_GUL_IPC_GET_SYS_MAP,
> > +		    &ipc_priv->sys_map);
> > +	if (ret) {
> > +		rte_bbdev_log(ERR,
> > +			"IOCTL_GUL_IPC_GET_SYS_MAP ioctl failed");
> > +		goto err;
> > +	}
> > +
> > +	phy_align =3D (ipc_priv->sys_map.mhif_start.host_phys % 0x1000);
> > +	ipc_priv->mhif_start.host_vaddr =3D
> > +		mmap(0, ipc_priv->sys_map.mhif_start.size + phy_align,
> > +		     (PROT_READ | PROT_WRITE), MAP_SHARED, ipc_priv-
> > >dev_mem,
> > +		     (ipc_priv->sys_map.mhif_start.host_phys - phy_align));
> > +	if (ipc_priv->mhif_start.host_vaddr =3D=3D MAP_FAILED) {
> > +		rte_bbdev_log(ERR, "MAP failed:");
> > +		ret =3D -errno;
> > +		goto err;
> > +	}
> > +
> > +	ipc_priv->mhif_start.host_vaddr =3D (void *) ((uint64_t)
> > +		(ipc_priv->mhif_start.host_vaddr) + phy_align);
> > +
> > +	phy_align =3D (ipc_priv->sys_map.peb_start.host_phys % 0x1000);
> > +	ipc_priv->peb_start.host_vaddr =3D
> > +		mmap(0, ipc_priv->sys_map.peb_start.size + phy_align,
> > +		     (PROT_READ | PROT_WRITE), MAP_SHARED, ipc_priv-
> > >dev_mem,
> > +		     (ipc_priv->sys_map.peb_start.host_phys - phy_align));
> > +	if (ipc_priv->peb_start.host_vaddr =3D=3D MAP_FAILED) {
> > +		rte_bbdev_log(ERR, "MAP failed:");
> > +		ret =3D -errno;
> > +		goto err;
> > +	}
> > +
> > +	ipc_priv->peb_start.host_vaddr =3D (void *)((uint64_t)
> > +		(ipc_priv->peb_start.host_vaddr) + phy_align);
> > +
> > +	phy_align =3D (ipc_priv->sys_map.modem_ccsrbar.host_phys %
> > 0x1000);
> > +	ipc_priv->modem_ccsrbar.host_vaddr =3D
> > +		mmap(0, ipc_priv->sys_map.modem_ccsrbar.size +
> > phy_align,
> > +		     (PROT_READ | PROT_WRITE), MAP_SHARED, ipc_priv-
> > >dev_mem,
> > +		     (ipc_priv->sys_map.modem_ccsrbar.host_phys -
> > phy_align));
> > +	if (ipc_priv->modem_ccsrbar.host_vaddr =3D=3D MAP_FAILED) {
> > +		rte_bbdev_log(ERR, "MAP failed:");
> > +		ret =3D -errno;
> > +		goto err;
> > +	}
> > +
> > +	ipc_priv->modem_ccsrbar.host_vaddr =3D (void *)((uint64_t)
> > +		(ipc_priv->modem_ccsrbar.host_vaddr) + phy_align);
> > +
> > +	ipc_priv->hugepg_start.modem_phys =3D
> > +		ipc_priv->sys_map.hugepg_start.modem_phys;
> > +
> > +	ipc_priv->mhif_start.host_phys =3D
> > +		ipc_priv->sys_map.mhif_start.host_phys;
> > +	ipc_priv->mhif_start.size =3D ipc_priv->sys_map.mhif_start.size;
> > +	ipc_priv->peb_start.host_phys =3D ipc_priv-
> > >sys_map.peb_start.host_phys;
> > +	ipc_priv->peb_start.size =3D ipc_priv->sys_map.peb_start.size;
> > +
> > +	rte_bbdev_log(INFO, "peb %lx %p %x",
> > +			ipc_priv->peb_start.host_phys,
> > +			ipc_priv->peb_start.host_vaddr,
> > +			ipc_priv->peb_start.size);
> > +	rte_bbdev_log(INFO, "hugepg %lx %p %x",
> > +			ipc_priv->hugepg_start.host_phys,
> > +			ipc_priv->hugepg_start.host_vaddr,
> > +			ipc_priv->hugepg_start.size);
> > +	rte_bbdev_log(INFO, "mhif %lx %p %x",
> > +			ipc_priv->mhif_start.host_phys,
> > +			ipc_priv->mhif_start.host_vaddr,
> > +			ipc_priv->mhif_start.size);
> > +	mhif =3D (struct gul_hif *)ipc_priv->mhif_start.host_vaddr;
> > +
> > +	/* offset is from start of PEB */
> > +	ipc_md =3D (ipc_metadata_t *)((uint64_t)ipc_priv-
> > >peb_start.host_vaddr +
> > +			mhif->ipc_regs.ipc_mdata_offset);
> > +
> > +	if (sizeof(ipc_metadata_t) !=3D mhif->ipc_regs.ipc_mdata_size) {
> > +		rte_bbdev_log(ERR,
> > +			"ipc_metadata_t =3D%lx, mhif-
> > >ipc_regs.ipc_mdata_size=3D%x",
> > +			sizeof(ipc_metadata_t), mhif-
> > >ipc_regs.ipc_mdata_size);
> > +		rte_bbdev_log(ERR, "--> mhif->ipc_regs.ipc_mdata_offset=3D
> > %x",
> > +			mhif->ipc_regs.ipc_mdata_offset);
> > +		rte_bbdev_log(ERR, "gul_hif size=3D%lx", sizeof(struct
> > gul_hif));
> > +		return IPC_MD_SZ_MISS_MATCH;
> > +	}
> > +
> > +	ipc_priv->instance =3D (ipc_instance_t *)
> > +		(&ipc_md->instance_list[ipc_priv->instance_id]);
> > +
> > +	rte_bbdev_log_debug("finish host init");
> > +
> > +	priv->ipc_priv =3D ipc_priv;
> > +
> > +	return 0;
> > +
> > +err:
> > +	rte_free(hp);
> > +	rte_free(ipc_priv);
> > +	rte_free(ipc_priv_ch);
> > +	if (dev_mem)
> > +		close(dev_mem);
> > +	if (dev_ipc)
> > +		close(dev_ipc);
> > +
> > +	return ret;
> > +}
> > +
> >  static inline int
> >  parse_u16_arg(const char *key, const char *value, void *extra_args)  {=
 @@ -
> > 123,6 +667,7 @@ la12xx_bbdev_create(struct rte_vdev_device *vdev,
> >  	struct rte_bbdev *bbdev;
> >  	const char *name =3D rte_vdev_device_name(vdev);
> >  	struct bbdev_la12xx_private *priv;
> > +	int ret;
> >
> >  	PMD_INIT_FUNC_TRACE();
> >
> > @@ -152,7 +697,13 @@ la12xx_bbdev_create(struct rte_vdev_device
> > *vdev,
> >
> >  	rte_bbdev_log(INFO, "Setting Up %s: DevId=3D%d, ModemId=3D%d",
> >  				name, bbdev->data->dev_id, priv-
> > >modem_id);
> > -	bbdev->dev_ops =3D NULL;
> > +	ret =3D setup_la12xx_dev(bbdev);
> > +	if (ret) {
> > +		rte_bbdev_log(ERR, "IPC Setup failed for %s", name);
> > +		rte_free(bbdev->data->dev_private);
> > +		return ret;
> > +	}
> > +	bbdev->dev_ops =3D &pmd_ops;
> >  	bbdev->device =3D &vdev->device;
> >  	bbdev->data->socket_id =3D 0;
> >  	bbdev->intr_handle =3D NULL;
> > diff --git a/drivers/baseband/la12xx/bbdev_la12xx.h
> > b/drivers/baseband/la12xx/bbdev_la12xx.h
> > index 5228502331..49c37fe2fe 100644
> > --- a/drivers/baseband/la12xx/bbdev_la12xx.h
> > +++ b/drivers/baseband/la12xx/bbdev_la12xx.h
> > @@ -5,16 +5,10 @@
> >  #ifndef __BBDEV_LA12XX_H__
> >  #define __BBDEV_LA12XX_H__
> >
> > -#define BBDEV_IPC_ENC_OP_TYPE	1
> > -#define BBDEV_IPC_DEC_OP_TYPE	2
> > -
> > -#define MAX_LDPC_ENC_FECA_QUEUES	4
> > -#define MAX_LDPC_DEC_FECA_QUEUES	4
> > -
> >  #define MAX_CHANNEL_DEPTH 16
> >  /* private data structure */
> >  struct bbdev_la12xx_private {
> > -	void *ipc_priv;
> > +	ipc_userspace_t *ipc_priv;
> >  	uint8_t num_valid_queues;
> >  	uint8_t max_nb_queues;
> >  	uint8_t num_ldpc_enc_queues;
> > @@ -52,5 +46,6 @@ struct bbdev_la12xx_q_priv {
> >
> >  #define lower_32_bits(x) ((uint32_t)((uint64_t)x))  #define
> > upper_32_bits(x) ((uint32_t)(((uint64_t)(x) >> 16) >> 16))
> > -
> > +#define join_32_bits(upper, lower) \
> > +	((size_t)(((uint64_t)(upper) << 32) | (uint32_t)(lower)))
> >  #endif
> > diff --git a/drivers/baseband/la12xx/bbdev_la12xx_ipc.h
> > b/drivers/baseband/la12xx/bbdev_la12xx_ipc.h
> > index 9aa5562981..5f613fb087 100644
> > --- a/drivers/baseband/la12xx/bbdev_la12xx_ipc.h
> > +++ b/drivers/baseband/la12xx/bbdev_la12xx_ipc.h
> > @@ -4,9 +4,182 @@
> >  #ifndef __BBDEV_LA12XX_IPC_H__
> >  #define __BBDEV_LA12XX_IPC_H__
> >
> > +#define LA12XX_MAX_QUEUES 20
> > +#define HOST_RX_QUEUEID_OFFSET LA12XX_MAX_QUEUES
> > +
> > +/** No. of max channel per instance */
> > +#define IPC_MAX_CHANNEL_COUNT	(64)
> > +
> >  /** No. of max channel per instance */
> >  #define IPC_MAX_DEPTH	(16)
> >
> > +/** No. of max IPC instance per modem */
> > +#define IPC_MAX_INSTANCE_COUNT	(1)
> > +
> > +/** Error codes */
> > +#define IPC_SUCCESS		(0) /** IPC operation success */
>=20
> Cosmetic, but such comments are expected to be as /**< IPC operation succ=
ess
> */

Okay

>=20
> > +#define IPC_INPUT_INVALID	(-1) /** Invalid input to API */
> > +#define IPC_CH_INVALID		(-2) /** Channel no is invalid */
> > +#define IPC_INSTANCE_INVALID	(-3) /** Instance no is invalid */
> > +#define IPC_MEM_INVALID		(-4) /** Insufficient memory */
> > +#define IPC_CH_FULL		(-5) /** Channel is full */
> > +#define IPC_CH_EMPTY		(-6) /** Channel is empty */
> > +#define IPC_BL_EMPTY		(-7) /** Free buffer list is empty */
> > +#define IPC_BL_FULL		(-8) /** Free buffer list is full */
> > +#define IPC_HOST_BUF_ALLOC_FAIL	(-9) /** DPDK malloc fail */
> > +#define IPC_MD_SZ_MISS_MATCH	(-10) /** META DATA size in mhif
> > miss matched*/
> > +#define IPC_MALLOC_FAIL		(-11) /** system malloc fail */
> > +#define IPC_IOCTL_FAIL		(-12) /** IOCTL call failed */
> > +#define IPC_MMAP_FAIL		(-14) /** MMAP fail */
> > +#define IPC_OPEN_FAIL		(-15) /** OPEN fail */
> > +#define IPC_EVENTFD_FAIL	(-16) /** eventfd initialization failed */
> > +#define IPC_NOT_IMPLEMENTED	(-17) /** IPC feature is not
> > implemented yet*/
> > +
> > +#define SET_HIF_HOST_RDY(hif, RDY_MASK) (hif->host_ready |=3D
> > RDY_MASK)
> > +#define CHK_HIF_MOD_RDY(hif, RDY_MASK) (hif->mod_ready &
> > RDY_MASK)
> > +
> > +/* Host Ready bits */
> > +#define HIF_HOST_READY_HOST_REGIONS	(1 << 0)
> > +#define HIF_HOST_READY_IPC_LIB		(1 << 12)
> > +#define HIF_HOST_READY_IPC_APP		(1 << 13)
> > +#define HIF_HOST_READY_FECA		(1 << 14)
> > +
> > +/* Modem Ready bits */
> > +#define HIF_MOD_READY_IPC_LIB		(1 << 5)
> > +#define HIF_MOD_READY_IPC_APP		(1 << 6)
> > +#define HIF_MOD_READY_FECA		(1 << 7)
> > +
> > +typedef void *ipc_t;
> > +
> > +struct ipc_msg {
> > +	int chid;
> > +	void *addr;
> > +	uint32_t len;
> > +	uint8_t flags;
> > +};
> > +
> > +typedef struct {
> > +	uint64_t host_phys;
> > +	uint32_t modem_phys;
> > +	void    *host_vaddr;
> > +	uint32_t size;
> > +} mem_range_t;
> > +
> > +#define GUL_IPC_MAGIC	'R'
> > +
> > +#define IOCTL_GUL_IPC_GET_SYS_MAP _IOW(GUL_IPC_MAGIC, 1, struct
> > ipc_msg
> > +*) #define IOCTL_GUL_IPC_CHANNEL_REGISTER _IOWR(GUL_IPC_MAGIC,
> > 4,
> > +struct ipc_msg *) #define IOCTL_GUL_IPC_CHANNEL_DEREGISTER \
> > +	_IOWR(GUL_IPC_MAGIC, 5, struct ipc_msg *) #define
> > +IOCTL_GUL_IPC_CHANNEL_RAISE_INTERRUPT _IOW(GUL_IPC_MAGIC, 6,
> > int *)
> > +
> > +/** buffer ring common metadata */
> > +typedef struct ipc_bd_ring_md {
> > +	volatile uint32_t pi;		/**< Producer index and flag (MSB)
> > +					  *  which flip for each Ring wrapping
> > +					  */
> > +	volatile uint32_t ci;		/**< Consumer index and flag (MSB)
> > +					  *  which flip for each Ring wrapping
> > +					  */
> > +	uint32_t ring_size;	/**< depth (Used to roll-over pi/ci) */
> > +	uint32_t msg_size;	/**< Size of the each buffer */
> > +} __rte_packed ipc_br_md_t;
> > +
> > +/** IPC buffer descriptor */
> > +typedef struct ipc_buffer_desc {
> > +	union {
> > +		uint64_t host_virt;	/**< msg's host virtual address */
> > +		struct {
> > +			uint32_t host_virt_l;
> > +			uint32_t host_virt_h;
> > +		};
> > +	};
> > +	uint32_t modem_ptr;	/**< msg's modem physical address */
> > +	uint32_t len;		/**< msg len */
> > +} __rte_packed ipc_bd_t;
> > +
> > +typedef struct ipc_channel {
> > +	uint32_t ch_id;		/**< Channel id */
> > +	ipc_br_md_t md;			/**< Metadata for BD ring */
> > +	ipc_bd_t bd_h[IPC_MAX_DEPTH];	/**< Buffer Descriptor on
> > Host */
> > +	ipc_bd_t bd_m[IPC_MAX_DEPTH];	/**< Buffer Descriptor on
> > Modem */
> > +	uint32_t op_type;		/**< Type of the BBDEV operation
> > +					  * supported on this channel
> > +					  */
> > +	uint32_t depth;			/**< Channel depth */
> > +	uint32_t feca_blk_id;	/**< FECA Transport Block ID for processing
> > */
> > +	uint32_t la12xx_core_id;/**< LA12xx core ID on which this will be
> > +				  * scheduled
> > +				  */
> > +	uint32_t feca_input_circ_size;	/**< FECA transport block input
> > +					  * circular buffer size
> > +					  */
> > +	uint32_t host_ipc_params;	/**< Address for host IPC
> > parameters */
> > +} __rte_packed ipc_ch_t;
> > +
> > +typedef struct ipc_instance {
> > +	uint32_t instance_id;		/**< instance id, use to init this
> > +					  * instance by ipc_init API
> > +					  */
> > +	uint32_t initialized;		/**< Set in ipc_init */
> > +	ipc_ch_t ch_list[IPC_MAX_CHANNEL_COUNT];
> > +		/**< Channel descriptors in this instance */ } __rte_packed
> > +ipc_instance_t;
> > +
> > +typedef struct ipc_metadata {
> > +	uint32_t ipc_host_signature; /**< IPC host signature, Set by host/L2
> > */
> > +	uint32_t ipc_geul_signature; /**< IPC geul signature, Set by modem
> > */
> > +	ipc_instance_t instance_list[IPC_MAX_INSTANCE_COUNT];
> > +} __rte_packed ipc_metadata_t;
> > +
> > +typedef struct ipc_channel_us_priv {
> > +	int32_t		eventfd;
> > +	uint32_t	channel_id;
> > +	/* In flight packets status for buffer list. */
> > +	uint8_t		bufs_inflight[IPC_MAX_DEPTH];
> > +} ipc_channel_us_t;
> > +
> > +typedef struct {
> > +	uint64_t host_phys;
> > +	uint32_t modem_phys;
> > +	uint32_t size;
> > +} mem_strt_addr_t;
> > +
> > +typedef struct {
> > +	mem_strt_addr_t modem_ccsrbar;
> > +	mem_strt_addr_t peb_start; /* PEB meta data */
> > +	mem_strt_addr_t mhif_start; /* MHIF meta daat */
> > +	mem_strt_addr_t hugepg_start; /* Modem to access hugepage */ }
> > +sys_map_t;
> > +
> > +typedef struct ipc_priv_t {
> > +	int instance_id;
> > +	int dev_ipc;
> > +	int dev_mem;
> > +	sys_map_t sys_map;
> > +	mem_range_t modem_ccsrbar;
> > +	mem_range_t peb_start;
> > +	mem_range_t mhif_start;
> > +	mem_range_t hugepg_start;
> > +	ipc_channel_us_t *channels[IPC_MAX_CHANNEL_COUNT];
> > +	ipc_instance_t	*instance;
> > +	ipc_instance_t	*instance_bk;
> > +} ipc_userspace_t;
> > +
> > +/** Structure specifying enqueue operation (enqueue at LA1224) */
> > +struct bbdev_ipc_enqueue_op {
> > +	/** Status of operation that was performed */
> > +	int32_t status;
> > +	/** CRC Status of SD operation that was performed */
> > +	int32_t crc_stat_addr;
> > +	/** HARQ Output buffer memory length for Shared Decode.
> > +	 * Filled by LA12xx.
> > +	 */
> > +	uint32_t out_len;
> > +	/** Reserved (for 8 byte alignment) */
> > +	uint32_t rsvd;
> > +};
> > +
> >  /* This shared memory would be on the host side which have copy of som=
e
> >   * of the parameters which are also part of Shared BD ring. Read acces=
s
> >   * of these parameters from the host side would not be over PCI.
> > @@ -14,7 +187,21 @@
> >  typedef struct host_ipc_params {
> >  	volatile uint32_t pi;
> >  	volatile uint32_t ci;
> > -	volatile uint32_t modem_ptr[IPC_MAX_DEPTH];
> > +	volatile uint32_t bd_m_modem_ptr[IPC_MAX_DEPTH];
> >  } __rte_packed host_ipc_params_t;
> >
> > +struct hif_ipc_regs {
> > +	uint32_t ipc_mdata_offset;
> > +	uint32_t ipc_mdata_size;
> > +} __rte_packed;
> > +
> > +struct gul_hif {
> > +	uint32_t ver;
> > +	uint32_t hif_ver;
> > +	uint32_t status;
> > +	volatile uint32_t host_ready;
> > +	volatile uint32_t mod_ready;
> > +	struct hif_ipc_regs ipc_regs;
> > +} __rte_packed;
> > +
> >  #endif
> > --
> > 2.17.1