Re: [dpdk-dev] [PATCH 27/44] event/octeontx2: add event timer support

[Jerin Jacob Kollanukkaran <jerinj@marvell.com>]

> -----Original Message-----
> From: pbhagavatula@marvell.com <pbhagavatula@marvell.com>
> Sent: Sunday, June 2, 2019 12:24 AM
> To: Jerin Jacob Kollanukkaran <jerinj@marvell.com>; Pavan Nikhilesh
> Bhagavatula <pbhagavatula@marvell.com>; Anatoly Burakov
> <anatoly.burakov@intel.com>
> Cc: dev@dpdk.org; Erik Gabriel Carrillo <erik.g.carrillo@intel.com>
> Subject: [dpdk-dev] [PATCH 27/44] event/octeontx2: add event timer support
> 
> From: Pavan Nikhilesh <pbhagavatula@marvell.com>
> 
> Add event timer adapter aka TIM initilization on SSO probe.
> 
> Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
> ---
> Cc: Erik Gabriel Carrillo <erik.g.carrillo@intel.com>
> 
>  drivers/event/octeontx2/Makefile         |  1 +
>  drivers/event/octeontx2/meson.build      |  1 +
>  drivers/event/octeontx2/otx2_evdev.c     |  2 +
>  drivers/event/octeontx2/otx2_tim_evdev.c | 61 ++++++++++++++++++++++++
> drivers/event/octeontx2/otx2_tim_evdev.h | 23 +++++++++
>  5 files changed, 88 insertions(+)
>  create mode 100644 drivers/event/octeontx2/otx2_tim_evdev.c
>  create mode 100644 drivers/event/octeontx2/otx2_tim_evdev.h
> 
> diff --git a/drivers/event/octeontx2/Makefile
> diff --git a/drivers/event/octeontx2/otx2_tim_evdev.h
> b/drivers/event/octeontx2/otx2_tim_evdev.h
> new file mode 100644
> index 000000000..14bbaf43c
> --- /dev/null
> +++ b/drivers/event/octeontx2/otx2_tim_evdev.h
> @@ -0,0 +1,23 @@
> +/* SPDX-License-Identifier: BSD-3-Clause
> + * Copyright(C) 2019 Marvell International Ltd.
> + */
> +
> +#ifndef __OTX2_TIM_EVDEV_H__
> +#define __OTX2_TIM_EVDEV_H__
> +
> +#include <rte_event_timer_adapter.h>
> +
> +#include "otx2_dev.h"
> +
> +#define OTX2_TIM_EVDEV_NAME otx2_tim_eventdev
> +
> +struct otx2_tim_evdev {
> +	struct rte_pci_device *pci_dev;
> +	struct otx2_mbox *mbox;
> +	uint16_t nb_rings;
> +	uintptr_t bar2;
> +};
> +
> +void otx2_tim_init(struct rte_pci_device *pci_dev, struct otx2_dev
> +*cmn_dev);

Please add counter part_fini as well.

> +
> +#endif /* __OTX2_TIM_EVDEV_H__ */
> --
> 2.21.0