On Mon, Aug 05, 2019 at 01:02:08PM +0000, Pascal Van Leeuwen wrote: > > From: Antoine Tenart <antoine.tenart@xxxxxxxxxxx> > > > > Using this function that is designed to sleep with a delay of 0 and > > designed to timeout with a value of 1 does not seem to follow what the > > function is designed for :) It could work but I would suggest to keep > > the polling as it is in the patch in this case. > > > Ok, will do. Does that mean that patch 4/4 is good as is? > i.e. can I add an "Acked-by: Antoine Tenart" line when I resubmit? Sure, the rest of the patch looked good and you can add: Acked-by: Antoine Tenart <antoine.tenart@xxxxxxxxxxx> Thanks! Antoine -- Antoine Ténart, Bootlin Embedded Linux and Kernel engineering https://bootlin.com