On 25-02-20, 09:14, Dave Jiang wrote: > > > On 2/25/20 1:14 AM, Vinod Koul wrote: > > The driver assigns result of check_vma() to rc and never checks the > > result, so remove the assignment > > > > drivers/dma/idxd/cdev.c: In function ‘idxd_cdev_mmap’: > > drivers/dma/idxd/cdev.c:136:6: warning: variable ‘rc’ set but not used [-Wunused-but-set-variable] > > 136 | int rc; > > | ^~ > > > > Signed-off-by: Vinod Koul <vkoul@xxxxxxxxxx> > > --- > > > > Dave, if you want the result of the check_vma() to be checked, feel free to > > send the patch for that > > Yes I will send a fix. Mistake from several code merges. Oops. Ok, no issues -- ~Vinod
