Hi

> -----Original Message-----
> From: Ferruh Yigit <[email protected]>
> Sent: Sunday, October 28, 2018 6:05 AM
> To: Gagandeep Singh <[email protected]>; [email protected]; Akhil Goyal
> <[email protected]>
> Cc: Hemant Agrawal <[email protected]>
> Subject: Re: [dpdk-dev] [PATCH v5 04/15] crypto/caam_jr: add UIO specific
> operations
> 
> On 10/22/2018 3:57 PM, Gagandeep Singh wrote:
> > caam_jr need support from kernel caam driver for job ring
> > initialisation. So to access register space for job ring and allow re
> > configure and map to userspace UIO interface is used. This also allows
> > to handle the caam interrupts from the user space.
> >
> > This patch adds UIO specific operations
> >
> > Signed-off-by: Gagandeep Singh <[email protected]>
> > Signed-off-by: Hemant Agrawal <[email protected]>
> > Acked-by: Akhil Goyal <[email protected]>
> 
> <...>
> 
> > +void
> > +free_job_ring(uint32_t uio_fd)
> > +{
> > +   struct uio_job_ring *job_ring = NULL;
> > +   int i;
> > +
> > +   if (!job_ring->uio_fd)
> > +           return;
> 
> It seems this function is not called. Can the intention be:
> 
>  if (!uio_fd)
>        return;

Patch has been sent http://patchwork.dpdk.org/patch/47557/ 

Reply via email to