Re: [PATCH 4/7] staging: mt7621-dma: Remove assignment in if

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Mar 21, 2018 at 09:21:55AM +1100, NeilBrown wrote:
> I don't like this change.  We now call vchan_find_desc() in situations
> where we didn't before.  It isn't a very expensive function so that
> doesn't matter a lot, but I think it is best not to do this.
> If you want to remove the warning (and I'm not convinced that we need
> to), please make it.
> 
>   ....
>   } else {
>       vdesc = vchan_find_desc(...);
>       if (vdesec)
>           state->residue = ...
>  }

Thank you for your reviewing all the patches. I'will change this patch
based on your advice and send a v2.

Christian
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux