On Fri, Sep 23, 2022 at 08:10:48PM +0300, Andy Shevchenko wrote: > The readx_poll_timeout() consolidates the necessary code under > macro. Replace current code with it. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > --- > v2: dropped TODO: in the Subject line > include/linux/serdev.h | 14 ++++---------- > 1 file changed, 4 insertions(+), 10 deletions(-) > > diff --git a/include/linux/serdev.h b/include/linux/serdev.h > index 66f624fc618c..69d9c3188065 100644 > --- a/include/linux/serdev.h > +++ b/include/linux/serdev.h > @@ -7,9 +7,11 @@ > > #include <linux/types.h> > #include <linux/device.h> > +#include <linux/iopoll.h> > #include <linux/uaccess.h> > #include <linux/termios.h> > #include <linux/delay.h> > +#include <vdso/time64.h> Ick, just for USEC_PER_MSEC? That should be in units.h, not burried in a vdso-only .h file. This feels wrong, sorry. greg k-h